mirror of
https://github.com/google/adk-docs.git
synced 2026-09-14 16:16:59 +08:00
Salman Chishti
68b5ece85a
* Fix python-tests workflow: split get_sample_dirs into its own job The build job declares `needs: get_sample_dirs`, but get_sample_dirs was defined as a step inside build rather than as a separate job. This caused GitHub Actions to reject the workflow with: The workflow must contain at least one job with no dependencies. This commit: - Extracts get_sample_dirs into its own job with proper outputs - Replaces undefined $PYTHON_DIR with the actual path (samples/python) - Formats the find output as JSON array for fromJson() compatibility - Adds an if condition to skip the build matrix when no samples exist Signed-off-by: Salman Muin Kayser Chishti <13schishti@gmail.com> * Update actions/checkout from v4 to v6 --------- Signed-off-by: Salman Muin Kayser Chishti <13schishti@gmail.com> Co-authored-by: Kristopher Overholt <koverholt@google.com>
104 lines
3.2 KiB
YAML
104 lines
3.2 KiB
YAML
# Copyright 2025 Google LLC
|
|
#
|
|
# Licensed under the Apache License, Version 2.0 (the "License");
|
|
# you may not use this file except in compliance with the License.
|
|
# You may obtain a copy of the License at
|
|
#
|
|
# http://www.apache.org/licenses/LICENSE-2.0
|
|
#
|
|
# Unless required by applicable law or agreed to in writing, software
|
|
# distributed under the License is distributed on an "AS IS" BASIS,
|
|
# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
|
|
# See the License for the specific language governing permissions and
|
|
# limitations under the License.
|
|
|
|
name: Python Tests
|
|
|
|
on:
|
|
push:
|
|
branches:
|
|
- main
|
|
paths:
|
|
- "samples/python/**"
|
|
- ".github/workflows/python-tests.yaml"
|
|
pull_request:
|
|
types:
|
|
- opened
|
|
- reopened
|
|
- synchronize
|
|
paths:
|
|
- "samples/python/**"
|
|
- ".github/workflows/python-tests.yaml"
|
|
schedule:
|
|
- cron: "0 0 * * 0"
|
|
|
|
jobs:
|
|
get_sample_dirs:
|
|
runs-on: ubuntu-latest
|
|
outputs:
|
|
sample_dirs: ${{ steps.get_dirs.outputs.sample_dirs }}
|
|
steps:
|
|
- uses: actions/checkout@v6
|
|
with:
|
|
ref: ${{ github.event.pull_request.head.sha }}
|
|
|
|
- name: Get Sample Directories
|
|
id: get_dirs
|
|
run: |
|
|
SAMPLE_DIRS=$( find samples/python -type d -not -path "*/.venv*" -exec test -e '{}'/requirements.txt \; -print | jq -R -s -c 'split("\n") | map(select(. != ""))' )
|
|
echo "sample_dirs=$SAMPLE_DIRS" >> $GITHUB_OUTPUT
|
|
echo "SAMPLE_DIRS: $SAMPLE_DIRS" # For debugging
|
|
|
|
build:
|
|
runs-on: ubuntu-latest
|
|
needs:
|
|
- get_sample_dirs
|
|
if: ${{ needs.get_sample_dirs.outputs.sample_dirs != '[]' }}
|
|
strategy:
|
|
matrix:
|
|
python-version: ["3.9", "3.10", "3.11", "3.12"]
|
|
sample_dir: ${{ fromJson(needs.get_sample_dirs.outputs.sample_dirs) }}
|
|
fail-fast: false # Important: Don't stop if one matrix configuration fails
|
|
|
|
steps:
|
|
- uses: actions/checkout@v6
|
|
with:
|
|
ref: ${{ github.event.pull_request.head.sha }}
|
|
|
|
- name: Set up Python ${{ matrix.python-version }}
|
|
uses: actions/setup-python@v6
|
|
with:
|
|
python-version: ${{ matrix.python-version }}
|
|
cache: "pip"
|
|
|
|
- name: Install dependencies
|
|
run: |
|
|
python -m pip install --upgrade pip
|
|
pip install pytest coverage
|
|
pip install -r ${{ matrix.sample_dir }}/requirements.txt
|
|
pip install -r ${{ matrix.sample_dir }}/requirements-test.txt
|
|
|
|
- name: Run pytest in ${{ matrix.sample_dir }}
|
|
run: |
|
|
cd ${{ matrix.sample_dir }}
|
|
pytest | tee pytest.txt
|
|
|
|
- name: Show failed tests and overall summary
|
|
run: |
|
|
grep -E "FAILED tests|ERROR tests|[0-9]+ passed," pytest.txt
|
|
failed_count=$(grep -E "FAILED tests|ERROR tests" pytest.txt | wc -l | tr -d '[:space:]')
|
|
if [[ $failed_count -gt 0 ]]; then
|
|
echo "$failed_count failed lines found! Task failed."
|
|
exit 1
|
|
fi
|
|
|
|
- name: Upload pytest test results
|
|
uses: actions/upload-artifact@v7
|
|
with:
|
|
name: pytest-results-${{ matrix.python-version }}-${{ matrix.sample_dir }}
|
|
path: |
|
|
pytest.txt
|
|
./htmlcov/
|
|
retention-days: 30
|
|
if: ${{ always() }}
|