RESOLVED FIXED 229323
Make UIEventAttribution tests less platform dependent 
https://bugs.webkit.org/show_bug.cgi?id=229323
Summary Make UIEventAttribution tests less platform dependent
Alex Christensen
Reported 2021-08-19 23:08:09 PDT
Make UIEventAttribution tests less platform dependent
Attachments
Patch (11.24 KB, patch)
2021-08-19 23:09 PDT, Alex Christensen 		no flags
DetailsFormatted DiffDiff
Patch (15.86 KB, patch)
2021-08-20 15:15 PDT, Alex Christensen 		no flags
DetailsFormatted DiffDiff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.
Alex Christensen
Comment 1 2021-08-19 23:09:00 PDT
Created attachment 435943 [details] Patch
John Wilander
Comment 2 2021-08-20 08:48:51 PDT
TestWebKitAPI.EventAttribution.Basic is timing out on iOS.
John Wilander
Comment 3 2021-08-20 08:53:20 PDT
Comment on attachment 435943 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=435943&action=review r=me as long as the broken iOS test is fixed. > Source/WebKit/UIProcess/API/Cocoa/WKWebViewTesting.mm:58 > +- (void)_addEventAttributionWithSourceID:(uint8_t)sourceID destinationURL:(NSURL *)destination sourceDescription:(NSString *)sourceDescription purchaser:(NSString *)purchaser reportEndpoint:(NSURL *)reportEndpoint nonce:(NSString *)nonce Should we call the nonce parameter optionalNonce? > Tools/TestWebKitAPI/Tests/WebKitCocoa/EventAttribution.mm:265 > + [webView _addEventAttributionWithSourceID:42 destinationURL:exampleURL() sourceDescription:@"test source description" purchaser:@"test purchaser" reportEndpoint:serverURL nonce:@"ABCDEFabcdef0123456789"]; Where did 42 come from? Did it come from MockEventAttribution before?
Alex Christensen
Comment 4 2021-08-20 13:41:54 PDT
This test seems to have been timing out since r277447, which was refactored significantly in r279514
Alex Christensen
Comment 5 2021-08-20 15:15:45 PDT
Created attachment 436034 [details] Patch
Alex Christensen
Comment 6 2021-08-20 15:50:29 PDT
Comment on attachment 436034 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=436034&action=review > Tools/TestWebKitAPI/Tests/WebKitCocoa/EventAttribution.mm:129 > +#if HAVE(UI_EVENT_ATTRIBUTION) I verified on Monterey that this is not needed. The test is no longer dependent on UIEventAttribution.
Alex Christensen
Comment 7 2021-08-20 16:02:12 PDT
http://trac.webkit.org/r281349
Radar WebKit Bug Importer
Comment 8 2021-08-20 16:03:36 PDT
<rdar://problem/82189489>
Note You need to log in before you can comment on or make changes to this bug.