From 9c429d0764ed91cf56efb8a47a35a556b54a86a4 Mon Sep 17 00:00:00 2001
From: Mac L <mjladson@pm.me>
Date: Thu, 9 Jun 2022 10:47:03 +0000
Subject: [PATCH] Only use authenticated endpoints during EE integration
 testing (#3253)

## Issue Addressed

Failures in our CI integration tests for Geth.

## Proposed Changes

Only connect to the authenticated execution endpoints during execution tests.
This is necessary now that it is impossible to connect to the `engine` api on an unauthenticated endpoint.
See https://github.com/ethereum/go-ethereum/pull/24997

## Additional Info

As these tests break semi-regularly, I have kept logs enabled to ease future debugging.
I've also updated the Nethermind tests, although these weren't broken. This should future-proof us if Nethermind decides to follow suit with Geth
---
 .../execution_engine_integration/src/execution_engine.rs | 6 ------
 testing/execution_engine_integration/src/main.rs         | 2 +-
 testing/execution_engine_integration/src/nethermind.rs   | 9 ++++++---
 testing/execution_engine_integration/src/test_rig.rs     | 2 +-
 4 files changed, 8 insertions(+), 11 deletions(-)

diff --git a/testing/execution_engine_integration/src/execution_engine.rs b/testing/execution_engine_integration/src/execution_engine.rs
index dd5d03be89b..7df88aa0d7c 100644
--- a/testing/execution_engine_integration/src/execution_engine.rs
+++ b/testing/execution_engine_integration/src/execution_engine.rs
@@ -22,7 +22,6 @@ pub struct ExecutionEngine<E> {
     engine: E,
     #[allow(dead_code)]
     datadir: TempDir,
-    http_port: u16,
     http_auth_port: u16,
     child: Child,
 }
@@ -46,16 +45,11 @@ impl<E: GenericExecutionEngine> ExecutionEngine<E> {
         Self {
             engine,
             datadir,
-            http_port,
             http_auth_port,
             child,
         }
     }
 
-    pub fn http_url(&self) -> SensitiveUrl {
-        SensitiveUrl::parse(&format!("http://127.0.0.1:{}", self.http_port)).unwrap()
-    }
-
     pub fn http_auth_url(&self) -> SensitiveUrl {
         SensitiveUrl::parse(&format!("http://127.0.0.1:{}", self.http_auth_port)).unwrap()
     }
diff --git a/testing/execution_engine_integration/src/main.rs b/testing/execution_engine_integration/src/main.rs
index 30c8132b7c0..a4ec0f92153 100644
--- a/testing/execution_engine_integration/src/main.rs
+++ b/testing/execution_engine_integration/src/main.rs
@@ -15,7 +15,7 @@ use nethermind::NethermindEngine;
 use test_rig::TestRig;
 
 /// Set to `false` to send logs to the console during tests. Logs are useful when debugging.
-const SUPPRESS_LOGS: bool = true;
+const SUPPRESS_LOGS: bool = false;
 
 fn main() {
     if cfg!(windows) {
diff --git a/testing/execution_engine_integration/src/nethermind.rs b/testing/execution_engine_integration/src/nethermind.rs
index 7fb07c9e5a4..be638fe0424 100644
--- a/testing/execution_engine_integration/src/nethermind.rs
+++ b/testing/execution_engine_integration/src/nethermind.rs
@@ -75,7 +75,7 @@ impl GenericExecutionEngine for NethermindEngine {
 
     fn start_client(
         datadir: &TempDir,
-        http_port: u16,
+        _http_port: u16,
         http_auth_port: u16,
         jwt_secret_path: PathBuf,
     ) -> Child {
@@ -89,11 +89,14 @@ impl GenericExecutionEngine for NethermindEngine {
             .arg("--Merge.TerminalTotalDifficulty")
             .arg("0")
             .arg("--JsonRpc.AdditionalRpcUrls")
-            .arg(format!("http://localhost:{}|http;ws|net;eth;subscribe;engine;web3;client|no-auth,http://localhost:{}|http;ws|net;eth;subscribe;engine;web3;client", http_port, http_auth_port))
+            .arg(format!(
+                "http://localhost:{}|http;ws|net;eth;subscribe;engine;web3;client",
+                http_auth_port
+            ))
             .arg("--JsonRpc.EnabledModules")
             .arg("net,eth,subscribe,web3,admin,engine")
             .arg("--JsonRpc.Port")
-            .arg(http_port.to_string())
+            .arg(http_auth_port.to_string())
             .arg("--Network.DiscoveryPort")
             .arg(network_port.to_string())
             .arg("--Network.P2PPort")
diff --git a/testing/execution_engine_integration/src/test_rig.rs b/testing/execution_engine_integration/src/test_rig.rs
index 77993df2b79..21162fea56d 100644
--- a/testing/execution_engine_integration/src/test_rig.rs
+++ b/testing/execution_engine_integration/src/test_rig.rs
@@ -68,7 +68,7 @@ impl<E: GenericExecutionEngine> TestRig<E> {
 
         let ee_b = {
             let execution_engine = ExecutionEngine::new(generic_engine);
-            let urls = vec![execution_engine.http_url()];
+            let urls = vec![execution_engine.http_auth_url()];
 
             let config = execution_layer::Config {
                 execution_endpoints: urls,