Skip to content

Add expose lifecycle E2E coverage - #81

Merged
casey-brooks merged 1 commit into
mainfrom
noa/issue-80
May 10, 2026
Merged

Add expose lifecycle E2E coverage#81
casey-brooks merged 1 commit into
mainfrom
noa/issue-80

Conversation

@casey-brooks

Copy link
Copy Markdown
Contributor

Summary

  • add full agyn expose lifecycle test (list/add/reach/remove/unreachable)
  • refactor expose helpers for deterministic in-pod HTTP server startup
  • share Ziti management address helper across suites

Testing

  • go test ./tests/... -tags 'e2e svc_agents_orchestrator' -run TestDoesNotExist
  • go vet -tags 'e2e svc_agents_orchestrator' ./tests/...

Closes #80

@casey-brooks

Copy link
Copy Markdown
Contributor Author

Summary

  • add full expose lifecycle test (list/add/reach/remove/unreachable)
  • refactor expose helpers and add shared Ziti management address helper

Testing

  • go test ./tests/... -tags 'e2e svc_agents_orchestrator' -run TestDoesNotExist
  • go vet -tags 'e2e svc_agents_orchestrator' ./tests/...
  • PYTHONPATH=/nix/store/jl0mxihyizv77l66mzbvmv49iiri72sd-python3.13-pyyaml-6.0.3/lib/python3.13/site-packages AGN_INIT_IMAGE=ghcr.io/agynio/agent-init-agn:latest CODEX_INIT_IMAGE=ghcr.io/agynio/agent-init-codex:latest AGN_EXPOSE_INIT_IMAGE=ghcr.io/agynio/agent-init-agn:latest TAGS=svc_agents_orchestrator DEVSPACE_NAMESPACE=platform devspace run test-e2e (failed: timed out after 5m; missing AGYN_MODEL_ID for go-terraform suite)

@casey-brooks

Copy link
Copy Markdown
Contributor Author

Update

  • rebased onto main and resolved expose test conflicts

Testing

  • buf generate
  • go test ./tests/... -tags 'e2e svc_agents_orchestrator' -run TestDoesNotExist
  • go vet -tags 'e2e svc_agents_orchestrator' ./tests/...

@noa-lucent noa-lucent left a comment

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.

Adds full agyn expose lifecycle coverage (list→add→reach→remove→list empty→unreachable) and refactors helpers for more deterministic HTTP server startup.

One small nit inline around simplifying the unreachable polling logic.

Comment thread suites/go-core/tests/expose_test.go
@casey-brooks
casey-brooks merged commit b71d612 into main May 10, 2026
1 check passed
@casey-brooks
casey-brooks deleted the noa/issue-80 branch May 10, 2026 23:15
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.

go-core E2E: cover agyn expose list/remove lifecycle (add→list→dial→remove→list/unreachable)

2 participants