-
Notifications
You must be signed in to change notification settings - Fork 4.7k
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
[6.0] Test failures on OSX: OSVersion_ValidVersion_OSX, CurrencyNegativePattern_Get_ReturnsExpected_ByLocale #80252
Comments
I couldn't figure out the best area label to add to this issue. If you have write-permissions please help me learn by adding exactly one area label. |
Tagging subscribers to this area: @dotnet/runtime-infrastructure Issue DetailsBuild InformationBuild: https://dev.azure.com/dnceng-public/cbb18261-c48f-4abb-8651-8cdcb5474649/_build/results?buildId=125170 Failing Configurations (3)
Related/potential cause: #80030 Error Message
Exception Message
CallStack
Exception Message
Stack Trace
|
Tagging subscribers to this area: @dotnet/area-system-globalization Issue DetailsBuild InformationBuild: https://dev.azure.com/dnceng-public/cbb18261-c48f-4abb-8651-8cdcb5474649/_build/results?buildId=125170 Failing Configurations (3)
Related/potential cause: #80030 Error Message
Exception Message
CallStack
Exception Message
Stack Trace
ReportSummary
|
Here's an example of a failing test happening in the internal->public merge PR for 6.0:
|
@carlossanlop this looks to me the OS detection is the culprit here and not the test itself. This may explain the other Environment test failure which is same. |
Tagging subscribers to this area: @dotnet/area-system-runtime Issue DetailsBuild InformationBuild: https://dev.azure.com/dnceng-public/cbb18261-c48f-4abb-8651-8cdcb5474649/_build/results?buildId=125170 Failing Configurations (3)
Related/potential cause: #80030 Error Message
Exception Message
CallStack
Exception Message
Stack Trace
ReportSummary
|
I agree with Tarek that this needs to be investigated, ideally before next servicing release's code-complete, as the failures might indicate a product regression. |
The OSVersion_ValidVersion_OSX test isn't that concerning, it's just a test issue regarding how the version is checked. The CurrencyNegativePattern_Get_ReturnsExpected_ByLocale was fixed by #63971 so we'd need to backport that to 6.0 |
@akoeplinger looks you are right. I was looking at the main branch code and not 6.0 release. Any idea why EnvironmentTest is failing too? I can port the globalization test fix to 6.0. |
@tarekgh I see we're at least missing this change to the test from this commit in 6.0 which makes it only look at the major version: 5da4464#diff-3b1296c3b8a733ad1c027de7b5c653d0843afcc959cf9b2656160a9dcf8c0aa5R193-R194 But that doesn't explain the full story since we're still getting |
I figured it out, will send a PR. I'll include the globalization fix in that. |
Fixed by #80560 |
Build Information
Build: https://dev.azure.com/dnceng-public/cbb18261-c48f-4abb-8651-8cdcb5474649/_build/results?buildId=125170
Build error leg or test failing:
System.Tests.EnvironmentTests.OSVersion_ValidVersion_OSX
,System.Globalization.Tests.NumberFormatInfoCurrencyNegativePattern.CurrencyNegativePattern_Get_ReturnsExpected_ByLocale(locale: \"es-BO\")
Pull request: #79549, #80190, #80209
Failing Configurations (3)
Related/potential cause: #80030
Error Message
System.Tests.EnvironmentTests.OSVersion_ValidVersion_OSX
Exception Message
CallStack
System.Globalization.Tests.NumberFormatInfoCurrencyNegativePattern.CurrencyNegativePattern_Get_ReturnsExpected_ByLocale(locale: \"es-BO\")
Exception Message
Stack Trace
Report
Summary
The text was updated successfully, but these errors were encountered: