Skip to content

Commit 23babe3

Browse files
committed
Code refactored by adding a functional interface for closing the driver
1 parent 0d6076b commit 23babe3

8 files changed

+4
-37
lines changed

src/main/java/com/maxsoft/webautomation/util/driver/ChromeDriverServiceImpl.java

-5
Original file line numberDiff line numberDiff line change
@@ -25,11 +25,6 @@ public void spinUpDriver() {
2525
.implicitlyWait(IMPLICIT_WAIT_IN_SECONDS);
2626
}
2727

28-
@Override
29-
public void closeDriver() {
30-
driver.quit();
31-
}
32-
3328
public WebDriver getDriver() {
3429
return driver;
3530
}

src/main/java/com/maxsoft/webautomation/util/driver/EdgeDriverServiceImpl.java

-5
Original file line numberDiff line numberDiff line change
@@ -25,11 +25,6 @@ public void spinUpDriver() {
2525
.implicitlyWait(IMPLICIT_WAIT_IN_SECONDS);
2626
}
2727

28-
@Override
29-
public void closeDriver() {
30-
driver.quit();
31-
}
32-
3328
public WebDriver getDriver() {
3429
return driver;
3530
}

src/main/java/com/maxsoft/webautomation/util/driver/FirefoxDriverServiceImpl.java

-5
Original file line numberDiff line numberDiff line change
@@ -25,11 +25,6 @@ public void spinUpDriver() {
2525
.implicitlyWait(IMPLICIT_WAIT_IN_SECONDS);
2626
}
2727

28-
@Override
29-
public void closeDriver() {
30-
driver.quit();
31-
}
32-
3328
public WebDriver getDriver() {
3429
return driver;
3530
}

src/main/java/com/maxsoft/webautomation/util/driver/HeadlessChromeDriverServiceImpl.java

-5
Original file line numberDiff line numberDiff line change
@@ -38,11 +38,6 @@ public void spinUpDriver() {
3838
.implicitlyWait(IMPLICIT_WAIT_IN_SECONDS);
3939
}
4040

41-
@Override
42-
public void closeDriver() {
43-
driver.quit();
44-
}
45-
4641
public WebDriver getDriver() {
4742
return driver;
4843
}

src/main/java/com/maxsoft/webautomation/util/driver/HeadlessFirefoxDriverServiceImpl.java

-5
Original file line numberDiff line numberDiff line change
@@ -32,11 +32,6 @@ public void spinUpDriver() {
3232
.implicitlyWait(IMPLICIT_WAIT_IN_SECONDS);
3333
}
3434

35-
@Override
36-
public void closeDriver() {
37-
driver.quit();
38-
}
39-
4035
public WebDriver getDriver() {
4136
return driver;
4237
}

src/main/java/com/maxsoft/webautomation/util/driver/InternetExplorerDriverServiceImpl.java

-5
Original file line numberDiff line numberDiff line change
@@ -25,11 +25,6 @@ public void spinUpDriver() {
2525
.implicitlyWait(IMPLICIT_WAIT_IN_SECONDS);
2626
}
2727

28-
@Override
29-
public void closeDriver() {
30-
driver.quit();
31-
}
32-
3328
public WebDriver getDriver() {
3429
return driver;
3530
}

src/main/java/com/maxsoft/webautomation/util/driver/SafariDriverServiceImpl.java

-5
Original file line numberDiff line numberDiff line change
@@ -25,11 +25,6 @@ public void spinUpDriver() {
2525
.implicitlyWait(IMPLICIT_WAIT_IN_SECONDS);
2626
}
2727

28-
@Override
29-
public void closeDriver() {
30-
driver.quit();
31-
}
32-
3328
public WebDriver getDriver() {
3429
return driver;
3530
}

src/main/java/com/maxsoft/webautomation/util/driver/WebDriverService.java

+4-2
Original file line numberDiff line numberDiff line change
@@ -14,7 +14,9 @@
1414
public interface WebDriverService {
1515
void spinUpDriver();
1616

17-
void closeDriver();
18-
1917
WebDriver getDriver();
18+
19+
default void closeDriver() {
20+
getDriver().quit();
21+
}
2022
}

0 commit comments

Comments
 (0)