Ignoring tests that don't pass in htmlunit with JS off

Author: barancev

java/client/test/org/openqa/selenium/AlertsTest.java

@@ -474,6 +474,7 @@ public void testIncludesAlertTextInUnhandledAlertException() {
     assertThat(t.getMessage(), containsString("cheese"));
   }
 
+  @JavascriptEnabled
   @NoDriverAfterTest
   @Test
   public void testCanQuitWhenAnAlertIsPresent() {

java/client/test/org/openqa/selenium/ContentEditableTest.java

@@ -110,6 +110,7 @@ public void testShouldBeAbleToTypeIntoContentEditableElementWithExistingValue()
     assertThat(editable.getText(), equalTo(initialText + ", edited"));
   }
 
+  @JavascriptEnabled
   @Test
   @Ignore(IE)
   @Ignore(value = SAFARI, reason = "cannot type on contentEditable with synthetic events, issue 3127")
@@ -126,6 +127,7 @@ public void testShouldBeAbleToTypeIntoTinyMCE() {
     assertThat(editable.getText(), equalTo("cheese"));
   }
 
+  @JavascriptEnabled
   @Test
   @Ignore(CHROME)
   @Ignore(IE)

java/client/test/org/openqa/selenium/FormHandlingTest.java

@@ -37,6 +37,7 @@
 import org.junit.Test;
 import org.openqa.selenium.testing.Ignore;
 import org.openqa.selenium.testing.JUnit4TestBase;
+import org.openqa.selenium.testing.JavascriptEnabled;
 import org.openqa.selenium.testing.NotYetImplemented;
 import org.openqa.selenium.testing.TestUtilities;
 
@@ -266,6 +267,7 @@ public void testEmptyTextBoxesShouldReturnAnEmptyStringNotNull() {
     assertEquals(emptyTextBox.getAttribute("value"), "");
   }
 
+  @JavascriptEnabled
   @Test
   @Ignore(PHANTOMJS)
   @Ignore(SAFARI)

java/client/test/org/openqa/selenium/ReferrerTest.java

@@ -50,6 +50,7 @@
 import org.openqa.selenium.support.ui.WebDriverWait;
 import org.openqa.selenium.testing.Ignore;
 import org.openqa.selenium.testing.JUnit4TestBase;
+import org.openqa.selenium.testing.JavascriptEnabled;
 import org.openqa.selenium.testing.NeedsLocalEnvironment;
 import org.openqa.selenium.testing.drivers.WebDriverBuilder;
 import org.seleniumhq.jetty9.server.Handler;
@@ -119,6 +120,7 @@ public static void readPages() throws IOException {
    * Tests navigation when all of the files are hosted on the same domain and the browser
    * does not have a proxy configured.
    */
+  @JavascriptEnabled
   @Test
   @NeedsLocalEnvironment
   public void basicHistoryNavigationWithoutAProxy() {
@@ -141,6 +143,7 @@ public void basicHistoryNavigationWithoutAProxy() {
   /**
    * Tests navigation across multiple domains when the browser does not have a proxy configured.
    */
+  @JavascriptEnabled
   @Test
   @NeedsLocalEnvironment
   public void crossDomainHistoryNavigationWithoutAProxy() {
@@ -171,6 +174,7 @@ public void crossDomainHistoryNavigationWithoutAProxy() {
    * Tests navigation when all of the files are hosted on the same domain and the browser is
    * configured to use a proxy that permits direct access to that domain.
    */
+  @JavascriptEnabled
   @Test
   @NeedsLocalEnvironment
   public void basicHistoryNavigationWithADirectProxy() {
@@ -199,6 +203,7 @@ public void basicHistoryNavigationWithADirectProxy() {
    * Tests navigation across multiple domains when the browser is configured to use a proxy that
    * permits direct access to those domains.
    */
+  @JavascriptEnabled
   @Test
   @NeedsLocalEnvironment
   public void crossDomainHistoryNavigationWithADirectProxy() {
@@ -233,6 +238,7 @@ public void crossDomainHistoryNavigationWithADirectProxy() {
    * Tests navigation across multiple domains when the browser is configured to use a proxy that
    * redirects the second domain to another host.
    */
+  @JavascriptEnabled
   @Test
   @Ignore(MARIONETTE)
   @NeedsLocalEnvironment
@@ -273,6 +279,7 @@ public void crossDomainHistoryNavigationWithAProxiedHost() {
    * intercepts requests to a specific host (www.example.com) - all other requests are permitted
    * to connect directly to the target server.
    */
+  @JavascriptEnabled
   @Test
   @Ignore(MARIONETTE)
   @NeedsLocalEnvironment
@@ -310,6 +317,7 @@ public void crossDomainHistoryNavigationWhenProxyInterceptsHostRequests() {
    * Tests navigation on a single domain where the browser is configured to use a proxy that
    * intercepts requests for page 2.
    */
+  @JavascriptEnabled
   @Test
   @Ignore(value = IE,
       reason = "IEDriver does not disable automatic proxy caching, causing this test to fail, issue 6629")

java/client/test/org/openqa/selenium/UnexpectedAlertBehaviorTest.java

@@ -36,6 +36,7 @@
 import org.openqa.selenium.support.ui.WebDriverWait;
 import org.openqa.selenium.testing.Ignore;
 import org.openqa.selenium.testing.JUnit4TestBase;
+import org.openqa.selenium.testing.JavascriptEnabled;
 import org.openqa.selenium.testing.NeedsLocalEnvironment;
 import org.openqa.selenium.testing.NotYetImplemented;
 import org.openqa.selenium.testing.TestUtilities;
@@ -63,12 +64,14 @@ public void canAcceptUnhandledAlert() {
     runScenarioWithUnhandledAlert(UnexpectedAlertBehaviour.ACCEPT, "This is a default value");
   }
 
+  @JavascriptEnabled
   @Test
   @Ignore(value = CHROME, reason = "Unstable Chrome behavior")
   public void canDismissUnhandledAlert() {
     runScenarioWithUnhandledAlert(UnexpectedAlertBehaviour.DISMISS, "null");
   }
 
+  @JavascriptEnabled
   @Test
   @Ignore(value = CHROME, reason = "Chrome uses IGNORE mode by default")
   public void dismissUnhandledAlertsByDefault() {

java/client/test/org/openqa/selenium/interactions/CombinedInputActionsTest.java

@@ -354,6 +354,7 @@ public void testCombiningShiftAndClickResultsInANewWindow() {
     assertEquals("Should not have navigated away.", originalTitle, driver.getTitle());
   }
 
+  @JavascriptEnabled
   @Test
   @Ignore(MARIONETTE)
   @Ignore(IE)