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 #576 from fabien/fix/scope-caching 

Don't cache static scope method results
This commit is contained in:
Miroslav Bajtoš 2015-07-03 10:49:41 +02:00
parent 88229fb5cf 5290559a42
commit 014e91ea0c
2 changed files with 23 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
lib/scope.js
View File

@ -52,7 +52,7 @@ ScopeDefinition.prototype.related = function(receiver, scopeParams, condOrRefres
var actualCond = {};
var actualRefresh = false;
var saveOnCache = true;
var saveOnCache = receiver instanceof DefaultModelBaseClass;
if (typeof condOrRefresh === 'function' &&
options === undefined && cb === undefined) {
// related(receiver, scopeParams, cb)

22
test/scope.test.js
View File

@ -76,6 +76,28 @@ describe('scope', function () {
});
});
it('should not cache any results', function (done) {
Station.scope('active', {where: {isActive: true}});
Station.active.create(function (err, s) {
if (err) return done(err);
s.isActive.should.be.true;
Station.active(function(err, ss) {
if (err) return done(err);
ss.should.have.lengthOf(1);
ss[0].id.should.eql(s.id);
s.updateAttribute('isActive', false, function(err, s) {
if (err) return done(err);
s.isActive.should.be.false;
Station.active(function(err, ss) {
if (err) return done(err);
ss.should.have.lengthOf(0);
done();
});
});
});
});
});
});
describe('scope - order', function () {