mirror of
https://github.com/TryGhost/Ghost.git
synced 2025-02-03 23:00:14 -05:00
refs #7116, refs #2001 - Changes the way Ghost errors are implemented to benefit from proper inheritance - Moves all error definitions into a single file - Changes the error constructor to take an options object, rather than needing the arguments to be passed in the correct order. - Provides a wrapper so that any errors that haven't already been converted to GhostErrors get converted before they are displayed. Summary of changes: * 🐛 set NODE_ENV in config handler * ✨ add GhostError implementation (core/server/errors.js) - register all errors in one file - inheritance from GhostError - option pattern * 🔥 remove all error files * ✨ wrap all errors into GhostError in case of HTTP * 🎨 adaptions - option pattern for errors - use GhostError when needed * 🎨 revert debug deletion and add TODO for error id's
189 lines
7.6 KiB
JavaScript
189 lines
7.6 KiB
JavaScript
var should = require('should'),
|
|
sinon = require('sinon'),
|
|
Promise = require('bluebird'),
|
|
|
|
// Stuff we are testing
|
|
versioning = require('../../server/data/schema').versioning,
|
|
db = require('../../server/data/db'),
|
|
errors = require('../../server/errors'),
|
|
|
|
sandbox = sinon.sandbox.create();
|
|
|
|
describe('Versioning', function () {
|
|
afterEach(function () {
|
|
sandbox.restore();
|
|
});
|
|
|
|
describe('getMigrationVersions', function () {
|
|
it('should output a single item if the from and to versions are the same', function () {
|
|
should.exist(versioning.getMigrationVersions);
|
|
versioning.getMigrationVersions('003', '003').should.eql(['003']);
|
|
versioning.getMigrationVersions('004', '004').should.eql(['004']);
|
|
});
|
|
|
|
it('should output an empty array if the toVersion is higher than the fromVersion', function () {
|
|
versioning.getMigrationVersions('003', '002').should.eql([]);
|
|
});
|
|
|
|
it('should output all the versions between two versions', function () {
|
|
versioning.getMigrationVersions('003', '004').should.eql(['003', '004']);
|
|
versioning.getMigrationVersions('003', '005').should.eql(['003', '004', '005']);
|
|
versioning.getMigrationVersions('003', '006').should.eql(['003', '004', '005', '006']);
|
|
versioning.getMigrationVersions('010', '011').should.eql(['010', '011']);
|
|
});
|
|
});
|
|
|
|
describe('getNewestDatabaseVersion', function () {
|
|
it('should return the correct version', function () {
|
|
var currentVersion = require('../../server/data/schema').defaultSettings.core.databaseVersion.defaultValue;
|
|
// This function has an internal cache, so we call it twice.
|
|
// First, to check that it fetches the correct version from default-settings.json.
|
|
versioning.getNewestDatabaseVersion().should.eql(currentVersion);
|
|
// Second, to check that it returns the same value from the cache.
|
|
versioning.getNewestDatabaseVersion().should.eql(currentVersion);
|
|
});
|
|
});
|
|
|
|
describe('getDatabaseVersion', function () {
|
|
var knexStub, knexMock, queryMock;
|
|
|
|
beforeEach(function () {
|
|
queryMock = {
|
|
where: sandbox.stub().returnsThis(),
|
|
first: sandbox.stub()
|
|
};
|
|
|
|
knexMock = sandbox.stub().returns(queryMock);
|
|
knexMock.schema = {
|
|
hasTable: sandbox.stub()
|
|
};
|
|
|
|
// this MUST use sinon, not sandbox, see sinonjs/sinon#781
|
|
knexStub = sinon.stub(db, 'knex', {get: function () { return knexMock; }});
|
|
});
|
|
|
|
afterEach(function () {
|
|
knexStub.restore();
|
|
});
|
|
|
|
it('should throw error if settings table does not exist', function (done) {
|
|
// Setup
|
|
knexMock.schema.hasTable.returns(new Promise.resolve(false));
|
|
|
|
// Execute
|
|
versioning.getDatabaseVersion().then(function () {
|
|
done('Should throw an error if the settings table does not exist');
|
|
}).catch(function (err) {
|
|
should.exist(err);
|
|
(err instanceof errors.DatabaseNotPopulatedError).should.eql(true);
|
|
|
|
knexStub.get.calledOnce.should.be.true();
|
|
knexMock.schema.hasTable.calledOnce.should.be.true();
|
|
knexMock.schema.hasTable.calledWith('settings').should.be.true();
|
|
queryMock.where.called.should.be.false();
|
|
queryMock.first.called.should.be.false();
|
|
done();
|
|
}).catch(done);
|
|
});
|
|
|
|
it('should lookup & return version, if settings table exists', function (done) {
|
|
// Setup
|
|
knexMock.schema.hasTable.returns(new Promise.resolve(true));
|
|
queryMock.first.returns(new Promise.resolve({value: '001'}));
|
|
|
|
// Execute
|
|
versioning.getDatabaseVersion().then(function (version) {
|
|
should.exist(version);
|
|
version.should.eql('001');
|
|
|
|
knexStub.get.calledTwice.should.be.true();
|
|
knexMock.schema.hasTable.calledOnce.should.be.true();
|
|
knexMock.schema.hasTable.calledWith('settings').should.be.true();
|
|
queryMock.where.called.should.be.true();
|
|
queryMock.first.called.should.be.true();
|
|
|
|
done();
|
|
}).catch(done);
|
|
});
|
|
|
|
// @TODO change this so we handle a non-existent version?
|
|
// There is an open bug in Ghost around this:
|
|
// https://github.com/TryGhost/Ghost/issues/7345
|
|
// I think it is a timing error
|
|
it.skip('should throw error if version does not exist', function (done) {
|
|
// Setup
|
|
knexMock.schema.hasTable.returns(new Promise.resolve(true));
|
|
queryMock.first.returns(new Promise.resolve());
|
|
|
|
// Execute
|
|
versioning.getDatabaseVersion().then(function () {
|
|
done('Should throw an error if version does not exist');
|
|
}).catch(function (err) {
|
|
should.exist(err);
|
|
(err instanceof errors.DatabaseVersionError).should.eql(true);
|
|
|
|
knexStub.get.calledTwice.should.be.true();
|
|
knexMock.schema.hasTable.calledOnce.should.be.true();
|
|
knexMock.schema.hasTable.calledWith('settings').should.be.true();
|
|
queryMock.where.called.should.be.true();
|
|
queryMock.first.called.should.be.true();
|
|
|
|
done();
|
|
}).catch(done);
|
|
});
|
|
|
|
// @TODO decide on a new scheme for database versioning and update
|
|
// how we validate those versions
|
|
it.skip('should throw error if version is not a number', function (done) {
|
|
// Setup
|
|
knexMock.schema.hasTable.returns(new Promise.resolve(true));
|
|
queryMock.first.returns(new Promise.resolve('Eyjafjallajökull'));
|
|
|
|
// Execute
|
|
versioning.getDatabaseVersion().then(function () {
|
|
done('Should throw an error if version is not a number');
|
|
}).catch(function (err) {
|
|
should.exist(err);
|
|
(err instanceof errors.DatabaseVersionError).should.eql(true);
|
|
|
|
knexStub.get.calledTwice.should.be.true();
|
|
knexMock.schema.hasTable.calledOnce.should.be.true();
|
|
knexMock.schema.hasTable.calledWith('settings').should.be.true();
|
|
queryMock.where.called.should.be.true();
|
|
queryMock.first.called.should.be.true();
|
|
|
|
done();
|
|
}).catch(done);
|
|
});
|
|
});
|
|
|
|
describe('setDatabaseVersion', function () {
|
|
var knexStub, knexMock, queryMock;
|
|
|
|
beforeEach(function () {
|
|
queryMock = {
|
|
where: sandbox.stub().returnsThis(),
|
|
update: sandbox.stub().returns(new Promise.resolve())
|
|
};
|
|
|
|
knexMock = sandbox.stub().returns(queryMock);
|
|
|
|
// this MUST use sinon, not sandbox, see sinonjs/sinon#781
|
|
knexStub = sinon.stub(db, 'knex', {get: function () { return knexMock; }});
|
|
});
|
|
|
|
afterEach(function () {
|
|
knexStub.restore();
|
|
});
|
|
|
|
it('should try to update the databaseVersion', function (done) {
|
|
versioning.setDatabaseVersion().then(function () {
|
|
knexStub.get.calledOnce.should.be.true();
|
|
queryMock.where.called.should.be.true();
|
|
queryMock.update.called.should.be.true();
|
|
|
|
done();
|
|
}).catch(done);
|
|
});
|
|
});
|
|
});
|