Remove hardcoded Sauce Connect version (#33)

* Remove manual Sauce Connect version override sauce-connect-launcher now fetches the "latest" version, which should be just fine. * add changelog bit
Polymer · Jun 8, 2018 · fb2831b · fb2831b
1 parent 7ba139f
commit fb2831b
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 12 deletions.
diff --git a/packages/wct-sauce/CHANGELOG.md b/packages/wct-sauce/CHANGELOG.md
@@ -2,6 +2,8 @@
 
 ## Unreleased
 
+- Remove hardcoded Sauce Connect version
+
 ## 2.0.0-pre.4 - 2018-01-09
 
 - Removed the Linux Chrome 'dev' version browser, which is not a thing that Sauce does.

diff --git a/packages/wct-sauce/lib/sauce.js b/packages/wct-sauce/lib/sauce.js
@@ -46,8 +46,6 @@ function startTunnel(config, emitter, done) {
     emitter.emit('log:info', 'Creating Sauce Connect tunnel');
     emitter.emit('log:info', 'Sauce Connect log:', chalk.magenta(logPath));
 
-    setSauceConnectDownloadVersion();
-
     sauceConnect(connectOptions, function(error, tunnel) {
       if (error) {
         emitter.emit('log:error', 'Sauce tunnel failed:');
@@ -87,14 +85,7 @@ function isTravisSauceConnectRunning() {
   }
 }
 
-// sauce-connect-launcher only checks the environment variable
-// $SAUCE_CONNECT_VERSION
-function setSauceConnectDownloadVersion() {
-  process.env.SAUCE_CONNECT_VERSION = '4.4.0';
-}
-
 module.exports = {
   startTunnel: startTunnel,
-  isTravisSauceConnectRunning: isTravisSauceConnectRunning,
-  setSauceConnectDownloadVersion: setSauceConnectDownloadVersion
+  isTravisSauceConnectRunning: isTravisSauceConnectRunning
 };
diff --git a/packages/wct-sauce/scripts/postinstall.js b/packages/wct-sauce/scripts/postinstall.js
@@ -43,8 +43,6 @@ var sauce = require('../lib/sauce');
 if (!(sauce.isTravisSauceConnectRunning() || process.env.SKIP_WCT_SAUCE_POSTINSTALL_DOWNLOAD)) {
   console.log('Prefetching the Sauce Connect binary.');
 
-  sauce.setSauceConnectDownloadVersion();
-
   requireSauceConnectLauncher(function(sauceConnectLauncher) {
     sauceConnectLauncher.download({
       logger: console.log.bind(console),