Inject has-one scope in where clause

lib/associations/has-one.js

@@ -21,6 +21,7 @@ var HasOne = function(srcModel, targetModel, options) {
   this.source = srcModel;
   this.target = targetModel;
   this.options = options;
+  this.scope = options.scope;
   this.isSingleAssociation = true;
   this.isSelfAssociation = (this.source === this.target);
   this.as = this.options.as;

test/integration/associations/scope.test.js

@@ -17,7 +17,8 @@ describe(Support.getTestDialectTeaser('associations'), function() {
       this.Comment = this.sequelize.define('comment', {
         title: Sequelize.STRING,
         commentable: Sequelize.STRING,
-        commentable_id: Sequelize.INTEGER
+        commentable_id: Sequelize.INTEGER,
+        isMain: Sequelize.BOOLEAN
       }, {
         instanceMethods: {
           getItem: function() {
@@ -29,12 +30,26 @@ describe(Support.getTestDialectTeaser('associations'), function() {
       this.Post.addScope('withComments', {
         include: [this.Comment]
       });
+      this.Post.addScope('withMainComment', {
+          include: [{
+            model: this.Comment,
+            as: 'mainComment'
+          }]
+      });
       this.Post.hasMany(this.Comment, {
         foreignKey: 'commentable_id',
         scope: {
           commentable: 'post'
         }
       });
+      this.Post.hasOne(this.Comment, {
+        foreignKey: 'commentable_id',
+        as: 'mainComment',
+        scope: {
+          commentable: 'post',
+          isMain: true
+        }
+      });
       this.Comment.belongsTo(this.Post, {
         foreignKey: 'commentable_id',
         as: 'post'
@@ -63,6 +78,41 @@ describe(Support.getTestDialectTeaser('associations'), function() {
       });
     });
 
+    describe('1:1', function() {
+      it('should create, find and include associations with scope values', function() {
+        var self = this;
+        return this.sequelize.sync({force: true}).then(function() {
+          return Promise.join(
+            self.Post.create(),
+            self.Comment.create({
+              title: 'I am a comment'
+            }),
+            self.Comment.create({
+              title: 'I am a main comment',
+              isMain: true
+            })
+          );
+        }).bind(this).spread(function(post) {
+          this.post = post;
+          return post.createComment({
+            title: 'I am a post comment'
+          });
+        }).then(function() {
+          return self.Post.scope('withMainComment').findById(this.post.id);
+        }).then(function(post) {
+          expect(post.mainComment).to.be.null;
+          return Promise.join(
+            post.createMainComment({
+              title: 'I am a main post comment'
+            }),
+            self.Post.scope('withMainComment').findById(this.post.id)
+          );
+        }).spread(function(mainComment, post) {
+          expect(post.mainComment.id).to.equal(mainComment.id);
+        });
+      });
+    });
+
     describe('1:M', function() {
       it('should create, find and include associations with scope values', function() {
         var self = this;

test/integration/model/scope/associations.test.js

@@ -86,7 +86,7 @@ describe(Support.getTestDialectTeaser('Model'), function() {
               where: {
                 active: false
               }
-            },
+            }
           }
         });
 
@@ -106,12 +106,12 @@ describe(Support.getTestDialectTeaser('Model'), function() {
             this.ScopeMe.create({ id: 4, username: 'fred', email: 'fred@foobar.com', access_level: 3, other_value: 7, parent_id: 1}),
             this.ScopeMe.create({ id: 5, username: 'bob', email: 'bob@foobar.com', access_level: 1, other_value: 9, parent_id: 5}),
             this.Company.create({ id: 1, active: true}),
-            this.Company.create({ id: 2, active: false}),
+            this.Company.create({ id: 2, active: false})
           ]);
         }).spread(function (u1, u2, u3, u4, u5, c1, c2) {
           return Promise.all([
             c1.setUsers([u1, u2, u3, u4]),
-            c2.setUsers([u5]),
+            c2.setUsers([u5])
           ]);
         });
       });
@@ -178,8 +178,8 @@ describe(Support.getTestDialectTeaser('Model'), function() {
               return this.ScopeMe.findById(1);
             }).then(function (user) {
               return user.getProfile({ scope: false });
-            }).then(function (project) {
-              expect(project).to.be.ok;
+            }).then(function (profile) {
+              expect(profile).to.be.ok;
             });
           });
 
@@ -217,8 +217,8 @@ describe(Support.getTestDialectTeaser('Model'), function() {
               return this.ScopeMe.findById(1);
             }).then(function (user) {
               return user.getProfile();
-            }).then(function (project) {
-              expect(project).not.to.be.ok;
+            }).then(function (profile) {
+              expect(profile).not.to.be.ok;
             });
           });
 
@@ -258,8 +258,8 @@ describe(Support.getTestDialectTeaser('Model'), function() {
               return this.ScopeMe.findById(1);
             }).then(function (user) {
               return user.getProfile({ scope: 'notActive' });
-            }).then(function (project) {
-              expect(project).not.to.be.ok;
+            }).then(function (profile) {
+              expect(profile).not.to.be.ok;
             });
           });