From 2befd05cafc211d474a9e9c1842b0fb567e3a8ce Mon Sep 17 00:00:00 2001
From: Corey Powell <mistdragon100@gmail.com>
Date: Mon, 23 Sep 2024 10:56:01 -0500
Subject: [PATCH] Added some tests for string_to_integer

And a decode test to ensure it works as intended with integers

EDIT: You know I should read properly next time
EDIT.2: Made the formatter happy
---
 test/geo/json_test.exs  | 96 ++++++++++++++++++++++++++++++++++-------
 test/geo/utils_test.exs | 17 ++++++++
 2 files changed, 97 insertions(+), 16 deletions(-)

diff --git a/test/geo/json_test.exs b/test/geo/json_test.exs
index 90726ff..3f73064 100644
--- a/test/geo/json_test.exs
+++ b/test/geo/json_test.exs
@@ -57,6 +57,45 @@ defmodule Geo.JSON.Test do
     assert_geojson_equal(exjson, new_exjson)
   end
 
+  test "GeoJson to Point (with integer components) and back" do
+    json = """
+      {
+        "type": "Point",
+        "coordinates": [100, 0]
+      }
+    """
+
+    exjson = Jason.decode!(json)
+    geom = Jason.decode!(json) |> Geo.JSON.decode!()
+
+    assert(geom.coordinates == {100.0, 0.0})
+
+    new_exjson = Geo.JSON.encode!(geom)
+    assert_geojson_equal(exjson, new_exjson)
+  end
+
+  test "GeoJson to Point (with string:integer components) and back" do
+    json = """
+      {
+        "type": "Point",
+        "coordinates": ["100", "0"]
+      }
+    """
+
+    exjson =
+      %{
+        "type" => "Point",
+        "coordinates" => [100.0, 0.0]
+      }
+
+    geom = Jason.decode!(json) |> Geo.JSON.decode!()
+
+    assert(geom.coordinates == {100.0, 0.0})
+
+    new_exjson = Geo.JSON.encode!(geom)
+    assert_geojson_equal(exjson, new_exjson)
+  end
+
   test "GeoJson Point without coordinates" do
     json = "{ \"type\": \"Point\", \"coordinates\": [] }"
     exjson = Jason.decode!(json)
@@ -387,23 +426,48 @@ defmodule Geo.JSON.Test do
     assert geom.geometries == []
   end
 
-  test "Decode seamlessly converts coordinates that are numbers-as-strings" do
-    check all(
-            x <- float(),
-            y <- float()
-          ) do
-      json = """
-        {
-          "properties": {},
-          "geometry": {
-            "type": "Point",
-            "coordinates": ["#{x}", "#{y}"]
-          },
-          "type": "Feature"
-        }
-      """
+  describe "decode seamlessly converts coordinates that are numbers-as-strings" do
+    test "works with floats" do
+      check all(
+              x <- float(),
+              y <- float()
+            ) do
+        json = """
+          {
+            "properties": {},
+            "geometry": {
+              "type": "Point",
+              "coordinates": ["#{x}", "#{y}"]
+            },
+            "type": "Feature"
+          }
+        """
+
+        assert %Geo.Point{coordinates: {^x, ^y}} = Jason.decode!(json) |> Geo.JSON.decode!()
+      end
+    end
 
-      assert %Geo.Point{coordinates: {^x, ^y}} = Jason.decode!(json) |> Geo.JSON.decode!()
+    test "works with integers" do
+      check all(
+              x <- integer(),
+              y <- integer()
+            ) do
+        json = """
+          {
+            "properties": {},
+            "geometry": {
+              "type": "Point",
+              "coordinates": ["#{x}", "#{y}"]
+            },
+            "type": "Feature"
+          }
+        """
+
+        # float coercion
+        fx = 0.0 + x
+        fy = 0.0 + y
+        assert %Geo.Point{coordinates: {^fx, ^fy}} = Jason.decode!(json) |> Geo.JSON.decode!()
+      end
     end
   end
 
diff --git a/test/geo/utils_test.exs b/test/geo/utils_test.exs
index d1c14c8..77a7234 100644
--- a/test/geo/utils_test.exs
+++ b/test/geo/utils_test.exs
@@ -2,6 +2,23 @@ defmodule Geo.Utils.Test do
   use ExUnit.Case, async: true
   use ExUnitProperties
 
+  describe "string_to_float/1" do
+    test "can convert a textual float" do
+      assert {:ok, +0.0} == Geo.Utils.string_to_float("0.0")
+      assert {:ok, 12.34} == Geo.Utils.string_to_float("12.34")
+    end
+
+    test "can convert a textual integer" do
+      assert {:ok, +0.0} == Geo.Utils.string_to_float("0")
+      assert {:ok, 12.0} == Geo.Utils.string_to_float("12")
+    end
+
+    test "can handle badly formatted float" do
+      assert {:error, :bad_arg} == Geo.Utils.string_to_float("0.x")
+      assert {:error, :bad_arg} == Geo.Utils.string_to_float("11f")
+    end
+  end
+
   test "Hex String to Float Conversion" do
     assert(Geo.Utils.hex_to_float("40000000") == 2.0)
     assert(Geo.Utils.hex_to_float("C0000000") == -2.0)