Bug 176736 - [iOS WK2] ExternalSourceDataTransferItemGetAsEntry tests produce inconsistent results on test automation bots
Summary: [iOS WK2] ExternalSourceDataTransferItemGetAsEntry tests produce inconsistent...
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebKit2 (show other bugs)
Version: WebKit Nightly Build
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Wenson Hsieh
URL:
Keywords: InRadar
Depends on:
Blocks:
 
Reported: 2017-09-11 14:22 PDT by Wenson Hsieh
Modified: 2017-09-27 13:01 PDT (History)
5 users (show)

See Also:


Attachments
Patch (4.86 KB, patch)
2017-09-11 14:27 PDT, Wenson Hsieh
aestes: review+
Details | Formatted Diff | Diff
Patch for landing (5.18 KB, patch)
2017-09-11 15:52 PDT, Wenson Hsieh
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Wenson Hsieh 2017-09-11 14:22:05 PDT
These tests are failing on the bots, since the order in which files are read and appear is different from the order observed locally on my machine. Instead, these tests should be tweaked so they don't spit out FileSystemEntry output in arbitrary order.
Comment 1 Wenson Hsieh 2017-09-11 14:27:41 PDT
Created attachment 320479 [details]
Patch
Comment 2 Megan Gardner 2017-09-11 15:01:43 PDT
Comment on attachment 320479 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=320479&action=review

Good fix, just make sure that what you're testing against is also guaranteed to be in alphabetical order. then r=me. Not sure if you need a WK2 review as well.

> Tools/TestWebKitAPI/Tests/ios/DataInteractionTests.mm:966
>      ];

I don't see that this list is sorted again in code. Can you either do that and/or make a comment that these are expected to be in alphabetical order?
Comment 3 Wenson Hsieh 2017-09-11 15:06:57 PDT
Comment on attachment 320479 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=320479&action=review

>> Tools/TestWebKitAPI/Tests/ios/DataInteractionTests.mm:966
>>      ];
> 
> I don't see that this list is sorted again in code. Can you either do that and/or make a comment that these are expected to be in alphabetical order?

I don't think sorting this test expectation is really needed, since it's hard-coded anyways to be in alphabetical order...I'll add a comment to make this more apparent, though.
Comment 4 Wenson Hsieh 2017-09-11 15:52:06 PDT
Created attachment 320493 [details]
Patch for landing
Comment 5 WebKit Commit Bot 2017-09-11 16:34:07 PDT
Comment on attachment 320493 [details]
Patch for landing

Clearing flags on attachment: 320493

Committed r221893: <http://trac.webkit.org/changeset/221893>
Comment 6 Radar WebKit Bug Importer 2017-09-27 13:01:24 PDT
<rdar://problem/34694490>