diff --git a/src/test/jtx/Env.h b/src/test/jtx/Env.h index 6e46a9cdd..2c3e1dbf6 100644 --- a/src/test/jtx/Env.h +++ b/src/test/jtx/Env.h @@ -83,9 +83,15 @@ supported_amendments() "Unknown feature: " + s + " in supportedAmendments."); } //@@start rng-test-environment-gating - // TODO: ConsensusEntropy injects a pseudo-tx every ledger which - // breaks existing test transaction count assumptions. Exclude from - // default test set until dedicated tests are written. + // ConsensusEntropy is a supported amendment, but it is intentionally + // excluded from the default jtx fixture set. + // + // Enabling it changes every closed ledger by injecting a deterministic + // ttCONSENSUS_ENTROPY pseudo-transaction. Many legacy Env tests make + // exact transaction-count / metadata-shape assertions and are not + // trying to exercise ledger-wide entropy materialization. Keep those + // default fixtures stable; tests that cover entropy opt in explicitly + // with `supported_amendments() | featureConsensusEntropy`. return FeatureBitset(feats) - featureConsensusEntropy; //@@end rng-test-environment-gating }();