search

BlinkUX / sequelize-mock

A simple mock interface specifically for testing code relying on Sequelize models
https://sequelize-mock.readthedocs.io
MIT License
139 stars 73 forks source link

Update model.js #88

Open luiztools opened 3 years ago

luiztools commented 3 years ago

Modern Sequelize doesn't have findById anymore, so, we need to change this to add support to findByPk function.

tawfiknasser commented 2 years ago

I agree that we need to support findByPk and it should replace it. but this is another functionality. primary key is not always id. Pk column can be called in many names (id, UUID, ID).

also we need to refactor the tests if we are changing the function name.