Skip to content

Commit 7038163

Browse files
committed
Adding information on the reasons to ignore tests
1 parent acb94a2 commit 7038163

File tree

3 files changed

+26
-22
lines changed

3 files changed

+26
-22
lines changed

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

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -153,7 +153,7 @@ public void testFindElementByLinkText() {
153153
}
154154

155155
@Test
156-
@Ignore(MARIONETTE)
156+
@Ignore(value = {MARIONETTE}, reason = "Marionette: https://bugzilla.mozilla.org/show_bug.cgi?id=1204496")
157157
public void testFindElementsByLinkTest() {
158158
driver.get(pages.nestedPage);
159159
WebElement element = driver.findElement(By.name("div1"));
@@ -291,7 +291,7 @@ public void testFindingByCssShouldNotIncludeParentElementIfSameTagType() {
291291
assertEquals("child", child.getAttribute("id"));
292292
}
293293

294-
@Ignore({REMOTE, MARIONETTE})
294+
@Ignore(value = {REMOTE, MARIONETTE}, reason = "Marionette: https://bugzilla.mozilla.org/show_bug.cgi?id=1204496")
295295
@Test
296296
public void testFindMultipleElements() {
297297
driver.get(pages.simpleTestPage);

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

Lines changed: 16 additions & 16 deletions
Original file line numberDiff line numberDiff line change
@@ -91,14 +91,14 @@ public void testShouldNotBeAbleToLocateByIdMultipleElementsThatDoNotExist() {
9191
}
9292

9393
@Test(expected = NoSuchElementException.class)
94-
@Ignore(MARIONETTE)
94+
@Ignore(value = MARIONETTE, reason = "https://bugzilla.mozilla.org/show_bug.cgi?id=1204504")
9595
public void testFindingASingleElementByEmptyIdShouldThrow() {
9696
driver.get(pages.formPage);
9797
driver.findElement(By.id(""));
9898
}
9999

100100
@Test
101-
@Ignore(MARIONETTE)
101+
@Ignore(value = MARIONETTE, reason = "https://bugzilla.mozilla.org/show_bug.cgi?id=1204504")
102102
public void testFindingMultipleElementsByEmptyIdShouldReturnEmptyList() {
103103
driver.get(pages.formPage);
104104
List<WebElement> elements = driver.findElements(By.id(""));
@@ -157,29 +157,29 @@ public void testShouldNotBeAbleToLocateByNameMultipleElementsThatDoNotExist() {
157157
}
158158

159159
@Test(expected = NoSuchElementException.class)
160-
@Ignore(MARIONETTE)
160+
@Ignore(value = MARIONETTE, reason = "https://bugzilla.mozilla.org/show_bug.cgi?id=1204504")
161161
public void testFindingASingleElementByEmptyNameShouldThrow() {
162162
driver.get(pages.formPage);
163163
driver.findElement(By.name(""));
164164
}
165165

166166
@Test
167-
@Ignore(MARIONETTE)
167+
@Ignore(value = MARIONETTE, reason = "https://bugzilla.mozilla.org/show_bug.cgi?id=1204504")
168168
public void testFindingMultipleElementsByEmptyNameShouldReturnEmptyList() {
169169
driver.get(pages.formPage);
170170
List<WebElement> elements = driver.findElements(By.name(""));
171171
assertThat(elements.size(), is(0));
172172
}
173173

174174
@Test(expected = NoSuchElementException.class)
175-
@Ignore(MARIONETTE)
175+
@Ignore(value = MARIONETTE, reason = "https://bugzilla.mozilla.org/show_bug.cgi?id=1204504")
176176
public void testFindingASingleElementByNameWithSpaceShouldThrow() {
177177
driver.get(pages.formPage);
178178
driver.findElement(By.name("nonexistent button"));
179179
}
180180

181181
@Test
182-
@Ignore(MARIONETTE)
182+
@Ignore(value = MARIONETTE, reason = "https://bugzilla.mozilla.org/show_bug.cgi?id=1204504")
183183
public void testFindingMultipleElementsByNameWithSpaceShouldReturnEmptyList() {
184184
driver.get(pages.formPage);
185185
List<WebElement> elements = driver.findElements(By.name("nonexistent button"));
@@ -218,14 +218,14 @@ public void testShouldNotBeAbleToLocateByTagNameMultipleElementsThatDoNotExist()
218218
}
219219

220220
@Test(expected = NoSuchElementException.class)
221-
@Ignore(MARIONETTE)
221+
@Ignore(value = MARIONETTE, reason = "https://bugzilla.mozilla.org/show_bug.cgi?id=1204504")
222222
public void testFindingASingleElementByEmptyTagNameShouldThrow() {
223223
driver.get(pages.formPage);
224224
driver.findElement(By.tagName(""));
225225
}
226226

227227
@Test
228-
@Ignore(MARIONETTE)
228+
@Ignore(value = MARIONETTE, reason = "https://bugzilla.mozilla.org/show_bug.cgi?id=1204504")
229229
public void testFindingMultipleElementsByEmptyTagNameShouldReturnEmptyList() {
230230
driver.get(pages.formPage);
231231
List<WebElement> elements = driver.findElements(By.tagName(""));
@@ -428,14 +428,14 @@ public void testShouldThrowAnExceptionWhenThereIsNoLinkToClick() {
428428
driver.findElement(By.xpath("//a[@id='Not here']"));
429429
}
430430

431-
@Ignore({MARIONETTE})
431+
@Ignore(value = MARIONETTE, reason = "https://bugzilla.mozilla.org/show_bug.cgi?id=1204504")
432432
@Test(expected = InvalidSelectorException.class)
433433
public void testShouldThrowInvalidSelectorExceptionWhenXPathIsSyntacticallyInvalidInDriverFindElement() {
434434
driver.get(pages.formPage);
435435
driver.findElement(By.xpath("this][isnot][valid"));
436436
}
437437

438-
@Ignore({MARIONETTE})
438+
@Ignore(value = MARIONETTE, reason = "https://bugzilla.mozilla.org/show_bug.cgi?id=1204504")
439439
@Test(expected = InvalidSelectorException.class)
440440
public void testShouldThrowInvalidSelectorExceptionWhenXPathIsSyntacticallyInvalidInDriverFindElements() {
441441
assumeFalse("Ignoring xpath error test in IE6", TestUtilities.isIe6(driver));
@@ -444,15 +444,15 @@ public void testShouldThrowInvalidSelectorExceptionWhenXPathIsSyntacticallyInval
444444
driver.findElements(By.xpath("this][isnot][valid"));
445445
}
446446

447-
@Ignore({MARIONETTE})
447+
@Ignore(value = MARIONETTE, reason = "https://bugzilla.mozilla.org/show_bug.cgi?id=1204504")
448448
@Test(expected = InvalidSelectorException.class)
449449
public void testShouldThrowInvalidSelectorExceptionWhenXPathIsSyntacticallyInvalidInElementFindElement() {
450450
driver.get(pages.formPage);
451451
WebElement body = driver.findElement(By.tagName("body"));
452452
body.findElement(By.xpath("this][isnot][valid"));
453453
}
454454

455-
@Ignore({MARIONETTE})
455+
@Ignore(value = MARIONETTE, reason = "https://bugzilla.mozilla.org/show_bug.cgi?id=1204504")
456456
@Test(expected = InvalidSelectorException.class)
457457
public void testShouldThrowInvalidSelectorExceptionWhenXPathIsSyntacticallyInvalidInElementFindElements() {
458458
assumeFalse("Ignoring xpath error test in IE6", TestUtilities.isIe6(driver));
@@ -462,14 +462,14 @@ public void testShouldThrowInvalidSelectorExceptionWhenXPathIsSyntacticallyInval
462462
body.findElements(By.xpath("this][isnot][valid"));
463463
}
464464

465-
@Ignore({MARIONETTE})
465+
@Ignore(value = MARIONETTE, reason = "https://bugzilla.mozilla.org/show_bug.cgi?id=1204504")
466466
@Test(expected = InvalidSelectorException.class)
467467
public void testShouldThrowInvalidSelectorExceptionWhenXPathReturnsWrongTypeInDriverFindElement() {
468468
driver.get(pages.formPage);
469469
driver.findElement(By.xpath("count(//input)"));
470470
}
471471

472-
@Ignore({MARIONETTE})
472+
@Ignore(value = MARIONETTE, reason = "https://bugzilla.mozilla.org/show_bug.cgi?id=1204504")
473473
@Test(expected = InvalidSelectorException.class)
474474
public void testShouldThrowInvalidSelectorExceptionWhenXPathReturnsWrongTypeInDriverFindElements() {
475475
assumeFalse("Ignoring xpath error test in IE6", TestUtilities.isIe6(driver));
@@ -478,7 +478,7 @@ public void testShouldThrowInvalidSelectorExceptionWhenXPathReturnsWrongTypeInDr
478478
driver.findElements(By.xpath("count(//input)"));
479479
}
480480

481-
@Ignore({MARIONETTE})
481+
@Ignore(value = MARIONETTE, reason = "https://bugzilla.mozilla.org/show_bug.cgi?id=1204504")
482482
@Test(expected = InvalidSelectorException.class)
483483
public void testShouldThrowInvalidSelectorExceptionWhenXPathReturnsWrongTypeInElementFindElement() {
484484
driver.get(pages.formPage);
@@ -487,7 +487,7 @@ public void testShouldThrowInvalidSelectorExceptionWhenXPathReturnsWrongTypeInEl
487487
body.findElement(By.xpath("count(//input)"));
488488
}
489489

490-
@Ignore({MARIONETTE})
490+
@Ignore(value = MARIONETTE, reason = "https://bugzilla.mozilla.org/show_bug.cgi?id=1204504")
491491
@Test(expected = InvalidSelectorException.class)
492492
public void testShouldThrowInvalidSelectorExceptionWhenXPathReturnsWrongTypeInElementFindElements() {
493493
assumeFalse("Ignoring xpath error test in IE6", TestUtilities.isIe6(driver));

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

Lines changed: 8 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -261,7 +261,8 @@ public void testShouldThrowAnExceptionWhenAFrameCannotBeFoundByIndex() {
261261
}
262262
}
263263

264-
@Ignore({CHROME, IE, PHANTOMJS, SAFARI, MARIONETTE})
264+
@Ignore(value = {CHROME, IE, PHANTOMJS, SAFARI, MARIONETTE},
265+
reason = "Marionette: https://github.com/jgraham/wires/issues/22")
265266
@Test
266267
public void testShouldBeAbleToSwitchToParentFrame() {
267268
driver.get(pages.framesetPage);
@@ -270,7 +271,8 @@ public void testShouldBeAbleToSwitchToParentFrame() {
270271
assertThat(driver.findElement(By.id("pageNumber")).getText(), equalTo("1"));
271272
}
272273

273-
@Ignore({CHROME, IE, PHANTOMJS, SAFARI, MARIONETTE})
274+
@Ignore(value = {CHROME, IE, PHANTOMJS, SAFARI, MARIONETTE},
275+
reason = "Marionette: https://github.com/jgraham/wires/issues/22")
274276
@Test
275277
public void testShouldBeAbleToSwitchToParentFrameFromASecondLevelFrame() {
276278
driver.get(pages.framesetPage);
@@ -280,15 +282,17 @@ public void testShouldBeAbleToSwitchToParentFrameFromASecondLevelFrame() {
280282
assertThat(driver.findElement(By.id("pageNumber")).getText(), equalTo("11"));
281283
}
282284

283-
@Ignore({CHROME, IE, PHANTOMJS, SAFARI, MARIONETTE})
285+
@Ignore(value = {CHROME, IE, PHANTOMJS, SAFARI, MARIONETTE},
286+
reason = "Marionette: https://github.com/jgraham/wires/issues/22")
284287
@Test
285288
public void testSwitchingToParentFrameFromDefaultContextIsNoOp() {
286289
driver.get(pages.xhtmlTestPage);
287290
driver.switchTo().parentFrame();
288291
assertEquals(driver.getTitle(), "XHTML Test Page");
289292
}
290293

291-
@Ignore({CHROME, IE, PHANTOMJS, SAFARI, MARIONETTE})
294+
@Ignore(value = {CHROME, IE, PHANTOMJS, SAFARI, MARIONETTE},
295+
reason = "Marionette: https://github.com/jgraham/wires/issues/22")
292296
@Test
293297
public void testShouldBeAbleToSwitchToParentFromAnIframe() {
294298
driver.get(pages.iframePage);

0 commit comments

Comments
 (0)