Skip to content

[codex] Centralize test infrastructure helpers - #207

Merged
jkendall327 merged 2 commits into
mainfrom
codex-centralize-test-infrastructure
May 31, 2026
Merged

jkendall327 merged 2 commits into
mainfrom
codex-centralize-test-infrastructure

Conversation

@jkendall327

Copy link
Copy Markdown
Owner

Summary

Centralizes repeated test doubles and helper infrastructure under Octans.Tests/Infrastructure.

  • Moves the shared channel spy into the infrastructure namespace.
  • Extracts repeated HTTP message handlers for simple stubs, downloader resolver tests, HTTP downloader unit tests, and download-manager integration tests.
  • Extracts related test doubles for download content, disk-space checks, completion notification tracking, navigation, and throwing filesystem behavior.
  • Updates affected tests to consume the centralized helpers while keeping purpose-specific HTTP fakes separate where they exercise different seams.

Validation

  • dotnet build
  • dotnet test

@jkendall327
jkendall327 marked this pull request as ready for review May 31, 2026 09:34
@jkendall327
jkendall327 merged commit a09f81f into main May 31, 2026
1 check passed
@jkendall327
jkendall327 deleted the codex-centralize-test-infrastructure branch May 31, 2026 09:34
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.

1 participant