From: Feross Aboukhadijeh <feross@feross.org>
Date: Fri, 2 Mar 2018 20:43:06 +0000 (-0800)
Subject: airtap: always run browser test
X-Git-Url: https://zoso.dev/?a=commitdiff_plain;h=aa18c5296f52e64cd694b77ae5df4dc89db0d95f;p=buffer.git

airtap: always run browser test

Rely on airtap to automatically skip browser tests when Travis secure variables are not available (in PRs)
---

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 () {