From 947ad9f14a2802e974bf62270c29a092fa894350 Mon Sep 17 00:00:00 2001 From: Michael Sproul Date: Wed, 27 Jul 2022 00:51:08 +0000 Subject: [PATCH] Allow syncing or accepted in integration test (#3378) ## Issue Addressed Unblock CI for this failure: https://github.com/sigp/lighthouse/runs/7529551988 The root cause is a disagreement between the test and Nethermind over whether the appropriate status for a payload with an unknown parent is SYNCING or ACCEPTED. According to the spec, SYNCING is correct so we should update the test to expect this correct behaviour. However Geth still returns `ACCEPTED`, so for now we allow either. --- testing/execution_engine_integration/src/test_rig.rs | 6 +++++- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/testing/execution_engine_integration/src/test_rig.rs b/testing/execution_engine_integration/src/test_rig.rs index 7dac2010b6..7126268c37 100644 --- a/testing/execution_engine_integration/src/test_rig.rs +++ b/testing/execution_engine_integration/src/test_rig.rs @@ -488,7 +488,11 @@ impl TestRig { .notify_new_payload(&second_payload) .await .unwrap(); - assert_eq!(status, PayloadStatus::Accepted); + // TODO: we should remove the `Accepted` status here once Geth fixes it + assert!(matches!( + status, + PayloadStatus::Syncing | PayloadStatus::Accepted + )); /* * Execution Engine B: