Browse Source

fix: re-enable mixed-sandbox tests

Shelley Vohr 6 years ago
parent
commit
590ae5cd29
1 changed files with 0 additions and 14 deletions
  1. 0 14
      spec/api-app-spec.js

+ 0 - 14
spec/api-app-spec.js

@@ -869,13 +869,6 @@ describe('app module', () => {
     })
 
     describe('when app.enableMixedSandbox() is called', () => {
-      // TODO(zcbenz): Find out why it fails in CI.
-      before(function () {
-        if (isCI && process.platform === 'win32') {
-          this.skip()
-        }
-      })
-
       it('adds --enable-sandbox to render processes created with sandbox: true', done => {
         const appPath = path.join(__dirname, 'fixtures', 'api', 'mixed-sandbox-app')
         appProcess = ChildProcess.spawn(remote.process.execPath, [appPath])
@@ -898,13 +891,6 @@ describe('app module', () => {
     })
 
     describe('when the app is launched with --enable-mixed-sandbox', () => {
-      // TODO(zcbenz): Find out why it fails in CI.
-      before(function () {
-        if (isCI && process.platform === 'win32') {
-          this.skip()
-        }
-      })
-
       it('adds --enable-sandbox to render processes created with sandbox: true', done => {
         const appPath = path.join(__dirname, 'fixtures', 'api', 'mixed-sandbox-app')
         appProcess = ChildProcess.spawn(remote.process.execPath, [appPath, '--enable-mixed-sandbox'])