Skip to content

Commit

Permalink
Fix multitarget test on AppVeyor
Browse files Browse the repository at this point in the history
  • Loading branch information
malept committed Apr 25, 2017
1 parent 080a486 commit 2af9efc
Show file tree
Hide file tree
Showing 2 changed files with 1 addition and 12 deletions.
1 change: 0 additions & 1 deletion package.json
Original file line number Diff line number Diff line change
Expand Up @@ -40,7 +40,6 @@
"eslint-plugin-promise": "^3.0.0",
"eslint-plugin-standard": "^2.0.0",
"eslint-plugin-tape": "^1.1.0",
"is-admin": "^2.0.0",
"nyc": "^10.0.0",
"pkg-up": "^1.0.0",
"rimraf": "^2.3.2",
Expand Down
12 changes: 1 addition & 11 deletions test/multitarget.js
Original file line number Diff line number Diff line change
@@ -1,7 +1,6 @@
'use strict'

const config = require('./config.json')
const isAdmin = require('is-admin')
const packager = require('..')
const util = require('./util')
const waterfall = require('run-waterfall')
Expand Down Expand Up @@ -36,7 +35,7 @@ function createMultiTest (arch, platform) {
}

util.packagerTest('all test', (t) => {
t.timeoutAfter(config.timeout * 7) // 5-7 packages will be built during this test
t.timeoutAfter(config.timeout * 7) // 7 packages will be built during this test

var opts = {
name: 'basicTest',
Expand All @@ -49,15 +48,6 @@ util.packagerTest('all test', (t) => {

waterfall([
function (cb) {
if (process.platform === 'win32') {
isAdmin().then((admin) => {
if (!admin) expectedAppCount = 5
cb()
})
} else {
cb()
}
}, function (cb) {
packager(opts, cb)
}, function (finalPaths, cb) {
// OS X only has 64-bit releases
Expand Down

0 comments on commit 2af9efc

Please sign in to comment.