association helper methods do not appear to be created.

PROBLEM

Note: I cannot tell if this is a docs issue, a bug, or user error.

I have read through other seemingly-related bug reports (#7738, #6187, #10955) and lots of documentation and cannot find an answer.

My question is basically identical to #10955. I have defined associations on my models but when I try to call course.getCoursePaths(), I get an error that getCoursePaths()is not a function on course.

Note that I have also tried all variations I can think of re: using hasOne instead of belongsTo and using plural/singular

I am using sequelize orm version 5.18.1 and node 10.16.0

MIGRATIONS

These execute happily

module.exports = {
  up: async (queryInterface, Sequelize) => {
    await queryInterface.createTable('courses', {
      id: {
        allowNull: false,
        autoIncrement: true,
        primaryKey: true,
        type: Sequelize.INTEGER,
      },
      ...other fields....
      created_at: {
        allowNull: false,
        type: Sequelize.DATE,
        defaultValue: Sequelize.literal('CURRENT_TIMESTAMP'),
      },
      updated_at: {
        allowNull: false,
        type: Sequelize.DATE,
        defaultValue: Sequelize.literal('CURRENT_TIMESTAMP'),
      },
    });
    await queryInterface.bulkInsert('courses', <some seed data>);

  },
  down: (queryInterface, Sequelize) => {
    return queryInterface.dropTable('courses');
  },
};

module.exports = {
  up: async (queryInterface, Sequelize) => {
    await queryInterface.createTable('course_paths', {
      id: {
        allowNull: false,
        autoIncrement: true,
        primaryKey: true,
        type: Sequelize.INTEGER,
      },
      ...other fields
      created_at: {
        allowNull: false,
        type: Sequelize.DATE,
        defaultValue: Sequelize.literal('CURRENT_TIMESTAMP'),
      },
      updated_at: {
        allowNull: false,
        type: Sequelize.DATE,
        defaultValue: Sequelize.literal('CURRENT_TIMESTAMP'),
      },
    });

    await queryInterface.bulkInsert('course_paths', <some seed data>);
  },
  down: (queryInterface, Sequelize) => {
    return queryInterface.dropTable('course_paths');
  },
};

module.exports = {
  up: async (queryInterface, Sequelize) => {
    await queryInterface.addColumn('courses', 'path_id', {
      type: Sequelize.INTEGER,
      references: {
        model: 'course_paths',
        key: 'id',
      },
    });

    await queryInterface.<INSERT SOME SEED ASSOCIATIONS>

    // ADD NON-NULL CONSTRAINT AFTER POPULATION
    await queryInterface.changeColumn('courses', 'path_id', {
      type: Sequelize.INTEGER,
      references: {
        model: 'course_paths',
        key: 'id',
      },
      allowNull: false,
    });
  },

  down: (queryInterface, Sequelize) => {
    return queryInterface.removeColumn('courses', 'path_id');
  },
};

MODELS

module.exports = (sequelize, DataTypes) => {
  const Course = sequelize.define(
    'course',
    {
      ....some other fields
      path_id: DataTypes.INTEGER,
    },
    {
      underscored: true,
    }
  );

  Course.associate = function(models) {
    // I HAVE ALSO TRIED THIS AS hasOne
    models.course.belongsTo(models.course_path, {
      // I HAVE ALSO TRIED THIS VALUE AS 'id' with sourceKey 'path_id', did not work
      foreignKey: 'path_id',
    });
  };
  return Course;
};

module.exports = (sequelize, DataTypes) => {
  const CoursePath = sequelize.define(
    'course_path',
    {
      ....you know, implicit integer id, other fields
      title: DataTypes.STRING,
    },
    {
      underscored: true,
    }
  );
  CoursePath.associate = function(models) {
    // I HAVE ALSO TRIED EXCLUDING THE FOLLOWING LINES
    models.course_path.hasMany(models.course, {
      foreignKey: 'path_id',
      sourceKey: 'id',
    });
  };
  return CoursePath;
};

CODE

const course = await db.course.findOne({
    where: {
      <some identifier>: <some course identifier>
    },
    // I HAVE ALSO TRIED OMITTING THIS
    include: [
      {model: db.course_path}
    ]
  });

// THIS IS WHERE IT ERRORS
// I HAVE ALSO TRIED getCoursePaths
// this succeeds with course.course_path
const path = await course.getCoursePath();

What is going on here? Why doesn’t getCoursePath() exist?

[ccemeraldeyes]

d’oh.

it’s getCourse_path.

[ccemeraldeyes]

@papb ty! I saw that in the docs in some examples camelCase is the expectation for field names so I guess that’s on us. Although none of the examples use multi-word field names! But yes, it would be great to convert entirely to camelCase for these methods 😃