Skip to content

Commit 552bc52

Browse files
committed
fix tests
Signed-off-by: Karim Taam <karim.t2am@gmail.com>
1 parent e12adea commit 552bc52

File tree

2 files changed

+23
-4
lines changed

2 files changed

+23
-4
lines changed

ethereum/core/src/test/java/org/hyperledger/besu/ethereum/MainnetBlockValidatorTest.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -101,7 +101,7 @@ public void setup() {
101101
when(blockBodyValidator.validateBody(any(), any(), any(), any(), any(), any()))
102102
.thenReturn(true);
103103
when(blockBodyValidator.validateBodyLight(any(), any(), any(), any())).thenReturn(true);
104-
when(blockProcessor.processBlock(protocolContext, any(), any(), any()))
104+
when(blockProcessor.processBlock(eq(protocolContext), any(), any(), any()))
105105
.thenReturn(successfulProcessingResult);
106106
when(blockProcessor.processBlock(any(), any(), any(), any()))
107107
.thenReturn(successfulProcessingResult);

ethereum/core/src/test/java/org/hyperledger/besu/ethereum/mainnet/parallelization/ParallelizedConcurrentTransactionProcessorTest.java

Lines changed: 22 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -19,13 +19,16 @@
1919
import static org.mockito.ArgumentMatchers.any;
2020
import static org.mockito.ArgumentMatchers.anyBoolean;
2121
import static org.mockito.ArgumentMatchers.eq;
22+
import static org.mockito.Mockito.mock;
2223
import static org.mockito.Mockito.times;
2324
import static org.mockito.Mockito.verify;
2425
import static org.mockito.Mockito.when;
2526

2627
import org.hyperledger.besu.datatypes.Address;
2728
import org.hyperledger.besu.datatypes.Hash;
2829
import org.hyperledger.besu.datatypes.Wei;
30+
import org.hyperledger.besu.ethereum.ProtocolContext;
31+
import org.hyperledger.besu.ethereum.chain.MutableBlockchain;
2932
import org.hyperledger.besu.ethereum.core.BlockHeader;
3033
import org.hyperledger.besu.ethereum.core.InMemoryKeyValueStorageProvider;
3134
import org.hyperledger.besu.ethereum.core.Transaction;
@@ -42,6 +45,7 @@
4245
import org.hyperledger.besu.ethereum.trie.diffbased.common.trielog.NoOpTrieLogManager;
4346
import org.hyperledger.besu.ethereum.trie.diffbased.common.worldview.accumulator.DiffBasedWorldStateUpdateAccumulator;
4447
import org.hyperledger.besu.ethereum.worldstate.DataStorageConfiguration;
48+
import org.hyperledger.besu.ethereum.worldstate.WorldStateArchive;
4549
import org.hyperledger.besu.evm.blockhash.BlockHashLookup;
4650
import org.hyperledger.besu.evm.internal.EvmConfiguration;
4751
import org.hyperledger.besu.evm.tracing.OperationTracer;
@@ -62,7 +66,9 @@
6266
class ParallelizedConcurrentTransactionProcessorTest {
6367

6468
@Mock private MainnetTransactionProcessor transactionProcessor;
69+
@Mock private BlockHeader chainHeadBlockHeader;
6570
@Mock private BlockHeader blockHeader;
71+
@Mock ProtocolContext protocolContext;
6672
@Mock private Transaction transaction;
6773
@Mock private PrivateMetadataUpdater privateMetadataUpdater;
6874
@Mock private TransactionCollisionDetector transactionCollisionDetector;
@@ -89,6 +95,19 @@ void setUp() {
8995
new NoOpTrieLogManager(),
9096
EvmConfiguration.DEFAULT,
9197
createStatefulConfigWithTrie());
98+
99+
when(chainHeadBlockHeader.getHash()).thenReturn(Hash.ZERO);
100+
when(chainHeadBlockHeader.getStateRoot()).thenReturn(Hash.EMPTY_TRIE_HASH);
101+
when(blockHeader.getParentHash()).thenReturn(Hash.ZERO);
102+
103+
when(transaction.detachedCopy()).thenReturn(transaction);
104+
105+
final MutableBlockchain blockchain = mock(MutableBlockchain.class);
106+
when(protocolContext.getBlockchain()).thenReturn(blockchain);
107+
when(blockchain.getChainHeadHeader()).thenReturn(chainHeadBlockHeader);
108+
final WorldStateArchive worldStateArchive = mock(WorldStateArchive.class);
109+
when(protocolContext.getWorldStateArchive()).thenReturn(worldStateArchive);
110+
when(worldStateArchive.getWorldState(any())).thenReturn(Optional.of(worldState));
92111
when(transactionCollisionDetector.hasCollision(any(), any(), any(), any())).thenReturn(false);
93112
}
94113

@@ -105,7 +124,7 @@ void testRunTransaction() {
105124
Collections.emptyList(), 0, 0, Bytes.EMPTY, ValidationResult.valid()));
106125

107126
processor.runTransaction(
108-
worldState,
127+
protocolContext,
109128
blockHeader,
110129
0,
111130
transaction,
@@ -151,7 +170,7 @@ void testRunTransactionWithFailure() {
151170
Optional.of(Bytes.EMPTY)));
152171

153172
processor.runTransaction(
154-
worldState,
173+
protocolContext,
155174
blockHeader,
156175
0,
157176
transaction,
@@ -180,7 +199,7 @@ void testRunTransactionWithConflict() {
180199
Collections.emptyList(), 0, 0, Bytes.EMPTY, ValidationResult.valid()));
181200

182201
processor.runTransaction(
183-
worldState,
202+
protocolContext,
184203
blockHeader,
185204
0,
186205
transaction,

0 commit comments

Comments
 (0)