Add new test, fix existing test, throw better exception.

NetTopologySuite · Sep 14, 2023 · ff417b6 · ff417b6
1 parent 2fe6026
commit ff417b6
Show file tree

Hide file tree

Showing 2 changed files with 26 additions and 2 deletions.
diff --git a/src/NetTopologySuite.IO.GeoJSON4STJ/Converters/StjFeatureConverter.cs b/src/NetTopologySuite.IO.GeoJSON4STJ/Converters/StjFeatureConverter.cs
@@ -1,4 +1,5 @@
 using System;
+using System.Text;
 using System.Text.Json;
 using System.Text.Json.Serialization;
 using NetTopologySuite.Features;
@@ -107,7 +108,12 @@ public override IFeature Read(ref Utf8JsonReader reader, Type objectType, JsonSe
                                 break;
 
                             case JsonTokenType.Number:
-                                throw new JsonException("Number value cannot be boxed as a decimal: " + reader.GetString());
+                                // "number" is technically SUPPOSED to be a float, so even though we
+                                // prefer reading it as a decimal, it should at least be safe enough
+                                // to use GetDouble() for the exception message, and certainly safer
+                                // than GetString() which an older version of this code used to use,
+                                // which ALWAYS throws for Number tokens (airbreather 2023-09-14).
+                                throw new JsonException("Number value cannot be boxed as a decimal: " + reader.GetDouble());
 
                             case JsonTokenType.String:
                                 feature.Id = reader.GetString();

diff --git a/test/NetTopologySuite.IO.GeoJSON4STJ.Test/Converters/FeatureConverterTest.cs b/test/NetTopologySuite.IO.GeoJSON4STJ.Test/Converters/FeatureConverterTest.cs
@@ -227,7 +227,25 @@ public void TestFeatureIdSerializedToRoot()
             };
 
             string expected = "{\"type\":\"Feature\",\"id\":1,\"geometry\":{\"type\":\"Point\",\"coordinates\":[0,0]},\"properties\":{\"name\":\"Test feature\"}}";
-            Assert.That(expected, Is.EqualTo(expected));
+            Assert.That(JsonSerializer.Serialize(feature, options), Is.EqualTo(expected));
+        }
+
+        [GeoJsonIssueNumber(132)]
+        [Test]
+        public void TestNumericFeatureIdMustBeValidDecimal()
+        {
+            string serialized = $@"
+{{
+    ""type"": ""Feature"",
+    ""id"": {double.MaxValue},
+    ""geometry"": {{
+        ""type"": ""Point"",
+        ""coordinates"": [0, 0]
+    }}
+}}
+            ";
+
+            Assert.That(() => JsonSerializer.Deserialize<Feature>(serialized, DefaultOptions), Throws.InstanceOf<JsonException>());
         }
     }
 }