fix(test): rename ReprTests to Repr_Tests and add missing \x1f test case

antonsynd · claude · antonsynd · commit 0e016a35da51 · 2026-04-13T18:13:52.000-04:00
Follow project naming convention (underscore-separated class names) and
add the missing US (0x1f) control character to the unit test theory set
for complete boundary coverage.

Co-Authored-By: Claude Opus 4.6 (1M context) &lt;noreply@anthropic.com&gt;
diff --git a/src/Sharpy.Core.Tests/ReprTests.cs b/src/Sharpy.Core.Tests/ReprTests.cs
@@ -3,7 +3,7 @@
 
 namespace Sharpy.Core.Tests;
 
-public class ReprTests
+public class Repr_Tests
 {
     [Theory]
     [InlineData("\0", "'\\x00'")]
@@ -12,6 +12,7 @@ public class ReprTests
     [InlineData("\f", "'\\x0c'")]
     [InlineData("\v", "'\\x0b'")]
     [InlineData("\x1b", "'\\x1b'")]
+    [InlineData("\x1f", "'\\x1f'")]
     [InlineData("\x7f", "'\\x7f'")]
     public void Repr_ControlCharacter_EscapedAsHex(string input, string expected)
     {

Original file line number	Diff line number	Diff line change
`@@ -3,7 +3,7 @@`
`3`	`3`
`4`	`4`	`namespace Sharpy.Core.Tests;`
`5`	`5`
`6`		`-public class ReprTests`
	`6`	`+public class Repr_Tests`
`7`	`7`	`{`
`8`	`8`	`[Theory]`
`9`	`9`	`[InlineData("\0", "'\\x00'")]`
`@@ -12,6 +12,7 @@ public class ReprTests`
`12`	`12`	`[InlineData("\f", "'\\x0c'")]`
`13`	`13`	`[InlineData("\v", "'\\x0b'")]`
`14`	`14`	`[InlineData("\x1b", "'\\x1b'")]`
	`15`	`+ [InlineData("\x1f", "'\\x1f'")]`
`15`	`16`	`[InlineData("\x7f", "'\\x7f'")]`
`16`	`17`	`public void Repr_ControlCharacter_EscapedAsHex(string input, string expected)`
`17`	`18`	`{`