[java] Let JUnit do its job

SeleniumHQ · Oct 23, 2018 · fb249b3 · fb249b3
1 parent 92f4ed8
commit fb249b3
Showing 1 changed file with 7 additions and 17 deletions.
diff --git a/java/client/test/org/openqa/selenium/AlertsTest.java b/java/client/test/org/openqa/selenium/AlertsTest.java
@@ -107,12 +107,9 @@ public void testShouldThrowIllegalArgumentExceptionWhenKeysNull() {
 
     driver.findElement(By.id("alert")).click();
     Alert alert = wait.until(alertIsPresent());
-    try {
-      assertThatExceptionOfType(IllegalArgumentException.class)
-          .isThrownBy(() -> alert.sendKeys(null));
-    } finally {
-      alert.accept();
-    }
+
+    assertThatExceptionOfType(IllegalArgumentException.class)
+        .isThrownBy(() -> alert.sendKeys(null));
   }
 
   @Test
@@ -138,11 +135,8 @@ public void testShouldGetTextOfAlertOpenedInSetTimeout() {
 
     driver.findElement(By.id("slow-alert")).click();
     Alert alert = wait.until(alertIsPresent());
-    try {
-      assertThat(alert.getText()).isEqualTo("Slow");
-    } finally {
-      alert.accept();
-    }
+
+    assertThat(alert.getText()).isEqualTo("Slow");
   }
 
   @Test
@@ -201,12 +195,8 @@ public void testSettingTheValueOfAnAlertThrows() {
     driver.findElement(By.id("alert")).click();
 
     Alert alert = wait.until(alertIsPresent());
-    try {
-      assertThatExceptionOfType(ElementNotInteractableException.class)
-          .isThrownBy(() -> alert.sendKeys("cheese"));
-    } finally {
-      alert.accept();
-    }
+    assertThatExceptionOfType(ElementNotInteractableException.class)
+        .isThrownBy(() -> alert.sendKeys("cheese"));
   }
 
   @Test