chore: remove disabled tests

Author: JamesPiechota

src/dev_codec_ans104.erl

@@ -447,24 +447,4 @@ field_and_tag_ordering_test() ->
     Wallet = hb:wallet(),
     SignedTABM = hb_message:commit(
         UnsignedTABM, #{priv_wallet => Wallet}, <<"[email protected]">>),
-    ?assert(hb_message:verify(SignedTABM)).
-
-%% @doc This test is disabled until we can resolve an exception that gets
-%% thrown during the `to/3` call.
-nested_data_test_disabled() ->
-    TABM = #{
-        <<"tag">> => <<"level-1">>,
-        <<"data">> => #{
-            <<"tag">> => <<"level-2">>,
-            <<"data">> => #{
-                <<"tag">> => <<"level-3">>,
-                <<"data">> => <<"nested-data">>
-            }  
-        }
-    },
-    Wallet = hb:wallet(),
-    SignedTABM = hb_message:commit(
-        TABM, #{priv_wallet => Wallet}, <<"[email protected]">>),
-    ?assert(hb_message:verify(SignedTABM)),
-    SignedTX = hb_util:ok(to(SignedTABM, #{}, #{})),
-    ?assert(ar_bundles:verify_item(SignedTX)).
+    ?assert(hb_message:verify(SignedTABM)).

src/dev_codec_tx.erl

@@ -610,17 +610,8 @@ nested_data_tabm_test() ->
         <<"type">> => <<"rsa-pss-sha256">>,
         <<"bundle">> => <<"true">>
     },
-    do_tabm_roundtrips(UnsignedTX, UnsignedTABM, NoLinksCommitment, true),
-
-    % XXX: Fails in some ans104 code, commenting out until we can resolve.
-    % LinksCommitment = #{
-    %     <<"commitment-device">> => <<"[email protected]">>,
-    %     <<"committed">> => [<<"data+link">>, <<"tag">>],
-    %     <<"type">> => <<"rsa-pss-sha256">>,
-    %     <<"bundle">> => <<"false">>
-    % },
-    % do_tabm_roundtrips(UnsignedTX, UnsignedTABM, LinksCommitment, false),
-    ok.
+    % only bundle true is supported
+    do_tabm_roundtrips(UnsignedTX, UnsignedTABM, NoLinksCommitment, true).
 
 nested_non_data_key_tabm_test() ->
     UnsignedTABM = #{
@@ -664,17 +655,8 @@ nested_non_data_key_tabm_test() ->
         <<"type">> => <<"rsa-pss-sha256">>,
         <<"bundle">> => <<"true">>
     },
-    do_tabm_roundtrips(UnsignedTX, UnsignedTABM, NoLinksCommitment, true),
-
-    % XXX: Fails in some ans104 code, commenting out until we can resolve.
-    % LinksCommitment = #{
-    %     <<"commitment-device">> => <<"[email protected]">>,
-    %     <<"committed">> => [<<"a1+link">>, <<"tag1">>],
-    %     <<"type">> => <<"rsa-pss-sha256">>,
-    %     <<"bundle">> => <<"false">>
-    % },
-    % do_tabm_roundtrips(UnsignedTX, UnsignedTABM, LinksCommitment, false),
-    ok.
+    % only bundle true is supported
+    do_tabm_roundtrips(UnsignedTX, UnsignedTABM, NoLinksCommitment, true).
 
 nested_multiple_tabm_test() ->
     UnsignedTABM = #{
@@ -736,17 +718,8 @@ nested_multiple_tabm_test() ->
         <<"type">> => <<"rsa-pss-sha256">>,
         <<"bundle">> => <<"true">>
     },
-    do_tabm_roundtrips(UnsignedTX, UnsignedTABM, NoLinksCommitment, true),
-
-    % XXX: Fails in some ans104 code, commenting out until we can resolve.
-    % LinksCommitment = #{
-    %     <<"commitment-device">> => <<"[email protected]">>,
-    %     <<"committed">> => [<<"a1+link">>, <<"tag1">>],
-    %     <<"type">> => <<"rsa-pss-sha256">>,
-    %     <<"bundle">> => <<"false">>
-    % },
-    % do_tabm_roundtrips(UnsignedTX, UnsignedTABM, LinksCommitment, false),
-    ok.
+    % only bundle true is supported
+    do_tabm_roundtrips(UnsignedTX, UnsignedTABM, NoLinksCommitment, true).
 
 %% @doc This test is disabled for now. Unclear whether we should deserialize
 %% data by default. If we don't need to deserialize data, then this test