Skip to content

test(orchestration): cover proxy status and method handling#44

Merged
DonPrus merged 4 commits into
nullclaw:mainfrom
vernonstinebaker:test/orchestration-proxy
May 6, 2026
Merged

test(orchestration): cover proxy status and method handling#44
DonPrus merged 4 commits into
nullclaw:mainfrom
vernonstinebaker:test/orchestration-proxy

Conversation

@vernonstinebaker

Copy link
Copy Markdown
Contributor

Summary

  • expand orchestration proxy coverage beyond the existing "not configured" checks
  • add regression tests for non-proxy 404 behavior and unsupported-method rejection before any upstream fetch occurs
  • add a tiny local fake upstream to verify that the proxy passes through upstream 409 status and JSON body correctly
  • keep the slice focused on high-risk proxy behavior without introducing new shared fixture infrastructure

Validation

  • zig build test -Dembed-ui=false -Dbuild-ui=false --summary all
  • npm --prefix ui ci --no-audit --no-fund
  • npm --prefix ui run build
  • bash tests/test_e2e.sh
  • zig fmt --check src/
  • git diff --check

Notes

  • this PR is test-only and does not change production proxy logic
  • the local fake upstream is intentionally embedded in the test module to keep the slice small; if more HTTP-backed integration tests are added later, Phase 6 can factor it into broader shared helpers
  • remaining Phase 5 slices are installer failure/cleanup coverage and service helper boundary coverage

@DonPrus DonPrus merged commit e403e0d into nullclaw:main May 6, 2026
4 checks passed
@vernonstinebaker vernonstinebaker deleted the test/orchestration-proxy branch May 6, 2026 19:28
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