add try and catch with finally to handle switching the locale default

Author: MustafaAgamy

.idea/kotlinc.xml

@@ -205,14 +205,19 @@ void canExecuteCdpCommands() {
   @Test
   @NoDriverBeforeTest
   void shouldLaunchSuccessfullyWithArabicDate() {
-    Locale arabicLocale = new Locale("ar", "EG");
-    Locale.setDefault(arabicLocale);
+    try {
+        Locale arabicLocale = new Locale("ar", "EG");
+        Locale.setDefault(arabicLocale);
 
-    int port = PortProber.findFreePort();
-    ChromeDriverService.Builder builder = new ChromeDriverService.Builder();
-    builder.usingPort(port);
-    builder.build();
+        int port = PortProber.findFreePort();
+        ChromeDriverService.Builder builder = new ChromeDriverService.Builder();
+        builder.usingPort(port);
+        builder.build();
 
-    Locale.setDefault(Locale.US);
+    } catch (Exception e) {
+        throw e;
+    } finally {
+        Locale.setDefault(Locale.US);
+    }
   }
 }

.idea/misc.xml

@@ -199,14 +199,19 @@ void canExecuteCdpCommands() {
   @Test
   @NoDriverBeforeTest
   void shouldLaunchSuccessfullyWithArabicDate() {
-    Locale arabicLocale = new Locale("ar", "EG");
-    Locale.setDefault(arabicLocale);
+    try {
+        Locale arabicLocale = new Locale("ar", "EG");
+        Locale.setDefault(arabicLocale);
 
-    int port = PortProber.findFreePort();
-    EdgeDriverService.Builder builder = new EdgeDriverService.Builder();
-    builder.usingPort(port);
-    builder.build();
+        int port = PortProber.findFreePort();
+        EdgeDriverService.Builder builder = new EdgeDriverService.Builder();
+        builder.usingPort(port);
+        builder.build();
 
-    Locale.setDefault(Locale.US);
+    } catch (Exception e) {
+        throw e;
+    } finally {
+        Locale.setDefault(Locale.US);
+    }
   }
 }

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

@@ -18,7 +18,6 @@
 package org.openqa.selenium.firefox;
 
 import static org.assertj.core.api.Assertions.assertThat;
-import static org.assertj.core.api.AssertionsForClassTypes.assertThatExceptionOfType;
 import static org.assertj.core.api.AssertionsForClassTypes.assertThatThrownBy;
 import static org.junit.jupiter.api.Assertions.fail;
 import static org.junit.jupiter.api.Assumptions.assumeTrue;
@@ -274,15 +273,20 @@ void canSetContext() {
   @Test
   @NoDriverBeforeTest
   void shouldLaunchSuccessfullyWithArabicDate() {
-    Locale arabicLocale = new Locale("ar", "EG");
-    Locale.setDefault(arabicLocale);
+    try {
+        Locale arabicLocale = new Locale("ar", "EG");
+        Locale.setDefault(arabicLocale);
 
-    int port = PortProber.findFreePort();
-    GeckoDriverService.Builder builder = new GeckoDriverService.Builder();
-    builder.usingPort(port);
-    builder.build();
+        int port = PortProber.findFreePort();
+        GeckoDriverService.Builder builder = new GeckoDriverService.Builder();
+        builder.usingPort(port);
+        builder.build();
 
-    Locale.setDefault(Locale.US);
+    } catch (Exception e) {
+        throw e;
+    } finally {
+        Locale.setDefault(Locale.US);
+    }
   }
 
   private static class CustomFirefoxProfile extends FirefoxProfile {}

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

@@ -148,15 +148,20 @@ void testPersistentHoverCanBeTurnedOff() throws Exception {
   @Test
   @NoDriverBeforeTest
   void shouldLaunchSuccessfullyWithArabicDate() {
-    Locale arabicLocale = new Locale("ar", "EG");
-    Locale.setDefault(arabicLocale);
+    try {
+        Locale arabicLocale = new Locale("ar", "EG");
+        Locale.setDefault(arabicLocale);
 
-    int port = PortProber.findFreePort();
-    InternetExplorerDriverService.Builder builder = new InternetExplorerDriverService.Builder();
-    builder.usingPort(port);
-    builder.build();
+        int port = PortProber.findFreePort();
+        InternetExplorerDriverService.Builder builder = new InternetExplorerDriverService.Builder();
+        builder.usingPort(port);
+        builder.build();
 
-    Locale.setDefault(Locale.US);
+    } catch (Exception e) {
+        throw e;
+    } finally {
+        Locale.setDefault(Locale.US);
+    }
   }
 
   private WebDriver newIeDriver() {

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

@@ -138,14 +138,19 @@ public void canAttachDebugger() {
   @Test
   @NoDriverBeforeTest
   void shouldLaunchSuccessfullyWithArabicDate() {
-    Locale arabicLocale = new Locale("ar", "EG");
-    Locale.setDefault(arabicLocale);
-
-    int port = PortProber.findFreePort();
-    SafariDriverService.Builder builder = new SafariDriverService.Builder();
-    builder.usingPort(port);
-    builder.build();
-
-    Locale.setDefault(Locale.US);
+    try {
+        Locale arabicLocale = new Locale("ar", "EG");
+        Locale.setDefault(arabicLocale);
+
+        int port = PortProber.findFreePort();
+        SafariDriverService.Builder builder = new SafariDriverService.Builder();
+        builder.usingPort(port);
+        builder.build();
+
+    } catch (Exception e) {
+        throw e;
+    } finally {
+        Locale.setDefault(Locale.US);
+    }
   }
 }