Skip to content

docs(temporal) WB-529 submission test plan - #131

Open
jimmeryn wants to merge 3 commits into
mainfrom
docs/WB-529-submission-test-plan
Open

docs(temporal) WB-529 submission test plan#131
jimmeryn wants to merge 3 commits into
mainfrom
docs/WB-529-submission-test-plan

Conversation

@jimmeryn

@jimmeryn jimmeryn commented Sep 9, 2026

Copy link
Copy Markdown

What

  • packages/temporal/TESTING.md — test plan: replay safety, limitations, suite/CI table
  • Cache-off replay test (maxCachedWorkflows: 0), the second side-effect technique Temporal's plugin guide names
  • replay-audit.md: runtime file list corrected, resolve-start-node.ts added, Object.keys verdict fixed
  • test/replay/README.md: four harness checks, not three; fan-out claim narrowed to what is true
  • README.md: one-line pointer to TESTING.md

Not in scope, and why

  • No stack-level e2e (HTTP → worker → Postgres). Needs its own CI job with containers; not a docs ticket
  • Cancellation and node-failure paths untested in the sandbox. Disclosed in TESTING.md
  • No count of audit sources in the doc. Ticket asked for "18 sources, 8 rules"; actual is 20 and 9. Counts rot in a permanent doc, so they go in the submission form instead
  • Submission form not filled. Blocked on WB-521 (owner, DocuSign), WB-531 (integration guide), WB-628 (npm publish). The Documentation review area cannot be answered until the last two land
  • CI re-downloads a 141 MB dev-server binary per temporal job, uncached. Separate ticket

@jimmeryn jimmeryn changed the title Docs/wb 529 submission test plan docs(teemporal) WB-529 submission test plan Sep 9, 2026
@jimmeryn jimmeryn changed the title docs(teemporal) WB-529 submission test plan docs(temporal) WB-529 submission test plan Sep 9, 2026
## Limitations

- **Cancellation is untested.** No test runs the cancellation branch in `run-workflow.ts`.
- **Node failure is untested in the sandbox.** The fixture executors in the replay test never fail. The failure branch of the workflow therefore runs only in the `execution-core` tests, and in the Activity tests.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This will become deprecated once WB-629 is finished 😊

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants