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

test: allow retries for flaky mas loginitem specs (#16358)

Browse source
This commit is contained in:
trop[bot] 2019-01-11 08:12:04 -08:00 committed by Shelley Vohr
parent 27bfeb6148
commit 79e55e5e85
1 changed files with 6 additions and 9 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

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

@ -432,7 +432,10 @@ describe('app module', () => {
})
})
describe('app.get/setLoginItemSettings API', () => {
describe('app.get/setLoginItemSettings API', function () {
// allow up to three retries to account for flaky mas results
this.retries(3)
const updateExe = path.resolve(path.dirname(process.execPath), '..', 'Update.exe')
const processStartArgs = [
'--processStart', `"${path.basename(process.execPath)}"`,
@ -440,9 +443,7 @@ describe('app module', () => {
]
before(function () {
if (process.platform === 'linux') {
this.skip()
}
if (process.platform === 'linux') this.skip()
})
beforeEach(() => {
@ -513,11 +514,7 @@ describe('app module', () => {
})
it('allows you to pass a custom executable and arguments', function () {
if (process.platform !== 'win32') {
// FIXME(alexeykuzmin): Skip the test.
// this.skip()
return
}
if (process.platform !== 'win32') this.skip()
app.setLoginItemSettings({openAtLogin: true, path: updateExe, args: processStartArgs})