Deleting all ignores for REMOTE, they all pass for me with legacy FirefoxDriver

Author: barancev

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

@@ -25,7 +25,6 @@
 import static org.junit.Assert.assertThat;
 import static org.openqa.selenium.testing.Driver.CHROME;
 import static org.openqa.selenium.testing.Driver.IE;
-import static org.openqa.selenium.testing.Driver.REMOTE;
 import static org.openqa.selenium.testing.TestUtilities.catchThrowable;
 
 import org.junit.Test;
@@ -305,7 +304,6 @@ public void testFindingByCssShouldNotIncludeParentElementIfSameTagType() {
   }
 
   @Test
-  @Ignore(REMOTE)
   public void testFindMultipleElements() {
     driver.get(pages.simpleTestPage);
     WebElement elem = driver.findElement(By.id("links"));
@@ -316,7 +314,6 @@ public void testFindMultipleElements() {
   }
 
   @Test
-  @Ignore(REMOTE)
   public void testLinkWithLeadingSpaces() {
     driver.get(pages.simpleTestPage);
     WebElement elem = driver.findElement(By.id("links"));
@@ -326,7 +323,6 @@ public void testLinkWithLeadingSpaces() {
   }
 
   @Test
-  @Ignore(REMOTE)
   public void testLinkWithTrailingSpace() {
     driver.get(pages.simpleTestPage);
     WebElement elem = driver.findElement(By.id("links"));

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

@@ -33,7 +33,6 @@
 import static org.openqa.selenium.testing.Driver.IE;
 import static org.openqa.selenium.testing.Driver.MARIONETTE;
 import static org.openqa.selenium.testing.Driver.PHANTOMJS;
-import static org.openqa.selenium.testing.Driver.REMOTE;
 import static org.openqa.selenium.testing.Driver.SAFARI;
 
 import org.junit.Before;
@@ -275,7 +274,6 @@ public void testsShouldNotGetCookiesRelatedToCurrentDomainWithoutLeadingPeriod()
   }
 
   @Test
-  @Ignore(REMOTE)
   public void testShouldBeAbleToIncludeLeadingPeriodInDomainName() throws Exception {
     String cookieName = "name";
     assertCookieIsNotPresentWithName(cookieName);

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

@@ -29,7 +29,6 @@
 import static org.openqa.selenium.testing.Driver.HTMLUNIT;
 import static org.openqa.selenium.testing.Driver.IE;
 import static org.openqa.selenium.testing.Driver.MARIONETTE;
-import static org.openqa.selenium.testing.Driver.REMOTE;
 import static org.openqa.selenium.testing.Driver.SAFARI;
 import static org.openqa.selenium.testing.TestUtilities.catchThrowable;
 import static org.openqa.selenium.testing.TestUtilities.isOldIe;
@@ -677,7 +676,6 @@ public void findsByLinkTextOnXhtmlPage() {
   }
 
   @Test
-  @Ignore(REMOTE)
   public void testLinkWithFormattingTags() {
     driver.get(pages.simpleTestPage);
     WebElement elem = driver.findElement(By.id("links"));

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

@@ -25,7 +25,6 @@
 import static org.openqa.selenium.testing.Driver.IE;
 import static org.openqa.selenium.testing.Driver.MARIONETTE;
 import static org.openqa.selenium.testing.Driver.PHANTOMJS;
-import static org.openqa.selenium.testing.Driver.REMOTE;
 import static org.openqa.selenium.testing.Driver.SAFARI;
 
 import com.google.common.base.Joiner;
@@ -163,7 +162,6 @@ public void canUsePACThatOnlyProxiesCertainHosts() throws Exception {
   @Ignore(IE)
   @Ignore(SAFARI)
   @Ignore(PHANTOMJS)
-  @Ignore(REMOTE)
   @NeedsLocalEnvironment
   public void canConfigureProxyWithRequiredCapability() {
     Proxy proxyToUse = proxyServer.asProxy();
@@ -182,7 +180,6 @@ public void canConfigureProxyWithRequiredCapability() {
   @Ignore(IE)
   @Ignore(SAFARI)
   @Ignore(PHANTOMJS)
-  @Ignore(REMOTE)
   @NeedsLocalEnvironment
   public void requiredProxyCapabilityShouldHavePriority() {
     ProxyServer desiredProxyServer = new ProxyServer();

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

@@ -22,7 +22,6 @@
 import static org.openqa.selenium.testing.Driver.FIREFOX;
 import static org.openqa.selenium.testing.Driver.MARIONETTE;
 import static org.openqa.selenium.testing.Driver.PHANTOMJS;
-import static org.openqa.selenium.testing.Driver.REMOTE;
 import static org.openqa.selenium.testing.Driver.SAFARI;
 import static org.openqa.selenium.testing.TestUtilities.catchThrowable;
 
@@ -33,7 +32,6 @@
 import org.openqa.selenium.testing.drivers.WebDriverBuilder;
 
 @RunWith(SeleniumTestRunner.class)
-@Ignore(REMOTE)
 public class SessionHandlingTest {
 
   @Test

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

@@ -30,7 +30,6 @@
 import static org.openqa.selenium.support.ui.ExpectedConditions.alertIsPresent;
 import static org.openqa.selenium.testing.Driver.IE;
 import static org.openqa.selenium.testing.Driver.MARIONETTE;
-import static org.openqa.selenium.testing.Driver.REMOTE;
 import static org.openqa.selenium.testing.TestUtilities.catchThrowable;
 
 import org.junit.Rule;
@@ -168,7 +167,6 @@ public void testShouldThrowNoSuchWindowExceptionOnAnyElementOperationIfAWindowIs
   @NoDriverAfterTest
   @Test
   @Ignore(IE)
-  @Ignore(REMOTE)
   public void testShouldBeAbleToIterateOverAllOpenWindows() {
     driver.get(pages.xhtmlTestPage);
     String original = driver.getWindowHandle();

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

@@ -31,7 +31,6 @@
 import static org.openqa.selenium.testing.Driver.IE;
 import static org.openqa.selenium.testing.Driver.MARIONETTE;
 import static org.openqa.selenium.testing.Driver.PHANTOMJS;
-import static org.openqa.selenium.testing.Driver.REMOTE;
 import static org.openqa.selenium.testing.Driver.SAFARI;
 import static org.openqa.selenium.testing.TestUtilities.getEffectivePlatform;
 import static org.openqa.selenium.testing.TestUtilities.getIEVersion;
@@ -150,7 +149,6 @@ public void testControlClickingOnMultiSelectionList() {
   @JavascriptEnabled
   @Test
   @Ignore(IE)
-  @Ignore(REMOTE)
   @Ignore(PHANTOMJS)
   @Ignore(MARIONETTE)
   public void testControlClickingOnCustomMultiSelectionList() {