Skip to content

test(core): simplify runner test fixtures#35832

Merged
kitlangton merged 2 commits into
v2from
concise-runner-tests
Jul 8, 2026
Merged

test(core): simplify runner test fixtures#35832
kitlangton merged 2 commits into
v2from
concise-runner-tests

Conversation

@kitlangton

Copy link
Copy Markdown
Contributor

Summary

  • replace repeated LLM event envelopes with small pure response helpers
  • collapse default prompt admissions into a focused admit helper
  • centralize runner fixture state reset and return the Session service from setup
  • preserve explicit raw streams and race controls where event ordering matters

Result

  • session-runner.test.ts: 4,345 to 3,711 lines
  • 634 net lines removed
  • no new fixture framework or test files
  • assertion count preserved

Verification

  • bun run test test/session-runner.test.ts test/session-runner-recorded.test.ts (106 passed)
  • bun run test test/session-runner.test.ts --randomize --rerun-each 5 (525 passed)
  • bun typecheck in packages/core
  • pre-push workspace typecheck (31 packages passed)

@kitlangton
kitlangton merged commit d27746e into v2 Jul 8, 2026
8 of 10 checks passed
@kitlangton
kitlangton deleted the concise-runner-tests branch July 8, 2026 01:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant