ci: move ci check for BS

Author: posva

test/e2e/nightwatch.browserstack.js

@@ -4,18 +4,6 @@
 
 const BS_USER = process.env.BS_USER
 const BS_KEY = process.env.BS_KEY
-const isCI = process.env.CI
-
-if (!BS_USER || !BS_KEY) {
-  console.log(
-    'Hey!\n',
-    'You are missing credentials for Browserstack.\n',
-    'If you are a contributor, this is normal, credentials are private. These tests must be run by a maintainer of vue-router',
-    'If you are a maintainer, you forgot to `source keys.env`!'
-  )
-  // fail if testing locally
-  process.exit(isCI ? 0 : 1)
-}
 
 const nwConf = {
   src_folders: ['test/e2e/specs'],

test/e2e/runner.js

@@ -81,6 +81,17 @@ process.mainModule.filename = resolve(
 )
 
 if (isLocal) {
+  if (isLocal && (!process.env.BS_USER || !process.env.BS_KEY)) {
+    console.log(
+      'Hey!\n',
+      'You are missing credentials for Browserstack.\n',
+      'If you are a contributor, this is normal, credentials are private. These tests must be run by a maintainer of vue-router',
+      'If you are a maintainer, you forgot to `source keys.env`!'
+    )
+    // fail if testing locally
+    process.exit(process.env.isCI ? 0 : 1)
+  }
+
   let bsLocal
   const browserstack = require('browserstack-local')
   Nightwatch.bs_local = bsLocal = new browserstack.Local()