Skip to content

Commit

Permalink
test: test on travis with node 10 (#2461)
Browse files Browse the repository at this point in the history
  • Loading branch information
dead-horse authored Jul 17, 2018
1 parent 38a472f commit b55b303
Show file tree
Hide file tree
Showing 5 changed files with 8 additions and 7 deletions.
2 changes: 1 addition & 1 deletion .travis.yml
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@ sudo: false
language: node_js
node_js:
- '8'
- '9'
- '10'
install:
- npm i npminstall && npminstall
script:
Expand Down
2 changes: 1 addition & 1 deletion appveyor.yml
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
environment:
matrix:
- nodejs_version: '8'
- nodejs_version: '9'
- nodejs_version: '10'

install:
- ps: Install-Product node $env:nodejs_version
Expand Down
1 change: 1 addition & 0 deletions test/doc.test.js
Original file line number Diff line number Diff line change
Expand Up @@ -24,6 +24,7 @@ describe('test/doc.test.js', () => {

it('should no broken url', async () => {
const result = await findlinks({ src: app.url, logger: console });
if (result.fail !== 0) console.log(result);
assert(result.fail === 0);
});
});
6 changes: 3 additions & 3 deletions test/lib/core/cookies.test.js
Original file line number Diff line number Diff line change
Expand Up @@ -21,7 +21,7 @@ describe('test/lib/core/cookies.test.js', () => {
const ctx = app.mockContext();
assert.throws(() => {
ctx.cookies.set('foo', 'bar', { secure: true });
}, 'Cannot send secure cookie over unencrypted connection');
}, /Cannot send secure cookie over unencrypted connection/);
});

it('should set cookie twice and not set domain when ctx.hostname=localhost', () => {
Expand Down Expand Up @@ -54,7 +54,7 @@ describe('test/lib/core/cookies.test.js', () => {
ctx.cookies.set('foo', 'bar', {
encrypt: true,
});
}, '.keys required for encrypt/sign cookies');
}, /\.keys required for encrypt\/sign cookies/);
});

it('should throw TypeError when get encrypt on keys not exists', () => {
Expand All @@ -65,7 +65,7 @@ describe('test/lib/core/cookies.test.js', () => {
ctx.cookies.get('foo', {
encrypt: true,
});
}, '.keys required for encrypt/sign cookies');
}, /\.keys required for encrypt\/sign cookies/);
});

it('should not set secure when request protocol is http', done => {
Expand Down
4 changes: 2 additions & 2 deletions test/lib/core/loader/load_plugin.test.js
Original file line number Diff line number Diff line change
Expand Up @@ -250,7 +250,7 @@ describe('test/lib/core/loader/load_plugin.test.js', () => {
logger,
});
appLoader.loadConfig();
}, 'sequencify plugins has problem, missing: [], recursive: [a,b,c,a]');
}, /sequencify plugins has problem, missing: \[\], recursive: \[a,b,c,a\]/);
});

it('should throw error when plugin dep not exists', function() {
Expand All @@ -262,7 +262,7 @@ describe('test/lib/core/loader/load_plugin.test.js', () => {
logger,
});
appLoader.loadConfig();
}, 'sequencify plugins has problem, missing: [a1], recursive: []\n\t>> Plugin [a1] is disabled or missed, but is required by [c]');
}, /sequencify plugins has problem, missing: \[a1\], recursive: \[\]\s+>> Plugin \[a1\] is disabled or missed, but is required by \[c\]/);
});

it('should auto fill plugin infos', () => {
Expand Down

0 comments on commit b55b303

Please sign in to comment.