Skip to content

Commit

Permalink
Use required plugins in ATs to not register tracer plugins (#91)
Browse files Browse the repository at this point in the history
Signed-off-by: Fabio Di Fabio <[email protected]>
  • Loading branch information
fab-10 authored Oct 3, 2024
1 parent 624b99d commit fcd99a4
Show file tree
Hide file tree
Showing 3 changed files with 9 additions and 10 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -41,8 +41,7 @@ public List<String> getTestCliOptions() {
@Override
@BeforeEach
public void setup() throws Exception {
minerNode = besu.createMinerNodeWithExtraCliOptions("miner1", getTestCliOptions());
cluster.start(minerNode);
super.setup();
minerNode.execute(minerTransactions.minerStop());
}

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -121,7 +121,14 @@ private BesuNode createCliqueNodeWithExtraCliOptionsAndRpcApis(
.jsonRpcTxPool()
.genesisConfigProvider(
validators -> Optional.of(provideGenesisConfig(validators, cliqueOptions)))
.extraCLIOptions(extraCliOptions);
.extraCLIOptions(extraCliOptions)
.requestedPlugins(
List.of(
"LineaExtraDataPlugin",
"LineaEstimateGasEndpointPlugin",
"LineaSetExtraDataEndpointPlugin",
"LineaTransactionPoolValidatorPlugin",
"LineaTransactionSelectorPlugin"));

return besu.create(nodeConfBuilder.build());
}
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -40,13 +40,6 @@ public TestCommandLineOptionsBuilder() {
cliOptions.setProperty(
"--plugin-linea-l1l2-bridge-contract=", "0x00000000000000000000000000000000DEADBEEF");
cliOptions.setProperty("--plugin-linea-l1l2-bridge-topic=", "0x123456");
// temporarily specify tracer mandatory options, until we improve the ATs to accept a list of
// enabled plugins
cliOptions.setProperty("--plugin-linea-conflated-trace-generation-traces-output-path=", ".");
cliOptions.setProperty("--plugin-linea-rpc-concurrent-requests-limit=", "1");
cliOptions.setProperty("--plugin-linea-tracer-readiness-server-port=", "1234");
cliOptions.setProperty("--plugin-linea-tracer-readiness-server-host=", "localhost");
cliOptions.setProperty("--plugin-linea-tracer-readiness-max-blocks-behind=", "1");
}

public TestCommandLineOptionsBuilder set(String option, String value) {
Expand Down

0 comments on commit fcd99a4

Please sign in to comment.