Add Formatting with Locale.ROOT for the port at :

- ChromeDriverService
- EdgeDriverService
- GeckoDriverService
- InternetExplorerDriverService

and to the getUrl method at:
- DriverService

Refactor the Arabic Tests in the following accordingly:
- ChromeDriverFunctionalTest
- EdgeDriverFunctionalTest
- FirefoxDriverTest
- InternetExplorerDriverTest
- SafariDriverTest

Author: MustafaAgamy

java/src/org/openqa/selenium/chrome/ChromeDriverService.java

@@ -284,7 +284,7 @@ protected void loadSystemProperties() {
     @Override
     protected List<String> createArgs() {
       List<String> args = new ArrayList<>();
-      args.add(String.format("--port=%d", getPort()));
+      args.add(String.format(Locale.ROOT, "--port=%d", getPort()));
 
       // Readable timestamp and append logs only work if log path is specified in args
       // Cannot use logOutput because goog:loggingPrefs requires --log-path get sent

java/src/org/openqa/selenium/edge/EdgeDriverService.java

@@ -278,7 +278,7 @@ protected void loadSystemProperties() {
     @Override
     protected List<String> createArgs() {
       List<String> args = new ArrayList<>();
-      args.add(String.format("--port=%d", getPort()));
+      args.add(String.format(Locale.ROOT, "--port=%d", getPort()));
 
       // Readable timestamp and append logs only work if log path is specified in args
       // Cannot use logOutput because goog:loggingPrefs requires --log-path get sent

java/src/org/openqa/selenium/firefox/GeckoDriverService.java

@@ -30,6 +30,7 @@
 import java.util.Arrays;
 import java.util.HashMap;
 import java.util.List;
+import java.util.Locale;
 import java.util.Map;
 import org.openqa.selenium.Capabilities;
 import org.openqa.selenium.WebDriverException;
@@ -219,7 +220,7 @@ protected void loadSystemProperties() {
     @Override
     protected List<String> createArgs() {
       List<String> args = new ArrayList<>();
-      args.add(String.format("--port=%d", getPort()));
+      args.add(String.format(Locale.ROOT, "--port=%d", getPort()));
 
       int wsPort = PortProber.findFreePort();
       args.add(String.format("--websocket-port=%d", wsPort));

java/src/org/openqa/selenium/ie/InternetExplorerDriverService.java

@@ -28,6 +28,7 @@
 import java.util.ArrayList;
 import java.util.HashMap;
 import java.util.List;
+import java.util.Locale;
 import java.util.Map;
 import org.openqa.selenium.Capabilities;
 import org.openqa.selenium.WebDriverException;
@@ -217,7 +218,7 @@ protected void loadSystemProperties() {
     @Override
     protected List<String> createArgs() {
       List<String> args = new ArrayList<>();
-      args.add(String.format("--port=%d", getPort()));
+      args.add(String.format(Locale.ROOT, "--port=%d", getPort()));
 
       if (logLevel != null) {
         args.add(String.format("--log-level=%s", logLevel));

java/src/org/openqa/selenium/remote/service/DriverService.java

@@ -138,7 +138,7 @@ protected Map<String, String> getEnvironment() {
   }
 
   protected URL getUrl(int port) throws IOException {
-    return new URL(String.format("http://localhost:%d", port));
+    return new URL(String.format(Locale.ROOT,"http://localhost:%d", port));
   }
 
   protected Capabilities getDefaultDriverOptions() {
@@ -505,17 +505,6 @@ public DS build() {
         port = PortProber.findFreePort();
       }
 
-      if (Locale.getDefault(Locale.Category.FORMAT).getLanguage().equals("ar")) {
-        throw new NumberFormatException(
-            String.format(
-                "Couldn't format the port numbers because the System Language is arabic: \""
-                    + String.format("--port=%d", port)
-                    + "\", please make sure to add the required arguments \"-Duser.language=en"
-                    + " -Duser.region=US\" to your JVM, for more info please visit :\n"
-                    + "  https://www.selenium.dev/documentation/webdriver/browsers/",
-                getPort()));
-      }
-
       if (timeout == null) {
         timeout = getDefaultTimeout();
       }

java/test/org/openqa/selenium/chrome/ChromeDriverFunctionalTest.java

@@ -204,22 +204,14 @@ void canExecuteCdpCommands() {
 
   @Test
   @NoDriverBeforeTest
-  void shouldThrowNumberFormatException() {
+  void shouldLaunchSuccessfullyWithArabicDate() {
     Locale arabicLocale = new Locale("ar", "EG");
     Locale.setDefault(arabicLocale);
 
     int port = PortProber.findFreePort();
     ChromeDriverService.Builder builder = new ChromeDriverService.Builder();
     builder.usingPort(port);
-
-    assertThatExceptionOfType(NumberFormatException.class)
-        .isThrownBy(builder::build)
-        .withMessage(
-            "Couldn't format the port numbers because the System Language is arabic: \""
-                + String.format("--port=%d", port)
-                + "\", please make sure to add the required arguments \"-Duser.language=en"
-                + " -Duser.region=US\" to your JVM, for more info please visit :\n"
-                + "  https://www.selenium.dev/documentation/webdriver/browsers/");
+    builder.build();
 
     Locale.setDefault(Locale.US);
   }

java/test/org/openqa/selenium/edge/EdgeDriverFunctionalTest.java

@@ -198,22 +198,14 @@ void canExecuteCdpCommands() {
 
   @Test
   @NoDriverBeforeTest
-  void shouldThrowNumberFormatException() {
+  void shouldLaunchSuccessfullyWithArabicDate() {
     Locale arabicLocale = new Locale("ar", "EG");
     Locale.setDefault(arabicLocale);
 
     int port = PortProber.findFreePort();
     EdgeDriverService.Builder builder = new EdgeDriverService.Builder();
     builder.usingPort(port);
-
-    assertThatExceptionOfType(NumberFormatException.class)
-        .isThrownBy(builder::build)
-        .withMessage(
-            "Couldn't format the port numbers because the System Language is arabic: \""
-                + String.format("--port=%d", port)
-                + "\", please make sure to add the required arguments \"-Duser.language=en"
-                + " -Duser.region=US\" to your JVM, for more info please visit :\n"
-                + "  https://www.selenium.dev/documentation/webdriver/browsers/");
+    builder.build();
 
     Locale.setDefault(Locale.US);
   }

java/test/org/openqa/selenium/firefox/FirefoxDriverTest.java

@@ -273,22 +273,14 @@ void canSetContext() {
 
   @Test
   @NoDriverBeforeTest
-  void shouldThrowNumberFormatException() {
+  void shouldLaunchSuccessfullyWithArabicDate() {
     Locale arabicLocale = new Locale("ar", "EG");
     Locale.setDefault(arabicLocale);
 
     int port = PortProber.findFreePort();
     GeckoDriverService.Builder builder = new GeckoDriverService.Builder();
     builder.usingPort(port);
-
-    assertThatExceptionOfType(NumberFormatException.class)
-        .isThrownBy(builder::build)
-        .withMessage(
-            "Couldn't format the port numbers because the System Language is arabic: \""
-                + String.format("--port=%d", port)
-                + "\", please make sure to add the required arguments \"-Duser.language=en"
-                + " -Duser.region=US\" to your JVM, for more info please visit :\n"
-                + "  https://www.selenium.dev/documentation/webdriver/browsers/");
+    builder.build();
 
     Locale.setDefault(Locale.US);
   }

java/test/org/openqa/selenium/ie/InternetExplorerDriverTest.java

@@ -147,22 +147,14 @@ void testPersistentHoverCanBeTurnedOff() throws Exception {
 
   @Test
   @NoDriverBeforeTest
-  void shouldThrowNumberFormatException() {
+  void shouldLaunchSuccessfullyWithArabicDate() {
     Locale arabicLocale = new Locale("ar", "EG");
     Locale.setDefault(arabicLocale);
 
     int port = PortProber.findFreePort();
     InternetExplorerDriverService.Builder builder = new InternetExplorerDriverService.Builder();
     builder.usingPort(port);
-
-    assertThatExceptionOfType(NumberFormatException.class)
-        .isThrownBy(builder::build)
-        .withMessage(
-            "Couldn't format the port numbers because the System Language is arabic: \""
-                + String.format("--port=%d", port)
-                + "\", please make sure to add the required arguments \"-Duser.language=en"
-                + " -Duser.region=US\" to your JVM, for more info please visit :\n"
-                + "  https://www.selenium.dev/documentation/webdriver/browsers/");
+    builder.build();
 
     Locale.setDefault(Locale.US);
   }

java/test/org/openqa/selenium/safari/SafariDriverTest.java

@@ -137,22 +137,14 @@ public void canAttachDebugger() {
 
   @Test
   @NoDriverBeforeTest
-  void shouldThrowNumberFormatException() {
+  void shouldLaunchSuccessfullyWithArabicDate() {
     Locale arabicLocale = new Locale("ar", "EG");
     Locale.setDefault(arabicLocale);
 
     int port = PortProber.findFreePort();
     SafariDriverService.Builder builder = new SafariDriverService.Builder();
     builder.usingPort(port);
-
-    assertThatExceptionOfType(NumberFormatException.class)
-        .isThrownBy(builder::build)
-        .withMessage(
-            "Couldn't format the port numbers because the System Language is arabic: \""
-                + String.format("--port=%d", port)
-                + "\", please make sure to add the required arguments \"-Duser.language=en"
-                + " -Duser.region=US\" to your JVM, for more info please visit :\n"
-                + "  https://www.selenium.dev/documentation/webdriver/browsers/");
+    builder.build();
 
     Locale.setDefault(Locale.US);
   }