🏗🚮 Stop testing on Sauce Labs and delete all related infrastructure (a…

…mpproject#29038)

.gitignore

@@ -24,9 +24,6 @@ test/coverage
 *.swp
 *.swo
 yarn-error.log
-sc-*-linux*
-sc-*-osx*
-sauce_connect_*
 deps.txt
 firebase
 .firebaserc

build-system/pr-check/utils.js

@@ -113,39 +113,6 @@ function printChangeSummary(fileName) {
   }
 }
 
-/**
- * Starts connection to Sauce Labs using account credentials from env vars.
- * @param {string} functionName
- */
-async function startSauceConnect(functionName) {
-  if (!process.env.SAUCE_USERNAME || !process.env.SAUCE_ACCESS_KEY) {
-    throw new Error('Missing Sauce Labs credentials');
-  }
-  const startScCmd = 'build-system/sauce_connect/start_sauce_connect.sh';
-  const fileLogPrefix = colors.bold(colors.yellow(`${functionName}:`));
-  console.log(
-    '\n' + fileLogPrefix,
-    'Starting Sauce Connect Proxy:',
-    colors.cyan(startScCmd)
-  );
-  execOrDie(startScCmd);
-}
-
-/**
- * Stops connection to Sauce Labs
- * @param {string} functionName
- */
-function stopSauceConnect(functionName) {
-  const stopScCmd = 'build-system/sauce_connect/stop_sauce_connect.sh';
-  const fileLogPrefix = colors.bold(colors.yellow(`${functionName}:`));
-  console.log(
-    '\n' + fileLogPrefix,
-    'Stopping Sauce Connect Proxy:',
-    colors.cyan(stopScCmd)
-  );
-  execOrDie(stopScCmd);
-}
-
 /**
  * Starts a timer to measure the execution time of the given function.
  * @param {string} functionName
@@ -400,8 +367,6 @@ module.exports = {
   processAndUploadDistOutput,
   startTimer,
   stopTimer,
-  startSauceConnect,
-  stopSauceConnect,
   stopTimedJob,
   timedExec,
   timedExecOrDie,

build-system/tasks/a4a.js

@@ -16,15 +16,12 @@
 'use strict';
 
 const argv = require('minimist')(process.argv.slice(2));
-const {
-  maybePrintArgvMessages,
-  shouldNotRun,
-} = require('./runtime-test/helpers');
 const {
   RuntimeTestRunner,
   RuntimeTestConfig,
 } = require('./runtime-test/runtime-test-base');
 const {css} = require('./css');
+const {maybePrintArgvMessages} = require('./runtime-test/helpers');
 
 class Runner extends RuntimeTestRunner {
   constructor(config) {
@@ -42,10 +39,6 @@ class Runner extends RuntimeTestRunner {
 }
 
 async function a4a() {
-  if (shouldNotRun()) {
-    return;
-  }
-
   maybePrintArgvMessages();
 
   const config = new RuntimeTestConfig('a4a');

build-system/tasks/integration.js

@@ -19,16 +19,13 @@ const argv = require('minimist')(process.argv.slice(2));
 const fs = require('fs-extra');
 const globby = require('globby');
 const log = require('fancy-log');
-const {
-  maybePrintArgvMessages,
-  shouldNotRun,
-} = require('./runtime-test/helpers');
 const {
   RuntimeTestRunner,
   RuntimeTestConfig,
 } = require('./runtime-test/runtime-test-base');
 const {buildRuntime} = require('../common/utils');
 const {cyan, green} = require('ansi-colors');
+const {maybePrintArgvMessages} = require('./runtime-test/helpers');
 
 class Runner extends RuntimeTestRunner {
   constructor(config) {
@@ -64,10 +61,6 @@ async function buildTransformedHtml() {
 }
 
 async function integration() {
-  if (shouldNotRun()) {
-    return;
-  }
-
   await buildTransformedHtml();
 
   maybePrintArgvMessages();
@@ -103,9 +96,6 @@ integration.flags = {
   'nobuild': '  Skips build step',
   'nohelp': '  Silence help messages that are printed prior to test run',
   'safari': '  Runs tests on Safari',
-  'saucelabs': '  Runs tests on Sauce Labs (requires setup)',
-  'stable': '  Runs Sauce Labs tests on stable browsers',
-  'beta': '  Runs Sauce Labs tests on beta browsers',
   'testnames': '  Lists the name of each test being run',
   'verbose': '  With logging enabled',
   'watch': '  Watches for changes in files, runs corresponding test(s)',