ConvertIcuTimeFormatString: convert narrow no-break spaces to spaces too.

[tmds]

Fixes #83571.

@stephentoub ptal.

[ghost]

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

Issue Details

---

Fixes #83571.

@stephentoub ptal.

Author:	tmds
Assignees:	-
Labels:	area-System.Globalization, community-contribution
Milestone:	-

[tarekgh]

we are discussing the right fix on the issue #83571

[tmds]

@tarekgh I've updated the implementation. Can you take a look?
I have not included a test yet, is there a test I can use as a base?

[tarekgh]

@tmds the change looks good to me, Thanks. We need to add tests especially parsing date/time strings including several types of spaces. You may look at the file

runtime/src/libraries/System.Runtime/tests/System/DateTimeTests.cs

Line 1968 in ce48579

public static IEnumerable<object[]> Parse_ValidInput_Succeeds_MemberData()

which has different test cases which you can mimic to test parsing. Also, it will be good if you test formatting too (which is the original reported issue).

[tarekgh]

@tmds did you have a chance to address the remaining feedback on this PR? Thanks!

[tmds]

@tarekgh I've added a test. ptal.

[tarekgh]

The CI failures are known unrelated issues. I logged #84323 too for the unknown one.

[tarekgh]

@tmds thanks a lot for the help with this issue.