Add test

Author: ferranbt

crates/op-rbuilder/src/integration/integration_test.rs

@@ -125,8 +125,24 @@ mod tests {
                 .await?
                 .expect("block");
 
-            // blocks should only include two
-            println!("block: {:?}", block.transactions.len());
+            // blocks should only include two transactions (deposit + builder)
+            assert_eq!(block.transactions.len(), 2);
+        }
+
+        // check that the builder emitted logs for the reverted transactions
+        // with the monitoring logic
+        // TODO: this is not ideal, lets find a different way to detect this
+        // Each time a transaction is dropped, it emits a log like this
+        // 'Transaction event received target="monitoring" tx_hash="<tx_hash>" kind="discarded"'
+        let builder_logs = std::fs::read_to_string(harness.builder_log_path)?;
+
+        for txn in pending_txn {
+            let txn_log = format!(
+                "Transaction event received target=\"monitoring\" tx_hash=\"{}\" kind=\"discarded\"",
+                txn.tx_hash()
+            );
+
+            assert!(builder_logs.contains(txn_log.as_str()));
         }
 
         Ok(())

crates/op-rbuilder/src/integration/mod.rs

@@ -270,16 +270,19 @@ impl TestHarnessBuilder {
 
         framework.start("op-reth", &reth).await.unwrap();
 
-        let _ = framework
+        let builder = framework
             .start("op-rbuilder", &op_rbuilder_config)
             .await
             .unwrap();
 
+        let builder_log_path = builder.log_path.clone();
+
         Ok(TestHarness {
             _framework: framework,
             builder_auth_rpc_port,
             builder_http_port,
             validator_auth_rpc_port,
+            builder_log_path,
         })
     }
 }
@@ -289,6 +292,7 @@ pub struct TestHarness {
     builder_auth_rpc_port: u16,
     builder_http_port: u16,
     validator_auth_rpc_port: u16,
+    builder_log_path: PathBuf,
 }
 
 impl TestHarness {

crates/op-rbuilder/src/integration/op_rbuilder.rs

@@ -118,6 +118,7 @@ impl Service for OpRbuilderConfig {
             .arg("--disable-discovery")
             .arg("--color")
             .arg("never")
+            .arg("--builder.log-pool-transactions")
             .arg("--port")
             .arg(self.network_port.expect("network_port not set").to_string())
             .arg("--ipcdisable");