Add pylint, pip caching improvements, and static analysis to GitHub workflow #214
Workflow file for this run
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| name: Python Tests | |
| on: | |
| workflow_dispatch: | |
| push: | |
| branches: [ main ] | |
| pull_request: | |
| branches: [ main ] | |
| permissions: | |
| contents: read | |
| checks: write | |
| pull-requests: write | |
| jobs: | |
| test: | |
| runs-on: ubuntu-latest | |
| strategy: | |
| matrix: | |
| python-version: [ '3.12', '3.13' , '3.14' ] | |
| steps: | |
| - uses: actions/checkout@v4 | |
| - name: Set up Python ${{ matrix.python-version }} | |
| uses: actions/setup-python@v5 | |
| with: | |
| python-version: ${{ matrix.python-version }} | |
| cache: pip | |
| cache-dependency-path: requirements.txt | |
| - name: Install dependencies | |
| run: | | |
| python -m pip install --upgrade pip | |
| pip install -r requirements.txt | |
| - name: Run pylint analysis | |
| run: | | |
| bash tests/python/run_pylint.sh | |
| - name: Upload pylint reports | |
| if: always() | |
| uses: actions/upload-artifact@v4 | |
| with: | |
| name: pylint-reports-${{ matrix.python-version }} | |
| path: tests/python/pylint/reports/ | |
| - name: Verify bytecode compilation | |
| run: | | |
| python -m compileall infrastructure samples setup shared tests | |
| # Run tests with continue-on-error so that coverage and PR comments are always published. | |
| # The final step will explicitly fail the job if any test failed, ensuring PRs cannot be merged with failing tests. | |
| - name: Run pytest with coverage and generate JUnit XML | |
| id: pytest | |
| run: | | |
| PYTHONPATH=$(pwd) COVERAGE_FILE=tests/python/.coverage-${{ matrix.python-version }} pytest --cov=shared/python --cov-config=tests/python/.coveragerc --cov-report=html:tests/python/htmlcov-${{ matrix.python-version }} --junitxml=tests/python/junit-${{ matrix.python-version }}.xml tests/python/ | |
| continue-on-error: true | |
| - name: Upload coverage HTML report | |
| if: always() | |
| uses: actions/upload-artifact@v4 | |
| with: | |
| name: coverage-html-${{ matrix.python-version }} | |
| path: tests/python/htmlcov-${{ matrix.python-version }}/ | |
| - name: Upload JUnit test results | |
| if: always() | |
| uses: actions/upload-artifact@v4 | |
| continue-on-error: true | |
| with: | |
| name: junit-results-${{ matrix.python-version }} | |
| path: tests/python/junit-${{ matrix.python-version }}.xml | |
| - name: Publish Unit Test Results to PR | |
| if: always() | |
| uses: EnricoMi/publish-unit-test-result-action@v2 | |
| with: | |
| files: tests/python/junit-${{ matrix.python-version }}.xml | |
| comment_title: Python ${{ matrix.python-version }} Test Results | |
| # Explicitly fail the job if any test failed (so PRs cannot be merged with failing tests). | |
| # This runs after all reporting steps, meaning coverage and PR comments are always published. | |
| - name: Fail if tests failed | |
| if: steps.pytest.outcome == 'failure' | |
| run: | | |
| echo "::error ::Unit tests failed. See above for details." | |
| exit 1 |