Compare commits
3 Commits
| Author | SHA1 | Date | |
|---|---|---|---|
| 0bf6fb0023 | |||
| 300449bfb0 | |||
| 868c808db7 |
@@ -1,6 +1,6 @@
|
|||||||
{
|
{
|
||||||
"name": "esqueue",
|
"name": "esqueue",
|
||||||
"version": "0.3.1",
|
"version": "0.3.2",
|
||||||
"description": "Job queue, powered by Elasticsearch",
|
"description": "Job queue, powered by Elasticsearch",
|
||||||
"main": "lib/index.js",
|
"main": "lib/index.js",
|
||||||
"scripts": {
|
"scripts": {
|
||||||
|
|||||||
@@ -180,7 +180,7 @@ export default class Job extends events.EventEmitter {
|
|||||||
}
|
}
|
||||||
|
|
||||||
_claimPendingJobs(jobs) {
|
_claimPendingJobs(jobs) {
|
||||||
if (jobs.length === 0) return;
|
if (!jobs || jobs.length === 0) return;
|
||||||
|
|
||||||
this._stopJobPolling();
|
this._stopJobPolling();
|
||||||
let claimed = false;
|
let claimed = false;
|
||||||
@@ -268,7 +268,7 @@ export default class Job extends events.EventEmitter {
|
|||||||
})
|
})
|
||||||
.catch((err) => {
|
.catch((err) => {
|
||||||
// ignore missing indices errors
|
// ignore missing indices errors
|
||||||
if (err.status === 404) return;
|
if (err.status === 404) return [];
|
||||||
|
|
||||||
this.debug('job querying failed', err);
|
this.debug('job querying failed', err);
|
||||||
this.emit('error', err);
|
this.emit('error', err);
|
||||||
|
|||||||
@@ -165,6 +165,24 @@ describe('Worker class', function () {
|
|||||||
.then(() => { done(); })
|
.then(() => { done(); })
|
||||||
.catch(() => done(new Error('should not reject')));
|
.catch(() => done(new Error('should not reject')));
|
||||||
});
|
});
|
||||||
|
|
||||||
|
it('should return an empty array on missing index', function (done) {
|
||||||
|
searchStub = sinon.stub(mockQueue.client, 'search', () => Promise.reject({
|
||||||
|
status: 404
|
||||||
|
}));
|
||||||
|
worker = new Worker(mockQueue, 'test', noop);
|
||||||
|
worker._getPendingJobs()
|
||||||
|
.then((res) => {
|
||||||
|
try {
|
||||||
|
expect(res).to.be.an(Array);
|
||||||
|
expect(res).to.have.length(0);
|
||||||
|
done();
|
||||||
|
} catch (e) {
|
||||||
|
done(e);
|
||||||
|
}
|
||||||
|
})
|
||||||
|
.catch(() => done(new Error('should not reject')));
|
||||||
|
});
|
||||||
});
|
});
|
||||||
|
|
||||||
describe('query parameters', function () {
|
describe('query parameters', function () {
|
||||||
|
|||||||
Reference in New Issue
Block a user