mirrors/electron
2
0
Fork 0
Code Issues Pull requests Projects Releases Packages 1 Wiki Activity Actions

Make sure the spec takes arguments into account.

Browse source
This commit is contained in:
CharlieHess 2017-01-26 12:35:46 -08:00 committed by Kevin Sawicki
parent 4fce00148d
commit 852e158cc6
2 changed files with 9 additions and 9 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

3
docs/api/app.md
View file

@ -765,6 +765,9 @@ Returns `Boolean` - Whether the current desktop environment is Unity launcher.
* `args` String[] (optional) _Windows_ - The command-line arguments to compare * `args` String[] (optional) _Windows_ - The command-line arguments to compare
against. Defaults to an empty array. against. Defaults to an empty array.
If you provided arguments to `app.setLoginItemSettings` you need to pass the
same arguments here for `openAtLogin` to be set correctly.
Returns `Object`: Returns `Object`:
* `openAtLogin` Boolean - `true` if the app is set to open at login. * `openAtLogin` Boolean - `true` if the app is set to open at login.

15
spec/api-app-spec.js
View file

@ -357,18 +357,15 @@ describe('app module', function () {
const updateExe = path.resolve(appFolder, '..', 'Update.exe') const updateExe = path.resolve(appFolder, '..', 'Update.exe')
const exeName = path.basename(process.execPath) const exeName = path.basename(process.execPath)
app.setLoginItemSettings({openAtLogin: true}, updateExe, [ const processStartArgs = [
'--processStart', `"${exeName}"`, '--processStart', `"${exeName}"`,
'--process-start-args', `"--hidden"` '--process-start-args', `"--hidden"`
]) ];
assert.deepEqual(app.getLoginItemSettings(), { app.setLoginItemSettings({openAtLogin: true}, updateExe, processStartArgs)
openAtLogin: true,
openAsHidden: false, assert(!app.getLoginItemSettings().openAtLogin)
wasOpenedAtLogin: false, assert(app.getLoginItemSettings(updateExe, processStartArgs))
wasOpenedAsHidden: false,
restoreState: false
})
}) })
}) })