Clear Update.exe login item settings in afterEach
authorKevin Sawicki <kevinsawicki@gmail.com>
Thu, 2 Feb 2017 16:01:47 +0000 (08:01 -0800)
committerKevin Sawicki <kevinsawicki@gmail.com>
Mon, 6 Feb 2017 16:34:35 +0000 (08:34 -0800)
spec/api-app-spec.js

index 53d8235f7b44bbeba2398f8482ae86977a0f1be3..085522a0cc9c10f555870e4054ec33fdb2b74974 100644 (file)
@@ -313,12 +313,20 @@ describe('app module', function () {
   describe('app.get/setLoginItemSettings API', function () {
     if (process.platform === 'linux') return
 
+    const updateExe = path.resolve(path.dirname(process.execPath), '..', 'Update.exe')
+    const processStartArgs = [
+      '--processStart', `"${path.basename(process.execPath)}"`,
+      '--process-start-args', `"--hidden"`
+    ]
+
     beforeEach(function () {
       app.setLoginItemSettings({openAtLogin: false})
+      app.setLoginItemSettings({openAtLogin: false, path: updateExe, args: processStartArgs})
     })
 
     afterEach(function () {
       app.setLoginItemSettings({openAtLogin: false})
+      app.setLoginItemSettings({openAtLogin: false, path: updateExe, args: processStartArgs})
     })
 
     it('returns the login item status of the app', function () {
@@ -353,15 +361,6 @@ describe('app module', function () {
     it('allows you to pass a custom executable and arguments', () => {
       if (process.platform !== 'win32') return
 
-      const appFolder = path.dirname(process.execPath)
-      const updateExe = path.resolve(appFolder, '..', 'Update.exe')
-      const exeName = path.basename(process.execPath)
-
-      const processStartArgs = [
-        '--processStart', `"${exeName}"`,
-        '--process-start-args', `"--hidden"`
-      ]
-
       app.setLoginItemSettings({openAtLogin: true, path: updateExe, args: processStartArgs})
 
       assert.equal(app.getLoginItemSettings().openAtLogin, false)