checkpoint: fix `current()`

Fix the query in `Checkpoint.current()` to correctly specify sorting
`seq DESC`. Before this change, the first checkpoint was returned as the
current one.
This commit is contained in:
Miroslav Bajtoš 2014-07-11 17:07:22 +02:00
parent be48a504e0
commit 31ef6394b4
2 changed files with 25 additions and 1 deletions

View File

@ -48,7 +48,7 @@ Checkpoint.current = function(cb) {
var Checkpoint = this;
this.find({
limit: 1,
sort: 'seq DESC'
order: 'seq DESC'
}, function(err, checkpoints) {
if(err) return cb(err);
var checkpoint = checkpoints[0];

24
test/checkpoint.test.js Normal file
View File

@ -0,0 +1,24 @@
var async = require('async');
var loopback = require('../');
// create a unique Checkpoint model
var Checkpoint = require('../lib/models/checkpoint').extend('TestCheckpoint');
Checkpoint.attachTo(loopback.memory());
describe('Checkpoint', function() {
describe('current()', function() {
it('returns the highest `seq` value', function(done) {
async.series([
Checkpoint.create.bind(Checkpoint),
Checkpoint.create.bind(Checkpoint),
function(next) {
Checkpoint.current(function(err, seq) {
if (err) next(err);
expect(seq).to.equal(2);
next();
});
}
], done);
});
});
});