Test that transaction type is correct if authorizations present in call param

Signed-off-by: Miroslav Kovar <miroslavkovar@protonmail.com>

Author: mirgee

ethereum/core/src/test/java/org/hyperledger/besu/ethereum/transaction/CallParameterTest.java

@@ -152,13 +152,11 @@ public void deserializesAuthorizationList() throws JsonProcessingException {
               assertThat(auth.r())
                   .isEqualTo(
                       new BigInteger(
-                          "462a70678128d9dd8f5b8010aaecddda1ba9ad767f0807c341f38d4dcb7eb893",
-                          16));
+                          "462a70678128d9dd8f5b8010aaecddda1ba9ad767f0807c341f38d4dcb7eb893", 16));
               assertThat(auth.s())
                   .isEqualTo(
                       new BigInteger(
-                          "645f7afd51a86bafe8939c8498fc89769918a38213859843ad7b19ffd4273a48",
-                          16));
+                          "645f7afd51a86bafe8939c8498fc89769918a38213859843ad7b19ffd4273a48", 16));
             });
   }
 }

ethereum/core/src/test/java/org/hyperledger/besu/ethereum/transaction/TransactionSimulatorTest.java

@@ -40,6 +40,7 @@
 import org.hyperledger.besu.ethereum.core.BlockHeader;
 import org.hyperledger.besu.ethereum.core.BlockHeaderFunctions;
 import org.hyperledger.besu.ethereum.core.BlockHeaderTestFixture;
+import org.hyperledger.besu.ethereum.core.CodeDelegation;
 import org.hyperledger.besu.ethereum.core.Difficulty;
 import org.hyperledger.besu.ethereum.core.MiningConfiguration;
 import org.hyperledger.besu.ethereum.core.MutableWorldState;
@@ -59,9 +60,11 @@
 import org.hyperledger.besu.evm.account.Account;
 import org.hyperledger.besu.evm.account.MutableAccount;
 import org.hyperledger.besu.evm.gascalculator.FrontierGasCalculator;
+import org.hyperledger.besu.evm.tracing.OperationTracer;
 import org.hyperledger.besu.evm.worldstate.WorldUpdater;
 
 import java.math.BigInteger;
+import java.util.List;
 import java.util.Map;
 import java.util.Optional;
 import java.util.OptionalLong;
@@ -923,6 +926,47 @@ public void shouldSimulateLegacyTransactionWhenBaseFeeNotZero() {
     assertThat(result.get().isSuccessful()).isTrue();
   }
 
+  @Test
+  public void shouldGuessDelegateCodeTransactionTypeWhenAuthorizationsPresent() {
+    final CodeDelegation delegation =
+        new CodeDelegation(BigInteger.ONE, Address.fromHexString("0x1"), 42L, FAKE_SIGNATURE);
+
+    final CallParameter callParameter =
+        legacyTransactionCallParameterBuilder()
+            .codeDelegationAuthorizations(List.of(delegation))
+            .build();
+
+    final BlockHeader blockHeader = mockBlockchainAndWorldState(callParameter);
+
+    mockProtocolSpecForProcessWithWorldUpdater();
+
+    final Transaction expectedTx =
+        Transaction.builder()
+            .chainId(BigInteger.ONE)
+            .nonce(1L)
+            .gasLimit(blockHeader.getGasLimit())
+            .sender(callParameter.getSender().orElseThrow())
+            .to(callParameter.getTo().orElseThrow())
+            .value(callParameter.getValue().orElseThrow())
+            .payload(callParameter.getPayload().orElseThrow())
+            .signature(FAKE_SIGNATURE)
+            .codeDelegations(List.of(delegation))
+            .guessType()
+            .build();
+
+    mockProcessorStatusForTransaction(expectedTx, Status.SUCCESSFUL);
+
+    final Optional<TransactionSimulatorResult> result =
+        uncappedTransactionSimulator.process(
+            callParameter,
+            TransactionValidationParams.transactionSimulator(),
+            OperationTracer.NO_TRACING,
+            blockHeader);
+
+    assertThat(result).isPresent();
+    assertThat(result.get().transaction().getType()).isEqualTo(TransactionType.DELEGATE_CODE);
+  }
+
   private BlockHeader mockBlockchainAndWorldState(final CallParameter callParameter) {
     final BlockHeader blockHeader =
         mockBlockHeader(Hash.ZERO, 1L, Wei.ONE, DEFAULT_BLOCK_GAS_LIMIT);