Skip to content

Conversation

Copy link
Contributor

Copilot AI commented Jan 14, 2026

Description

The fuzzer compared culture names using .ToLower(), which fails when CultureInfo normalizes aliases to canonical forms (e.g., "tur" → "tr", "bul" → "bg").

Changes:

  • Use CultureInfo.GetCultureInfo() to obtain normalized culture names for comparison
  • Handle invariant culture ("c"/"C" → ""), null, and empty string cases
  • Add test ToAssemblyNameNormalizesCultureAliases documenting alias normalization behavior

Customer Impact

Internal fuzzing infrastructure failure. No customer impact.

Regression

No, pre-existing issue in fuzzer test logic.

Testing

  • Manually verified culture normalization: tur→tr, bul→bg, c→""
  • Added test cases for Turkish and Bulgarian aliases
  • Verified null/empty string preservation

Risk

Low. Changes isolated to fuzzer test code, no production code modified.

Original prompt

The AssemblyNameInfoFuzzer is reporting failures around culture names.
E.g.

Unhandled exception. System.Exception: Expected=bul Actual=bg
   at DotnetFuzzing.Assert.<Equal>g__Throw|1_0[T](T expected, T actual) in C:\MihaZupan\runtime-debug\src\libraries\Fuzzing\DotnetFuzzing\Assert.cs:line 31
   at DotnetFuzzing.Assert.Equal[T](T expected, T actual) in C:\MihaZupan\runtime-debug\src\libraries\Fuzzing\DotnetFuzzing\Assert.cs:line 27
   at DotnetFuzzing.Fuzzers.AssemblyNameInfoFuzzer.Test(Span`1 span) in C:\MihaZupan\runtime-debug\src\libraries\Fuzzing\DotnetFuzzing\Fuzzers\AssemblyNameInfoFuzzer.cs:line 79

✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

Copilot AI changed the title [WIP] Fix culture name mismatches in AssemblyNameInfoFuzzer Fix AssemblyNameInfoFuzzer culture name normalization Jan 14, 2026
Copilot AI requested a review from MihaZupan January 14, 2026 14:04
@dotnet-policy-service
Copy link
Contributor

Tagging subscribers to this area: @dotnet/area-system-reflection-metadata
See info in area-owners.md if you want to be subscribed.

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