From bf608fa271a4fcbb82490b7b6dc8d80a1f985ab8 Mon Sep 17 00:00:00 2001 From: AJ Isaacs Date: Tue, 29 Sep 2026 22:02:32 -0400 Subject: [PATCH] test(math): name fraction culture coverage accurately --- OpenNest.Tests/Math/FractionTests.cs | 7 ++++--- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/OpenNest.Tests/Math/FractionTests.cs b/OpenNest.Tests/Math/FractionTests.cs index 7c7dd8d..085f084 100644 --- a/OpenNest.Tests/Math/FractionTests.cs +++ b/OpenNest.Tests/Math/FractionTests.cs @@ -29,14 +29,15 @@ public class FractionTests [Theory] [InlineData("en-US")] [InlineData("fr-FR")] - public void Parse_UsesCurrentCultureForOrdinaryIntegerComponents(string culture) + public void Parse_AcceptsOrdinaryFractionsAcrossCultures(string culture) { var original = CultureInfo.CurrentCulture; try { CultureInfo.CurrentCulture = CultureInfo.GetCultureInfo(culture); - // The implementation uses current-culture double.Parse, not invariant parsing. - // Ordinary digits are invariant-parseable too; exotic numeric syntax is not this contract. + // Compatibility coverage: ordinary digits parse identically under these cultures. + // This does not prove current-culture vs invariant parsing; exotic numeric syntax + // is not this contract. var expected = double.Parse("1") + System.Math.Round(double.Parse("3") / double.Parse("4"), 8); Assert.Equal(expected, Fraction.Parse("1 3/4"), 8); }