complete tests

Consensys · Dec 13, 2024 · ba4f460 · ba4f460
1 parent e42508b
commit ba4f460
Showing 1 changed file with 11 additions and 2 deletions.
diff --git a/...nsition/src/test/java/tech/pegasys/teku/statetransition/blobs/BlobSidecarManagerTest.java b/...nsition/src/test/java/tech/pegasys/teku/statetransition/blobs/BlobSidecarManagerTest.java
@@ -14,6 +14,7 @@
 package tech.pegasys.teku.statetransition.blobs;
 
 import static org.assertj.core.api.Assertions.assertThat;
+import static org.assertj.core.api.Assertions.assertThatThrownBy;
 import static org.mockito.ArgumentMatchers.any;
 import static org.mockito.ArgumentMatchers.eq;
 import static org.mockito.Mockito.mock;
@@ -340,11 +341,19 @@ void createAvailabilityCheckerAndValidateImmediately_shouldReturnNotRequiredWhen
     when(blockBlobSidecarsTracker.add(any())).thenReturn(true);
     when(blockBlobSidecarsTracker.isComplete()).thenReturn(true);
 
-    final SafeFuture<BlobSidecarsAndValidationResult> result =
-        SafeFuture.completedFuture(BlobSidecarsAndValidationResult.validResult(blobSidecars));
+    final SafeFuture<BlobSidecarsAndValidationResult> result = new SafeFuture<>();
 
     when(forkChoiceBlobSidecarsAvailabilityChecker.getAvailabilityCheckResult()).thenReturn(result);
 
+    assertThatThrownBy(
+            () ->
+                blobSidecarManager.createAvailabilityCheckerAndValidateImmediately(
+                    block, blobSidecars))
+        .isInstanceOf(IllegalStateException.class)
+        .hasMessage("Availability check should be done immediately");
+
+    result.complete(BlobSidecarsAndValidationResult.validResult(blobSidecars));
+
     assertThat(
             blobSidecarManager.createAvailabilityCheckerAndValidateImmediately(block, blobSidecars))
         .isSameAs(result.join());