From b809b82cbd06b56383e176a4efebb9b9ab8055ef Mon Sep 17 00:00:00 2001 From: Nate Fischer Date: Sat, 17 Mar 2018 03:02:21 -0700 Subject: [PATCH] test: increase timeouts for flaky tests (#15) This increases timeouts for the 2 flaky tests. This also fixes a timeout issue for the "escapes quotes" test on Windows, since we only invoked the done() callback from the unix code branch (a bug in #11). Fixes #14 --- test/test.js | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/test/test.js b/test/test.js index 8a3f232..df930f2 100644 --- a/test/test.js +++ b/test/test.js @@ -252,7 +252,7 @@ describe('proxy', () => { fs.existsSync(fa).should.equal(true); fs.existsSync(fb).should.equal(true); done(); - }); + }).timeout(5000); it('avoids globs', (done) => { const fa = 'a.txt'; @@ -268,7 +268,7 @@ describe('proxy', () => { // These files are still ok fs.existsSync(fa).should.equal(true); done(); - }); + }).timeout(5000); it('escapes quotes', (done) => { if (unix()) { @@ -277,12 +277,12 @@ describe('proxy', () => { fs.existsSync(fquote).should.equal(true); shell[delVarName](fquote); fs.existsSync(fquote).should.equal(false); - done(); } else { // Windows doesn't support `"` as a character in a filename, see // https://msdn.microsoft.com/en-us/library/windows/desktop/aa365247(v=vs.85).aspx console.log('skipping test'); } + done(); }); }); });