refactor openExterna spec to fix side effects

electron · Dec 22, 2018 · 4ae0b27 · 4ae0b27
1 parent c6b798e
commit 4ae0b27
Showing 1 changed file with 26 additions and 16 deletions.
diff --git a/spec/api-shell-spec.js b/spec/api-shell-spec.js
@@ -17,7 +17,24 @@ describe('shell module', () => {
   }
 
   describe('shell.openExternal()', () => {
-    it('opens an external link', function (done) {
+    let envVars = {}
+
+    beforeEach(() => {
+      // keep original process.env values to prevent side effect
+      envVars = {
+        display: process.env.DISPLAY,
+        de: process.env.DE,
+        browser: process.env.BROWSER
+      }
+    })
+
+    afterEach(() => {
+      process.env.BROWSER = envVars.browser
+      process.env.DE = envVars.de
+      process.env.DISPLAY = envVars.browser
+    })
+
+    it('opens an external link asynchronously', function (done) {
       if (process.platform !== 'linux') this.skip()
 
       // set env vars for xdg-open
@@ -28,25 +45,18 @@ describe('shell module', () => {
 
       shell.openExternal(url).then(() => done())
     })
-  })
 
-  describe('shell.openExternalSync()', () => {
-    it('opens an external link', function () {
+    it('opens an external link synchronously', function () {
       if (process.platform !== 'linux') this.skip()
 
+      // set env vars for xdg-open
+      const url = 'http://www.example.com'
       process.env.DE = 'generic'
-      const tests = [
-        { BROWSER: '/bin/true', expected: true },
-        { BROWSER: '/bin/false', expected: false }
-      ]
-
-      for (const test of tests) {
-        const { BROWSER, expected } = test
-        process.env.DISPLAY = ''
-        process.env.BROWSER = BROWSER
-        const actual = shell.openExternalSync('http://www.example.com')
-        assert.strictEqual(expected, actual)
-      }
+      process.env.BROWSER = '/bin/true'
+      process.env.DISPLAY = ''
+
+      const success = shell.openExternalSync(url)
+      assert.strictEqual(true, success)
     })
   })