[Java] PR Linting (#70)

Co-authored-by: Logan Graham <logan.graham@saucelabs.com>

.github/workflows/java-build.yml

@@ -30,8 +30,9 @@ jobs:
           distribution: 'temurin'
           java-version: ${{ matrix.version }}
           cache: 'maven'
-      - name: Format / Lint Check
-        run: mvn fmt:check --file pom.xml
-        if: matrix.version == '11'
       - name: Build with Maven
         run: mvn -B package --file pom.xml
+        if: matrix.version == '8'
+      - name: Build and Lint with Maven
+        run: mvn -B verify --file pom.xml
+        if: matrix.version == '11'

visual-java/pom.xml

@@ -6,7 +6,7 @@
 
   <groupId>com.saucelabs.visual</groupId>
   <artifactId>java-client</artifactId>
-  <version>0.9.0</version>
+  <version>0.7.0</version>
   <name>visual-java-client</name>
   <description>Java library to interact with Sauce Visual</description>
 

visual-java/src/main/java/com/saucelabs/visual/VisualApi.java

@@ -521,7 +521,7 @@ private List<ElementIn> extractIgnoreElements(CheckOptions options) {
     List<ElementIn> result = new ArrayList<>();
     for (int i = 0; i < ignoredElements.size(); i++) {
       WebElement element = ignoredElements.get(i);
-      if (validateRemoteElement(element) == null) {
+      if (validate(element) == null) {
         throw new VisualApiException("options.ignoreElement[" + i + "] does not exist (yet)");
       }
       result.add(VisualRegion.ignoreChangesFor(element).toElementIn());
@@ -530,14 +530,7 @@ private List<ElementIn> extractIgnoreElements(CheckOptions options) {
   }
 
   private WebElement validate(WebElement element) {
-    if (element == null || !element.isDisplayed() || element.getRect() == null) {
-      return null;
-    }
-    return element;
-  }
-
-  private WebElement validateRemoteElement(WebElement element) {
-    if (element != null && element instanceof RemoteWebElement && element.isDisplayed()) {
+    if (element instanceof RemoteWebElement && element.isDisplayed()) {
       return element;
     }
     return null;