Merge pull request #1157 from strongloop/1150_backport
Apply style guide to test names
This commit is contained in:
Amirali Jafarian
GitHub
commit
adcebab371
|
|
@ -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,7 +765,8 @@ describe('Models attached to a dataSource', function() {
|
|||
Post.destroyAll(done);
|
||||
});
|
||||
|
||||
it('updateOrCreate should update the instance', function(done) {
|
||||
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) {
|
||||
|
|
@ -779,14 +780,13 @@ describe('Models attached to a dataSource', function() {
|
|||
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) {
|
||||
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;
|
||||
|
|
@ -810,7 +810,7 @@ describe('Models attached to a dataSource', function() {
|
|||
});
|
||||
});
|
||||
|
||||
it('updateOrCreate should create a new instance if it does not exist', function(done) {
|
||||
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);
|
||||
|
|
@ -824,13 +824,14 @@ describe('Models attached to a dataSource', function() {
|
|||
p.content.should.be.equal(post.content);
|
||||
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) {
|
||||
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) {
|
||||
|
|
@ -844,14 +845,13 @@ describe('Models attached to a dataSource', function() {
|
|||
should.not.exist(p._id);
|
||||
p.content.should.be.equal(post.content);
|
||||
p.title.should.be.equal('b');
|
||||
|
||||
done();
|
||||
});
|
||||
});
|
||||
});
|
||||
});
|
||||
|
||||
it('save should update the instance without removing existing properties', function(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) {
|
||||
|
|
@ -865,7 +865,26 @@ describe('Models attached to a dataSource', function() {
|
|||
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(post.title);
|
||||
p.id.should.be.equal(post.id);
|
||||
done();
|
||||
});
|
||||
});
|
||||
|
|
@ -894,31 +913,32 @@ 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() {
|
||||
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() {
|
||||
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() {
|
||||
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() {
|
||||
it('tests unsupported types by array', function() {
|
||||
var ds = new DataSource('memory');
|
||||
var result = ds.supportTypes(['rdbms', 'memory']);
|
||||
assert(!result);
|
||||
|
|
@ -926,6 +946,7 @@ describe('DataSource connector types', function() {
|
|||
result = ds.supportTypes(['rdbms']);
|
||||
assert(!result);
|
||||
});
|
||||
});
|
||||
});
|
||||
|
||||
describe('DataSource constructor', function() {
|
||||
|
|
@ -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', {
|
||||
|
|
|
|||
Loading…
Reference in New Issue