-
Notifications
You must be signed in to change notification settings - Fork 345
Implement numeric 'R' format #50
Comments
@KrzysztofCwalina I am trying to get the tests failing right now. I uncommented the line in Program.cs for the test generator and ran it and got an exception in GenerateNumericTypesTests(). When the StreamWriter is instantiated it uses "......\tests\NumberFormattingTests.Generated.cs" which doesn't seem to point to the correct folder and throws a DirectoryNotFoundException. I looked around and the correct folder seems to be: "corefxlab\tests\System.Text.Formatting.Tests" Am I misunderstanding how this is supposed to work? This is my first time working in the project. Any help would be appreciated! Thanks! |
So if I change the directories to: "..........\tests\System.Text.Formatting.Tests\TimeSpanFormattingTests.Generated.cs" Then build the project, the files seem to get written to the correct location. Running build produces a ton of output. I checked the code of the PS file and figured out I can run: Then I get several issues as expected with generated methods named *FormatR(). I have two issues:
Thanks again. |
@davidfowl Could you provide some direction? Thanks! |
Try everything again, we recently fixed some stuff that should make it easier to run tests from within VS (and on the command line). Though I'd have to take a look at that program that generates tests. |
@KrzysztofCwalina I don't think "R" is a valid format specifier for integral types. Can you clarify what the intent of this test is? |
I don't remember the exact context, but I see a comment in the Program.cs file that says // TODO: all of the formats should work. If we don't think we need R for integral types, then we should change the comment and/or cleanup the Program.cs file as it implies that many of the tests should be enabled in the future. |
Repro:
Result:
Some formatting tests will fail.
The text was updated successfully, but these errors were encountered: