[java] Ignoring a couple of Chrome tests

diemol · diemol · commit 409c3899282e · 2022-02-02T10:54:27.000+01:00
Due to https://bugs.chromium.org/p/chromedriver/issues/detail?id=3999
diff --git a/java/test/org/openqa/selenium/TypingTest.java b/java/test/org/openqa/selenium/TypingTest.java
@@ -22,10 +22,11 @@
 import static org.assertj.core.api.Assertions.assertThatExceptionOfType;
 import static org.junit.Assume.assumeFalse;
 import static org.openqa.selenium.WaitingConditions.elementValueToEqual;
-import static org.openqa.selenium.testing.drivers.Browser.IE;
+import static org.openqa.selenium.testing.TestUtilities.getEffectivePlatform;
+import static org.openqa.selenium.testing.drivers.Browser.CHROME;
 import static org.openqa.selenium.testing.drivers.Browser.FIREFOX;
+import static org.openqa.selenium.testing.drivers.Browser.IE;
 import static org.openqa.selenium.testing.drivers.Browser.SAFARI;
-import static org.openqa.selenium.testing.TestUtilities.getEffectivePlatform;
 
 import org.junit.Test;
 import org.openqa.selenium.support.ui.ExpectedConditions;
@@ -36,6 +37,17 @@
 
 public class TypingTest extends JUnit4TestBase {
 
+  private static void checkRecordedKeySequence(WebElement element, int expectedKeyCode) {
+    assertThat(element.getText().trim()).contains(
+      String.format("down: %1$d", expectedKeyCode),
+      String.format("up: %1$d", expectedKeyCode));
+  }
+
+  private static String getValueText(WebElement el) {
+    // Standardize on \n and strip any trailing whitespace.
+    return el.getAttribute("value").replace("\r\n", "\n").trim();
+  }
+
   @Test
   public void testShouldFireKeyPressEvents() {
     driver.get(pages.javascriptPage);
@@ -237,12 +249,6 @@ public void testShouldFireFocusKeyEventsInTheRightOrder() {
     assertThat(result.getText().trim()).isEqualTo("focus keydown keypress keyup");
   }
 
-  private static void checkRecordedKeySequence(WebElement element, int expectedKeyCode) {
-    assertThat(element.getText().trim()).contains(
-        String.format("down: %1$d", expectedKeyCode),
-        String.format("up: %1$d", expectedKeyCode));
-  }
-
   @Test
   public void testShouldReportKeyCodeOfArrowKeys() {
     assumeFalse(Browser.detect() == Browser.LEGACY_OPERA &&
@@ -422,7 +428,8 @@ public void testSpecialSpaceKeys() {
   @Test
   @NotYetImplemented(value = FIREFOX)
   @NotYetImplemented(value = SAFARI, reason = "Enters dot instead of comma")
-  public void testNumberpadKeys() {
+  @Ignore(value = CHROME, reason = "https://bugs.chromium.org/p/chromedriver/issues/detail?id=3999")
+  public void testNumberPadKeys() {
     driver.get(pages.javascriptPage);
 
     WebElement element = driver.findElement(By.id("keyReporter"));
@@ -482,6 +489,9 @@ public void testChordControlHomeShiftEndDelete() {
     assertThat(element.getAttribute("value")).isEqualTo("");
   }
 
+  // control-x control-v here for cut & paste tests, these work on windows
+  // and linux, but not on the MAC.
+
   @Test
   @NotYetImplemented(value = FIREFOX, reason = "https://github.com/mozilla/geckodriver/issues/646")
   public void testChordReveseShiftHomeSelectionDeletes() {
@@ -510,10 +520,8 @@ public void testChordReveseShiftHomeSelectionDeletes() {
     assertThat(element.getAttribute("value")).isEqualTo("");
   }
 
-  // control-x control-v here for cut & paste tests, these work on windows
-  // and linux, but not on the MAC.
-
   @Test
+  @Ignore(value = CHROME, reason = "https://bugs.chromium.org/p/chromedriver/issues/detail?id=3999")
   @NotYetImplemented(value = FIREFOX, reason = "https://github.com/mozilla/geckodriver/issues/646")
   public void testChordControlCutAndPaste() {
     assumeFalse("FIXME: macs don't have HOME keys, would PGUP work?",
@@ -670,9 +678,4 @@ public void canTypeMultipleNewLineCharactersIntoTextArea() {
     element.sendKeys("\n\n\n");
     shortWait.until(ExpectedConditions.attributeToBe(element, "value", "\n\n\n"));
   }
-
-  private static String getValueText(WebElement el) {
-    // Standardize on \n and strip any trailing whitespace.
-    return el.getAttribute("value").replace("\r\n", "\n").trim();
-  }
 }
