[iOS] WKAttachmentTestsIOS.InsertDroppedItemProvidersInOrder fails after r231396
authorwenson_hsieh@apple.com <wenson_hsieh@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 8 May 2018 20:16:44 +0000 (20:16 +0000)
committerwenson_hsieh@apple.com <wenson_hsieh@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 8 May 2018 20:16:44 +0000 (20:16 +0000)
https://bugs.webkit.org/show_bug.cgi?id=185414

Reviewed by Tim Horton.

Adjust a test expectation to check that the resulting DOM contains an attachment element,
anchor element, and another attachment element in that order. This ensures that the test
still passes, even after the anchor element is followed by a line break.

* TestWebKitAPI/Tests/WebKitCocoa/WKAttachmentTests.mm:
(TestWebKitAPI::TEST):

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

Tools/ChangeLog
Tools/TestWebKitAPI/Tests/WebKitCocoa/WKAttachmentTests.mm

index eccdf8a..5fb10fe 100644 (file)
@@ -1,3 +1,17 @@
+2018-05-08  Wenson Hsieh  <wenson_hsieh@apple.com>
+
+        [iOS] WKAttachmentTestsIOS.InsertDroppedItemProvidersInOrder fails after r231396
+        https://bugs.webkit.org/show_bug.cgi?id=185414
+
+        Reviewed by Tim Horton.
+
+        Adjust a test expectation to check that the resulting DOM contains an attachment element,
+        anchor element, and another attachment element in that order. This ensures that the test
+        still passes, even after the anchor element is followed by a line break.
+
+        * TestWebKitAPI/Tests/WebKitCocoa/WKAttachmentTests.mm:
+        (TestWebKitAPI::TEST):
+
 2018-05-08  Commit Queue  <commit-queue@webkit.org>
 
         Unreviewed, rolling out r231491.
index b9a32c9..4689da4 100644 (file)
@@ -1338,11 +1338,7 @@ TEST(WKAttachmentTestsIOS, InsertDroppedItemProvidersInOrder)
             NSLog(@"Error: %@", error);
     }
 
-    NSArray *observedElementTags = (NSArray *)[webView objectByEvaluatingJavaScript:@"Array.from(document.body.children).map(e => e.tagName)"];
-    NSArray *expectedElementTags = @[ @"ATTACHMENT", @"A", @"ATTACHMENT" ];
-    EXPECT_TRUE([observedElementTags isEqualToArray:expectedElementTags]);
-    if (![observedElementTags isEqualToArray:expectedElementTags])
-        NSLog(@"Observed elements: %@ did not match expectations: %@", observedElementTags, expectedElementTags);
+    [webView expectElementTagsInOrder:@[ @"ATTACHMENT", @"A", @"ATTACHMENT" ]];
 
     EXPECT_WK_STREQ("first.txt", [webView stringByEvaluatingJavaScript:@"document.querySelectorAll('attachment')[0].getAttribute('title')"]);
     EXPECT_WK_STREQ("text/plain", [webView stringByEvaluatingJavaScript:@"document.querySelectorAll('attachment')[0].getAttribute('type')"]);