Use co-mocha for tests to simplify async flow. Fixes #19

Author: techniq

mocha.start.js

@@ -6,6 +6,8 @@ assert.equalObjects = function (a, b, m) {
   assert.deepEqual(JSON.parse(JSON.stringify(a)), JSON.parse(JSON.stringify(b)), m || 'Objects should be equal!');
 };
 var mocha = require('mocha');
+var coMocha = require('co-mocha');
+coMocha(mocha);
 var sinon = require('sinon');
 var JSData = require('js-data');
 JSData.DSUtils.Promise = require('bluebird');

package.json

@@ -30,6 +30,7 @@
     "babel-loader": "5.3.2",
     "bluebird": "2.10.0",
     "chai": "3.2.0",
+    "co-mocha": "^1.1.2",
     "grunt": "0.4.5",
     "grunt-contrib-watch": "0.6.1",
     "grunt-karma-coveralls": "2.5.4",

test/create.spec.js

@@ -1,27 +1,23 @@
 describe('DSSqlAdapter#create', function () {
-  it('should create a user in a sql db', function () {
-    var id;
-    return adapter.create(User, {name: 'John'}).then(function (user) {
-      id = user.id;
-      assert.equal(user.name, 'John');
-      assert.isDefined(user.id);
-      return adapter.find(User, user.id);
-    })
-      .then(function (user) {
-        assert.equal(user.name, 'John');
-        assert.isDefined(user.id);
-        assert.equalObjects(user, {id: id, name: 'John', age: null, profileId: null});
-        return adapter.destroy(User, user.id);
-      })
-      .then(function (user) {
-        assert.isFalse(!!user);
-        return adapter.find(User, id);
-      })
-      .then(function () {
-        throw new Error('Should not have reached here!');
-      })
-      .catch(function (err) {
-        assert.equal(err.message, 'Not Found!');
-      });
+  it('should create a user in a sql db', function* () {
+    var createUser = yield adapter.create(User, {name: 'John'});
+    var id = createUser.id;
+    assert.equal(createUser.name, 'John');
+    assert.isDefined(createUser.id);
+
+    var findUser = yield adapter.find(User, createUser.id);
+    assert.equal(findUser.name, 'John');
+    assert.isDefined(findUser.id);
+    assert.equalObjects(findUser, {id: id, name: 'John', age: null, profileId: null});
+
+    var destoryUser = yield adapter.destroy(User, findUser.id);
+    assert.isFalse(!!destoryUser);
+
+    try {
+      var findUser2 = yield adapter.find(User, id);
+      throw new Error('Should not have reached here!');
+    } catch(err) {
+      assert.equal(err.message, 'Not Found!');
+    }
   });
 });

test/destroy.spec.js

@@ -1,20 +1,16 @@
 describe('DSSqlAdapter#destroy', function () {
-  it('should destroy a user from a Sql db', function () {
-    var id;
-    return adapter.create(User, {name: 'John'})
-      .then(function (user) {
-        id = user.id;
-        return adapter.destroy(User, user.id);
-      })
-      .then(function (user) {
-        assert.isFalse(!!user);
-        return adapter.find(User, id);
-      })
-      .then(function () {
-        throw new Error('Should not have reached here!');
-      })
-      .catch(function (err) {
-        assert.equal(err.message, 'Not Found!');
-      });
+  it('should destroy a user from a Sql db', function* () {
+    var createUser = yield adapter.create(User, {name: 'John'})
+    var id = createUser.id;
+
+    var destroyUser = yield adapter.destroy(User, createUser.id);
+    assert.isFalse(!!destroyUser);
+
+    try {
+      var findUser = yield adapter.find(User, id);
+      throw new Error('Should not have reached here!');
+    } catch (err) {
+      assert.equal(err.message, 'Not Found!');
+    }
   });
 });

test/destroyAll.spec.js

@@ -1,24 +1,14 @@
 describe('DSSqlAdapter#destroyAll', function () {
-  it('should destroy all items', function () {
-    var id;
-    return adapter.create(User, {name: 'John'})
-      .then(function (user) {
-        id = user.id;
-        return adapter.findAll(User, {
-          name: 'John'
-        });
-      }).then(function (users) {
-        assert.equal(users.length, 1);
-        assert.equalObjects(users[0], {id: id, name: 'John', age: null, profileId: null});
-        return adapter.destroyAll(User, {
-          name: 'John'
-        });
-      }).then(function () {
-        return adapter.findAll(User, {
-          name: 'John'
-        });
-      }).then(function (users) {
-        assert.equal(users.length, 0);
-      });
+  it('should destroy all items', function* () {
+    var createUser = yield adapter.create(User, {name: 'John'});
+    var id = createUser.id;
+
+    var findUsers = yield adapter.findAll(User, { name: 'John' });
+    assert.equal(findUsers.length, 1);
+    assert.equalObjects(findUsers[0], {id: id, name: 'John', age: null, profileId: null});
+
+    var destroyUser = yield adapter.destroyAll(User, { name: 'John' });
+    var findUsers2 = yield adapter.findAll(User, { name: 'John' });
+    assert.equal(findUsers2.length, 0);
   });
 });

test/find.spec.js

@@ -1,74 +1,57 @@
 var Promise = require('bluebird');
 describe('DSSqlAdapter#find', function () {
-  it('should find a user in a Sql db', function () {
-    var id, id2, _user, _post, _comments;
-    return adapter.create(User, {name: 'John'})
-      .then(function (user) {
-        _user = user;
-        id = user.id;
-        assert.equal(user.name, 'John');
-        assert.isDefined(user.id);
-        return adapter.find(User, user.id);
-      })
-      .then(function (user) {
-        assert.equal(user.name, 'John');
-        assert.isDefined(user.id);
-        assert.equalObjects(user, {id: id, name: 'John', age: null, profileId: null});
-        return adapter.create(Post, {
-          content: 'test',
-          userId: user.id
-        });
-      })
-      .then(function (post) {
-        _post = post;
-        id2 = post.id;
-        assert.equal(post.content, 'test');
-        assert.isDefined(post.id);
-        assert.isDefined(post.userId);
-        return Promise.all([
-          adapter.create(Comment, {
-            content: 'test2',
-            postId: post.id,
-            userId: _user.id
-          }),
-          adapter.create(Comment, {
-            content: 'test3',
-            postId: post.id,
-            userId: _user.id
-          })
-        ]);
-      })
-      .then(function (comments) {
-        _comments = comments;
-        _comments.sort(function (a, b) {
-          return a.content > b.content;
-        });
-        return adapter.find(Post, _post.id, {with: ['user', 'comment']});
-      })
-      .then(function (post) {
-        post.comments.sort(function (a, b) {
-          return a.content > b.content;
-        });
-        assert.equalObjects(post.user, _user);
-        assert.equalObjects(post.comments, _comments);
-        return adapter.destroyAll(Comment);
-      })
-      .then(function () {
-        return adapter.destroy(Post, id2);
-      })
-      .then(function () {
-        return adapter.destroy(User, id);
-      })
-      .then(function (user) {
-        assert.isFalse(!!user);
-        return adapter.find(User, id);
-      })
-      .then(function () {
-        throw new Error('Should not have reached here!');
-      })
-      .catch(function (err) {
-        console.log(err.stack);
-        assert.equal(err.message, 'Not Found!');
-      });
+  it('should find a user in a Sql db', function* () {
+    var user = yield adapter.create(User, {name: 'John'});
+    var userId = user.id;
+    assert.equal(user.name, 'John');
+    assert.isDefined(user.id);
+
+    var user2 = yield adapter.find(User, user.id);
+    assert.equal(user2.name, 'John');
+    assert.isDefined(user2.id);
+    assert.equalObjects(user2, {id: userId, name: 'John', age: null, profileId: null});
+
+    var post = yield adapter.create(Post, { content: 'test', userId: userId });
+    var postId = post.id;
+    assert.equal(post.content, 'test');
+    assert.isDefined(post.id);
+    assert.isDefined(post.userId);
+
+    var comments = yield [
+      adapter.create(Comment, {
+        content: 'test2',
+        postId: post.id,
+        userId: user.id
+      }),
+      adapter.create(Comment, {
+        content: 'test3',
+        postId: post.id,
+        userId: user.id
+      })
+    ];
+
+    comments.sort(function (a, b) {
+      return a.content > b.content;
+    });
+
+    var findPost = yield adapter.find(Post, postId, {with: ['user', 'comment']});
+    findPost.comments.sort(function (a, b) {
+      return a.content > b.content;
+    });
+    assert.equalObjects(findPost.user, user);
+    assert.equalObjects(findPost.comments, comments);
+
+    yield adapter.destroyAll(Comment);
+    yield adapter.destroy(Post, postId);
+    var destroyUser = yield adapter.destroy(User, userId);
+    assert.isFalse(!!destroyUser);
+
+    try {
+      yield adapter.find(User, userId);
+      throw new Error('Should not have reached here!');
+    } catch (err) {
+      console.log(err.stack);
+      assert.equal(err.message, 'Not Found!');
+    }
   });
 });