archive-gh-me/serverless
1
0
Fork 0
mirror of https://github.com/serverless/serverless.git synced 2026-01-25 15:07:39 +00:00
Code Issues Projects Releases Wiki Activity

Merge pull request #3931 from horike37/issue-3920

Browse Source 
fixed unhandled rejection errors in Utils test
This commit is contained in:
Eslam λ Hefnawy 2017-07-12 19:38:16 +07:00 committed by GitHub
commit b6cff4ebfd
1 changed files with 40 additions and 16 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

56
lib/classes/Utils.test.js
View File

@ -2,7 +2,7 @@
const path = require('path');
const os = require('os');
const expect = require('chai').expect;
const chai = require('chai');
const sinon = require('sinon');
const Serverless = require('../../lib/Serverless');
const testUtils = require('../../tests/utils');
@ -10,6 +10,8 @@ const configUtils = require('../utils/config');
const serverlessVersion = require('../../package.json').version;
const segment = require('../utils/segment');
const proxyquire = require('proxyquire');
chai.use(require('chai-as-promised'));
const expect = require('chai').expect;
describe('Utils', () => {
let utils;
@ -83,7 +85,7 @@ describe('Utils', () => {
serverless.utils.writeFileSync(tmpFilePath, { foo: 'bar' });
return serverless.yamlParser.parse(tmpFilePath).then((obj) => {
return expect(serverless.yamlParser.parse(tmpFilePath)).to.be.fulfilled.then((obj) => {
expect(obj.foo).to.equal('bar');
});
});
@ -93,7 +95,7 @@ describe('Utils', () => {
serverless.utils.writeFileSync(tmpFilePath, { foo: 'bar' });
return serverless.yamlParser.parse(tmpFilePath).then((obj) => {
return expect(serverless.yamlParser.parse(tmpFilePath)).to.be.fulfilled.then((obj) => {
expect(obj.foo).to.equal('bar');
});
});
@ -108,7 +110,8 @@ describe('Utils', () => {
const tmpFilePath = testUtils.getTmpFilePath('anything.json');
// note: use return when testing promises otherwise you'll have unhandled rejection errors
return serverless.utils.writeFile(tmpFilePath, { foo: 'bar' }).then(() => {
return expect(serverless.utils.writeFile(tmpFilePath, { foo: 'bar' }))
.to.be.fulfilled.then(() => {
const obj = serverless.utils.readFileSync(tmpFilePath);
expect(obj.foo).to.equal('bar');
@ -178,7 +181,7 @@ describe('Utils', () => {
serverless.utils.writeFileSync(tmpFilePath, { foo: 'bar' });
// note: use return when testing promises otherwise you'll have unhandled rejection errors
return serverless.utils.readFile(tmpFilePath).then((obj) => {
return expect(serverless.utils.readFile(tmpFilePath)).to.be.fulfilled.then((obj) => {
expect(obj.foo).to.equal('bar');
});
});
@ -349,7 +352,7 @@ describe('Utils', () => {
// help is a whitelisted option
serverless.processedInput.options = options;
return utils.logStat(serverless).then(() => {
return expect(utils.logStat(serverless)).to.be.fulfilled.then(() => {
expect(trackStub.calledOnce).to.equal(true);
expect(getConfigStub.calledOnce).to.equal(true);
@ -365,7 +368,7 @@ describe('Utils', () => {
frameworkId: '1234wasd',
});
return utils.logStat(serverless).then(() => {
return expect(utils.logStat(serverless)).to.be.fulfilled.then(() => {
expect(getConfigStub.calledOnce).to.equal(true);
expect(isDockerContainerStub.calledOnce).to.equal(true);
expect(trackStub.calledOnce).to.equal(true);
@ -398,9 +401,12 @@ describe('Utils', () => {
],
},
},
package: {
path: 'foo',
},
};
return utils.logStat(serverless).then(() => {
return expect(utils.logStat(serverless)).to.be.fulfilled.then(() => {
expect(getConfigStub.calledOnce).to.equal(true);
expect(trackStub.calledOnce).to.equal(true);
@ -436,10 +442,13 @@ describe('Utils', () => {
],
},
},
package: {
path: 'foo',
},
};
return utils.logStat(serverless).then(() => {
return expect(utils.logStat(serverless)).to.be.fulfilled.then(() => {
expect(getConfigStub.calledOnce).to.equal(true);
expect(trackStub.calledOnce).to.equal(true);
@ -473,9 +482,12 @@ describe('Utils', () => {
],
},
},
package: {
path: 'foo',
},
};
return utils.logStat(serverless).then(() => {
return expect(utils.logStat(serverless)).to.be.fulfilled.then(() => {
expect(getConfigStub.calledOnce).to.equal(true);
expect(trackStub.calledOnce).to.equal(true);
@ -511,9 +523,12 @@ describe('Utils', () => {
],
},
},
package: {
path: 'foo',
},
};
return utils.logStat(serverless).then(() => {
return expect(utils.logStat(serverless)).to.be.fulfilled.then(() => {
expect(getConfigStub.calledOnce).to.equal(true);
expect(trackStub.calledOnce).to.equal(true);
@ -549,9 +564,12 @@ describe('Utils', () => {
],
},
},
package: {
path: 'foo',
},
};
return utils.logStat(serverless).then(() => {
return expect(utils.logStat(serverless)).to.be.fulfilled.then(() => {
expect(getConfigStub.calledOnce).to.equal(true);
expect(trackStub.calledOnce).to.equal(true);
@ -585,9 +603,12 @@ describe('Utils', () => {
],
},
},
package: {
path: 'foo',
},
};
return utils.logStat(serverless).then(() => {
return expect(utils.logStat(serverless)).to.be.fulfilled.then(() => {
expect(getConfigStub.calledOnce).to.equal(true);
expect(trackStub.calledOnce).to.equal(true);
@ -623,9 +644,12 @@ describe('Utils', () => {
],
},
},
package: {
path: 'foo',
},
};
return utils.logStat(serverless).then(() => {
return expect(utils.logStat(serverless)).to.be.fulfilled.then(() => {
expect(getConfigStub.calledOnce).to.equal(true);
expect(trackStub.calledOnce).to.equal(true);
@ -648,7 +672,7 @@ describe('Utils', () => {
},
};
return utils.logStat(serverless).then(() => {
return expect(utils.logStat(serverless)).to.be.fulfilled.then(() => {
expect(getConfigStub.calledOnce).to.equal(true);
expect(trackStub.calledOnce).to.equal(true);
@ -711,7 +735,7 @@ describe('Utils', () => {
package: {},
};
return utils.logStat(serverless).then(() => {
return expect(utils.logStat(serverless)).to.be.fulfilled.then(() => {
expect(trackStub.calledOnce).to.equal(true);
expect(getConfigStub.calledOnce).to.equal(true);