Browse Source

spec: add more assertions to help smoke out the close all window failures (#13868)

Samuel Attard 6 years ago
parent
commit
4990b3990b
1 changed files with 3 additions and 2 deletions
  1. 3 2
      spec/api-app-spec.js

+ 3 - 2
spec/api-app-spec.js

@@ -168,8 +168,9 @@ describe('app module', () => {
       const electronPath = remote.getGlobal('process').execPath
 
       appProcess = ChildProcess.spawn(electronPath, [appPath])
-      appProcess.on('close', code => {
-        expect(code).to.equal(123)
+      appProcess.on('close', (code, signal) => {
+        expect(signal).to.equal(null, 'exit signal should be null, if you see this please tag @MarshallOfSound')
+        expect(code).to.equal(123, 'exit code should be 123, if you see this please tag @MarshallOfSound')
         done()
       })
     })