Fix raise on missing values

ExWeb3 · Jan 29, 2024 · 83d526d · 83d526d
1 parent 392f133
commit 83d526d
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 0 deletions.
diff --git a/lib/ethers/transaction.ex b/lib/ethers/transaction.ex
@@ -200,6 +200,8 @@ defmodule Ethers.Transaction do
     tx
     |> Map.from_struct()
     |> Map.new(fn
+      {k, nil} -> {k, nil}
+      {k, ""} -> {k, nil}
       {k, v} when k in @integer_type_values -> {k, Utils.hex_to_integer!(v)}
       {k, v} when k in @binary_type_values -> {k, Utils.hex_decode!(v)}
       {k, v} -> {k, v}

diff --git a/test/ethers/transaction_test.exs b/test/ethers/transaction_test.exs
@@ -55,5 +55,13 @@ defmodule Ethers.TransactionTest do
       assert is_binary(decoded.signature_r)
       assert is_binary(decoded.signature_s)
     end
+
+    test "does not fail with missing values" do
+      assert %{signature_recovery_id: nil} =
+               Transaction.decode_values(%{@transaction_fixture | signature_recovery_id: nil})
+
+      assert %{signature_recovery_id: nil} =
+               Transaction.decode_values(%{@transaction_fixture | signature_recovery_id: ""})
+    end
   end
 end