From 845818be44dd2424fe7bdf19cfd49523a1a2a9b0 Mon Sep 17 00:00:00 2001 From: Liang Wu Date: Wed, 11 Feb 2026 21:39:15 -0800 Subject: [PATCH] ci: only use `--extra test` for unit tests This can fix unit test github action error by removing `--extra eval`. `--extra a2a` is not needed because it's included in `test`. All tests are still passing. Co-authored-by: Liang Wu PiperOrigin-RevId: 869014318 --- AGENTS.md | 5 ++--- CONTRIBUTING.md | 6 +++--- scripts/unittests.sh | 2 +- 3 files changed, 6 insertions(+), 7 deletions(-) diff --git a/AGENTS.md b/AGENTS.md index 1ec9dcb8..c6b9c89d 100644 --- a/AGENTS.md +++ b/AGENTS.md @@ -187,8 +187,7 @@ uv venv --python "python3.11" ".venv" source .venv/bin/activate uv sync --all-extras ``` -**Minimal setup for testing only (matches CI):** `bash uv sync --extra test ---extra eval --extra a2a` +**Minimal setup for testing only (matches CI):** `bash uv sync --extra test` **Virtual Environment Usage (Required):** - **Always use** `.venv/bin/python` or `.venv/bin/pytest` directly - **Or activate** with `source .venv/bin/activate` @@ -363,7 +362,7 @@ catching issues with the public API. **Quick start:** Run all tests with: `bash pytest tests/unittests` **Recommended:** Match CI configuration before submitting PRs: `bash uv sync ---extra test --extra eval --extra a2a && pytest tests/unittests` +--extra test && pytest tests/unittests` **Additional options:** ```bash diff --git a/CONTRIBUTING.md b/CONTRIBUTING.md index 70dab562..78172029 100644 --- a/CONTRIBUTING.md +++ b/CONTRIBUTING.md @@ -180,11 +180,11 @@ part before or alongside your code PR. pytest ./tests/unittests ``` - NOTE: for accurate repro of test failure, only include `test`, `eval` and - `a2a` as extra dependencies. + NOTE: for accurate repro of test failure, only include `test` as extra + dependencies. ```shell - uv sync --extra test --extra eval --extra a2a + uv sync --extra test pytest ./tests/unittests ``` diff --git a/scripts/unittests.sh b/scripts/unittests.sh index bafcb118..b4892c29 100755 --- a/scripts/unittests.sh +++ b/scripts/unittests.sh @@ -94,7 +94,7 @@ for version in "${versions_to_run[@]}"; do # 3. perform the unit tests echo "Setting up test environment in .unittest_venv..." - uv sync --extra test --extra eval --extra a2a --active + uv sync --extra test --active echo "Running unit tests..." TEST_EXIT_CODE=0