Add comments to clarify the futurePParams issue

IntersectMBO · Aug 7, 2024 · 99c1712 · 99c1712
1 parent a43008d
commit 99c1712
Showing 1 changed file with 5 additions and 0 deletions.
diff --git a/cardano-testnet/test/cardano-testnet-test/Cardano/Testnet/Test/Cli/Query.hs b/cardano-testnet/test/cardano-testnet-test/Cardano/Testnet/Test/Cli/Query.hs
@@ -85,6 +85,8 @@ hprop_cli_queries = integrationWorkspace "cli-queries" $ \tempAbsBasePath' -> H.
         { cardanoEpochLength = 100
         , cardanoSlotLength = 0.1
         , cardanoNodeEra = cEra
+        -- We change slotCoeff because epochLength must be equal to:
+        -- securityParam * 10 / slotCoeff
         , cardanoActiveSlotsCoeff = 0.5
         }
 
@@ -352,6 +354,9 @@ hprop_cli_queries = integrationWorkspace "cli-queries" $ \tempAbsBasePath' -> H.
                then Just slotNo
                else Nothing
 
+  -- We wait till a slot after: 4 * securityParam / slotCoeff
+  -- If we query 'govState' before that we get 'PotentialPParamsUpdate'
+  -- in 'futurePParams' field
   areFuturePParamsStable :: ShelleyGenesis StandardCrypto -> SlotNo -> Bool
   areFuturePParamsStable
     ShelleyGenesis{ Api.sgActiveSlotsCoeff = activeSlotsCoeff