minor refactoring + first tests

spec/associations/has-many.spec.js

@@ -3,14 +3,14 @@ if (typeof require === 'function') {
       , Sequelize = require("../../index")
       , config    = require("../config/config")
       , sequelize = new Sequelize(config.database, config.username, config.password, {
-          logging: false
+          logging: true
         })
 }
 
 buster.spec.expose()
 buster.testRunner.timeout = 500
 
-describe('Associations', function() {
+describe('HasMany', function() {
   before(function(done) {
     var self = this
 
@@ -18,10 +18,10 @@ describe('Associations', function() {
       .dropAllTables()
       .success(done)
       .error(function(err) { console.log(err) })
-  }),
+  })
 
-  describe('hasMany', function() {
-    describe('hasAssociation', function() {
+  describe('Many To One', function() {
+    describe('hasSingle', function() {
       before(function(done) {
         var self = this
 
@@ -33,7 +33,6 @@ describe('Associations', function() {
         sequelize.sync({ force: true }).success(done)
       })
 
-      describe('hasLabel', function() {
       it('does not have any labels assigned to it initially', function(done) {
         var self = this
 
@@ -82,7 +81,18 @@ describe('Associations', function() {
       })
     })
 
-      describe('hasLabels', function() {
+    describe('hasAll', function() {
+      before(function(done) {
+        var self = this
+
+        this.Article = sequelize.define('Article', { 'title': Sequelize.STRING })
+        this.Label = sequelize.define('Label', { 'text': Sequelize.STRING })
+
+        this.Article.hasMany(this.Label)
+
+        sequelize.sync({ force: true }).success(done)
+      })
+
       it('answers false if only some labels have been assigned', function(done) {
         var self = this
 
@@ -123,7 +133,7 @@ describe('Associations', function() {
     })
   })
 
-    describe('many to many', function() {
+  describe('(N:M)', function() {
     it("removes the reference id, which was added in the first place", function() {
       var User = sequelize.define('User', { username: Sequelize.STRING })
         , Task = sequelize.define('Task', { title: Sequelize.STRING })
@@ -134,6 +144,48 @@ describe('Associations', function() {
       Task.hasMany(User)
       expect(Task.attributes.UserId).not.toBeDefined()
     })
+
+    it("=>clears associations when passing null to the set-method", function(done) {
+      var User = sequelize.define('User', { username: Sequelize.STRING })
+        , Task = sequelize.define('Task', { title: Sequelize.STRING })
+
+      User.hasMany(Task)
+      Task.hasMany(User)
+
+      sequelize.sync().success(function() {
+        console.log('sync ok')
+        User.create({ username: 'foo' }).success(function(user) {
+          console.log('user was created')
+          Task.create({ title: 'task' }).run().success(function(task) {
+            console.log('task was created')
+            expect(1).toEqual(1)
+            return done()
+            task.setUsers([ user ])
+              .success(function() {
+                console.log('set users done')
+
+                task.getUsers().success(function(_users) {
+                  expect(_users.length).toEqual(1)
+
+                  task.setUsers(null).success(function() {
+
+                    task.getUsers().success(function(_users) {
+                      expect(_users.length).toEqual(0)
+                      done()
+                    })
+                  }).error(function(err) {
+                    console.log(err)
+                  })
+                }).error(function(err) {
+                  console.log(err)
+                })
+              })
+              .error(function(err) {
+                console.log(err)
+              })
+          })
+        })
+      })
     })
   })
 })