[Java] Migrate the String.Format to String.Format(Locale.US) to handle different langauges for Edge, Chrome and Firefox (#13934)

* *Changes Done:
- Migrate the String.Format to Formatter.Format in :
- ChromeDriverSerivce.java
- EdgeDriverService.java
- RemoteWebDriver.java

*Issue Explaination:
- Having System Date & Language in Arabic caused with driver setup.
Using String.Format for ports and some other lines caused them to be localized in the system language and in arabic case it caused the numbers and some other character to be localized in arabic,
e.g. the port. And that caused an issues when appending the port to the URL in order to setup the browser

*Fix explaination:
- Now it's utilizaing the Formatter.Format and forcing the localization to be in English.US in the Formatter Constructor

* - Apply Suggested Code Enhancements from "codiumai-pr-agent-pro" to the classes changed:

- ChromeDriverSerivce.java
- EdgeDriverService.java
- RemoteWebDriver.java

*Changes Done:
- Add try with resoruces to handle memory leaks where it's applicables
- Changing String.ValueOf to ToString()

* * Refactor The Formatting Logic for better optimzation in classes:
- ChromeDriverService.java
- EdgeDriverService.java
- RemoteWebDriver.java
- DriverService.java
* Add Tests for both Edge and Chrome:
- ChromeArabicDateTest.java at test/chrome
- EdgeArabicDateTest.java at test/edge

* - Added Logic for formatting the required configurations in GekoDriverService.java
- Added Test Class for FirefoxArabicDateTest.java
-Refactored the code of EdgeArabicDateTest.java And ChromeArabicDateTest.java to follow the same approach

* Refactor the Formatter to String.format with Localization in
- ChromeDriverService.java
- EdgeDriverService.java
- GeckoDriverService.java
- RemoteWebDriver.java
- DriverService.java

* Fix spacing

* Refactor FirefoxArabicDateTest :
- Test added to LARGE_TESTS in BUILD.bazel
- Refacotor the Test Logic

* Fix spacing in FirefoxArabicDateTest.java

* Tests Refactoring :
- EdgeArabicDateTest.java
- ChomeArabicDateTest.java
- FirefoxArabicDateTest.java

* [java] Fix formatting

* Revert The Changes done to:

- DriverService.java
- RemoteWebDriver.java

Change the logic to Locale.setDefault in :
- ChomeDriverSerivce.java
- EdgeDriverService.java
- GeckDriverService.java

* Fix Spaces

* Fix Formatting

* - Delete the Added Arabic Test Classes
- Methods are added to the Functional Test Class for each driver accordingly

* - Reverse changes made to class files

* Fixing the Tests

---------

Co-authored-by: Puja Jagani <[email protected]>

Author: MustafaAgamy

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

@@ -27,6 +27,7 @@
 import com.google.common.util.concurrent.Uninterruptibles;
 import java.time.Duration;
 import java.util.List;
+import java.util.Locale;
 import java.util.Map;
 import org.assertj.core.api.Assumptions;
 import org.junit.jupiter.api.Test;
@@ -40,6 +41,7 @@
 import org.openqa.selenium.chromium.HasCdp;
 import org.openqa.selenium.chromium.HasNetworkConditions;
 import org.openqa.selenium.chromium.HasPermissions;
+import org.openqa.selenium.net.PortProber;
 import org.openqa.selenium.remote.RemoteWebDriverBuilder;
 import org.openqa.selenium.remote.http.ClientConfig;
 import org.openqa.selenium.testing.Ignore;
@@ -200,4 +202,21 @@ void canExecuteCdpCommands() {
 
     assertThat(driver.getTitle()).isEqualTo("Hello WebDriver");
   }
+
+  @Test
+  @NoDriverBeforeTest
+  void shouldLaunchSuccessfullyWithArabicDate() {
+    Locale arabicLocale = new Locale("ar", "EG");
+    Locale.setDefault(arabicLocale);
+    Locale.setDefault(Locale.US);
+
+    int port = PortProber.findFreePort();
+    ChromeDriverService.Builder builder = new ChromeDriverService.Builder();
+    builder.usingPort(port);
+    ChromeDriverService service = builder.build();
+
+    driver = new ChromeDriver(service, (ChromeOptions) CHROME.getCapabilities());
+    driver.get(pages.simpleTestPage);
+    assertThat(driver.getTitle()).isEqualTo("Hello WebDriver");
+  }
 }

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

@@ -26,6 +26,7 @@
 
 import java.time.Duration;
 import java.util.List;
+import java.util.Locale;
 import java.util.Map;
 import org.junit.jupiter.api.Test;
 import org.openqa.selenium.Capabilities;
@@ -38,6 +39,7 @@
 import org.openqa.selenium.chromium.HasCdp;
 import org.openqa.selenium.chromium.HasNetworkConditions;
 import org.openqa.selenium.chromium.HasPermissions;
+import org.openqa.selenium.net.PortProber;
 import org.openqa.selenium.remote.RemoteWebDriverBuilder;
 import org.openqa.selenium.remote.http.ClientConfig;
 import org.openqa.selenium.testing.Ignore;
@@ -193,4 +195,22 @@ void canExecuteCdpCommands() {
 
     assertThat(driver.getTitle()).isEqualTo("Hello WebDriver");
   }
+
+  @Test
+  @NoDriverBeforeTest
+  void shouldLaunchSuccessfullyWithArabicDate() {
+    Locale arabicLocale = new Locale("ar", "EG");
+    Locale.setDefault(arabicLocale);
+    Locale.setDefault(Locale.US);
+
+    int port = PortProber.findFreePort();
+    EdgeDriverService.Builder builder = new EdgeDriverService.Builder();
+    builder.usingPort(port);
+    EdgeDriverService service = builder.build();
+
+    driver = new EdgeDriver(service, (EdgeOptions) EDGE.getCapabilities());
+
+    driver.get(pages.simpleTestPage);
+    assertThat(driver.getTitle()).isEqualTo("Hello WebDriver");
+  }
 }

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

@@ -37,6 +37,7 @@
 import java.io.IOException;
 import java.lang.reflect.Field;
 import java.time.Duration;
+import java.util.Locale;
 import org.junit.jupiter.api.Test;
 import org.junit.jupiter.api.Timeout;
 import org.mockito.ArgumentMatchers;
@@ -50,6 +51,7 @@
 import org.openqa.selenium.PageLoadStrategy;
 import org.openqa.selenium.SessionNotCreatedException;
 import org.openqa.selenium.WebElement;
+import org.openqa.selenium.net.PortProber;
 import org.openqa.selenium.remote.CapabilityType;
 import org.openqa.selenium.remote.Command;
 import org.openqa.selenium.remote.CommandExecutor;
@@ -268,5 +270,22 @@ void canSetContext() {
     assertThat(context.getContext()).isEqualTo(FirefoxCommandContext.CHROME);
   }
 
+  @Test
+  @NoDriverBeforeTest
+  void shouldLaunchSuccessfullyWithArabicDate() {
+    Locale arabicLocale = new Locale("ar", "EG");
+    Locale.setDefault(arabicLocale);
+    Locale.setDefault(Locale.US);
+
+    int port = PortProber.findFreePort();
+    GeckoDriverService.Builder builder = new GeckoDriverService.Builder();
+    builder.usingPort(port);
+    GeckoDriverService service = builder.build();
+
+    driver = new FirefoxDriver(service, (FirefoxOptions) FIREFOX.getCapabilities());
+    driver.get(pages.simpleTestPage);
+    assertThat(driver.getTitle()).isEqualTo("Hello WebDriver");
+  }
+
   private static class CustomFirefoxProfile extends FirefoxProfile {}
 }