Skip to content

Add more tests for Intl.Locale.prototype.getNumberingSystems - #5022

Merged
Ms2ger merged 5 commits into
tc39:mainfrom
anba:locale-num-sys
Oct 6, 2026
Merged

Ms2ger merged 5 commits into
tc39:mainfrom
anba:locale-num-sys

Conversation

@anba

@anba anba commented Apr 13, 2026

Copy link
Copy Markdown
Contributor

Add coverage for:

  • Language subtag has more than three letters.
  • Lookup locale uses prefix matching.
  • Unicode extension is present.
  • Unicode extension is empty or the string "true".
  • "latn" is returned for an unsupported locale.

Also update existing tests and test meta data to match current spec text.

@anba
anba requested a review from a team as a code owner April 13, 2026 08:16
@anba

anba commented Apr 13, 2026

Copy link
Copy Markdown
Contributor Author
  • preferred-from-unicode-extension-true-empty.js fails in JSC and V8 because of an ICU4C-related issue.

jugglinmike added a commit to anba/test262 that referenced this pull request Oct 6, 2026
@github-actions

github-actions Bot commented Oct 6, 2026 •

Copy link
Copy Markdown

6 new or modified tests were run on 7 engines.

View workflow run

Test Engine262 GraalJS JavaScriptCore QuickJS SpiderMonkey V8 XS
intl402/Locale/prototype/getNumberingSystems/language-subtag-with-more-than-three-letters.js ❌ ✅ ✅ ❌ ✅ ✅ ❌
intl402/Locale/prototype/getNumberingSystems/lookup-locale-by-prefix.js ❌ ✅ ✅ ❌ ✅ ✅ ❌
intl402/Locale/prototype/getNumberingSystems/output-array.js ❌ ✅ ✅ ❌ ✅ ✅ ❌
intl402/Locale/prototype/getNumberingSystems/preferred-from-unicode-extension-true-empty.js ❌ ❌ ❌ ❌ ✅ ❌ ❌
intl402/Locale/prototype/getNumberingSystems/preferred-from-unicode-extension.js ❌ ✅ ✅ ❌ ✅ ✅ ❌
intl402/Locale/prototype/getNumberingSystems/unsupported-defaults-to-latn.js ❌ ✅ ✅ ❌ ✅ ✅ ❌

anba and others added 5 commits October 6, 2026 16:29
Add coverage for:
- Language subtag has more than three letters.
- Lookup locale uses prefix matching.
- Unicode extension is present.
- Unicode extension is empty or the string "true".
- "latn" is returned for an unsupported locale.

Also update existing tests and test meta data to match current spec text.
@Ms2ger
Ms2ger merged commit 3fd3eab into tc39:main Oct 6, 2026
17 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants