Layout Test accessibility/mac/async-increment-decrement-action.html is flaky
authorn_wang@apple.com <n_wang@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 20 Apr 2018 21:02:55 +0000 (21:02 +0000)
committern_wang@apple.com <n_wang@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 20 Apr 2018 21:02:55 +0000 (21:02 +0000)
https://bugs.webkit.org/show_bug.cgi?id=184834
<rdar://problem/39604753>

Reviewed by Chris Fleizach.

Changed the test to just make sure the increment/decrement actions are async, that
they won't block the test by changing the object's value.

* accessibility/mac/async-increment-decrement-action-expected.txt:
* accessibility/mac/async-increment-decrement-action.html:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@230855 268f45cc-cd09-0410-ab3c-d52691b4dbfc

LayoutTests/ChangeLog
LayoutTests/accessibility/mac/async-increment-decrement-action-expected.txt
LayoutTests/accessibility/mac/async-increment-decrement-action.html

index 28335ea..8fd72c4 100644 (file)
@@ -1,3 +1,17 @@
+2018-04-20  Nan Wang  <n_wang@apple.com>
+
+        Layout Test accessibility/mac/async-increment-decrement-action.html is flaky
+        https://bugs.webkit.org/show_bug.cgi?id=184834
+        <rdar://problem/39604753>
+
+        Reviewed by Chris Fleizach.
+
+        Changed the test to just make sure the increment/decrement actions are async, that
+        they won't block the test by changing the object's value.
+
+        * accessibility/mac/async-increment-decrement-action-expected.txt:
+        * accessibility/mac/async-increment-decrement-action.html:
+
 2018-04-20  Daniel Bates  <dabates@apple.com>
 
         Remove Strong Password decoration when text field type changes
index 0c90da6..85e17fa 100644 (file)
@@ -5,10 +5,10 @@ On success, you will see a series of "PASS" messages, followed by "TEST COMPLETE
 
 
 PASS obj.intValue is 25
-PASS obj.intValue is 25
-PASS obj.intValue is 50
 PASS obj.intValue is 50
 PASS obj.intValue is 25
+PASS obj.intValue is 25
+PASS obj.intValue is 25
 PASS successfullyParsed is true
 
 TEST COMPLETE
index 8e2d9b8..b897f59 100644 (file)
     description("This tests that increment and decrement actions are async.");
 
     if (window.accessibilityController) {
-        jsTestIsAsync = true;
 
         var obj = accessibilityController.accessibleElementById("range1");
         
         shouldBe("obj.intValue", "25");
 
-        obj.asyncIncrement();
+        // sync version
+        obj.increment();
+        shouldBe("obj.intValue", "50");
+        obj.decrement();
         shouldBe("obj.intValue", "25");
 
-        setTimeout(function() {
-            shouldBe("obj.intValue", "50");
-
-            obj.asyncDecrement();
-            shouldBe("obj.intValue", "50");
-
-            setTimeout(function() {
-                shouldBe("obj.intValue", "25");
-                finishJSTest();
-            }, 5);
-        }, 5);
+        // async version
+        // The increment/decrement actions are now async so that they won't 
+        // block the test run loop by changing the value.
+        obj.asyncIncrement();
+        shouldBe("obj.intValue", "25");
+        obj.asyncDecrement();
+        shouldBe("obj.intValue", "25");
     }
 
 </script>