Skip to content

Commit 1d3df8c

Browse files
committed
Fixing event order test (and unignoring a couple of tests in chrome)
1 parent 37d0874 commit 1d3df8c

File tree

1 file changed

+3
-3
lines changed

1 file changed

+3
-3
lines changed

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

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -29,7 +29,6 @@
2929
import static org.openqa.selenium.WaitingConditions.elementTextToEqual;
3030
import static org.openqa.selenium.WaitingConditions.elementValueToEqual;
3131
import static org.openqa.selenium.support.ui.ExpectedConditions.visibilityOfElementLocated;
32-
import static org.openqa.selenium.testing.Ignore.Driver.CHROME;
3332
import static org.openqa.selenium.testing.Ignore.Driver.HTMLUNIT;
3433
import static org.openqa.selenium.testing.Ignore.Driver.IE;
3534
import static org.openqa.selenium.testing.Ignore.Driver.MARIONETTE;
@@ -49,7 +48,7 @@
4948

5049
public class CorrectEventFiringTest extends JUnit4TestBase {
5150

52-
@Ignore(value = {CHROME, MARIONETTE}, reason = "Webkit bug 22261")
51+
@Ignore(value = {MARIONETTE})
5352
@JavascriptEnabled
5453
@Test
5554
public void testShouldFireFocusEventWhenClicking() {
@@ -125,7 +124,7 @@ public void testShouldNotThrowIfEventHandlerThrows() {
125124
}
126125
}
127126

128-
@Ignore(value = {CHROME, MARIONETTE}, reason = "Webkit bug 22261")
127+
@Ignore(value = {HTMLUNIT, MARIONETTE})
129128
@JavascriptEnabled
130129
@Test
131130
public void testShouldFireEventsInTheRightOrder() {
@@ -141,6 +140,7 @@ public void testShouldFireEventsInTheRightOrder() {
141140

142141
assertTrue(event + " did not fire at all", index != -1);
143142
assertTrue(event + " did not fire in the correct order", index > lastIndex);
143+
lastIndex = index;
144144
}
145145
}
146146

0 commit comments

Comments
 (0)