Improve test coverage for numeric string object key fix

Author: lpgauth

.github/workflows/ci.yml

@@ -37,6 +37,8 @@ jobs:
     strategy:
       matrix:
         include:
+          - otp: "28"
+            elixir: "1.18"
           - otp: "27"
             elixir: "1.18"
           - otp: "26"

test/property_test.exs

@@ -613,15 +613,6 @@ defmodule Torque.PropertyTest do
       {:ok, doc2} = Torque.parse(~s({"a":[1,2,3]}))
       assert {:error, :no_such_field} = Torque.get(doc2, "/a/0/x")
     end
-
-    property "numeric string keys on objects always return no_such_field" do
-      check all(n <- integer(0..100)) do
-        json = ~s({"#{n}": "val"})
-        {:ok, doc} = Torque.parse(json)
-        # A purely numeric segment on an object (not an array) never matches
-        assert {:error, :no_such_field} = Torque.get(doc, "/#{n}")
-      end
-    end
   end
 
   # ---- get/3 raises on nesting_too_deep ----

test/torque_test.exs

@@ -139,13 +139,21 @@ defmodule Torque.PointerTest do
       assert {:ok, "ten"} = Torque.get(doc, "/10")
     end
 
-    test "nested numeric string object key id reachable via JSON Pointer" do
+    test "nested numeric string object key is reachable via JSON Pointer" do
       {:ok, doc} = Torque.parse(~s({"k1":"v1","k2":{"10":"ten","n1":"nv1"}}))
       assert {:ok, "v1"} = Torque.get(doc, "/k1")
       assert {:ok, %{"10" => "ten", "n1" => "nv1"}} = Torque.get(doc, "/k2")
       assert {:ok, "ten"} = Torque.get(doc, "/k2/10")
       assert {:ok, "nv1"} = Torque.get(doc, "/k2/n1")
     end
+
+    test "numeric segment dispatches on node type" do
+      {:ok, obj_doc} = Torque.parse(~s({"0":"from-object"}))
+      {:ok, arr_doc} = Torque.parse(~s(["from-array"]))
+
+      assert {:ok, "from-object"} = Torque.get(obj_doc, "/0")
+      assert {:ok, "from-array"} = Torque.get(arr_doc, "/0")
+    end
   end
 
   describe "get_many/2" do
@@ -164,6 +172,11 @@ defmodule Torque.PointerTest do
       assert [] = Torque.get_many(doc, [])
     end
 
+    test "numeric string object keys" do
+      {:ok, doc} = Torque.parse(~s({"1":"one","2":"two"}))
+      assert [{:ok, "one"}, {:ok, "two"}] = Torque.get_many(doc, ["/1", "/2"])
+    end
+
     test "all fields", %{doc: doc} do
       paths = [
         "/id",