From 69ce75c662756f30456c4ca8d216a9517730462c Mon Sep 17 00:00:00 2001
From: Amir Jafarian <amirali.jfr61@gmail.com>
Date: Thu, 27 Oct 2016 17:59:11 -0400
Subject: [PATCH] Apply style guide to test names

*  Apply style guide to test names in test/loopback-dl.test.js
Backport of #1150
---
 test/loopback-dl.test.js | 409 ++++++++++++++++++++-------------------
 1 file changed, 215 insertions(+), 194 deletions(-)

diff --git a/test/loopback-dl.test.js b/test/loopback-dl.test.js
index d770697f..fc0e5a7c 100644
--- a/test/loopback-dl.test.js
+++ b/test/loopback-dl.test.js
@@ -13,8 +13,8 @@ var jdb = require('../');
 var ModelBuilder = jdb.ModelBuilder;
 var DataSource = jdb.DataSource;
 
-describe('ModelBuilder define model', function() {
-  it('should be able to define plain models', function(done) {
+describe('ModelBuilder', function() {
+  it('supports plain models', function(done) {
     var modelBuilder = new ModelBuilder();
 
     var User = modelBuilder.define('User', {
@@ -44,7 +44,7 @@ describe('ModelBuilder define model', function() {
     done(null, User);
   });
 
-  it('should not take unknown properties in strict mode', function(done) {
+  it('ignores unknown properties in strict mode', function(done) {
     var modelBuilder = new ModelBuilder();
 
     var User = modelBuilder.define('User', {name: String, bio: String}, {strict: true});
@@ -61,7 +61,7 @@ describe('ModelBuilder define model', function() {
     done(null, User);
   });
 
-  it('should ignore non-predefined properties in strict mode', function(done) {
+  it('ignores non-predefined properties in strict mode', function(done) {
     var modelBuilder = new ModelBuilder();
 
     var User = modelBuilder.define('User', {name: String, bio: String}, {strict: true});
@@ -85,7 +85,7 @@ describe('ModelBuilder define model', function() {
     done(null, User);
   });
 
-  it('should throw when unknown properties are used if strict=throw', function(done) {
+  it('throws an error when unknown properties are used if strict=throw', function(done) {
     var modelBuilder = new ModelBuilder();
 
     var User = modelBuilder.define('User', {name: String, bio: String}, {strict: 'throw'});
@@ -99,7 +99,7 @@ describe('ModelBuilder define model', function() {
     done(null, User);
   });
 
-  it('should be able to define open models', function(done) {
+  it('supports open models', function(done) {
     var modelBuilder = new ModelBuilder();
 
     var User = modelBuilder.define('User', {}, {strict: false});
@@ -114,7 +114,7 @@ describe('ModelBuilder define model', function() {
     done(null, User);
   });
 
-  it('should take non-predefined properties in non-strict mode', function(done) {
+  it('accepts non-predefined properties in non-strict mode', function(done) {
     var modelBuilder = new ModelBuilder();
 
     var User = modelBuilder.define('User', {name: String, bio: String}, {strict: false});
@@ -139,7 +139,7 @@ describe('ModelBuilder define model', function() {
     done(null, User);
   });
 
-  it('should use false as the default value for strict', function(done) {
+  it('uses non-strict mode by default', function(done) {
     var modelBuilder = new ModelBuilder();
 
     var User = modelBuilder.define('User', {});
@@ -154,7 +154,7 @@ describe('ModelBuilder define model', function() {
     done(null, User);
   });
 
-  it('should be able to define nesting models', function(done) {
+  it('supports nested model definitions', function(done) {
     var modelBuilder = new ModelBuilder();
 
     // simplier way to describe model
@@ -216,7 +216,7 @@ describe('ModelBuilder define model', function() {
     done(null, User);
   });
 
-  it('should be able to reference models by name before they are defined', function(done) {
+  it('allows models to be referenced by name before they are defined', function(done) {
     var modelBuilder = new ModelBuilder();
 
     var User = modelBuilder.define('User', {name: String, address: 'Address'});
@@ -248,7 +248,7 @@ describe('ModelBuilder define model', function() {
     done(null, User);
   });
 
-  it('should define an id property for composite ids', function() {
+  it('defines an id property for composite ids', function() {
     var modelBuilder = new ModelBuilder();
     var Follow = modelBuilder.define('Follow', {
       followerId: {type: String, id: 1},
@@ -273,7 +273,7 @@ describe('DataSource ping', function() {
     cb(new Error('bad connection 2'));
   };
 
-  it('should report connection errors during ping', function(done) {
+  it('reports connection errors during ping', function(done) {
     ds.ping(function(err) {
       (!!err).should.be.true;
       err.message.should.be.eql('bad connection 2');
@@ -281,7 +281,7 @@ describe('DataSource ping', function() {
     });
   });
 
-  it('should cancel invocation after timeout', function(done) {
+  it('cancels invocation after timeout', function(done) {
     ds.connected = false; // Force connect
     var Post = ds.define('Post', {
       title: {type: String, length: 255},
@@ -295,7 +295,7 @@ describe('DataSource ping', function() {
 });
 
 describe('DataSource define model', function() {
-  it('should be able to define plain models', function() {
+  it('supports plain model definitions', function() {
     var ds = new DataSource('memory');
 
 // define models
@@ -399,7 +399,7 @@ describe('DataSource define model', function() {
     });
   });
 
-  it('should emit events during attach', function() {
+  it('emits events during attach', function() {
     var ds = new DataSource('memory');
     var modelBuilder = new ModelBuilder();
 
@@ -427,7 +427,7 @@ describe('DataSource define model', function() {
     assert.equal(dataSourceAttached, 1);
   });
 
-  it('should not take unknown properties in strict mode', function(done) {
+  it('ignores unknown properties in strict mode', function(done) {
     var ds = new DataSource('memory');
 
     var User = ds.define('User', {name: String, bio: String}, {strict: true});
@@ -444,7 +444,7 @@ describe('DataSource define model', function() {
     });
   });
 
-  it('should throw when unknown properties are used if strict=throw', function(done) {
+  it('throws an error when unknown properties are used if strict=throw', function(done) {
     var ds = new DataSource('memory');
 
     var User = ds.define('User', {name: String, bio: String}, {strict: 'throw'});
@@ -459,7 +459,7 @@ describe('DataSource define model', function() {
   });
 
   describe('strict mode "validate"', function() {
-    it('should report validation error for unknown properties', function() {
+    it('reports validation errors for unknown properties', function() {
       var ds = new DataSource('memory');
       var User = ds.define('User', {name: String}, {strict: 'validate'});
       var user = new User({name: 'Joe', age: 20});
@@ -469,7 +469,7 @@ describe('DataSource define model', function() {
     });
   });
 
-  it('should be able to define open models', function(done) {
+  it('supports open model definitions', function(done) {
     var ds = new DataSource('memory');
 
     var User = ds.define('User', {}, {strict: false});
@@ -491,7 +491,7 @@ describe('DataSource define model', function() {
     });
   });
 
-  it('should use false as the default value for strict', function(done) {
+  it('uses non-strict mode by default', function(done) {
     var ds = new DataSource('memory');
 
     var User = ds.define('User', {});
@@ -506,7 +506,7 @@ describe('DataSource define model', function() {
     });
   });
 
-  it('should use true as the default value for strict for relational DBs', function(done) {
+  it('uses strict mode by default for relational DBs', function(done) {
     var ds = new DataSource('memory');
     ds.connector.relational = true; // HACK
 
@@ -524,7 +524,7 @@ describe('DataSource define model', function() {
     done(null, User);
   });
 
-  it('should throw when unknown properties are used if strict=false for relational DBs', function(done) {
+  it('throws an error with unknown properties in non-strict mode for relational DBs', function(done) {
     var ds = new DataSource('memory');
     ds.connector.relational = true; // HACK
 
@@ -539,7 +539,7 @@ describe('DataSource define model', function() {
     done(null, User);
   });
 
-  it('should change the property value for save if strict=false', function(done) {
+  it('changes the property value for save in non-strict mode', function(done) {
     var ds = new DataSource('memory');// define models
     var Post = ds.define('Post');
 
@@ -573,7 +573,7 @@ describe('DataSource define model', function() {
     user.toObject(false).should.have.property('age', 20);
   });
 
-  it('should update the instance with unknown properties', function(done) {
+  it('updates instances with unknown properties in non-strict mode', function(done) {
     var ds = new DataSource('memory');// define models
     var Post = ds.define('Post', {
       title: {type: String, length: 255, index: true},
@@ -632,7 +632,7 @@ describe('DataSource define model', function() {
     done();
   });
 
-  it('should allow an explicit remoting path', function() {
+  it('allows an explicit remoting path', function() {
     var ds = new DataSource('memory');
 
     var User = ds.define('User', {name: String, bio: String}, {
@@ -641,7 +641,7 @@ describe('DataSource define model', function() {
     User.http.path.should.equal('/accounts');
   });
 
-  it('should allow an explicit remoting path with leading /', function() {
+  it('allows an explicit remoting path with leading /', function() {
     var ds = new DataSource('memory');
 
     var User = ds.define('User', {name: String, bio: String}, {
@@ -651,8 +651,8 @@ describe('DataSource define model', function() {
   });
 });
 
-describe('Load models with base', function() {
-  it('should set up base class via base option', function() {
+describe('Model loaded with a base', function() {
+  it('has a base class according to the base option', function() {
     var ds = new ModelBuilder();
 
     var User = ds.define('User', {name: String});
@@ -677,7 +677,7 @@ describe('Load models with base', function() {
     }
   });
 
-  it('should inherit properties from base option', function() {
+  it('inherits properties from base model', function() {
     var ds = new ModelBuilder();
 
     var User = ds.define('User', {name: String});
@@ -688,7 +688,7 @@ describe('Load models with base', function() {
     Customer.definition.properties.name.should.have.property('type', String);
   });
 
-  it('should inherit properties by clone from base option', function() {
+  it('inherits properties by clone from base model', function() {
     var ds = new ModelBuilder();
 
     var User = ds.define('User', {name: String});
@@ -704,7 +704,7 @@ describe('Load models with base', function() {
       Customer2.definition.properties.name);
   });
 
-  it('should revert properties from base model', function() {
+  it('can remove properties from base model', function() {
     var ds = new ModelBuilder();
 
     var User = ds.define('User', {username: String, email: String});
@@ -727,7 +727,7 @@ describe('Load models with base', function() {
     u.should.have.property('email', 'x@y.com');
   });
 
-  it('should set up base class via parent arg', function() {
+  it('can configure base class via parent argument', function() {
     var ds = new ModelBuilder();
 
     var User = ds.define('User', {name: String});
@@ -765,107 +765,126 @@ describe('Models attached to a dataSource', function() {
     Post.destroyAll(done);
   });
 
-  it('updateOrCreate should update the instance', function(done) {
-    Post.create({title: 'a', content: 'AAA'}, function(err, post) {
-      post.title = 'b';
-      Post.updateOrCreate(post, function(err, p) {
-        should.not.exist(err);
-        p.id.should.be.equal(post.id);
-        p.content.should.be.equal(post.content);
-        should.not.exist(p._id);
-
-        Post.findById(post.id, function(err, p) {
+  describe('updateOrCreate', function() {
+    it('updates instances', function(done) {
+      Post.create({title: 'a', content: 'AAA'}, function(err, post) {
+        post.title = 'b';
+        Post.updateOrCreate(post, function(err, p) {
+          should.not.exist(err);
           p.id.should.be.equal(post.id);
-          should.not.exist(p._id);
           p.content.should.be.equal(post.content);
-          p.title.should.be.equal('b');
+          should.not.exist(p._id);
 
-          done();
+          Post.findById(post.id, function(err, p) {
+            p.id.should.be.equal(post.id);
+            should.not.exist(p._id);
+            p.content.should.be.equal(post.content);
+            p.title.should.be.equal('b');
+            done();
+          });
         });
       });
     });
-  });
 
-  it('updateOrCreate should update the instance without removing existing properties', function(done) {
-    Post.create({title: 'a', content: 'AAA', comments: ['Comment1']}, function(err, post) {
-      post = post.toObject();
-      delete post.title;
-      delete post.comments;
-      Post.updateOrCreate(post, function(err, p) {
-        should.not.exist(err);
-        p.id.should.be.equal(post.id);
-        p.content.should.be.equal(post.content);
-        should.not.exist(p._id);
-
-        Post.findById(post.id, function(err, p) {
+    it('updates instances without removing existing properties', function(done) {
+      Post.create({title: 'a', content: 'AAA', comments: ['Comment1']}, function(err, post) {
+        post = post.toObject();
+        delete post.title;
+        delete post.comments;
+        Post.updateOrCreate(post, function(err, p) {
+          should.not.exist(err);
           p.id.should.be.equal(post.id);
-          should.not.exist(p._id);
           p.content.should.be.equal(post.content);
-          p.title.should.be.equal('a');
-          p.comments.length.should.be.equal(1);
-          p.comments[0].should.be.equal('Comment1');
-          done();
+          should.not.exist(p._id);
+
+          Post.findById(post.id, function(err, p) {
+            p.id.should.be.equal(post.id);
+            should.not.exist(p._id);
+            p.content.should.be.equal(post.content);
+            p.title.should.be.equal('a');
+            p.comments.length.should.be.equal(1);
+            p.comments[0].should.be.equal('Comment1');
+            done();
+          });
         });
       });
     });
-  });
 
-  it('updateOrCreate should create a new instance if it does not exist', function(done) {
-    var post = {id: 123, title: 'a', content: 'AAA'};
-    Post.updateOrCreate(post, function(err, p) {
-      should.not.exist(err);
-      p.title.should.be.equal(post.title);
-      p.content.should.be.equal(post.content);
-      p.id.should.be.equal(post.id);
-
-      Post.findById(p.id, function(err, p) {
-        p.id.should.be.equal(post.id);
-        should.not.exist(p._id);
-        p.content.should.be.equal(post.content);
+    it('creates a new instance if it does not exist', function(done) {
+      var post = {id: 123, title: 'a', content: 'AAA'};
+      Post.updateOrCreate(post, function(err, p) {
+        should.not.exist(err);
         p.title.should.be.equal(post.title);
-        p.id.should.be.equal(post.id);
-
-        done();
-      });
-    });
-  });
-
-  it('save should update the instance with the same id', function(done) {
-    Post.create({title: 'a', content: 'AAA'}, function(err, post) {
-      post.title = 'b';
-      post.save(function(err, p) {
-        should.not.exist(err);
-        p.id.should.be.equal(post.id);
         p.content.should.be.equal(post.content);
-        should.not.exist(p._id);
+        p.id.should.be.equal(post.id);
 
-        Post.findById(post.id, function(err, p) {
+        Post.findById(p.id, function(err, p) {
           p.id.should.be.equal(post.id);
           should.not.exist(p._id);
           p.content.should.be.equal(post.content);
-          p.title.should.be.equal('b');
-
+          p.title.should.be.equal(post.title);
+          p.id.should.be.equal(post.id);
           done();
         });
       });
     });
   });
 
-  it('save should update the instance without removing existing properties', function(done) {
-    Post.create({title: 'a', content: 'AAA'}, function(err, post) {
-      delete post.title;
-      post.save(function(err, p) {
-        should.not.exist(err);
-        p.id.should.be.equal(post.id);
-        p.content.should.be.equal(post.content);
-        should.not.exist(p._id);
+  describe('save', function() {
+    it('updates instance with the same id', function(done) {
+      Post.create({title: 'a', content: 'AAA'}, function(err, post) {
+        post.title = 'b';
+        post.save(function(err, p) {
+          should.not.exist(err);
+          p.id.should.be.equal(post.id);
+          p.content.should.be.equal(post.content);
+          should.not.exist(p._id);
 
-        Post.findById(post.id, function(err, p) {
+          Post.findById(post.id, function(err, p) {
+            p.id.should.be.equal(post.id);
+            should.not.exist(p._id);
+            p.content.should.be.equal(post.content);
+            p.title.should.be.equal('b');
+            done();
+          });
+        });
+      });
+    });
+
+    it('updates the instance without removing existing properties', function(done) {
+      Post.create({title: 'a', content: 'AAA'}, function(err, post) {
+        delete post.title;
+        post.save(function(err, p) {
+          should.not.exist(err);
+          p.id.should.be.equal(post.id);
+          p.content.should.be.equal(post.content);
+          should.not.exist(p._id);
+
+          Post.findById(post.id, function(err, p) {
+            p.id.should.be.equal(post.id);
+            should.not.exist(p._id);
+            p.content.should.be.equal(post.content);
+            p.title.should.be.equal('a');
+            done();
+          });
+        });
+      });
+    });
+
+    it('creates a new instance if it does not exist', function(done) {
+      var post = new Post({id: '123', title: 'a', content: 'AAA'});
+      post.save(post, function(err, p) {
+        should.not.exist(err);
+        p.title.should.be.equal(post.title);
+        p.content.should.be.equal(post.content);
+        p.id.should.be.equal(post.id);
+
+        Post.findById(p.id, function(err, p) {
           p.id.should.be.equal(post.id);
           should.not.exist(p._id);
           p.content.should.be.equal(post.content);
-          p.title.should.be.equal('a');
-
+          p.title.should.be.equal(post.title);
+          p.id.should.be.equal(post.id);
           done();
         });
       });
@@ -894,37 +913,39 @@ describe('Models attached to a dataSource', function() {
 });
 
 describe('DataSource connector types', function() {
-  it('should return an array of types', function() {
+  it('returns an array of types using getTypes', function() {
     var ds = new DataSource('memory');
     var types = ds.getTypes();
     assert.deepEqual(types, ['db', 'nosql', 'memory']);
   });
 
-  it('should test supported types by string', function() {
-    var ds = new DataSource('memory');
-    var result = ds.supportTypes('db');
-    assert(result);
-  });
+  describe('supportTypes', function() {
+    it('tests supported types by string', function() {
+      var ds = new DataSource('memory');
+      var result = ds.supportTypes('db');
+      assert(result);
+    });
 
-  it('should test supported types by array', function() {
-    var ds = new DataSource('memory');
-    var result = ds.supportTypes(['db', 'memory']);
-    assert(result);
-  });
+    it('tests supported types by array', function() {
+      var ds = new DataSource('memory');
+      var result = ds.supportTypes(['db', 'memory']);
+      assert(result);
+    });
 
-  it('should test unsupported types by string', function() {
-    var ds = new DataSource('memory');
-    var result = ds.supportTypes('rdbms');
-    assert(!result);
-  });
+    it('tests unsupported types by string', function() {
+      var ds = new DataSource('memory');
+      var result = ds.supportTypes('rdbms');
+      assert(!result);
+    });
 
-  it('should test unsupported types by array', function() {
-    var ds = new DataSource('memory');
-    var result = ds.supportTypes(['rdbms', 'memory']);
-    assert(!result);
+    it('tests unsupported types by array', function() {
+      var ds = new DataSource('memory');
+      var result = ds.supportTypes(['rdbms', 'memory']);
+      assert(!result);
 
-    result = ds.supportTypes(['rdbms']);
-    assert(!result);
+      result = ds.supportTypes(['rdbms']);
+      assert(!result);
+    });
   });
 });
 
@@ -944,56 +965,56 @@ describe('DataSource constructor', function() {
     };
   };
 
-  it('should resolve connector by path', function() {
+  it('resolves connector by path', function() {
     var connector = DataSource._resolveConnector(__dirname + '/../lib/connectors/memory');
     assert(connector.connector);
   });
-  it('should resolve connector by internal name', function() {
+  it('resolves connector by internal name', function() {
     var connector = DataSource._resolveConnector('memory');
     assert(connector.connector);
   });
-  it('should try to resolve connector by module name starts with loopback-connector-', function() {
+  it('resolves connector by module name starting with loopback-connector-', function() {
     var connector = DataSource._resolveConnector('loopback-connector-xyz', loader);
     assert(connector.connector);
   });
-  it('should try to resolve connector by short module name with full name first', function() {
+  it('resolves connector by short module name with full name first', function() {
     var connector = DataSource._resolveConnector('xyz', loader);
     assert(connector.connector);
     assert.equal(connector.connector.name, 'loopback-connector-xyz');
   });
-  it('should try to resolve connector by short module name', function() {
+  it('resolves connector by short module name', function() {
     var connector = DataSource._resolveConnector('abc', loader);
     assert(connector.connector);
     assert.equal(connector.connector.name, 'abc');
   });
-  it('should try to resolve connector by short module name for known connectors', function() {
+  it('resolves connector by short module name for known connectors', function() {
     var connector = DataSource._resolveConnector('oracle', loader);
     assert(connector.connector);
     assert.equal(connector.connector.name, 'loopback-connector-oracle');
   });
-  it('should try to resolve connector by full module name', function() {
+  it('resolves connector by full module name', function() {
     var connector = DataSource._resolveConnector('loopback-xyz', loader);
     assert(connector.connector);
   });
-  it('should fail to resolve connector by module name starts with loopback-connector-', function() {
+  it('fails to resolve connector by module name starting with loopback-connector-', function() {
     var connector = DataSource._resolveConnector('loopback-connector-xyz');
     assert(!connector.connector);
     assert(connector.error.indexOf('loopback-connector-xyz') !== -1);
   });
-  it('should fail to resolve connector by short module name', function() {
+  it('fails resolve invalid connector by short module name', function() {
     var connector = DataSource._resolveConnector('xyz');
     assert(!connector.connector);
     assert(connector.error.indexOf('loopback-connector-xyz') !== -1);
   });
-  it('should fail to resolve connector by full module name', function() {
+  it('fails to resolve invalid connector by full module name', function() {
     var connector = DataSource._resolveConnector('loopback-xyz');
     assert(!connector.connector);
     assert(connector.error.indexOf('loopback-connector-loopback-xyz') !== -1);
   });
 });
 
-describe('Load models with relations', function() {
-  it('should set up relations', function(done) {
+describe('Model define with relations configuration', function() {
+  it('sets up hasMany relations', function(done) {
     var ds = new DataSource('memory');
 
     var Post = ds.define('Post', {userId: Number, content: String});
@@ -1007,7 +1028,7 @@ describe('Load models with relations', function() {
     done();
   });
 
-  it('should set up belongsTo relations', function(done) {
+  it('sets up belongsTo relations', function(done) {
     var ds = new DataSource('memory');
 
     var User = ds.define('User', {name: String});
@@ -1021,7 +1042,7 @@ describe('Load models with relations', function() {
     done();
   });
 
-  it('should set up referencesMany relations', function(done) {
+  it('sets up referencesMany relations', function(done) {
     var ds = new DataSource('memory');
 
     var Post = ds.define('Post', {userId: Number, content: String});
@@ -1035,7 +1056,7 @@ describe('Load models with relations', function() {
     done();
   });
 
-  it('should set up embedsMany relations', function(done) {
+  it('sets up embedsMany relations', function(done) {
     var ds = new DataSource('memory');
 
     var Post = ds.define('Post', {userId: Number, content: String});
@@ -1049,7 +1070,7 @@ describe('Load models with relations', function() {
     done();
   });
 
-  it('should set up polymorphic relations', function(done) {
+  it('sets up polymorphic relations', function(done) {
     var ds = new DataSource('memory');
 
     var Author = ds.define('Author', {name: String}, {relations: {
@@ -1093,7 +1114,7 @@ describe('Load models with relations', function() {
     done();
   });
 
-  it('should set up foreign key with the correct type', function(done) {
+  it('creates a foreign key with the correct type', function(done) {
     var ds = new DataSource('memory');
 
     var User = ds.define(
@@ -1113,7 +1134,7 @@ describe('Load models with relations', function() {
     done();
   });
 
-  it('should set up hasMany and belongsTo relations', function(done) {
+  it('sets up related hasMany and belongsTo relations', function(done) {
     var ds = new DataSource('memory');
 
     var User = ds.define(
@@ -1182,7 +1203,7 @@ describe('Load models with relations', function() {
     done();
   });
 
-  it('should throw if a relation is missing type', function(done) {
+  it('throws an error if a relation is missing type', function(done) {
     var ds = new DataSource('memory');
 
     var Post = ds.define('Post', {userId: Number, content: String});
@@ -1198,7 +1219,7 @@ describe('Load models with relations', function() {
     }
   });
 
-  it('should throw if the relation type is invalid', function(done) {
+  it('throws an error if a relation type is invalid', function(done) {
     var ds = new DataSource('memory');
 
     var Post = ds.define('Post', {userId: Number, content: String});
@@ -1214,7 +1235,7 @@ describe('Load models with relations', function() {
     }
   });
 
-  it('should handle hasMany through', function(done) {
+  it('sets up hasMany through relations', function(done) {
     var ds = new DataSource('memory');
     var Physician = ds.createModel('Physician', {
       name: String,
@@ -1248,7 +1269,7 @@ describe('Load models with relations', function() {
     done();
   });
 
-  it('should handle hasMany through options', function(done) {
+  it('sets up hasMany through relations with options', function(done) {
     var ds = new DataSource('memory');
     var Physician = ds.createModel('Physician',
       {name: String},
@@ -1297,7 +1318,7 @@ describe('Load models with relations', function() {
     done();
   });
 
-  it('should set up relations after attach', function(done) {
+  it('sets up relations after attach', function(done) {
     var ds = new DataSource('memory');
     var modelBuilder = new ModelBuilder();
 
@@ -1316,8 +1337,8 @@ describe('Load models with relations', function() {
   });
 });
 
-describe('Model with scopes', function() {
-  it('should create scopes', function(done) {
+describe('Model define with scopes configuration', function() {
+  it('creates scopes', function(done) {
     var ds = new DataSource('memory');
     var User = ds.define('User', {name: String, vip: Boolean, age: Number},
       {scopes: {vips: {where: {vip: true}}, top5: {limit: 5, order: 'age'}}});
@@ -1362,7 +1383,7 @@ describe('DataAccessObject', function() {
     error = null;
   });
 
-  it('should be able to coerce where clause for string types', function() {
+  it('coerces where clause for string types', function() {
     where = model._coerce({id: 1});
     assert.deepEqual(where, {id: '1'});
     where = model._coerce({id: '1'});
@@ -1381,7 +1402,7 @@ describe('DataAccessObject', function() {
     assert.deepEqual(where, {id: '1'});
   });
 
-  it('should be able to coerce where clause for number types', function() {
+  it('coerces where clause for number types', function() {
     where = model._coerce({age: '10'});
     assert.deepEqual(where, {age: 10});
 
@@ -1398,12 +1419,12 @@ describe('DataAccessObject', function() {
     assert.deepEqual(where, {age: {between: [10, 20]}});
   });
 
-  it('should be able to coerce where clause for array types', function() {
+  it('coerces where clause for array types', function() {
     where = model._coerce({scores: ['10', '20']});
     assert.deepEqual(where, {scores: [10, 20]});
   });
 
-  it('should be able to coerce where clause for date types', function() {
+  it('coerces where clause for date types', function() {
     var d = new Date();
     where = model._coerce({date: d});
     assert.deepEqual(where, {date: d});
@@ -1412,7 +1433,7 @@ describe('DataAccessObject', function() {
     assert.deepEqual(where, {date: d});
   });
 
-  it('should be able to coerce where clause for boolean types', function() {
+  it('coerces where clause for boolean types', function() {
     where = model._coerce({vip: 'true'});
     assert.deepEqual(where, {vip: true});
 
@@ -1435,12 +1456,12 @@ describe('DataAccessObject', function() {
     assert.deepEqual(where, {vip: false});
   });
 
-  it('should be able to coerce where clause with and operators', function() {
+  it('coerces where clause with and operators', function() {
     where = model._coerce({and: [{age: '10'}, {vip: 'true'}]});
     assert.deepEqual(where, {and: [{age: 10}, {vip: true}]});
   });
 
-  it('should be able to coerce where clause with or operators', function() {
+  it('coerces where clause with or operators', function() {
     where = model._coerce({or: [{age: '10'}, {vip: 'true'}]});
     assert.deepEqual(where, {or: [{age: 10}, {vip: true}]});
   });
@@ -1455,7 +1476,7 @@ describe('DataAccessObject', function() {
     assert.deepEqual(where, {and: [{age: 10}], vip: true});
   });
 
-  it('should throw if the where property is not an object', function() {
+  it('throws an error if the where property is not an object', function() {
     try {
       // The where clause has to be an object
       model._coerce('abc');
@@ -1465,7 +1486,7 @@ describe('DataAccessObject', function() {
     assert(error, 'An error should have been thrown');
   });
 
-  it('should throw if the where property is an array', function() {
+  it('throws an error if the where property is an array', function() {
     try {
       // The where clause cannot be an array
       model._coerce([
@@ -1477,7 +1498,7 @@ describe('DataAccessObject', function() {
     assert(error, 'An error should have been thrown');
   });
 
-  it('should throw if the and operator does not take an array', function() {
+  it('throws an error if the and operator is not configured with an array', function() {
     try {
       // The and operator only takes an array of objects
       model._coerce({and: {x: 1}});
@@ -1487,7 +1508,7 @@ describe('DataAccessObject', function() {
     assert(error, 'An error should have been thrown');
   });
 
-  it('should throw if the or operator does not take an array', function() {
+  it('throws an error if the or operator does not take an array', function() {
     try {
       // The or operator only takes an array of objects
       model._coerce({or: {x: 1}});
@@ -1497,7 +1518,7 @@ describe('DataAccessObject', function() {
     assert(error, 'An error should have been thrown');
   });
 
-  it('should throw if the or operator does not take an array of objects', function() {
+  it('throws an error if the or operator not configured with an array of objects', function() {
     try {
       // The or operator only takes an array of objects
       model._coerce({or: ['x']});
@@ -1521,7 +1542,7 @@ describe('DataAccessObject', function() {
     assert(error, 'An error should have been thrown');
   });
 
-  it('should throw if filter property is not an object', function() {
+  it('throws an error if the filter property is not an object', function() {
     var filter = null;
     try {
       // The filter clause has to be an object
@@ -1532,7 +1553,7 @@ describe('DataAccessObject', function() {
     assert(error, 'An error should have been thrown');
   });
 
-  it('should throw if filter.limit property is not a number', function() {
+  it('throws an error if the filter.limit property is not a number', function() {
     try {
       // The limit param must be a valid number
       filter = model._normalize({limit: 'x'});
@@ -1542,7 +1563,7 @@ describe('DataAccessObject', function() {
     assert(error, 'An error should have been thrown');
   });
 
-  it('should throw if filter.limit property is nagative', function() {
+  it('throws an error if the filter.limit property is nagative', function() {
     try {
       // The limit param must be a valid number
       filter = model._normalize({limit: -1});
@@ -1552,7 +1573,7 @@ describe('DataAccessObject', function() {
     assert(error, 'An error should have been thrown');
   });
 
-  it('should throw if filter.limit property is not an integer', function() {
+  it('throws an error if the filter.limit property is not an integer', function() {
     try {
       // The limit param must be a valid number
       filter = model._normalize({limit: 5.8});
@@ -1562,7 +1583,7 @@ describe('DataAccessObject', function() {
     assert(error, 'An error should have been thrown');
   });
 
-  it('should throw if filter.offset property is not a number', function() {
+  it('throws an error if filter.offset property is not a number', function() {
     try {
       // The limit param must be a valid number
       filter = model._normalize({offset: 'x'});
@@ -1572,7 +1593,7 @@ describe('DataAccessObject', function() {
     assert(error, 'An error should have been thrown');
   });
 
-  it('should throw if filter.skip property is not a number', function() {
+  it('throws an error if the filter.skip property is not a number', function() {
     try {
       // The limit param must be a valid number
       filter = model._normalize({skip: '_'});
@@ -1582,17 +1603,17 @@ describe('DataAccessObject', function() {
     assert(error, 'An error should have been thrown');
   });
 
-  it('should normalize limit/offset/skip', function() {
+  it('normalizes limit/offset/skip', function() {
     var filter = model._normalize({limit: '10', skip: 5});
     assert.deepEqual(filter, {limit: 10, offset: 5, skip: 5});
   });
 
-  it('should set the default value for limit', function() {
+  it('uses a default value for limit', function() {
     var filter = model._normalize({skip: 5});
     assert.deepEqual(filter, {limit: 100, offset: 5, skip: 5});
   });
 
-  it('should apply settings for handling undefined', function() {
+  it('applies settings for handling undefined', function() {
     var filter = model._normalize({filter: {x: undefined}});
     assert.deepEqual(filter, {filter: {}});
 
@@ -1608,35 +1629,35 @@ describe('DataAccessObject', function() {
     (function() { model._normalize({filter: {x: undefined}}); }).should.throw(/`undefined` in query/);
   });
 
-  it('should skip GeoPoint', function() {
+  it('does not coerce GeoPoint', function() {
     where = model._coerce({location: {near: {lng: 10, lat: 20}, maxDistance: 20}});
     assert.deepEqual(where, {location: {near: {lng: 10, lat: 20}, maxDistance: 20}});
   });
 
-  it('should skip null values', function() {
+  it('does not coerce null values', function() {
     where = model._coerce({date: null});
     assert.deepEqual(where, {date: null});
   });
 
-  it('should skip undefined values', function() {
+  it('does not coerce undefined values', function() {
     where = model._coerce({date: undefined});
     assert.deepEqual(where, {date: undefined});
   });
 
-  it('should skip conversion if a simple property produces NaN for numbers',
+  it('does not coerce to a number for a simple value that produces NaN',
     function() {
       where = model._coerce({age: 'xyz'});
       assert.deepEqual(where, {age: 'xyz'});
     });
 
-  it('should skip conversion if an array property produces NaN for numbers',
+  it('does not coerce to a number for a simple value in an array that produces NaN',
     function() {
       where = model._coerce({age: {inq: ['xyz', '12']}});
       assert.deepEqual(where, {age: {inq: ['xyz', 12]}});
     });
 
   // settings
-  it('should get settings in priority',
+  it('gets settings in priority',
     function() {
       ds.settings.test = 'test';
       assert.equal(model._getSetting('test'), ds.settings.test, 'Should get datasource setting');
@@ -1650,7 +1671,7 @@ describe('DataAccessObject', function() {
     });
 });
 
-describe('Load models from json', function() {
+describe('ModelBuilder processing json files', function() {
   var path = require('path'),
     fs = require('fs');
 
@@ -1674,7 +1695,7 @@ describe('Load models from json', function() {
     return modelBuilder.buildModels(schemas, createModel);
   }
 
-  it('should be able to define models from json', function() {
+  it('defines models', function() {
     var models = loadSchemasSync(path.join(__dirname, 'test1-schemas.json'));
 
     models.should.have.property('AnonymousModel_0');
@@ -1694,7 +1715,7 @@ describe('Load models from json', function() {
     }
   });
 
-  it('should be able to define models from json using dataSource', function() {
+  it('attaches models to a specified dataSource', function() {
     var ds = new DataSource('memory');
 
     var models = loadSchemasSync(path.join(__dirname, 'test2-schemas.json'), ds);
@@ -1704,7 +1725,7 @@ describe('Load models from json', function() {
     assert.equal(models.Address.dataSource, ds);
   });
 
-  it('should allow customization of default model base class', function() {
+  it('allows customization of default model base class', function() {
     var modelBuilder = new ModelBuilder();
 
     var User = modelBuilder.define('User', {
@@ -1721,7 +1742,7 @@ describe('Load models from json', function() {
     assert(Customer.prototype instanceof User);
   });
 
-  it('should allow model base class', function() {
+  it('accepts a model base class', function() {
     var modelBuilder = new ModelBuilder();
 
     var User = modelBuilder.define('User', {
@@ -1737,7 +1758,7 @@ describe('Load models from json', function() {
     assert(Customer.prototype instanceof User);
   });
 
-  it('should be able to extend models', function(done) {
+  it('allows model extension', function(done) {
     var modelBuilder = new ModelBuilder();
 
     var User = modelBuilder.define('User', {
@@ -1781,7 +1802,7 @@ describe('Load models from json', function() {
     done(null, customer);
   });
 
-  it('should be able to extend models with merged settings', function(done) {
+  it('allows model extension with merged settings', function(done) {
     var modelBuilder = new ModelBuilder();
 
     var User = modelBuilder.define('User', {
@@ -1874,29 +1895,29 @@ describe('Load models from json', function() {
 });
 
 describe('DataSource constructor', function() {
-  it('Takes url as the settings', function() {
+  it('takes url as the settings', function() {
     var ds = new DataSource('memory://localhost/mydb?x=1');
     assert.equal(ds.connector.name, 'memory');
   });
 
-  it('Takes connector name', function() {
+  it('takes connector name', function() {
     var ds = new DataSource('memory');
     assert.equal(ds.connector.name, 'memory');
   });
 
-  it('Takes settings object', function() {
+  it('takes settings object', function() {
     var ds = new DataSource({connector: 'memory'});
     assert.equal(ds.connector.name, 'memory');
   });
 
-  it('Takes settings object and name', function() {
+  it('takes settings object and name', function() {
     var ds = new DataSource('x', {connector: 'memory'});
     assert.equal(ds.connector.name, 'memory');
   });
 });
 
 describe('ModelBuilder options.models', function() {
-  it('should inject model classes from models', function() {
+  it('injects model classes from models', function() {
     var builder = new ModelBuilder();
     var M1 = builder.define('M1');
     var M2 = builder.define('M2', {}, {models: {
@@ -1906,7 +1927,7 @@ describe('ModelBuilder options.models', function() {
     assert.equal(M2.M1, M1, 'M1 should be injected to M2');
   });
 
-  it('should inject model classes by name in the models', function() {
+  it('injects model classes by name in the models', function() {
     var builder = new ModelBuilder();
     var M1 = builder.define('M1');
     var M2 = builder.define('M2', {}, {models: {
@@ -1916,7 +1937,7 @@ describe('ModelBuilder options.models', function() {
     assert.equal(M2.M1, M1, 'M1 should be injected to M2');
   });
 
-  it('should inject model classes by name in the models before the class is defined',
+  it('injects model classes by name in the models before the class is defined',
     function() {
       var builder = new ModelBuilder();
       var M2 = builder.define('M2', {}, {models: {
@@ -1928,7 +1949,7 @@ describe('ModelBuilder options.models', function() {
       assert.equal(M2.M1, M1, 'M1 should be injected to M2');
     });
 
-  it('should use false strict mode for embedded models by default', function() {
+  it('uses non-strict mode for embedded models by default', function() {
     var builder = new ModelBuilder();
     var M1 = builder.define('testEmbedded', {
       name: 'string',
@@ -1946,7 +1967,7 @@ describe('ModelBuilder options.models', function() {
     assert.equal(m1.address.number, 5512, 'm1 should contain number property in address');
   });
 
-  it('should use the strictEmbeddedModels setting (true) when applied on modelBuilder', function() {
+  it('uses the strictEmbeddedModels setting (true) when applied on modelBuilder', function() {
     var builder = new ModelBuilder();
     builder.settings.strictEmbeddedModels = true;
     var M1 = builder.define('testEmbedded', {