mirror of
https://github.com/TryGhost/Ghost.git
synced 2025-02-10 23:36:14 -05:00
f08a55c21f
refs: https://github.com/TryGhost/Team/issues/856 refs: https://github.com/TryGhost/Team/issues/756 - The .test.js extension is better than _spec.js as it's more obvious that it's an extension - It also meaans we can use the --extension parameter in mocha, which should result in a better default behaviour for `yarn test` - It also highlights that some of our tests were named incorrectly and were not (and still will not be) run (see https://github.com/TryGhost/Team/issues/856) - Note: even with this change, `yarn test` is throwing errors, I believe because of this issue https://github.com/TryGhost/Team/issues/756
36 lines
1.1 KiB
JavaScript
36 lines
1.1 KiB
JavaScript
const should = require('should');
|
|
const sinon = require('sinon');
|
|
const errors = require('@tryghost/errors');
|
|
const helpers = require('../../../../../core/frontend/services/routing/helpers');
|
|
|
|
describe('handleError', function () {
|
|
let next;
|
|
|
|
beforeEach(function () {
|
|
next = sinon.spy();
|
|
});
|
|
|
|
afterEach(function () {
|
|
sinon.restore();
|
|
});
|
|
|
|
it('should call next with no args for 404 errors', function () {
|
|
const notFoundError = new errors.NotFoundError({message: 'Something wasn\'t found'});
|
|
helpers.handleError(next)(notFoundError);
|
|
|
|
next.calledOnce.should.be.true();
|
|
next.firstCall.args.should.be.empty();
|
|
});
|
|
|
|
it('should call next with error for other errors', function () {
|
|
const otherError = new Error();
|
|
otherError.message = 'Something wasn\'t allowed';
|
|
|
|
helpers.handleError(next)(otherError);
|
|
|
|
next.calledOnce.should.be.true();
|
|
next.firstCall.args.should.have.lengthOf(1);
|
|
next.firstCall.args[0].should.be.an.Object();
|
|
next.firstCall.args[0].should.be.instanceof(Error);
|
|
});
|
|
});
|