docs(e2e): update READMEs for the consolidated suite layout#24518
Open
spalladino wants to merge 1 commit into
Open
docs(e2e): update READMEs for the consolidated suite layout#24518spalladino wants to merge 1 commit into
spalladino wants to merge 1 commit into
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Updates every README in the e2e test section to describe the suite as it will look once the nine open
round-2 consolidation PRs land. This is docs-only — no code, config, or
bootstrap.shchanges.What changed per README
end-to-end/README.md(root): repointed theLOG_LEVELexample off the deletedproving/empty_blocks.test.tstoproving/default_node.test.ts; linked the new p2p README from thecategory table (the p2p row now reads "yes" and points at
src/p2p/README.md); and renamed thesetupBlockProducersubmission-window literal1024to the named constantNO_REORG_SUBMISSION_EPOCHS.end-to-end/src/single-node/README.md: rewrote thecross-chain/row for the merged layout(
CrossChainMessagingTest+message_test_helpers.ts, and thel1_to_l2/l1_to_l2_inbox_drift/l2_to_l1/token_bridgefiles); foldedpublic_payments/sponsored_paymentsintoprivate_payments.paralleland fixed the stalefailures.parallelname tofailures; replacedslasher_config/sequencer_configwith the mergedruntime_config; replacedworld_state_pruning/empty_blockswith the mergeddefault_node; added the previously-missingsnapshot_syncto thesync/row; and applied theNO_REORG_SUBMISSION_EPOCHSrename.end-to-end/src/multi-node/README.md: namedequivocation_offensesin the slashing row (in placeof the separate
duplicate_proposal/duplicate_attestationfiles) and applied theNO_REORG_SUBMISSION_EPOCHSrename in theMOCK_GOSSIP_MULTI_VALIDATOR_OPTSpreset.end-to-end/src/p2p/README.md(new): created with the content added by test(e2e): extract p2p gossip scenario skeleton and add category README #24500.end-to-end/src/automine/README.mdandend-to-end/src/infra/README.md: untouched — both arebehavior-level (no file names) or cover a directory none of the nine PRs change, so they remain accurate
as-landed.
Assumptions and merge order
test(e2e): name shared timing presets and collapse proof-submission constants #24494, test(e2e): unify slashing timing profiles and merge duplicate suites #24495, test(e2e): dedupe multi-node block-production and governance test setups #24498, test(e2e): extract p2p gossip scenario skeleton and add category README #24500, test(e2e): split node-killing HA test into its own composed suite file #24503. The layout it documents is the union of the current base branch
and all nine diffs.
branch, so this PR and the other one merge cleanly in either order: the new p2p README is test(e2e): extract p2p gossip scenario skeleton and add category README #24500's file
as-is, and the multi-node slashing row already matches the base branch (the "name both inactivity files"
fix from test(e2e): unify slashing timing profiles and merge duplicate suites #24495 has already propagated to
merge-train/spartan-v5).