Fix ignore logic

cgoldberg · cgoldberg · commit bf53c3d4f7bd · 2025-10-20T08:32:23.000-04:00
diff --git a/java/test/org/openqa/selenium/testing/IgnoreComparator.java b/java/test/org/openqa/selenium/testing/IgnoreComparator.java
@@ -52,7 +52,7 @@ public boolean shouldIgnore(Stream<Ignore> ignoreList) {
     return ignoreList.anyMatch(
         driver ->
             (ignored.contains(driver.value()) || driver.value() == Browser.ALL)
-                && ((!driver.gitHubActions() || TestUtilities.isOnGitHubActions()))
+                && (!driver.gitHubActions() || TestUtilities.isOnGitHubActions())
                 && isOpen(driver.issue()));
   }
 

Original file line number	Diff line number	Diff line change
`@@ -52,7 +52,7 @@ public boolean shouldIgnore(Stream<Ignore> ignoreList) {`
`52`	`52`	`return ignoreList.anyMatch(`
`53`	`53`	`driver ->`
`54`	`54`	`(ignored.contains(driver.value()) \|\| driver.value() == Browser.ALL)`
`55`		`- && ((!driver.gitHubActions() \|\| TestUtilities.isOnGitHubActions()))`
	`55`	`+ && (!driver.gitHubActions() \|\| TestUtilities.isOnGitHubActions())`
`56`	`56`	`&& isOpen(driver.issue()));`
`57`	`57`	`}`
`58`	`58`