[java] enable safari for CookieImplementationTest (#15544)

* [java] enable safari for CookieImplementationTest

Author: Delta456

java/test/org/openqa/selenium/CookieImplementationTest.java

@@ -38,7 +38,6 @@
 import org.openqa.selenium.testing.JupiterTestBase;
 import org.openqa.selenium.testing.NeedsSecureServer;
 import org.openqa.selenium.testing.NotWorkingInRemoteBazelBuilds;
-import org.openqa.selenium.testing.NotYetImplemented;
 import org.openqa.selenium.testing.SwitchToTopAfterTest;
 
 @NeedsSecureServer
@@ -74,7 +73,6 @@ public void setUp() {
   }
 
   @Test
-  @NotYetImplemented(SAFARI)
   public void testShouldGetCookieByName() {
     String key = generateUniqueKey();
     String value = "set";
@@ -87,7 +85,6 @@ public void testShouldGetCookieByName() {
   }
 
   @Test
-  @NotYetImplemented(SAFARI)
   @NotWorkingInRemoteBazelBuilds(FIREFOX)
   public void testShouldBeAbleToAddCookie() {
     String key = generateUniqueKey();
@@ -104,7 +101,6 @@ public void testShouldBeAbleToAddCookie() {
   }
 
   @Test
-  @NotYetImplemented(SAFARI)
   public void testGetAllCookies() {
     String key1 = generateUniqueKey();
     String key2 = generateUniqueKey();
@@ -130,7 +126,6 @@ public void testGetAllCookies() {
   }
 
   @Test
-  @NotYetImplemented(SAFARI)
   public void testDeleteAllCookies() {
     addCookieOnServerSide(new Cookie("foo", "set"));
     assertSomeCookiesArePresent();
@@ -144,7 +139,6 @@ public void testDeleteAllCookies() {
   }
 
   @Test
-  @NotYetImplemented(SAFARI)
   public void testDeleteCookieWithName() {
     String key1 = generateUniqueKey();
     String key2 = generateUniqueKey();
@@ -166,7 +160,6 @@ public void testDeleteCookieWithName() {
   }
 
   @Test
-  @NotYetImplemented(SAFARI)
   public void testShouldNotDeleteCookiesWithASimilarName() {
     String cookieOneName = "fish";
     Cookie cookie1 = new Cookie.Builder(cookieOneName, "cod").build();
@@ -187,7 +180,6 @@ public void testShouldNotDeleteCookiesWithASimilarName() {
   }
 
   @Test
-  @NotYetImplemented(SAFARI)
   public void testAddCookiesWithDifferentPathsThatAreRelatedToOurs() {
     driver.get(domainHelper.getUrlForFirstValidHostname("/common/animals"));
     Cookie cookie1 = new Cookie.Builder("fish", "cod").path("/common/animals").build();
@@ -224,7 +216,6 @@ public void testGetCookiesInAFrame() {
   }
 
   @Test
-  @NotYetImplemented(SAFARI)
   public void testCannotGetCookiesWithPathDifferingOnlyInCase() {
     String cookieName = "fish";
     Cookie cookie = new Cookie.Builder(cookieName, "cod").path("/Common/animals").build();
@@ -235,7 +226,6 @@ public void testCannotGetCookiesWithPathDifferingOnlyInCase() {
   }
 
   @Test
-  @NotYetImplemented(SAFARI)
   public void testShouldNotGetCookieOnDifferentDomain() {
     assumeTrue(domainHelper.checkHasValidAlternateHostname());
 
@@ -249,7 +239,6 @@ public void testShouldNotGetCookieOnDifferentDomain() {
   }
 
   @Test
-  @NotYetImplemented(SAFARI)
   @NotWorkingInRemoteBazelBuilds(FIREFOX)
   public void testShouldBeAbleToAddToADomainWhichIsRelatedToTheCurrentDomain() {
     String cookieName = "name";
@@ -289,7 +278,6 @@ void testShouldBeAbleToIncludeLeadingPeriodInDomainName() {
   }
 
   @Test
-  @NotYetImplemented(SAFARI)
   @NotWorkingInRemoteBazelBuilds(FIREFOX)
   public void testShouldBeAbleToSetDomainToTheCurrentDomain() throws Exception {
     URI url = new URI(driver.getCurrentUrl());
@@ -304,7 +292,6 @@ public void testShouldBeAbleToSetDomainToTheCurrentDomain() throws Exception {
   }
 
   @Test
-  @NotYetImplemented(SAFARI)
   @NotWorkingInRemoteBazelBuilds(CHROME)
   @NotWorkingInRemoteBazelBuilds(EDGE)
   @NotWorkingInRemoteBazelBuilds(FIREFOX)
@@ -336,7 +323,6 @@ public void testShouldWalkThePathToDeleteACookie() {
   }
 
   @Test
-  @NotYetImplemented(SAFARI)
   @NotWorkingInRemoteBazelBuilds(FIREFOX)
   public void testShouldIgnoreThePortNumberOfTheHostWhenSettingTheCookie() throws Exception {
     URI uri = new URI(driver.getCurrentUrl());
@@ -352,7 +338,6 @@ public void testShouldIgnoreThePortNumberOfTheHostWhenSettingTheCookie() throws
   }
 
   @Test
-  @NotYetImplemented(SAFARI)
   @NotWorkingInRemoteBazelBuilds(CHROME)
   @NotWorkingInRemoteBazelBuilds(EDGE)
   @NotWorkingInRemoteBazelBuilds(FIREFOX)
@@ -383,7 +368,6 @@ public void testCookieEqualityAfterSetAndGet() {
   }
 
   @Test
-  @NotYetImplemented(SAFARI)
   public void testRetainsCookieExpiry() {
     Cookie addedCookie =
         new Cookie.Builder("fish", "cod")
@@ -470,7 +454,6 @@ public void testRetainsHttpOnlyFlag() {
   }
 
   @Test
-  @NotYetImplemented(SAFARI)
   public void testSettingACookieThatExpiredInThePast() {
     long expires = System.currentTimeMillis() - 1000;
     Cookie cookie = new Cookie.Builder("expired", "yes").expiresOn(new Date(expires)).build();
@@ -483,7 +466,6 @@ public void testSettingACookieThatExpiredInThePast() {
   }
 
   @Test
-  @NotYetImplemented(SAFARI)
   public void testCanSetCookieWithoutOptionalFieldsSet() {
     String key = generateUniqueKey();
     String value = "foo";
@@ -496,7 +478,6 @@ public void testCanSetCookieWithoutOptionalFieldsSet() {
   }
 
   @Test
-  @NotYetImplemented(SAFARI)
   public void testDeleteNotExistedCookie() {
     String key = generateUniqueKey();
     assertCookieIsNotPresentWithName(key);
@@ -637,7 +618,6 @@ private void addCookieOnServerSide(Cookie cookie) {
   }
 
   @Test
-  @NotYetImplemented(SAFARI)
   public void deleteAllCookies() {
     assumeTrue(domainHelper.checkHasValidAlternateHostname());