Run more crashReporter specs on CI
authorKevin Sawicki <kevinsawicki@gmail.com>
Tue, 29 Nov 2016 22:18:42 +0000 (14:18 -0800)
committerKevin Sawicki <kevinsawicki@gmail.com>
Tue, 29 Nov 2016 22:18:42 +0000 (14:18 -0800)
spec/api-crash-reporter-spec.js
spec/chromium-spec.js

index df4bbfa..781a524 100644 (file)
@@ -9,6 +9,8 @@ const {closeWindow} = require('./window-helpers')
 const {remote} = require('electron')
 const {app, BrowserWindow, crashReporter} = remote.require('electron')
 
+const isCI = remote.getGlobal('isCi')
+
 describe('crashReporter module', function () {
   var fixtures = path.resolve(__dirname, 'fixtures')
   var w = null
@@ -33,12 +35,9 @@ describe('crashReporter module', function () {
     return
   }
 
-  var isCI = remote.getGlobal('isCi')
-  if (isCI) {
-    return
-  }
-
   it('should send minidump when renderer crashes', function (done) {
+    if (isCi) return
+
     this.timeout(120000)
 
     var called = false
index 1a39edd..ff27d0e 100644 (file)
@@ -61,11 +61,9 @@ describe('chromium feature', function () {
       w.loadURL(url)
     })
 
-    if (isCI && process.platform === 'win32') {
-      return
-    }
-
     it('is set correctly when window is inactive', function (done) {
+      if (isCI && process.platform === 'win32') return done()
+
       w = new BrowserWindow({
         show: false
       })