[java] Ignoring test for Firefox and fixing window size

Author: diemol

java/test/org/openqa/selenium/WindowTest.java

@@ -128,7 +128,7 @@ public void testCanMaximizeTheWindow() {
 
   @SwitchToTopAfterTest
   @Test
-  @Ignore(value = CHROME, travis = true)
+  @Ignore(value = CHROME, gitHubActions = true)
   @Ignore(value = FIREFOX, gitHubActions = true)
   public void testCanMaximizeTheWindowFromFrame() {
     // Browser window cannot be resized or moved on ANDROID (and most mobile platforms
@@ -144,15 +144,15 @@ public void testCanMaximizeTheWindowFromFrame() {
 
   @SwitchToTopAfterTest
   @Test
-  @Ignore(value = CHROME, travis = true)
+  @Ignore(value = CHROME, gitHubActions = true)
   @Ignore(value = FIREFOX, gitHubActions = true)
   public void testCanMaximizeTheWindowFromIframe() {
     // Browser window cannot be resized or moved on ANDROID (and most mobile platforms
     // though others aren't defined in org.openqa.selenium.Platform).
     assumeFalse(TestUtilities.getEffectivePlatform(driver).is(ANDROID));
 
     driver.get(pages.iframePage);
-    changeSizeTo(new Dimension(640, 325));
+    changeSizeTo(new Dimension(640, 400));
 
     driver.switchTo().frame("iframe1-name");
     enlargeBy(WebDriver.Window::maximize);
@@ -165,7 +165,7 @@ public void canMinimizeTheWindow() {
     // though others aren't defined in org.openqa.selenium.Platform).
     assumeFalse(TestUtilities.getEffectivePlatform(driver).is(ANDROID));
 
-    changeSizeTo(new Dimension(640, 323));
+    changeSizeTo(new Dimension(640, 400));
     driver.manage().window().minimize();
     // TODO: how to verify the result of this operation?
   }
@@ -180,7 +180,7 @@ public void canFullscreenTheWindow() {
     assumeFalse(TestUtilities.getEffectivePlatform(driver).is(ANDROID));
 
     try {
-      changeSizeTo(new Dimension(640, 323));
+      changeSizeTo(new Dimension(640, 400));
       enlargeBy(WebDriver.Window::fullscreen);
     } finally {
       driver.manage().window().setSize(new Dimension(640, 323));
@@ -198,7 +198,7 @@ public void canFullscreenTheWindowFromFrame() {
     assumeFalse(TestUtilities.getEffectivePlatform(driver).is(ANDROID));
 
     driver.get(pages.framesetPage);
-    changeSizeTo(new Dimension(640, 324));
+    changeSizeTo(new Dimension(640, 400));
 
     driver.switchTo().frame("fourth");
     enlargeBy(WebDriver.Window::fullscreen);
@@ -215,7 +215,7 @@ public void canFullscreenTheWindowFromIframe() {
     assumeFalse(TestUtilities.getEffectivePlatform(driver).is(ANDROID));
 
     driver.get(pages.iframePage);
-    changeSizeTo(new Dimension(640, 325));
+    changeSizeTo(new Dimension(640, 400));
 
     driver.switchTo().frame("iframe1-name");
     enlargeBy(WebDriver.Window::fullscreen);

java/test/org/openqa/selenium/devtools/WindowSwitchingTest.java

@@ -1,18 +1,21 @@
 package org.openqa.selenium.devtools;
 
+import static org.assertj.core.api.Assertions.assertThat;
+import static org.openqa.selenium.testing.drivers.Browser.FIREFOX;
+
 import org.junit.Test;
 import org.openqa.selenium.WebDriver;
 import org.openqa.selenium.WindowType;
 import org.openqa.selenium.devtools.idealized.target.model.TargetInfo;
 import org.openqa.selenium.remote.Augmenter;
+import org.openqa.selenium.testing.Ignore;
 
 import java.util.List;
 
-import static org.assertj.core.api.Assertions.assertThat;
-
 public class WindowSwitchingTest extends DevToolsTestBase {
 
   @Test
+  @Ignore(FIREFOX)
   public void shouldBeAbleToSwitchWindowsAndCloseTheOriginal() {
     WebDriver driver = new Augmenter().augment(this.driver);