Bug 212943 – [Intl] Update tests to support ICU 67

• Status: RESOLVED
• Resolution: FIXED
• Component: New Bugs

[Ross Kirsling]

[Intl] Update tests to support ICU 67

[Ross Kirsling]

Created attachment 401403 [details]
Patch

[Ross Kirsling]

Comment on attachment 401403 [details]
Patch

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

> JSTests/stress/intl-collator.js:106
>  // Ignores irrelevant extension keys.
> -shouldBe(testCollator(Intl.Collator('en-u-aa-aaaa-kn-true-bb-bbbb-co-eor-cc-cccc-y-yyd'), [{locale: 'en-u-co-eor-kn-true', collation: 'eor', numeric: true}, {locale: 'en-u-kn-true', numeric: true}]), true);
> +shouldBe(testCollator(Intl.Collator('en-u-aa-aaaa-kf-upper-bb-bbbb-co-eor-cc-cccc-y-yyd'), [{locale: 'en-u-co-eor-kf-upper', collation: 'eor', caseFirst: 'upper'}, {locale: 'en-u-kf-upper', caseFirst: 'upper'}]), true);
>  
>  // Ignores other extensions.
> -shouldBe(testCollator(Intl.Collator('en-u-kn-true-a-aa'), [{locale: 'en-u-kn-true', numeric: true}]), true);
> -shouldBe(testCollator(Intl.Collator('en-a-aa-u-kn-true'), [{locale: 'en-u-kn-true', numeric: true}]), true);
> -shouldBe(testCollator(Intl.Collator('en-a-aa-u-kn-true-b-bb'), [{locale: 'en-u-kn-true', numeric: true}]), true);
> +shouldBe(testCollator(Intl.Collator('en-u-kf-upper-a-aa'), [{locale: 'en-u-kf-upper', caseFirst: 'upper'}]), true);
> +shouldBe(testCollator(Intl.Collator('en-a-aa-u-kf-upper'), [{locale: 'en-u-kf-upper', caseFirst: 'upper'}]), true);
> +shouldBe(testCollator(Intl.Collator('en-a-aa-u-kf-upper-b-bb'), [{locale: 'en-u-kf-upper', caseFirst: 'upper'}]), true);

FWIW: I switched these particular cases to be version-independent instead, but could've gone either way.

[EWS]

Committed r262763: <https://trac.webkit.org/changeset/262763>

All reviewed patches have been landed. Closing bug and clearing flags on attachment 401403 [details].

[Radar WebKit Bug Importer]

<rdar://problem/64145730>