[java] Fixing method in SafariDriver

diemol · diemol · commit 11f7dde16635 · 2022-02-04T18:05:58.000+01:00
I missed to push this diff :facepalm
diff --git a/java/src/org/openqa/selenium/safari/SafariDriver.java b/java/src/org/openqa/selenium/safari/SafariDriver.java
@@ -18,6 +18,7 @@
 package org.openqa.selenium.safari;
 
 import com.google.common.collect.ImmutableMap;
+
 import org.openqa.selenium.Beta;
 import org.openqa.selenium.Capabilities;
 import org.openqa.selenium.WebDriverException;
@@ -68,7 +69,7 @@ public SafariDriver(Capabilities desiredCapabilities) {
    * @param safariOptions safari specific options / capabilities for the driver
    */
   public SafariDriver(SafariOptions safariOptions) {
-    this(SafariDriverService.createDefaultService(safariOptions), safariOptions);
+    this(SafariDriverService.createDefaultService(), safariOptions);
   }
 
   /**
@@ -91,6 +92,11 @@ public SafariDriver(SafariDriverService safariServer, SafariOptions safariOption
     debugger = new AddHasDebugger().getImplementation(getCapabilities(), getExecuteMethod());
   }
 
+  @Beta
+  public static RemoteWebDriverBuilder builder() {
+    return RemoteWebDriver.builder().oneOf(new SafariOptions());
+  }
+
   @Override
   public void setPermissions(String permission, boolean value) {
     Require.nonNull("Permission Name", permission);
@@ -109,6 +115,13 @@ public void attachDebugger() {
     debugger.attachDebugger();
   }
 
+  @Override
+  public void setFileDetector(FileDetector detector) {
+    throw new WebDriverException(
+        "Setting the file detector only works on remote webdriver instances obtained " +
+        "via RemoteWebDriver");
+  }
+
   private static class SafariDriverCommandExecutor extends DriverCommandExecutor {
     public SafariDriverCommandExecutor(DriverService service) {
       super(service, getExtraCommands());
@@ -121,16 +134,4 @@ private static Map<String, CommandInfo> getExtraCommands() {
         .build();
     }
   }
-
-  @Override
-  public void setFileDetector(FileDetector detector) {
-    throw new WebDriverException(
-        "Setting the file detector only works on remote webdriver instances obtained " +
-        "via RemoteWebDriver");
-  }
-
-  @Beta
-  public static RemoteWebDriverBuilder builder() {
-    return RemoteWebDriver.builder().oneOf(new SafariOptions());
-  }
 }
