Bug 210921 - Update test262 to commit 31dabb5618e2
Summary: Update test262 to commit 31dabb5618e2
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Tools / Tests (show other bugs)
Version: WebKit Nightly Build
Hardware: All All
: P2 Enhancement
Assignee: Alexey Shvayka
URL:
Keywords: InRadar
Depends on:
Blocks:
 
Reported: 2020-04-23 10:17 PDT by Alexey Shvayka
Modified: 2020-04-23 22:20 PDT (History)
3 users (show)

See Also:

Attachments
Patch (351.73 KB, patch)
2020-04-23 10:56 PDT, Alexey Shvayka 		no flags Details | Formatted Diff | Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Alexey Shvayka 2020-04-23 10:17:54 PDT 
Update test262 to commit 31dabb5618e2
Comment 1 Alexey Shvayka 2020-04-23 10:56:52 PDT 
Created attachment 397360 [details]
Patch
Comment 2 EWS 2020-04-23 12:57:17 PDT 
Committed r260591: <https://trac.webkit.org/changeset/260591>

All reviewed patches have been landed. Closing bug and clearing flags on attachment 397360 [details].
Comment 3 Radar WebKit Bug Importer 2020-04-23 12:58:42 PDT 
<rdar://problem/62258557>
Comment 4 Ross Kirsling 2020-04-23 15:01:55 PDT 
Whoops, we need to adjust the expectations since the bot is using ICU 64.

Committed r260601: <https://trac.webkit.org/changeset/260601>
Comment 5 Alexey Shvayka 2020-04-23 22:00:37 PDT 
(In reply to Ross Kirsling from comment #4)
> Whoops, we need to adjust the expectations since the bot is using ICU 64.

Thank you, Ross.
I will make sure to install Catalina before next sync.
Would be nice if it was possible to conditionally skip test262 files though.
Comment 6 Ross Kirsling 2020-04-23 22:20:22 PDT 
(In reply to Alexey Shvayka from comment #5)
> (In reply to Ross Kirsling from comment #4)
> > Whoops, we need to adjust the expectations since the bot is using ICU 64.
> 
> Thank you, Ross.
> I will make sure to install Catalina before next sync.
> Would be nice if it was possible to conditionally skip test262 files though.

Agreed. :(