add support for new decimal

ExpressApp · Dec 25, 2024 · e2590f0 · e2590f0
1 parent b294d93
commit e2590f0
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 3 deletions.
diff --git a/lib/construct/type.ex b/lib/construct/type.ex
@@ -218,7 +218,11 @@ defmodule Construct.Type do
   def cast(:reference, term) when is_reference(term), do: {:ok, term}
 
   def cast(:decimal, term) when is_binary(term) do
-    validate_decimal(apply(Decimal, :parse, [term]))
+    case apply(Decimal, :parse, [term]) do
+      {:ok, term} -> validate_decimal({:ok, term})
+      {%{__struct__: Decimal} = term, _} -> validate_decimal({:ok, term})
+      :error -> :error
+    end
   end
   def cast(:decimal, term) when is_integer(term) do
     {:ok, apply(Decimal, :new, [term])}

diff --git a/test/construct/type_test.exs b/test/construct/type_test.exs
@@ -136,9 +136,9 @@ defmodule Construct.TypeTest do
           == Type.cast(:decimal, 1)
       assert {:ok, Decimal.new("1")}
           == Type.cast(:decimal, Decimal.new("1"))
-      assert :error
+      assert {:ok, Decimal.new("NaN")}
           == Type.cast(:decimal, "nan")
-      assert :error
+      assert {:ok, Decimal.new("NaN")}
           == Type.cast(:decimal, Decimal.new("NaN"))
       assert :error
           == Type.cast(:decimal, Decimal.new("Infinity"))