verdnatura-mirror
/
loopback-datasource-juggler
mirror of https://github.com/loopbackio/loopback-datasource-juggler
0
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

Merge pull request #866 from strongloop/fix_Mongo_comptaibility_replaceORCreate 

Fix Mongo compatibility issue
This commit is contained in:
Amir-61 2016-03-30 11:25:22 -04:00
parent eed69a2573 5b6bffb5b3
commit 049e888039
2 changed files with 48 additions and 22 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
test/memory.test.js
View File

@ -853,7 +853,7 @@ describe('Optimized connector', function() {
}.bind(this));
};
require('./persistence-hooks.suite')(ds, should);
require('./persistence-hooks.suite')(ds, should, {replaceOrCreateReportsNewInstance: true});
});
describe('Unoptimized connector', function() {
@ -862,7 +862,7 @@ describe('Unoptimized connector', function() {
ds.connector.updateOrCreate = false;
ds.connector.findOrCreate = false;
require('./persistence-hooks.suite')(ds, should);
require('./persistence-hooks.suite')(ds, should, {replaceOrCreateReportsNewInstance: true});
});
describe('Memory connector with options', function() {

66
test/persistence-hooks.suite.js
View File

@ -1,7 +1,11 @@
var ValidationError = require('../').ValidationError;
var traverse = require('traverse');
module.exports = function(dataSource, should) {
module.exports = function(dataSource, should, connectorCapabilities) {
if (!connectorCapabilities) connectorCapabilities = {};
if (connectorCapabilities.replaceOrCreateReportsNewInstance === undefined) {
console.warn('The connector does not support a recently added feature: replaceOrCreateReportsNewInstance');
}
describe('Persistence hooks', function() {
var observedContexts, expectedError, observersCalled;
var TestModel, existingInstance;
@ -2403,13 +2407,19 @@ module.exports = function(dataSource, should) {
{ id: 'new-id', name: 'a name' },
function(err, instance) {
if (err) return done(err);
observedContexts.should.eql(aTestModelCtx({
var expected = {
data: {
id: 'new-id',
name: 'a name'
},
isNewInstance: true
}));
}
};
expected.isNewInstance =
connectorCapabilities.replaceOrCreateReportsNewInstance ?
true : undefined;
observedContexts.should.eql(aTestModelCtx(expected));
done();
});
});
@ -2422,14 +2432,19 @@ module.exports = function(dataSource, should) {
function(err, instance) {
if (err) return done(err);
var expected = {
data: {
id: existingInstance.id,
name: 'replaced name'
}
};
expected.isNewInstance =
connectorCapabilities.replaceOrCreateReportsNewInstance ?
false : undefined;
if (dataSource.connector.replaceOrCreate) {
observedContexts.should.eql(aTestModelCtx({
data: {
id: existingInstance.id,
name: 'replaced name'
},
isNewInstance: false
}));
observedContexts.should.eql(aTestModelCtx(expected));
} else {
// TODO: Please see loopback-datasource-juggler/issues#836
//
@ -2479,14 +2494,20 @@ module.exports = function(dataSource, should) {
{ id: existingInstance.id, name: 'replaced name' },
function(err, instance) {
if (err) return done(err);
observedContexts.should.eql(aTestModelCtx({
var expected = {
instance: {
id: existingInstance.id,
name: 'replaced name',
extra: undefined
},
isNewInstance: false
}));
}
};
expected.isNewInstance =
connectorCapabilities.replaceOrCreateReportsNewInstance ?
false : undefined;
observedContexts.should.eql(aTestModelCtx(expected));
done();
});
});
@ -2498,14 +2519,19 @@ module.exports = function(dataSource, should) {
{ id: 'new-id', name: 'a name' },
function(err, instance) {
if (err) return done(err);
observedContexts.should.eql(aTestModelCtx({
var expected = {
instance: {
id: instance.id,
name: 'a name',
extra: undefined
},
isNewInstance: true
}));
}
};
expected.isNewInstance =
connectorCapabilities.replaceOrCreateReportsNewInstance ?
true : undefined;
observedContexts.should.eql(aTestModelCtx(expected));
done();
});
});