Merge pull request #130 from mikkolehtinen/master
Fix for redis adapter when finding records filtered with multiple attributes
This commit is contained in:
commit
605f45dd35
|
@ -389,6 +389,7 @@ BridgeToRedis.prototype.all = function all(model, filter, callback) {
|
|||
if (indexes && indexes.length) {
|
||||
innerSetUsed = true;
|
||||
if (indexes.length > 1) {
|
||||
indexes.unshift(dest);
|
||||
trans.sinterstore(indexes);
|
||||
} else {
|
||||
dest = indexes[0];
|
||||
|
|
|
@ -418,6 +418,16 @@ function testOrm(schema) {
|
|||
|
||||
});
|
||||
|
||||
|
||||
it('should find records filtered with multiple attributes', function (test) {
|
||||
Post.create({title: 'title', content: 'content', published: true, date: 1}, function (err, post) {
|
||||
Post.all({where: {title: 'title', date: 1}}, function (err, res) {
|
||||
test.ok(res.length > 0, 'Exact match with string returns dataset');
|
||||
test.done();
|
||||
});
|
||||
});
|
||||
});
|
||||
|
||||
it('should handle hasMany relationship', function (test) {
|
||||
User.create(function (err, u) {
|
||||
if (err) return console.log(err);
|
||||
|
|
Loading…
Reference in New Issue