avoid unpacking phase 1 tests to reduce Azure CI disk usage (#1736)

beacon_chain/block_pools/clearance.nim

@@ -344,9 +344,7 @@ proc isValidBeaconBlock*(
   if parent_ref.isNil:
     # Pending dag gets checked via `ChainDAGRef.add(...)` later, and relevant
     # checks are performed there. In usual paths beacon_node adds blocks via
-    # ChainDAGRef.add(...) directly, with no additional validity checks. TODO,
-    # not specific to this, but by the pending dag keying on the htr of the
-    # BeaconBlock, not SignedBeaconBlock, opens up certain spoofing attacks.
+    # ChainDAGRef.add(...) directly, with no additional validity checks.
     debug "parent unknown, putting block in quarantine",
       current_slot = shortLog(current_slot)
     if not quarantine.add(dag, signed_beacon_block):

vendor/nim-eth2-scenarios

@@ -1 +1 @@
-Subproject commit 2a3f363c436c8d00733ef1152f9670e0b3ca411c
+Subproject commit 8cd0714c4b2ffd18229d637ec8b0e8140d3b725e