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"]
Summary: JSC test stress/intl-locale-info.js is failing: Error: expected ["islamic-uma...
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: JavaScriptCore (show other bugs)
Version: WebKit Nightly Build
Hardware: PC Linux
: P2 Normal
Assignee: Nobody
URL:
Keywords: InRadar
Depends on:
Blocks:
 
Reported: 2021-08-22 08:18 PDT by Michael Catanzaro
Modified: 2021-08-23 04:59 PDT (History)
4 users (show)

See Also:

Attachments
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 Michael Catanzaro 2021-08-22 08:18:55 PDT 
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: shouldBe@intl-locale-info.js:3:24
stress/intl-locale-info.js.default: global code@intl-locale-info.js: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.
Comment 1 Yusuke Suzuki 2021-08-22 15:05:36 PDT 
Yes, it looks like depending on ICU version. On older ICU, it is failing now.
Comment 2 Yusuke Suzuki 2021-08-22 15:08:00 PDT 
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?
Comment 3 Yusuke Suzuki 2021-08-22 15:10:17 PDT 
Committed r281421 (240811@main): <https://commits.webkit.org/240811@main>
Comment 4 Radar WebKit Bug Importer 2021-08-22 15:11:18 PDT 
<rdar://problem/82222734>
Comment 5 Michael Catanzaro 2021-08-23 04:59:47 PDT 
(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!