Fixing some small codacy issues

Author: travis-ci

src/main/java/uk/co/evoco/webdriver/WebDriverListener.java

@@ -1,7 +1,11 @@
 package uk.co.evoco.webdriver;
 
 import org.apache.commons.io.FileUtils;
-import org.openqa.selenium.*;
+import org.openqa.selenium.By;
+import org.openqa.selenium.TakesScreenshot;
+import org.openqa.selenium.OutputType;
+import org.openqa.selenium.WebDriver;
+import org.openqa.selenium.WebElement;
 import org.openqa.selenium.support.events.WebDriverEventListener;
 import org.openqa.selenium.support.ui.ExpectedConditions;
 import org.openqa.selenium.support.ui.WebDriverWait;

src/main/java/uk/co/evoco/webdriver/configuration/TolerantActionExceptions.java

@@ -13,7 +13,7 @@ public class TolerantActionExceptions {
      *
      * @return the tolerant action wait time in seconds
      */
-    String getWaitTimeoutInSeconds() {
+    public String getWaitTimeoutInSeconds() {
         return waitTimeoutInSeconds;
     }
 

src/main/java/uk/co/evoco/webdriver/configuration/driver/ConfiguredFirefoxDriver.java

@@ -3,6 +3,8 @@
 import com.fasterxml.jackson.databind.JsonNode;
 import io.github.bonigarcia.wdm.WebDriverManager;
 import org.openqa.selenium.WebDriver;
+import org.openqa.selenium.chrome.ChromeDriver;
+import org.openqa.selenium.chrome.ChromeOptions;
 import org.openqa.selenium.firefox.FirefoxDriver;
 import org.openqa.selenium.firefox.FirefoxOptions;
 import org.openqa.selenium.remote.RemoteWebDriver;
@@ -59,6 +61,7 @@ public FirefoxOptions getOptions() {
                     break;
                 default:
                     firefoxOptions.addPreference(key, value.asText());
+                    break;
             }
         }
 

src/main/java/uk/co/evoco/webdriver/utils/TolerantInteraction.java

@@ -181,6 +181,8 @@ private void interact(Select selectBox, String value, SelectBoxInteractionType s
             case BY_VISIBLE_TEXT:
                 selectBox.selectByVisibleText(value);
                 break;
+            default:
+                throw new WebDriverException("Must be one of BY_VALUE or BY_VISIBLE_TEXT");
         }
     }
 

src/test/java/uk/co/evoco/testdata/DatesTests.java

@@ -3,8 +3,6 @@
 import com.fasterxml.jackson.core.JsonProcessingException;
 import org.junit.jupiter.api.Test;
 
-import java.util.Date;
-
 import static org.hamcrest.CoreMatchers.is;
 import static org.hamcrest.MatcherAssert.assertThat;
 import static uk.co.evoco.testdata.Dates.*;

src/test/java/uk/co/evoco/webdriver/utils/WebDriverUtilsIT.java

@@ -5,7 +5,6 @@
 import org.junit.jupiter.api.BeforeEach;
 import org.junit.jupiter.api.Test;
 import org.openqa.selenium.By;
-import org.openqa.selenium.StaleElementReferenceException;
 import org.openqa.selenium.TimeoutException;
 import org.openqa.selenium.WebElement;
 import uk.co.evoco.tests.BaseAbstractTest;