Fix coreclr Test570 due to ICU on Windows feature

[safern]

Fixes: #35039

[ghost]

Tagging subscribers to this area: @ViktorHofer
Notify danmosemsft if you want to be subscribed.

[safern]

/azp run runtime-coreclr outerloop

[azure-pipelines]

Azure Pipelines could not run because the pipeline triggers exclude this branch/path.

[safern]

I queued a build from this branch: https://dev.azure.com/dnceng/public/_build/results?buildId=616798&view=results

[GrabYourPitchforks]

I reviewed the "build 18362" part of the change and confirmed that it matches the 19H1 RTM build. So there's my minor contribution. :)

[safern]

I reviewed the "build 18362" part of the change

Thanks 😄

[jkotas]

It would be better to fix this by moving the test to live under libraries where it belongs.

[jkotas]

If we have appropriate coverage under libraries, just delete the test.

[safern]

We already test this in Libraries:

runtime/src/libraries/System.Globalization/tests/NumberFormatInfo/NumberFormatInfoNumberDecimalDigits.cs

Line 15 in 1db45ed

yield return new object[] { CultureInfo.GetCultureInfo("en-US").NumberFormat, 2, 3 };

It seems like for PercentDecimalDigits we don't have such test case:
https://github.com/dotnet/runtime/blob/4f9ae42d861fcb4be2fcd5d3d55d5f227d30e723/src/libraries/System.Globalization/tests/NumberFormatInfo/NumberFormatInfoPercentDecimalDigits.cs

I guess I can just delete this test and add a test case in NumberFormatInfoPercentDecimalDigits tests

[safern]

Updated to remove the test and add the missing test data in libraries test.

[jkotas]

Thanks!

[jkotas]

The test failures are issues #35452 #35438 #35451