Unignoring a bunch of tests in marionette, file uploading and d-n-d w…

…ork now
SeleniumHQ · Aug 17, 2017 · 4be56fb · 4be56fb
1 parent fe0476d
commit 4be56fb
Show file tree

Hide file tree

Showing 6 changed files with 0 additions and 10 deletions.
diff --git a/java/client/test/org/openqa/selenium/CorrectEventFiringTest.java b/java/client/test/org/openqa/selenium/CorrectEventFiringTest.java
@@ -345,7 +345,6 @@ public void testSubmittingFormFromFormInputTextElementShouldFireOnSubmitForThatF
 
   @Test
   @Ignore(value = SAFARI, reason = "Does not yet support file uploads, issue 4220")
-  @NotYetImplemented(value = MARIONETTE, reason = "https://github.com/mozilla/geckodriver/issues/644")
   public void testUploadingFileShouldFireOnChangeEvent() throws IOException {
     driver.get(pages.formPage);
     WebElement uploadElement = driver.findElement(By.id("upload"));

diff --git a/java/client/test/org/openqa/selenium/FormHandlingTest.java b/java/client/test/org/openqa/selenium/FormHandlingTest.java
@@ -159,7 +159,6 @@ public void testShouldEnterDataIntoFormFields() {
 
   @Test
   @Ignore(value = SAFARI, reason = "issue 4220")
-  @NotYetImplemented(value = MARIONETTE, reason = "https://github.com/mozilla/geckodriver/issues/644")
   public void testShouldBeAbleToAlterTheContentsOfAFileUploadInputElement() throws IOException {
     driver.get(pages.formPage);
     WebElement uploadElement = driver.findElement(By.id("upload"));
@@ -176,7 +175,6 @@ public void testShouldBeAbleToAlterTheContentsOfAFileUploadInputElement() throws
 
   @Test
   @Ignore(value = SAFARI, reason = "issue 4220")
-  @NotYetImplemented(value = MARIONETTE, reason = "https://github.com/mozilla/geckodriver/issues/644")
   public void testShouldBeAbleToSendKeysToAFileUploadInputElementInAnXhtmlDocument()
       throws IOException {
     assumeFalse("IE before 9 doesn't handle pages served with an XHTML content type,"
@@ -198,7 +196,6 @@ public void testShouldBeAbleToSendKeysToAFileUploadInputElementInAnXhtmlDocument
 
   @Test
   @Ignore(value = SAFARI, reason = "issue 4220")
-  @NotYetImplemented(value = MARIONETTE, reason = "https://github.com/mozilla/geckodriver/issues/644")
   public void testShouldBeAbleToUploadTheSameFileTwice() throws IOException {
     File file = File.createTempFile("test", "txt");
     file.deleteOnExit();

diff --git a/java/client/test/org/openqa/selenium/PageLoadingTest.java b/java/client/test/org/openqa/selenium/PageLoadingTest.java
@@ -440,7 +440,6 @@ public void testShouldTimeoutIfAPageTakesTooLongToLoad() {
 
   @NoDriverAfterTest // Subsequent tests sometimes fail on Firefox.
   @Test
-  @Ignore(value = MARIONETTE)
   @Ignore(value = SAFARI, reason = "issue 687, comment 41")
   @NeedsLocalEnvironment
   public void testShouldTimeoutIfAPageTakesTooLongToLoadAfterClick() {

diff --git a/java/client/test/org/openqa/selenium/UploadTest.java b/java/client/test/org/openqa/selenium/UploadTest.java
@@ -60,7 +60,6 @@ public void setUp() throws Exception {
 
   @SwitchToTopAfterTest
   @Test
-  @NotYetImplemented(value = MARIONETTE, reason = "https://github.com/mozilla/geckodriver/issues/644")
   public void testFileUploading() throws Exception {
     assumeFalse(
         "This test as written assumes a file on local disk is accessible to the browser. "
@@ -84,7 +83,6 @@ public void testFileUploading() throws Exception {
   @Ignore(IE)
   @Ignore(PHANTOMJS)
   @Ignore(SAFARI)
-  @NotYetImplemented(value = MARIONETTE, reason = "https://github.com/mozilla/geckodriver/issues/644")
   public void testCleanFileInput() throws Exception {
     driver.get(pages.uploadPage);
     WebElement element = driver.findElement(By.id("upload"));

diff --git a/java/client/test/org/openqa/selenium/interactions/BasicMouseInterfaceTest.java b/java/client/test/org/openqa/selenium/interactions/BasicMouseInterfaceTest.java
@@ -401,7 +401,6 @@ public void testMovingMouseToRelativeZeroElementOffset() {
 
   @NeedsFreshDriver({IE, CHROME})
   @Test
-  @Ignore(MARIONETTE)
   @NotYetImplemented(HTMLUNIT)
   public void testMoveRelativeToBody() {
     try {

diff --git a/java/client/test/org/openqa/selenium/interactions/DragAndDropTest.java b/java/client/test/org/openqa/selenium/interactions/DragAndDropTest.java
@@ -53,7 +53,6 @@
 public class DragAndDropTest extends JUnit4TestBase {
 
   @Test
-  @Ignore(MARIONETTE)
   public void testDragAndDropRelative() {
     assumeFalse("See issue 2281", TestUtilities.getEffectivePlatform().is(Platform.MAC));
     assumeFalse(Browser.detect() == Browser.opera &&
@@ -73,7 +72,6 @@ public void testDragAndDropRelative() {
   }
 
   @Test
-  @Ignore(MARIONETTE)
   public void testDragAndDropToElement() {
     driver.get(pages.dragAndDropPage);
     WebElement img1 = driver.findElement(By.id("test1"));