Test suite audit: fix names, remove vacuous property, add coverage

Author: lpgauth

test/decode_test.exs

@@ -108,6 +108,5 @@ defmodule Torque.DecodeTest do
         Torque.decode!("{invalid}")
       end
     end
-
   end
 end

test/encode_test.exs

@@ -96,7 +96,7 @@ defmodule Torque.EncodeTest do
       assert [%{"id" => 1}, %{"id" => 2}] = Jason.decode!(json)
     end
 
-    test "atom values encoded as strings" do
+    test "atom map values encoded as strings" do
       assert {:ok, json} = Torque.encode(%{status: :active})
       assert %{"status" => "active"} = Jason.decode!(json)
     end
@@ -120,6 +120,12 @@ defmodule Torque.EncodeTest do
         Torque.encode!(self())
       end
     end
+
+    test "invalid UTF-8 binary raises" do
+      assert_raise ArgumentError, ~r/invalid_utf8/, fn ->
+        Torque.encode!(<<0x80>>)
+      end
+    end
   end
 
   describe "encode_to_iodata/1" do
@@ -138,5 +144,11 @@ defmodule Torque.EncodeTest do
         Torque.encode_to_iodata(self())
       end
     end
+
+    test "invalid UTF-8 binary raises ArgumentError" do
+      assert_raise ArgumentError, ~r/invalid_utf8/, fn ->
+        Torque.encode_to_iodata(<<0x80>>)
+      end
+    end
   end
 end

test/torque_test.exs test/pointer_test.exstest/torque_test.exs renamed to test/pointer_test.exs

@@ -541,6 +541,13 @@ defmodule Torque.PropertyTest do
       assert {:error, :malformed_proplist} = Torque.encode({[{:a, :b, :c}]})
     end
 
+    # BEAM rejects non-finite floats (both arithmetic overflow and binary_to_term
+    # validation), so this NIF code path is not reachable from Elixir.
+    @tag :skip
+    test "non-finite float returns non_finite_float" do
+      assert {:error, :non_finite_float} = Torque.encode(:infinity)
+    end
+
     test "encode_to_iodata raises with unsupported_type message" do
       assert_raise ArgumentError, ~r/unsupported_type/, fn ->
         Torque.encode_to_iodata(self())
@@ -589,17 +596,6 @@ defmodule Torque.PropertyTest do
       end
     end
 
-    property "numeric-looking string keys work alongside integer array indexes" do
-      check all(n <- integer(0..100)) do
-        # Object key that looks like an integer — should be treated as a string key
-        json = ~s({"#{n}": "string_val"})
-        {:ok, doc} = Torque.parse(json)
-        # Accessing "/N" on an object should still find the string key "N"
-        result = Torque.get(doc, "/#{n}")
-        assert match?({:ok, _}, result) or match?({:error, :no_such_field}, result)
-      end
-    end
-
     test "path without leading slash returns no_such_field" do
       {:ok, doc} = Torque.parse(~s({"a":1}))
       assert {:error, :no_such_field} = Torque.get(doc, "a")