[iOS] API test WKAttachmentTests.InsertAndRemoveDuplicateAttachment is failing after...
authorwenson_hsieh@apple.com <wenson_hsieh@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Mon, 4 Feb 2019 18:59:40 +0000 (18:59 +0000)
committerwenson_hsieh@apple.com <wenson_hsieh@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Mon, 4 Feb 2019 18:59:40 +0000 (18:59 +0000)
https://bugs.webkit.org/show_bug.cgi?id=194207

Reviewed by Tim Horton.

Fix this API test by making it robust against smart paste on iOS. Currently, this test pastes an attachment
element and assumes that the pasted attachment is adjacent to the original one. However, after enabling smart
paste, we now insert an extra space in between. Instead of executing "DeleteBackward" twice to test removing the
attachment elements, we delete backwards once to remove the pasted attachment, and move the selection to the
start and delete forwards to remove the original attachment.

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

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

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

index 9884c45..52ddd42 100644 (file)
@@ -1,3 +1,19 @@
+2019-02-04  Wenson Hsieh  <wenson_hsieh@apple.com>
+
+        [iOS] API test WKAttachmentTests.InsertAndRemoveDuplicateAttachment is failing after r240902
+        https://bugs.webkit.org/show_bug.cgi?id=194207
+
+        Reviewed by Tim Horton.
+
+        Fix this API test by making it robust against smart paste on iOS. Currently, this test pastes an attachment
+        element and assumes that the pasted attachment is adjacent to the original one. However, after enabling smart
+        paste, we now insert an extra space in between. Instead of executing "DeleteBackward" twice to test removing the
+        attachment elements, we delete backwards once to remove the pasted attachment, and move the selection to the
+        start and delete forwards to remove the original attachment.
+
+        * TestWebKitAPI/Tests/WebKitCocoa/WKAttachmentTests.mm:
+        (TestWebKitAPI::TEST):
+
 2019-02-04  Youenn Fablet  <youenn@apple.com>
 
         Capture state should be managed consistently when doing process swapping
index 722b68c..840f567 100644 (file)
@@ -1114,7 +1114,8 @@ TEST(WKAttachmentTests, InsertAndRemoveDuplicateAttachment)
     }
     {
         ObserveAttachmentUpdatesForScope observer(webView.get());
-        [webView _synchronouslyExecuteEditCommand:@"DeleteBackward" argument:nil];
+        [webView evaluateJavaScript:@"getSelection().setPosition(document.body)" completionHandler:nil];
+        [webView _synchronouslyExecuteEditCommand:@"DeleteForward" argument:nil];
         observer.expectAttachmentUpdates(@[ originalAttachment.get() ], @[ ]);
     }