statetest: Error on parsing Transction with mixed legacy/1559 contents

Author: axic

test/statetest/statetest_loader.cpp

@@ -195,6 +195,9 @@ static void from_json_tx_common(const json::json& j, state::Transaction& o)
         o.kind = state::Transaction::Kind::legacy;
         o.max_gas_price = from_json<intx::uint256>(*gas_price_it);
         o.max_priority_gas_price = o.max_gas_price;
+        if (j.contains("maxFeePerGas") || j.contains("maxPriorityFeePerGas"))
+            throw std::invalid_argument(
+                "Misformatted transaction -- contains both legacy and 1559 fees");
     }
     else
     {

test/unittests/statetest_loader_tx_test.cpp

@@ -101,14 +101,6 @@ TEST(statetest_loader, tx_confusing)
         "accessList": []
     })";
 
-    const auto tx = test::from_json<state::Transaction>(json::json::parse(input));
-    EXPECT_EQ(tx.kind, state::Transaction::Kind::legacy);
-    EXPECT_EQ(tx.data, (bytes{0xb0, 0xb1}));
-    EXPECT_EQ(tx.gas_limit, 0x9091);
-    EXPECT_EQ(tx.value, 0xe0e1);
-    EXPECT_EQ(tx.sender, 0xa0a1_address);
-    EXPECT_EQ(tx.to, 0xc0c1_address);
-    EXPECT_EQ(tx.max_gas_price, 0x8081);
-    EXPECT_EQ(tx.max_priority_gas_price, 0x8081);
-    EXPECT_TRUE(tx.access_list.empty());
+    EXPECT_THROW(
+        test::from_json<state::Transaction>(json::json::parse(input)), std::invalid_argument);
 }