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

Failing Test: ICU4N.Dev.Test.Collate.CollationServiceTest::TestRegisterFactory() #37

Open
NightOwl888 opened this issue Oct 22, 2021 · 0 comments
Labels
is:bug Something isn't working pri:normal up for grabs This issue is open to be worked on by anyone

Comments

@NightOwl888
Copy link
Owner

NightOwl888 commented Oct 22, 2021

ICU4N.Dev.Test.Collate.CollationServiceTest::TestRegisterFactory() is failing on Linux due to the Collator.GetDisplayName(UCultureInfo) method not returning the correct name for a custom UCultureInfo instance registered with Collator.GetDisplayName().

Impact

The test is known to fail on Ubuntu 18.0.4 and Ubuntu 20.0.4 and only on .NET 5 and higher. While this makes it highly likely the issue is related to the ICU integration in .NET, changing to NLS seems to have no effect on the problem.

More investigation is needed, but due to the fact this is only returning display text for a culture it doesn't seem to be a blocker for releases at this point.

@NightOwl888 NightOwl888 added is:bug Something isn't working pri:normal up for grabs This issue is open to be worked on by anyone labels Oct 24, 2021
@NightOwl888 NightOwl888 changed the title Failing Test: ICU4N.Dev.Test.Collate.CollationServiceTest::TestRegister() Failing Test: ICU4N.Dev.Test.Collate.CollationServiceTest::TestRegisterFactory() Oct 24, 2021
NightOwl888 added a commit that referenced this issue Oct 24, 2021
…gnore test failures on Linux until #37 can be resolved
NightOwl888 added a commit that referenced this issue Jan 14, 2022
…isabled test on .NET Core 3.x because it is failing on Linux (see #37)
NightOwl888 added a commit that referenced this issue Sep 19, 2024
NightOwl888 added a commit that referenced this issue Sep 19, 2024
…sterFactory(): Enabled testing on Linux. Fixes #37.
NightOwl888 added a commit that referenced this issue Sep 19, 2024
…TestRegisterFactory(): Enabled testing on Linux. Fixes #37."

This reverts commit f861ee3.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
is:bug Something isn't working pri:normal up for grabs This issue is open to be worked on by anyone
Projects
None yet
Development

No branches or pull requests

1 participant