From 5940231b7633e891cd765b80cea343d7502cf37f Mon Sep 17 00:00:00 2001 From: Cheng Zhao Date: Mon, 25 Sep 2017 10:49:54 +0900 Subject: [PATCH] spec: Do not assume which process exits first in singleton test --- spec/api-app-spec.js | 14 +++++++++----- 1 file changed, 9 insertions(+), 5 deletions(-) diff --git a/spec/api-app-spec.js b/spec/api-app-spec.js index 035520bb41..8a2062d78c 100644 --- a/spec/api-app-spec.js +++ b/spec/api-app-spec.js @@ -162,19 +162,23 @@ describe('app module', function () { this.timeout(120000) const appPath = path.join(__dirname, 'fixtures', 'api', 'singleton') // First launch should exit with 0. - let secondLaunched = false + let exited = 0 const first = ChildProcess.spawn(remote.process.execPath, [appPath]) first.once('exit', (code) => { - assert.ok(secondLaunched) assert.equal(code, 0) - done() + exited++ + if (exited === 2) { + done() + } }) // Second launch should exit with 1. const second = ChildProcess.spawn(remote.process.execPath, [appPath]) second.once('exit', (code) => { - assert.ok(!secondLaunched) assert.equal(code, 1) - secondLaunched = true + exited++ + if (exited === 2) { + done() + } }) }) })