[java] Fix test ignorance custom logic (#11007)

bonigarcia · web-flow · commit 9850c95bee3e · 2022-09-06T19:42:18.000-05:00
diff --git a/java/test/org/openqa/selenium/CookieImplementationTest.java b/java/test/org/openqa/selenium/CookieImplementationTest.java
@@ -203,10 +203,7 @@ public void testAddCookiesWithDifferentPathsThatAreRelatedToOurs() {
 
   @SwitchToTopAfterTest
   @Test
-  @NotYetImplemented(value = CHROME, reason = "https://bugs.chromium.org/p/chromedriver/issues/detail?id=3153")
-  @NotYetImplemented(value = EDGE, reason = "https://bugs.chromium.org/p/chromedriver/issues/detail?id=3153")
   @Ignore(SAFARI)
-  @NotYetImplemented(value = FIREFOX, reason = "https://github.com/mozilla/geckodriver/issues/1104")
   public void testGetCookiesInAFrame() {
     driver.get(domainHelper.getUrlForFirstValidHostname("/common/animals"));
     Cookie cookie1 = new Cookie.Builder("fish", "cod").path("/common/animals").build();
@@ -424,7 +421,6 @@ public void testRetainsCookieSecure() {
 
   @Test
   @Ignore(SAFARI)
-  @NotYetImplemented(CHROME)
   public void canHandleHttpOnlyCookie() {
     Cookie addedCookie =
       new Cookie.Builder("fish", "cod")
diff --git a/java/test/org/openqa/selenium/testing/IgnoreComparator.java b/java/test/org/openqa/selenium/testing/IgnoreComparator.java
@@ -45,7 +45,7 @@ public boolean shouldIgnore(Ignore ignore) {
     return ignore != null && shouldIgnore(Stream.of(ignore));
   }
 
-  private boolean shouldIgnore(Stream<Ignore> ignoreList) {
+  public boolean shouldIgnore(Stream<Ignore> ignoreList) {
     return ignoreList.anyMatch(
       driver -> (ignored.contains(driver.value()) || driver.value() == Browser.ALL)
                 && ((!driver.travis() || TestUtilities.isOnTravis())
diff --git a/java/test/org/openqa/selenium/testing/TestIgnorance.java b/java/test/org/openqa/selenium/testing/TestIgnorance.java
@@ -25,6 +25,7 @@
 import java.lang.reflect.Method;
 import java.util.Arrays;
 import java.util.HashSet;
+import java.util.List;
 import java.util.Optional;
 import java.util.Set;
 
@@ -72,10 +73,14 @@ public boolean isIgnored(ExtensionContext extensionContext) {
     Optional<Method> testMethod = extensionContext.getTestMethod();
 
     // Ignored because of Selenium's custom extensions
-    boolean ignored = findAnnotation(testClass, IgnoreList.class).isPresent() ||
-      !findRepeatableAnnotations(testClass, Ignore.class).isEmpty() ||
-      findAnnotation(testMethod, IgnoreList.class).isPresent() ||
-      !findRepeatableAnnotations(testMethod, Ignore.class).isEmpty();
+    Optional<IgnoreList> ignoreListClass = findAnnotation(testClass, IgnoreList.class);
+    List<Ignore> ignoreClass = findRepeatableAnnotations(testClass, Ignore.class);
+    Optional<IgnoreList> ignoreListMethod = findAnnotation(testMethod, IgnoreList.class);
+    List<Ignore> ignoreMethod = findRepeatableAnnotations(testMethod, Ignore.class);
+    boolean ignored = (ignoreListClass.isPresent() && ignoreComparator.shouldIgnore(ignoreListClass.get())) ||
+      (!ignoreClass.isEmpty() && ignoreComparator.shouldIgnore(ignoreClass.stream())) ||
+      (ignoreListMethod.isPresent() && ignoreComparator.shouldIgnore(ignoreListMethod.get())) ||
+      (!ignoreMethod.isEmpty() && ignoreComparator.shouldIgnore(ignoreMethod.stream()));
 
     // Ignored because of Jupiter's @Disabled
     ignored |= findAnnotation(testClass, Disabled.class).isPresent();

Original file line number	Diff line number	Diff line change
`@@ -45,7 +45,7 @@ public boolean shouldIgnore(Ignore ignore) {`
`45`	`45`	`return ignore != null && shouldIgnore(Stream.of(ignore));`
`46`	`46`	`}`
`47`	`47`
`48`		`- private boolean shouldIgnore(Stream<Ignore> ignoreList) {`
	`48`	`+ public boolean shouldIgnore(Stream<Ignore> ignoreList) {`
`49`	`49`	`return ignoreList.anyMatch(`
`50`	`50`	`driver -> (ignored.contains(driver.value()) \|\| driver.value() == Browser.ALL)`
`51`	`51`	`&& ((!driver.travis() \|\| TestUtilities.isOnTravis())`