Use skip for setAppDetails spec

This commit is contained in:
Kevin Sawicki 2016-11-28 14:30:43 -08:00
parent 5abecadfd3
commit 3897a88082

View file

@ -519,9 +519,9 @@ describe('browser-window module', function () {
}) })
describe('BrowserWindow.setAppDetails(options)', function () { describe('BrowserWindow.setAppDetails(options)', function () {
if (process.platform !== 'win32') return
it('supports setting the app details', function () { it('supports setting the app details', function () {
if (process.platform !== 'win32') return this.skip()
const iconPath = path.join(fixtures, 'assets', 'icon.ico') const iconPath = path.join(fixtures, 'assets', 'icon.ico')
assert.doesNotThrow(function () { assert.doesNotThrow(function () {
@ -540,9 +540,10 @@ describe('browser-window module', function () {
}) })
w.setAppDetails({}) w.setAppDetails({})
}) })
assert.throws(function () { assert.throws(function () {
w.setAppDetails() w.setAppDetails()
}, /Insufficient number of arguments./) }, /Insufficient number of arguments\./)
}) })
}) })