-
Notifications
You must be signed in to change notification settings - Fork 4.8k
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
Incorrect value in FirstDayOfWeek test #93354
Comments
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=432866 Error MessageFill the error message using step by step known issues guidance. {
"ErrorMessage": "Expected: Monday",
"ErrorPattern": "",
"BuildRetry": false,
"ExcludeConsoleLog": false
}
Known issue validationBuild: 🔎 https://dev.azure.com/dnceng-public/public/_build/results?buildId=432866
|
Tagging subscribers to 'arch-wasm': @lewing Issue DetailsBuild InformationBuild: https://dev.azure.com/dnceng-public/cbb18261-c48f-4abb-8651-8cdcb5474649/_build/results?buildId=432866 Error MessageFill the error message using step by step known issues guidance. {
"ErrorMessage": "Expected: Monday",
"ErrorPattern": "",
"BuildRetry": false,
"ExcludeConsoleLog": false
}
Known issue validationBuild: 🔎 https://dev.azure.com/dnceng-public/public/_build/results?buildId=432866 Report
Summary
|
@SmartmanApps thank you for trying to help. However, it is not that simple. This failure is in test that checks all locales available on WASM (~200) and makes sure native JS code returns the correct day of the week for them. I wish all locales had the same response to Line 20 in 8c21ead
Edit: it fails on "fr-FR" where ICU and ISO standard is Monday. |
Is this happening consistently or just occasionally? |
Oddly, I don't see any failures in rolling builds in the last 2 weeks! |
And the tests ran on the last PR that bumped the chrome version, where they didn't fail either. |
Occasionally but quite often. It never fails for me when I try to reproduce. Pavel has an idea that it might depend on location of data center where CI tests are run, so next step would be to try to repro changing TZ on the machine to see if I find the one that fails. |
Build Information
Build: https://dev.azure.com/dnceng-public/cbb18261-c48f-4abb-8651-8cdcb5474649/_build/results?buildId=432866
Build error leg or test failing: System.Globalization.Tests.DateTimeFormatInfoFirstDayOfWeek.FirstDayOfWeek
Pull request: #93224
Error Message
Fill the error message using step by step known issues guidance.
Known issue validation
Build: 🔎 https://dev.azure.com/dnceng-public/public/_build/results?buildId=432866
Error message validated:
Expected: Monday
Result validation: ✅ Known issue matched with the provided build.
Validation performed at: 10/11/2023 5:08:28 PM UTC
Report
Summary
The text was updated successfully, but these errors were encountered: