From ec45857a50d96a53cf186b343afff38a4d5e7e71 Mon Sep 17 00:00:00 2001 From: dapplion <35266934+dapplion@users.noreply.github.com> Date: Mon, 8 Jun 2026 12:59:40 +0200 Subject: [PATCH] Skip pre-Gloas block-level data-column tests under FORK_NAME=gloas Gloas carries data columns in the payload envelope, so the test harness produces no block-level columns for a Gloas block. Two tests assumed columns are present and failed under the newly-enabled FORK_NAME=gloas network test matrix: accept_processed_gossip_data_columns_without_import and data_column_reconstruction_at_deadline. Gate them on the same fork condition as their siblings, extended to skip Gloas; fulu coverage is unchanged. --- .../network/src/network_beacon_processor/tests.rs | 10 ++++++++-- 1 file changed, 8 insertions(+), 2 deletions(-) diff --git a/beacon_node/network/src/network_beacon_processor/tests.rs b/beacon_node/network/src/network_beacon_processor/tests.rs index 8ccfe38fa3..c6115271e8 100644 --- a/beacon_node/network/src/network_beacon_processor/tests.rs +++ b/beacon_node/network/src/network_beacon_processor/tests.rs @@ -907,7 +907,10 @@ async fn data_column_reconstruction_at_slot_start() { // reconstruction deadline. #[tokio::test] async fn data_column_reconstruction_at_deadline() { - if test_spec::().fulu_fork_epoch.is_none() { + let spec = test_spec::(); + // Pre-Gloas data-column path: a Gloas block carries its columns in the payload envelope, so the + // harness produces no block-level data columns and this gossip/reconstruction flow doesn't apply. + if spec.fulu_fork_epoch.is_none() || spec.gloas_fork_epoch.is_some() { return; }; @@ -1094,7 +1097,10 @@ async fn import_gossip_block_unacceptably_early() { /// Data columns that have already been processed but unobserved should be propagated without re-importing. #[tokio::test] async fn accept_processed_gossip_data_columns_without_import() { - if test_spec::().fulu_fork_epoch.is_none() { + let spec = test_spec::(); + // Pre-Gloas data-column path: a Gloas block carries its columns in the payload envelope, so the + // harness produces no block-level data columns and this gossip flow doesn't apply. + if spec.fulu_fork_epoch.is_none() || spec.gloas_fork_epoch.is_some() { return; };