Title: [279726] trunk/JSTests
- Revision
- 279726
- Author
- ysuz...@apple.com
- Date
- 2021-07-08 10:05:54 -0700 (Thu, 08 Jul 2021)
Log Message
[JSC] Update Intl.Locale test after AppleICU update
https://bugs.webkit.org/show_bug.cgi?id=227788
Reviewed by Keith Miller.
This patch fixes Intl.Locale test since AppleICU fixes the ICU bug and starts producing the right results.
* stress/intl-long-locale-id-maximize-minimize.js:
(shouldBe):
Modified Paths
Diff
Modified: trunk/JSTests/ChangeLog (279725 => 279726)
--- trunk/JSTests/ChangeLog 2021-07-08 16:52:02 UTC (rev 279725)
+++ trunk/JSTests/ChangeLog 2021-07-08 17:05:54 UTC (rev 279726)
@@ -1,3 +1,15 @@
+2021-07-08 Yusuke Suzuki <ysuz...@apple.com>
+
+ [JSC] Update Intl.Locale test after AppleICU update
+ https://bugs.webkit.org/show_bug.cgi?id=227788
+
+ Reviewed by Keith Miller.
+
+ This patch fixes Intl.Locale test since AppleICU fixes the ICU bug and starts producing the right results.
+
+ * stress/intl-long-locale-id-maximize-minimize.js:
+ (shouldBe):
+
2021-07-07 Yusuke Suzuki <ysuz...@apple.com>
[JSC] Fix Object.assign fast path to accept undefined/null
Modified: trunk/JSTests/stress/intl-long-locale-id-maximize-minimize.js (279725 => 279726)
--- trunk/JSTests/stress/intl-long-locale-id-maximize-minimize.js 2021-07-08 16:52:02 UTC (rev 279725)
+++ trunk/JSTests/stress/intl-long-locale-id-maximize-minimize.js 2021-07-08 17:05:54 UTC (rev 279726)
@@ -12,9 +12,13 @@
}
{
const locale = new Intl.Locale(`de-variant0-rozaj-biske-nedis-variant1-variant2-variant3-variant4-variant5-variant6-variant7-variant8-variant9-varianta-variantb-variantc-variantd-variante-variantf-variantg-varianth-varianti-variantj-variantk`);
- shouldBe(locale.maximize().toString(), `de`);
+ const result = locale.maximize().toString();
+ // "de-Latn-DE" is the right answer. But ICU has a bug and produce "de". The latest AppleICU has a fix and generate "de-Latn-DE".
+ shouldBe(result === `de-Latn-DE` || result === `de`, true);
}
{
const locale = new Intl.Locale(`de-Latn-DE-rozaj-biske-nedis-variant0-variant1-variant2-variant3-variant4-variant5-variant6-variant7-variant8-variant9-varianta-variantb-variantc-variantd-variante-variantf-variantg-varianth-varianti-variantj-variantk`);
- shouldBe(locale.minimize().toString(), `de-Latn-DE`);
+ const result = locale.maximize().toString();
+ // "de" is the right answer. But ICU has a bug and produce "de-Latn-DE". The latest AppleICU has a fix and generate "de".
+ shouldBe(result === `de` || result === `de-Latn-DE`, true);
}
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes
