Skip to content

ci: run Python tests for the antigravity harness sidecar#296

Open
joycel-github wants to merge 1 commit into
mainfrom
ci-python-tests-inline-deps
Open

ci: run Python tests for the antigravity harness sidecar#296
joycel-github wants to merge 1 commit into
mainfrom
ci-python-tests-inline-deps

Conversation

@joycel-github

@joycel-github joycel-github commented Jul 11, 2026

Copy link
Copy Markdown
Collaborator
  • Wires the 17 antigravity harness sidecar tests (python/antigravity/harness_server_test.py) into CI. They have never run automatically — .github/workflows/go.yml only runs go test ./..., which is how the stale assertions fixed in harness/python: fix stale thought summary test assertions #291 slipped through.
  • Adds .github/workflows/python.yml, a dedicated Python workflow kept separate from the Go-only go.yml so each can evolve independently.
  • Adds a test-python Makefile target for local parity.

The 17 tests in python/antigravity/harness_server_test.py have never
run in CI. .github/workflows/go.yml only runs 'go test ./...' and the
Makefile 'test' target likewise only runs Go. That let PR #244 land
with 2 stale assertions unnoticed (fixed in #291).

Adds:
- .github/workflows/python.yml -- dedicated Python workflow (kept
  separate from the Go-only go.yml so each can evolve independently).
- Makefile 'test-python' target for local parity.

Test deps are inlined in the workflow rather than added as a separate
requirements-test.txt -- the diff is 2 packages ('pytest>=7.0',
'pytest-timeout>=2.0'), not worth its own file. Version floors match
the upstream google-antigravity SDK's dev extras so this doesn't
drift as pytest evolves.

--timeout=30 --timeout-method=thread guards against hung gRPC servers
so a stuck test can't eat the whole workflow budget.
@joycel-github joycel-github marked this pull request as ready for review July 11, 2026 00:41
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