Bug 229385 – JSC test stress/intl-locale-info.js is failing: Error: expected ["islamic-umalqura","islamic-rgsa","islamic","gregory"] but got ["islamic-umalqura","gregory","islamic","islamic-rgsa"]

• Status: RESOLVED
• Resolution: FIXED
• Component: JavaScriptCore

[Michael Catanzaro]

The new JSC stress test stress/intl-locale-info.js is failing on my internal CI:

Running stress/intl-locale-info.js.default
stress/intl-locale-info.js.default: Exception: Error: expected ["islamic-umalqura","islamic-rgsa","islamic","gregory"] but got ["islamic-umalqura","gregory","islamic","islamic-rgsa"]
stress/intl-locale-info.js.default: [e-mail]:3:24
stress/intl-locale-info.js.default: global [e-mail]:37:13
stress/intl-locale-info.js.default: ERROR: Unexpected exit code: 3
FAIL: stress/intl-locale-info.js.default

It seems the test depends on the order of the locales returned by ICU, so we'll need a more permissive check here.

[Yusuke Suzuki]

Yes, it looks like depending on ICU version. On older ICU, it is failing now.

[Yusuke Suzuki]

I will quickly fix it. But it is possible the subsequent tests would fail after it is fixed. It is hard to fix it in my side since I don't have old ICU environment.
@Michael If the other failures happened, can you fix it?

[Yusuke Suzuki]

Committed r281421 (240811@main): <https://commits.webkit.org/240811@main>

[Radar WebKit Bug Importer]

<rdar://problem/82222734>

[Michael Catanzaro]

(In reply to Yusuke Suzuki from comment #2)
> I will quickly fix it. But it is possible the subsequent tests would fail
> after it is fixed. It is hard to fix it in my side since I don't have old
> ICU environment.
> @Michael If the other failures happened, can you fix it?

Turns out there are no other failures. Thanks Yusuke!