Skip to content

[wasm] Fix invariant globalization test - #56121

Merged
radical merged 1 commit into
dotnet:mainfrom
radical:fix-invar-tests
Jul 22, 2021
Merged

[wasm] Fix invariant globalization test#56121
radical merged 1 commit into
dotnet:mainfrom
radical:fix-invar-tests

Conversation

@radical

Copy link
Copy Markdown
Member

The following commit changed the behavior to throw CultureNotFoundException
when creating cultures in invariant mode.

commit 04dac7b0fede29d44f896c5fd793754f83974175
Author: Tarek Mahmoud Sayed <tarekms@microsoft.com>
Date: Thu Jul 1 11:55:05 2021 -0700
Allow restricting cultures creation with any arbitrary names in Globalization Invariant Mode (#54247)

This commit updates the corresponding test in Wasm.Build.Tests to
handle that explicitly.

Fixes#55838

The following commit changed the behavior to throw `CultureNotFoundException`
when creating cultures in invariant mode.
```
commit 04dac7b
Author: Tarek Mahmoud Sayed <tarekms@microsoft.com>
Date: Thu Jul 1 11:55:05 2021 -0700
Allow restricting cultures creation with any arbitrary names in Globalization Invariant Mode (dotnet#54247)
```
This commit updates the corresponding test in `Wasm.Build.Tests` to
handle that explicitly.
Fixesdotnet#55838
@radicalradical added the arch-wasm WebAssembly architecture label Jul 22, 2021
@ghost

Copy link
Copy Markdown

I couldn't figure out the best area label to add to this PR. If you have write-permissions please help me learn by adding exactly one area label.

@ghost

Copy link
Copy Markdown

Tagging subscribers to 'arch-wasm': @lewing
See info in area-owners.md if you want to be subscribed.

Issue Details

The following commit changed the behavior to throw CultureNotFoundException
when creating cultures in invariant mode.

commit 04dac7b0fede29d44f896c5fd793754f83974175
Author: Tarek Mahmoud Sayed <tarekms@microsoft.com>
Date: Thu Jul 1 11:55:05 2021 -0700
Allow restricting cultures creation with any arbitrary names in Globalization Invariant Mode (#54247)

This commit updates the corresponding test in Wasm.Build.Tests to
handle that explicitly.

Fixes #55838

Author:radical
Assignees:-
Labels:

arch-wasm

Milestone:-

@tarekghtarekgh added this to the 6.0.0 milestone Jul 22, 2021
@radical

Copy link
Copy Markdown
MemberAuthor

Runtime test failure is #11063

@radical
radical merged commit 6d15a02 into dotnet:mainJul 22, 2021
@radical
radical deleted the fix-invar-tests branch July 22, 2021 16:37
@ghostghost locked as resolved and limited conversation to collaborators Aug 21, 2021
Sign up for freeto subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

arch-wasmWebAssembly architecture

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[wasm] improve Wasm.Build.Tests to expect System.Globalization.CultureNotFoundException

2 participants

@radical@tarekgh