Skip to content

Re-enable CurrentCulture_Default_Is_Specific test on iOS CI#130298

Open
matouskozak wants to merge 1 commit into
dotnet:mainfrom
matouskozak:matouskozak-ios-globalization-test-check
Open

Re-enable CurrentCulture_Default_Is_Specific test on iOS CI#130298
matouskozak wants to merge 1 commit into
dotnet:mainfrom
matouskozak:matouskozak-ios-globalization-test-check

Conversation

@matouskozak

Copy link
Copy Markdown
Member

This PR re-enables the CurrentCulture_Default_Is_Specific test on iOS/MacCatalyst/tvOS to check if the globalization behavior difference on iOS simulators on CI (#111501) is still reproducing.

The test was disabled in #111032 because CultureInfo.CurrentCulture was returning en (neutral) on CI simulators instead of en-US (specific) as expected. This PR removes the SkipOnPlatform attribute to verify if the issue persists on current CI infrastructure.

…tvOS

Remove SkipOnPlatform attribute to verify if the globalization behavior
difference on iOS simulators on CI (issue dotnet#111501) is still reproducing.

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

This PR re-enables the CurrentCulture_Default_Is_Specific globalization test on Apple platforms by removing the iOS-family skip, so CI will once again validate that the default CultureInfo.CurrentCulture is specific (non-neutral).

Changes:

  • Remove the [SkipOnPlatform(...)] suppression for iOS/MacCatalyst/tvOS on CurrentCulture_Default_Is_Specific.

@matouskozak

Copy link
Copy Markdown
Member Author

/azp run runtime-ioslikesimulator

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines successfully started running 1 pipeline(s).

@tarekgh tarekgh added os-ios Apple iOS os-tvos Apple tvOS labels Jul 7, 2026
@tarekgh tarekgh added this to the 11.0.0 milestone Jul 7, 2026
@dotnet-policy-service

Copy link
Copy Markdown
Contributor

Tagging subscribers to 'os-ios': @vitek-karas, @kotlarmilos, @steveisok, @akoeplinger
See info in area-owners.md if you want to be subscribed.

@dotnet-policy-service

Copy link
Copy Markdown
Contributor

Tagging subscribers to 'os-tvos': @vitek-karas, @kotlarmilos, @steveisok, @akoeplinger
See info in area-owners.md if you want to be subscribed.

@matouskozak

Copy link
Copy Markdown
Member Author

Test is passing
image

@tarekgh tarekgh left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM if the test is not failing anymore.

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.

3 participants