docs(product): port the full-system test-strategy design record from orion (amended) - #232
Open
seal-agent wants to merge 2 commits into
Open
docs(product): port the full-system test-strategy design record from orion (amended)#232seal-agent wants to merge 2 commits into
seal-agent wants to merge 2 commits into
Conversation
…orion (amended) Ports the frozen orion #740 record into compass as docs/designs/product/compass-test-strategy/design.md, re-grounded against shipped compass CI, with Matt-ruled amendments: - pgtest runs INLINE in the existing CI job (no separate lane/task) — the shipped ci.yml shape is ratified (D-A1). - CI Postgres is the shipped digest-pinned postgres:16-alpine service container, not the orion embedded-postmaster design (D-A2). - DL renumbered to compass range DL-174..DL-181; rows appended to docs/designs/product/DECISIONS.md. - The bun TS-client contract lane is scoped as a follow-up filed against the compass-dogfood-e2e harness record, not built here. - Shipped-vs-unexecuted marked: only require-live teeth, the thin runner.Dial whole-flow test, and the bun contract lane remain. Ledger-impact: adds DL-174..DL-181 (net-new, no supersessions). Spec-impact: none. Co-authored-by: Matt Wilkinson <matt@sealedsecurity.com>
Review fixes on the compass-test-strategy port (PR #232), no design change. Renumber the Approach section and pyramid table to the canonical Plan scheme (S1 require-live teeth, S2 runner.Dial whole-flow test, S3 bun contract lane): the whole-flow seam was stale-labeled S3 and the bun lane S5 from the orion ancestor, so a reader tracing an S-token from the Approach landed on the wrong Plan deliverable. Every S-reference now resolves to one deliverable across Approach, pyramid, Plan, Tasks, and Decisions; no S4/S5 token remains. Reorder the two amendment subsections so section number matches decision label and ledger order: A1 = the inline-CI-job amendment (D-A1, DL-175), A2 = the service-container amendment (D-A2, DL-176), ascending. Flip the two DECISIONS.md anchors (DL-175 to the A1 slug, DL-176 to the A2 slug) to follow. Spec-impact: none. Co-authored-by: Matt Wilkinson <matt@sealedsecurity.com>
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
1 participant
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.
Ports the frozen orion #740 record into compass as docs/designs/product/compass-test-strategy/design.md, re-grounded against shipped compass CI, with Matt-ruled amendments:
Ledger-impact: adds DL-174..DL-181 (net-new, no supersessions).
Spec-impact: none.
Co-authored-by: Matt Wilkinson matt@sealedsecurity.com