encounter/adk-python
0
0
Fork 0
mirror of https://github.com/encounter/adk-python.git synced 2026-07-09 18:19:28 -07:00
Code Issues Packages Projects Releases Wiki Activity

ci: only use --extra test for unit tests

Browse Source 
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 <wuliang@google.com>
PiperOrigin-RevId: 869014318
This commit is contained in:
Liang Wu
2026-02-11 21:39:48 -08:00
committed by Copybara-Service
parent 9f7d5b3f14
commit 845818be44
3 changed files with 6 additions and 7 deletions
Download Patch File Download Diff File Expand all files Collapse all files
AGENTS.md
+2 -3
View File
@@ -187,8 +187,7 @@ uv venv --python "python3.11" ".venv" source .venv/bin/activate
uv sync --all-extras ``` uv sync --all-extras ```
**Minimal setup for testing only (matches CI):** `bash uv sync --extra test **Minimal setup for testing only (matches CI):** `bash uv sync --extra test`
--extra eval --extra a2a`
**Virtual Environment Usage (Required):** - **Always use** `.venv/bin/python` or **Virtual Environment Usage (Required):** - **Always use** `.venv/bin/python` or
`.venv/bin/pytest` directly - **Or activate** with `source .venv/bin/activate` `.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` **Quick start:** Run all tests with: `bash pytest tests/unittests`
**Recommended:** Match CI configuration before submitting PRs: `bash uv sync **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 **Additional options:** ```bash
CONTRIBUTING.md
+3 -3
View File
@@ -180,11 +180,11 @@ part before or alongside your code PR.
pytest ./tests/unittests pytest ./tests/unittests
``` ```
NOTE: for accurate repro of test failure, only include `test`, `eval` and NOTE: for accurate repro of test failure, only include `test` as extra
`a2a` as extra dependencies. dependencies.
```shell ```shell
uv sync --extra test --extra eval --extra a2a uv sync --extra test
pytest ./tests/unittests pytest ./tests/unittests
``` ```
scripts/unittests.sh
+1 -1
View File
@@ -94,7 +94,7 @@ for version in "${versions_to_run[@]}"; do
# 3. perform the unit tests # 3. perform the unit tests
echo "Setting up test environment in .unittest_venv..." 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..." echo "Running unit tests..."
TEST_EXIT_CODE=0 TEST_EXIT_CODE=0