From aa18c5296f52e64cd694b77ae5df4dc89db0d95f Mon Sep 17 00:00:00 2001 From: Feross Aboukhadijeh Date: Fri, 2 Mar 2018 12:43:06 -0800 Subject: [PATCH] airtap: always run browser test Rely on airtap to automatically skip browser tests when Travis secure variables are not available (in PRs) --- bin/test.js | 10 ++-------- 1 file changed, 2 insertions(+), 8 deletions(-) diff --git a/bin/test.js b/bin/test.js index fca4045..93d4c55 100644 --- a/bin/test.js +++ b/bin/test.js @@ -4,16 +4,10 @@ var cp = require('child_process') var fs = require('fs') var path = require('path') -var shouldRunBrowserTests = !process.env.TRAVIS_PULL_REQUEST || - process.env.TRAVIS_PULL_REQUEST === 'false' - var node = cp.spawn('npm', ['run', 'test-node'], { stdio: 'inherit' }) node.on('close', function (code) { - if (code === 0 && shouldRunBrowserTests) { - runBrowserTests() - } else { - process.exit(code) - } + if (code !== 0) return process.exit(code) + runBrowserTests() }) function runBrowserTests () { -- 2.34.1