Skip to content

Commit 0c0afc4

Browse files
committed
WebDriverResource: catch any WebDriverException thrown on quit
1 parent 27d9994 commit 0c0afc4

File tree

1 file changed

+12
-1
lines changed

1 file changed

+12
-1
lines changed

src/main/java/me/alb_i986/selenium/junit/rules/WebDriverResource.java

Lines changed: 12 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -4,6 +4,9 @@
44
import org.openqa.selenium.WebDriver;
55
import org.openqa.selenium.WebDriverException;
66

7+
import java.util.logging.Level;
8+
import java.util.logging.Logger;
9+
710
import me.alb_i986.selenium.WebDriverFactory;
811
import me.alb_i986.selenium.WebDriverProvider;
912

@@ -27,6 +30,8 @@
2730
*/
2831
public class WebDriverResource extends ExternalResource implements WebDriverProvider {
2932

33+
private static final Logger LOGGER = Logger.getLogger(WebDriverResource.class.getName());
34+
3035
private final WebDriverFactory driverFactory;
3136
private WebDriver driver;
3237

@@ -56,12 +61,18 @@ protected void before() throws Throwable {
5661

5762
/**
5863
* Quits the driver.
64+
* <p>
65+
* Any {@link WebDriverException} thrown is caught and logged.
5966
*
6067
* @see WebDriver#quit()
6168
*/
6269
@Override
6370
protected void after() {
64-
driver.quit();
71+
try {
72+
driver.quit();
73+
} catch (WebDriverException e) {
74+
LOGGER.log(Level.WARNING, "Quitting driver failed", e);
75+
}
6576
}
6677

6778
/**

0 commit comments

Comments
 (0)