RESOLVED FIXED209922
Move Intl tests from LayoutTests to JSTests
https://bugs.webkit.org/show_bug.cgi?id=209922
Summary Move Intl tests from LayoutTests to JSTests
Ross Kirsling
Reported 2020-04-02 13:22:43 PDT
Move Intl API tests from LayoutTests to JSTests
Attachments
Patch (655.43 KB, patch)
2020-04-02 13:34 PDT, Ross Kirsling
no flags
Patch (655.47 KB, patch)
2020-04-02 14:50 PDT, Ross Kirsling
no flags
Ross Kirsling
Comment 1 2020-04-02 13:34:11 PDT
Ross Kirsling
Comment 2 2020-04-02 13:39:57 PDT
Sorry if this is a bit tricky to review due to updating the assertion style of the test cases. I do recommend that we migrate tests from LayoutTests/js to JSTests/stress any time we're bitten by them as a general practice, but I might not necessarily recommend taking the time to update the assertion style like this in the future.
Ross Kirsling
Comment 3 2020-04-02 14:36:39 PDT
Grr, I tried to relax the skips a bit, but apparently that part will need to wait until after bug 209694.
Ross Kirsling
Comment 4 2020-04-02 14:50:07 PDT
Yusuke Suzuki
Comment 5 2020-04-03 03:23:17 PDT
Comment on attachment 395304 [details] Patch r=me
EWS
Comment 6 2020-04-03 11:34:58 PDT
Committed r259480: <https://trac.webkit.org/changeset/259480> All reviewed patches have been landed. Closing bug and clearing flags on attachment 395304 [details].
Radar WebKit Bug Importer
Comment 7 2020-04-03 11:35:16 PDT
Note You need to log in before you can comment on or make changes to this bug.