remove legacy SafariDriver references in lib/test (#2839)

SeleniumHQ · Sep 30, 2016 · 36d5777 · 36d5777
1 parent e97639b
commit 36d5777
Showing 1 changed file with 2 additions and 18 deletions.
diff --git a/javascript/node/selenium-webdriver/lib/test/index.js b/javascript/node/selenium-webdriver/lib/test/index.js
@@ -31,7 +31,6 @@ var build = require('./build'),
 
 
 const LEGACY_FIREFOX = 'legacy-' + webdriver.Browser.FIREFOX;
-const LEGACY_SAFARI = 'legacy-' + webdriver.Browser.SAFARI;
 
 
 /**
@@ -46,8 +45,7 @@ var NATIVE_BROWSERS = [
   webdriver.Browser.IE,
   webdriver.Browser.OPERA,
   webdriver.Browser.PHANTOM_JS,
-  webdriver.Browser.SAFARI,
-  LEGACY_SAFARI
+  webdriver.Browser.SAFARI
 ];
 
 
@@ -83,8 +81,7 @@ var browsersToTest = (function() {
       parts[0] = webdriver.Browser.IE;
     }
 
-    if (parts[0] === LEGACY_FIREFOX ||
-      parts[0] === LEGACY_SAFARI) {
+    if (parts[0] === LEGACY_FIREFOX) {
       return;
     }
 
@@ -175,14 +172,6 @@ function TestEnvironment(browserName, server) {
         parts[0] = webdriver.Browser.FIREFOX;
       }
 
-      if (parts[0] === LEGACY_SAFARI) {
-        var options = builder.getSafariOptions() || new safari.Options();
-        options.useLegacyDriver(true);
-        builder.setSafariOptions(options);
-
-        parts[0] = webdriver.Browser.SAFARI;
-      }
-
       builder.forBrowser(parts[0], parts[1], parts[2]);
       if (server) {
         builder.usingServer(server.address());
@@ -243,11 +232,6 @@ function suite(fn, opt_options) {
               return build.of('//javascript/firefox-driver:webdriver')
                   .onlyOnce().go();
             });
-          } else if (browser === LEGACY_SAFARI) {
-            testing.before(function() {
-              return build.of('//javascript/safari-driver:client')
-                  .onlyOnce().go();
-            });
           }
         }