mirror of
https://github.com/crewAIInc/crewAI.git
synced 2026-08-10 16:32:28 +00:00
Some checks failed
CodeQL Advanced / Analyze (actions) (push) Has been cancelled
CodeQL Advanced / Analyze (python) (push) Has been cancelled
Check Documentation Broken Links / Check broken links (push) Has been cancelled
Vulnerability Scan / Detect changes (push) Has been cancelled
Vulnerability Scan / pip-audit (push) Has been cancelled
Skipped matrix jobs never post status for branch-protection names like tests (3.10). Add lightweight skip jobs with the same names so Actions-only and docs-only PRs can merge without waiting forever. Co-authored-by: Cursor Agent <cursoragent@cursor.com> Co-authored-by: Rip&Tear <theCyberTech@users.noreply.github.com>
158 lines
5.2 KiB
YAML
158 lines
5.2 KiB
YAML
name: Run Tests
|
||
|
||
on: [pull_request]
|
||
|
||
permissions:
|
||
contents: read
|
||
|
||
jobs:
|
||
changes:
|
||
name: Detect changes
|
||
runs-on: ubuntu-latest
|
||
outputs:
|
||
code: ${{ steps.filter.outputs.code }}
|
||
steps:
|
||
- uses: actions/checkout@34e114876b0b11c390a56381ad16ebd13914f8d5 # v4.3.1
|
||
- uses: dorny/paths-filter@d1c1ffe0248fe513906c8e24db8ea791d46f8590 # v3
|
||
id: filter
|
||
with:
|
||
# Exclusion-only patterns match every non-excluded file under the
|
||
# default "some" quantifier. Require all patterns (including "**")
|
||
# so docs/markdown/Actions-only PRs correctly set code=false.
|
||
predicate-quantifier: every
|
||
filters: |
|
||
code:
|
||
- '**'
|
||
- '!docs/**'
|
||
- '!**/*.md'
|
||
- '!.github/**'
|
||
|
||
tests-matrix:
|
||
name: tests (${{ matrix.python-version }})
|
||
needs: changes
|
||
if: needs.changes.outputs.code == 'true'
|
||
runs-on: ubuntu-latest
|
||
timeout-minutes: 15
|
||
strategy:
|
||
fail-fast: true
|
||
matrix:
|
||
python-version: ['3.10', '3.11', '3.12', '3.13']
|
||
group: [1, 2, 3, 4, 5, 6, 7, 8]
|
||
steps:
|
||
- name: Checkout code
|
||
uses: actions/checkout@34e114876b0b11c390a56381ad16ebd13914f8d5 # v4.3.1
|
||
with:
|
||
fetch-depth: 0 # Fetch all history for proper diff
|
||
|
||
- name: Restore global uv cache
|
||
id: cache-restore
|
||
uses: actions/cache/restore@0057852bfaa89a56745cba8c7296529d2fc39830 # v4.3.0
|
||
with:
|
||
path: |
|
||
~/.cache/uv
|
||
~/.local/share/uv
|
||
.venv
|
||
key: uv-main-py${{ matrix.python-version }}-${{ hashFiles('uv.lock') }}
|
||
restore-keys: |
|
||
uv-main-py${{ matrix.python-version }}-
|
||
|
||
- name: Install uv
|
||
uses: astral-sh/setup-uv@d0cc045d04ccac9d8b7881df0226f9e82c39688e # v6
|
||
with:
|
||
version: "0.11.3"
|
||
python-version: ${{ matrix.python-version }}
|
||
enable-cache: false
|
||
|
||
- name: Install the project
|
||
run: uv sync --all-groups --all-extras
|
||
|
||
- name: Restore test durations
|
||
uses: actions/cache/restore@0057852bfaa89a56745cba8c7296529d2fc39830 # v4.3.0
|
||
with:
|
||
path: .test_durations_py*
|
||
key: test-durations-py${{ matrix.python-version }}
|
||
|
||
- name: Run tests (group ${{ matrix.group }} of 8)
|
||
run: |
|
||
PYTHON_VERSION_SAFE=$(echo "${{ matrix.python-version }}" | tr '.' '_')
|
||
DURATION_FILE="../../.test_durations_py${PYTHON_VERSION_SAFE}"
|
||
|
||
# Temporarily always skip cached durations to fix test splitting
|
||
# When durations don't match, pytest-split runs duplicate tests instead of splitting
|
||
echo "Using even test splitting (duration cache disabled until fix merged)"
|
||
DURATIONS_ARG=""
|
||
|
||
# Original logic (disabled temporarily):
|
||
# if [ ! -f "$DURATION_FILE" ]; then
|
||
# echo "No cached durations found, tests will be split evenly"
|
||
# DURATIONS_ARG=""
|
||
# elif git diff origin/${{ github.base_ref }}...HEAD --name-only 2>/dev/null | grep -q "^tests/.*\.py$"; then
|
||
# echo "Test files have changed, skipping cached durations to avoid mismatches"
|
||
# DURATIONS_ARG=""
|
||
# else
|
||
# echo "No test changes detected, using cached test durations for optimal splitting"
|
||
# DURATIONS_ARG="--durations-path=${DURATION_FILE}"
|
||
# fi
|
||
|
||
cd lib/crewai && uv run pytest \
|
||
-vv \
|
||
--splits 8 \
|
||
--group ${{ matrix.group }} \
|
||
$DURATIONS_ARG \
|
||
--durations=10 \
|
||
--maxfail=3
|
||
|
||
- name: Run tool tests (group ${{ matrix.group }} of 8)
|
||
run: |
|
||
cd lib/crewai-tools && uv run pytest \
|
||
-vv \
|
||
--splits 8 \
|
||
--group ${{ matrix.group }} \
|
||
--durations=10 \
|
||
--maxfail=3
|
||
|
||
|
||
- name: Save uv caches
|
||
if: steps.cache-restore.outputs.cache-hit != 'true'
|
||
uses: actions/cache/save@0057852bfaa89a56745cba8c7296529d2fc39830 # v4.3.0
|
||
with:
|
||
path: |
|
||
~/.cache/uv
|
||
~/.local/share/uv
|
||
.venv
|
||
key: uv-main-py${{ matrix.python-version }}-${{ hashFiles('uv.lock') }}
|
||
|
||
# Report the required check names (tests 3.10–3.13) when the matrix is skipped.
|
||
# Branch protection expects these names; a skipped matrix never reports them.
|
||
tests-skip:
|
||
name: tests (${{ matrix.python-version }})
|
||
needs: changes
|
||
if: needs.changes.outputs.code != 'true'
|
||
runs-on: ubuntu-latest
|
||
strategy:
|
||
matrix:
|
||
python-version: ['3.10', '3.11', '3.12', '3.13']
|
||
steps:
|
||
- name: Skip non-code change
|
||
run: echo "Non-code change, skipping tests"
|
||
|
||
# Summary job to provide single status for branch protection
|
||
tests:
|
||
name: tests
|
||
runs-on: ubuntu-latest
|
||
needs: [changes, tests-matrix, tests-skip]
|
||
if: always()
|
||
steps:
|
||
- name: Check results
|
||
run: |
|
||
if [ "${{ needs.changes.outputs.code }}" != "true" ]; then
|
||
echo "Non-code change, skipping tests"
|
||
exit 0
|
||
fi
|
||
if [ "${{ needs.tests-matrix.result }}" == "success" ]; then
|
||
echo "All tests passed"
|
||
else
|
||
echo "Tests failed"
|
||
exit 1
|
||
fi
|