TRAVIS_PULL_REQUEST is always set, so check not 'false'

Andy Earnshaw · Andy Earnshaw · commit 30b8a07ef96f · 2015-06-24T12:55:51.000+01:00
diff --git a/tests/saucelabs.js b/tests/saucelabs.js
@@ -447,7 +447,7 @@ function main(tunnelReady) {
 drawStatus();
 
 // Avoid "The Travis CI build failed" message on PRs
-if (process.env.TRAVIS_PULL_REQUEST) {
+if (process.env.TRAVIS_PULL_REQUEST && process.env.TRAVIS_PULL_REQUEST !== 'false') {
     console.warn('Unable to run Sauce Labs tests for pull requests.');
     process.exit(0);
 }

Original file line number	Diff line number	Diff line change
`@@ -447,7 +447,7 @@ function main(tunnelReady) {`
`447`	`447`	`drawStatus();`
`448`	`448`
`449`	`449`	`// Avoid "The Travis CI build failed" message on PRs`
`450`		`-if (process.env.TRAVIS_PULL_REQUEST) {`
	`450`	`+if (process.env.TRAVIS_PULL_REQUEST && process.env.TRAVIS_PULL_REQUEST !== 'false') {`
`451`	`451`	`console.warn('Unable to run Sauce Labs tests for pull requests.');`
`452`	`452`	`process.exit(0);`
`453`	`453`	`}`