Attachment 346250 Details for Bug 188107 – Adjust layout test

[patch] Adjust layout test

bug-188107-20180731203511.patch (text/plain), 3.85 KB, created by Wenson Hsieh on 2018-07-31 20:35:12 PDT

(hide)

Description:

Filename:

MIME Type:

Creator: Wenson Hsieh

Created: 2018-07-31 20:35:12 PDT

Size: 3.85 KB

patch

obsolete

>Subversion Revision: 234439
>diff --git a/LayoutTests/ChangeLog b/LayoutTests/ChangeLog
>index ca20d1559eac38794790f81d77a6cfa14035ab41..000eb9622cb217c203a6b5c165bf8d240c334887 100644
>--- a/LayoutTests/ChangeLog
>+++ b/LayoutTests/ChangeLog
>@@ -1,3 +1,18 @@
>+2018-07-31  Wenson Hsieh  <wenson_hsieh@apple.com>
>+
>+        Tidy up a layout test introduced in r234436.
>+        https://bugs.webkit.org/show_bug.cgi?id=188107
>+        <rdar://problem/42354250>
>+
>+        Reviewed by NOBODY (OOPS!).
>+
>+        The expected output of a test that uses js-test.js should include a description, plus a line that states that
>+        there will be a series of PASS messages, PASS/FAIL output, a successfully parsed line and finally a TEST
>+        COMPLETE line. This patch adjusts a newly introduced test to follow this convention.
>+
>+        * fast/forms/ios/click-should-not-suppress-misspelling-expected.txt:
>+        * fast/forms/ios/click-should-not-suppress-misspelling.html:
>+
> 2018-07-31  Wenson Hsieh  <wenson_hsieh@apple.com>
> 
>         [iOS] Spelling suggestions cannot be selected in focused form controls when zoomed in
>diff --git a/LayoutTests/fast/forms/ios/click-should-not-suppress-misspelling-expected.txt b/LayoutTests/fast/forms/ios/click-should-not-suppress-misspelling-expected.txt
>index 691e2b4d8afc63bed86fe3bfaf0f9f507d213f3c..7f7d6103c155c8734517ec977e94963ca4fcffc2 100644
>--- a/LayoutTests/fast/forms/ios/click-should-not-suppress-misspelling-expected.txt
>+++ b/LayoutTests/fast/forms/ios/click-should-not-suppress-misspelling-expected.txt
>@@ -1,6 +1,11 @@
>-PASS successfullyParsed is true
>+To manually test, tap the input field to bring up the keyboard, and then tap on a part of the word in the input field that is far away from the editing caret. As a result, the contents of the input should be selected and marked as a misspelled word.
>+
>+On success, you will see a series of "PASS" messages, followed by "TEST COMPLETE".
>+
> 
>-TEST COMPLETE
> PASS input.selectionStart is 0
> PASS input.selectionEnd is 7
>+PASS successfullyParsed is true
>+
>+TEST COMPLETE
> 
>diff --git a/LayoutTests/fast/forms/ios/click-should-not-suppress-misspelling.html b/LayoutTests/fast/forms/ios/click-should-not-suppress-misspelling.html
>index 07024853b06bf3c3d12346f6c1c3f727c1d5a8c2..4a5546bffbaf64a5b8ad8c0471c53ff3fccfbd3d 100644
>--- a/LayoutTests/fast/forms/ios/click-should-not-suppress-misspelling.html
>+++ b/LayoutTests/fast/forms/ios/click-should-not-suppress-misspelling.html
>@@ -18,32 +18,27 @@ input {
> </head>
> <body>
> <input id="input" value="covfefe">
>-</body>
> <script>
> const input = document.getElementById("input");
>+jsTestIsAsync = true;
> 
>-addEventListener("load", async () => {
>-    if (window.testRunner) {
>-        testRunner.dumpAsText();
>-        testRunner.waitUntilDone();
>-    } else {
>-        description(`To manually test, tap the input field to bring up the keyboard, and then tap on a part of the word in
>-            the input field that is far away from the editing caret. As a result, the contents of the input should be
>-            selected and marked as a misspelled word.`);
>-        return;
>-    }
>+description(`To manually test, tap the input field to bring up the keyboard, and then tap on a part of the word in
>+    the input field that is far away from the editing caret. As a result, the contents of the input should be
>+    selected and marked as a misspelled word.`);
> 
>-    await UIHelper.activateAndWaitForInputSessionAt(100, 150);
>+(async function() {
>+    await UIHelper.activateAndWaitForInputSessionAt(100, 200);
> 
>     input.addEventListener("click", () => {
>         setTimeout(() => {
>             shouldBe("input.selectionStart", "0");
>             shouldBe("input.selectionEnd", "7");
>-            testRunner.notifyDone();
>+            finishJSTest();
>         });
>     });
> 
>-    await UIHelper.tapAt(800, 150);
>-});
>+    UIHelper.tapAt(800, 200);
>+})();
> </script>
>+</body>
> </html>

Flags:

dbates: review+

Actions: View | Formatted Diff | Diff

Attachments on bug 188107: 345938 | 346248 | 346250 | 346258