Unignoring visibility tests that don't require javascript

Author: barancev

Diff for: java/client/test/org/openqa/selenium/VisibilityTest.java

@@ -44,7 +44,6 @@
 
 public class VisibilityTest extends JUnit4TestBase {
 
-  @JavascriptEnabled
   @Test
   public void testShouldAllowTheUserToTellIfAnElementIsDisplayedOrNot() {
     driver.get(pages.javascriptPage);
@@ -56,7 +55,6 @@ public void testShouldAllowTheUserToTellIfAnElementIsDisplayedOrNot() {
     assertThat(driver.findElement(By.id("hidden")).isDisplayed(), is(false));
   }
 
-  @JavascriptEnabled
   @Test
   public void testVisibilityShouldTakeIntoAccountParentVisibility() {
     driver.get(pages.javascriptPage);
@@ -68,7 +66,6 @@ public void testVisibilityShouldTakeIntoAccountParentVisibility() {
     assertFalse(hiddenLink.isDisplayed());
   }
 
-  @JavascriptEnabled
   @Test
   public void testShouldCountElementsAsVisibleIfStylePropertyHasBeenSet() {
     driver.get(pages.javascriptPage);
@@ -95,7 +92,6 @@ public void testShouldModifyTheVisibilityOfAnElementDynamically() {
     assertFalse(element.isDisplayed());
   }
 
-  @JavascriptEnabled
   @Test
   public void testHiddenInputElementsAreNeverVisible() {
     driver.get(pages.javascriptPage);
@@ -105,7 +101,6 @@ public void testHiddenInputElementsAreNeverVisible() {
     assertFalse(shown.isDisplayed());
   }
 
-  @JavascriptEnabled
   @Test
   public void testShouldNotBeAbleToClickOnAnElementThatIsNotDisplayed() {
     driver.get(pages.javascriptPage);
@@ -119,7 +114,6 @@ public void testShouldNotBeAbleToClickOnAnElementThatIsNotDisplayed() {
     }
   }
 
-  @JavascriptEnabled
   @Test
   public void testShouldNotBeAbleToTypeAnElementThatIsNotDisplayed() {
     driver.get(pages.javascriptPage);
@@ -135,7 +129,7 @@ public void testShouldNotBeAbleToTypeAnElementThatIsNotDisplayed() {
     assertThat(element.getAttribute("value"), is(not("You don't see me")));
   }
 
-  @JavascriptEnabled
+  @JavascriptEnabled // element.getSize() requires Javascript in HtmlUnit
   @Ignore({IE})
   @Test
   public void testZeroSizedDivIsShownIfDescendantHasSize() {
@@ -289,7 +283,6 @@ public void testShouldShowElementNotVisibleWhenParentElementHasHiddenAttribute()
    * @see <a href="http://code.google.com/p/selenium/issues/detail?id=1610">
    *      http://code.google.com/p/selenium/issues/detail?id=1610</a>
    */
-  @JavascriptEnabled
   @Ignore({IE, HTMLUNIT, MARIONETTE})
   @Test
   public void testShouldBeAbleToClickOnElementsWithOpacityZero() {
@@ -302,7 +295,6 @@ public void testShouldBeAbleToClickOnElementsWithOpacityZero() {
     assertEquals("1", element.getCssValue("opacity"));
   }
 
-  @JavascriptEnabled
   @Ignore({MARIONETTE})
   @Test
   public void testShouldBeAbleToSelectOptionsFromAnInvisibleSelect() {
@@ -322,7 +314,6 @@ public void testShouldBeAbleToSelectOptionsFromAnInvisibleSelect() {
     assertTrue("Oranges should be selected", oranges.isSelected());
   }
 
-  @JavascriptEnabled
   @Test
   public void testCorrectlyDetectMapElementsAreShown() {
     driver.get(pages.mapVisibilityPage);