fix: coverage issue with saucelabs

angular · Mar 9, 2017 · 6fed865 · 6fed865
1 parent ce0e933
commit 6fed865
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 3 deletions.
diff --git a/scripts/ci/build-and-test.sh b/scripts/ci/build-and-test.sh
@@ -26,7 +26,7 @@ else
 fi
 
 # Don't upload coverage for both test modes (browserstack and saucelabs) and inside of PRs.
-if [[ "$MODE" == "saucelabs_required" ]] && [ "$TRAVIS_PULL_REQUEST" = "false" ]; then
+if [[ "$MODE" == "browserstack_required" ]] && [ "$TRAVIS_PULL_REQUEST" != "false" ]; then
   $(npm bin)/gulp ci:coverage
 fi
 

diff --git a/test/karma.conf.js b/test/karma.conf.js
@@ -100,7 +100,7 @@ module.exports = (config) => {
   if (process.env['TRAVIS']) {
     let buildId = `TRAVIS #${process.env.TRAVIS_BUILD_NUMBER} (${process.env.TRAVIS_BUILD_ID})`;
 
-    if (process.env['TRAVIS_PULL_REQUEST'] === 'false') {
+    if (process.env['TRAVIS_PULL_REQUEST'] !== 'false') {
       config.preprocessors['dist/@angular/material/**/!(*+(.|-)spec).js'] = ['coverage'];
       config.reporters.push('coverage');
     }

diff --git a/tools/gulp/tasks/coverage.ts b/tools/gulp/tasks/coverage.ts
@@ -9,7 +9,7 @@ task('coverage:upload', () => {
     throw new Error('No coverage file has been found!');
   }
 
-  if (!isTravisPushBuild()) {
+  if (isTravisPushBuild()) {
     throw new Error('Coverage results will be only uploaded inside of Travis Push builds.');
   }