Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

CLDR-17812 v46 vxml: Update TestExampleGenerator/TestUnicodeSetExamples to match hi change in #3883 #3888

Conversation

pedberg-icu
Copy link
Contributor

CLDR-17812

  • This PR completes the ticket.

merge into #3880

Updated the hi test in TestExampleGenerator/TestUnicodeSetExamples to match the hii exemplar set change in #3883

ALLOW_MANY_COMMITS=true

@pedberg-icu pedberg-icu self-assigned this Jul 19, 2024
@@ -1638,7 +1638,7 @@ public void TestUnicodeSetExamples() {
"hi",
"//ldml/characters/exemplarCharacters[@type=\"auxiliary\"]",
"[ॄ‌‍]",
"〖‎🗝️ ॑ ॒ ॠ ॡ ॻ ॼ ॾ ॿ ॢ ॣ〗〖❰ZWNJ❱ ≡ cursive non-joiner〗〖❰ZWJ❱ ≡ cursive joiner〗〖❬internal: ❭[ॄ‌‍]〗"
"〖‎🗝️ ॑ ॒ ॠ ॡ ॻ ॼ ॾ ॿ ॢ ॣ〗〖❰ZWNJ❱ ≡ cursive non-joiner〗〖❰ZWJ❱ ≡ cursive joiner〗〖❬internal: ❭[ॄ‌‍]〗"
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this would be a good example (heh) of where the test would be better off to use canned input data - a static exemplar set content rather than using live input.

TestCLDRFile.testSourceLocale() is an example of where there's a whole separate cldr tree with stabilized data. Maybe we could come up with something similar for examples.

@srl295 srl295 merged commit 587bde6 into unicode-org:brs/v46vxml Jul 19, 2024
8 of 10 checks passed
srl295 added a commit that referenced this pull request Jul 20, 2024
32eea30 CLDR-17812 move to -z FINAL_TESTING (#3890)
b9780d0 CLDR-17812 yo_BJ: regen from yo (#3892)
c7b9d67 CLDR-17812 blo: drop incomplete groups (#3891)
9f3f4d8 CLDR-17812 v46 vxml fixes for ddls (#3889)
587bde6 CLDR-17812 v46 vxml: Update TestExampleGenerator/TestUnicodeSetExamples to match hi change in #3883 (#3888)
5f124f6 CLDR-17812 v46 vxml: Add {0} for de speed-light-speed one, fix error wording in TestCheckCLDR/TestPlaceholders (#3887)
dec0953 CLDR-17812 v46 vxml: update locale for TestAliases/testCountBase (#3886)
9d5d8a8 CLDR-17812 v46 vxml: annotations: ff: reinstate empty file (#3882)
e11c389 CLDR-17812 v46 vxml: Regenerate personName test data (#3885)
93ab67c CLDR-17812 v46 vxml: update exemplars to match vetter input (#3883)
2575193 CLDR-17812 v46 vxml: update testdata to match vetter input (#3884)
a51bdf0 CLDR-17812 ii: drop yiii numbering system (#3881)
ee497ab CLDR-17812 Generate VXML (#3879)

Co-authored-by: Steven R. Loomis <[email protected]>
Co-authored-by: Peter Edberg <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants