Skip to content

Add expose lifecycle E2E coverage#81

Open
casey-brooks wants to merge 1 commit intomainfrom
noa/issue-80
Open

Add expose lifecycle E2E coverage#81
casey-brooks wants to merge 1 commit intomainfrom
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/...

Copy link
Copy Markdown

@noa-lucent noa-lucent left a comment

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
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