[java] Add JSpecify nullable annotations to SafariDriverService parameters (#16000)

Add JSpecify nullable annotations to SafariDriverService parameters

Co-authored-by: Diego Molina <[email protected]>

Author: iampopovich

java/src/org/openqa/selenium/safari/BUILD.bazel

@@ -14,5 +14,6 @@ java_export(
         "//java:auto-service",
         "//java/src/org/openqa/selenium:core",
         "//java/src/org/openqa/selenium/remote",
+        "@maven//:org_jspecify_jspecify",
     ],
 )

java/src/org/openqa/selenium/safari/SafariDriverService.java

@@ -32,6 +32,7 @@
 import java.util.HashMap;
 import java.util.List;
 import java.util.Map;
+import org.jspecify.annotations.Nullable;
 import org.openqa.selenium.Capabilities;
 import org.openqa.selenium.WebDriverException;
 import org.openqa.selenium.net.PortProber;
@@ -64,11 +65,11 @@ public class SafariDriverService extends DriverService {
    * @throws IOException If an I/O error occurs.
    */
   public SafariDriverService(
-      File executable,
+      @Nullable File executable,
       int port,
-      Duration timeout,
-      List<String> args,
-      Map<String, String> environment)
+      @Nullable Duration timeout,
+      @Nullable List<String> args,
+      @Nullable Map<String, String> environment)
       throws IOException {
     super(
         executable,
@@ -126,7 +127,7 @@ protected void waitUntilAvailable() {
   public static class Builder
       extends DriverService.Builder<SafariDriverService, SafariDriverService.Builder> {
 
-    private Boolean diagnose;
+    private @Nullable Boolean diagnose;
 
     @Override
     public int score(Capabilities capabilities) {
@@ -139,13 +140,13 @@ public int score(Capabilities capabilities) {
       return score;
     }
 
-    public Builder withLogging(Boolean logging) {
+    public Builder withLogging(@Nullable Boolean logging) {
       this.diagnose = logging;
       return this;
     }
 
     @Override
-    public Builder withLogFile(File logFile) {
+    public Builder withLogFile(@Nullable File logFile) {
       throw new WebDriverException(
           "Can not set log location for Safari; use withLogging(true) and locate log in"
               + " ~/Library/Logs/com.apple.WebDriver/");
@@ -169,7 +170,11 @@ protected List<String> createArgs() {
 
     @Override
     protected SafariDriverService createDriverService(
-        File exe, int port, Duration timeout, List<String> args, Map<String, String> environment) {
+        @Nullable File exe,
+        int port,
+        @Nullable Duration timeout,
+        @Nullable List<String> args,
+        @Nullable Map<String, String> environment) {
       try {
         withLogOutput(OutputStream.nullOutputStream());
         return new SafariDriverService(exe, port, timeout, args, environment);