More test coverage, populate associations only if config setting is true

cesardeazevedo / sails-hook-sequelize-blueprints

Sails blueprints for sequelize ORM

MIT License

32 stars 19 forks source link

More test coverage, populate associations only if config setting is true #7

Closed joshgagnon closed 9 years ago

joshgagnon commented 9 years ago

I was having a test fail, so I needed to add notNull constraint to image.user association. Also having include: [{all: true}] can lead to performance issues if there are many associations, so it now just does a simple check of the config settings. Ideally it should mimic the behaviour of sails' ActionUtil.populateEach', but this is a start.