Stop quoting provider errors in API responses (CodeQL) #569
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: Tests | |
| on: | |
| push: | |
| branches: [main, develop] | |
| pull_request: | |
| branches: [main, develop] | |
| workflow_dispatch: | |
| jobs: | |
| lint: | |
| name: Lint & Format | |
| runs-on: ubuntu-latest | |
| steps: | |
| - uses: actions/checkout@v6 | |
| - name: Set up Python | |
| uses: actions/setup-python@v6 | |
| with: | |
| python-version: '3.12' | |
| - name: Install uv | |
| uses: astral-sh/setup-uv@v7 | |
| - name: Install linting tools | |
| run: | | |
| uv pip install --system ruff | |
| - name: Lint with Ruff | |
| run: ruff check src tests | |
| - name: Format check with Ruff | |
| run: ruff format --check src tests | |
| unit-tests: | |
| name: Unit Tests | |
| runs-on: ubuntu-latest | |
| strategy: | |
| matrix: | |
| python-version: ['3.12'] | |
| steps: | |
| - uses: actions/checkout@v6 | |
| - name: Set up Python ${{ matrix.python-version }} | |
| uses: actions/setup-python@v6 | |
| with: | |
| python-version: ${{ matrix.python-version }} | |
| - name: Set up Node.js for hed-lsp | |
| uses: actions/setup-node@v6 | |
| with: | |
| node-version: '22' | |
| - name: Build hed-lsp (for tests/lsp/ real-LSP tests) | |
| env: | |
| HED_LSP_REF: 36447afb5e48bfecc45af0464b652614ba571414 | |
| run: | | |
| git clone https://github.com/hed-standard/hed-lsp.git "$RUNNER_TEMP/hed-lsp" | |
| git -C "$RUNNER_TEMP/hed-lsp" checkout --quiet "${HED_LSP_REF}" | |
| cd "$RUNNER_TEMP/hed-lsp" | |
| corepack enable | |
| corepack prepare pnpm@10.33.4 --activate | |
| pnpm install --frozen-lockfile | |
| pnpm run -r compile | |
| echo "HED_LSP_SERVER_JS=$RUNNER_TEMP/hed-lsp/server/out/server.js" >> "$GITHUB_ENV" | |
| - name: Install uv | |
| uses: astral-sh/setup-uv@v7 | |
| - name: Install dependencies | |
| run: | | |
| uv pip install --system -e ".[dev]" | |
| - name: Run unit tests (excluding integration) | |
| run: | | |
| pytest tests/ -v -m "not integration" --cov=src --cov-report=xml:coverage-unit.xml --cov-report=term-missing | |
| - name: Upload coverage to Codecov | |
| uses: codecov/codecov-action@v5 | |
| with: | |
| token: ${{ secrets.CODECOV_TOKEN }} | |
| files: ./coverage-unit.xml | |
| flags: unittests | |
| name: codecov-unit | |
| fail_ci_if_error: false | |
| verbose: true | |
| # Check which files changed to decide if integration tests should run | |
| check-changes: | |
| name: Check Changed Files | |
| runs-on: ubuntu-latest | |
| outputs: | |
| integration_needed: ${{ steps.changes.outputs.integration_needed }} | |
| steps: | |
| - uses: actions/checkout@v6 | |
| with: | |
| fetch-depth: 0 | |
| - name: Check for integration-related changes | |
| id: changes | |
| run: | | |
| # For push events, compare with previous commit | |
| # For PRs, compare with base branch | |
| if [ "${{ github.event_name }}" = "pull_request" ]; then | |
| BASE=${{ github.event.pull_request.base.sha }} | |
| else | |
| BASE=${{ github.event.before }} | |
| fi | |
| # Get changed files | |
| CHANGED=$(git diff --name-only $BASE ${{ github.sha }} 2>/dev/null || echo "") | |
| echo "Changed files:" | |
| echo "$CHANGED" | |
| # Integration/standalone tests should run if these paths change: | |
| # - Integration test files (test_integration*.py, test_standalone*.py) | |
| # - Agent code (workflow, annotation, evaluation, keyword extraction, etc.) | |
| # - Validation code | |
| # - Anthropic LLM utility | |
| # - CLI code (for CLI integration tests) | |
| # - Semantic search code | |
| INTEGRATION_PATTERNS="tests/test_.*integration|tests/test_standalone|src/agents/|src/validation/|src/utils/anthropic|src/utils/semantic|src/cli/" | |
| if echo "$CHANGED" | grep -qE "$INTEGRATION_PATTERNS"; then | |
| echo "integration_needed=true" >> $GITHUB_OUTPUT | |
| echo "Integration-related files changed - tests will run" | |
| else | |
| echo "integration_needed=false" >> $GITHUB_OUTPUT | |
| echo "No integration-related files changed - tests will be skipped" | |
| fi | |
| # Standalone tests verify LLM behavior hasn't regressed across PRs. | |
| # Only runs on PRs targeting main when LangGraph components are touched, | |
| # to avoid unnecessary API costs on feature branch PRs. | |
| standalone-tests: | |
| name: Standalone Tests (PR to main) | |
| runs-on: ubuntu-latest | |
| needs: [check-changes] | |
| if: | | |
| github.event_name == 'pull_request' && | |
| github.event.pull_request.base.ref == 'main' && | |
| needs.check-changes.outputs.integration_needed == 'true' | |
| steps: | |
| - uses: actions/checkout@v6 | |
| - name: Set up Python | |
| uses: actions/setup-python@v6 | |
| with: | |
| python-version: '3.12' | |
| - name: Install uv | |
| uses: astral-sh/setup-uv@v7 | |
| - name: Install dependencies | |
| run: | | |
| uv pip install --system -e ".[dev]" | |
| uv pip install --system pytest-timeout | |
| - name: Run standalone tests only | |
| env: | |
| ANTHROPIC_API_KEY: ${{ secrets.ANTHROPIC_API_KEY }} | |
| ANTHROPIC_BASE_URL: ${{ vars.ANTHROPIC_BASE_URL }} | |
| ANTHROPIC_WORKSPACE_ID: ${{ vars.ANTHROPIC_WORKSPACE_ID }} | |
| run: | | |
| if [ -n "$ANTHROPIC_API_KEY" ]; then | |
| echo "Running standalone tests..." | |
| pytest tests/ -v -m standalone --timeout=180 --cov=src --cov-report=xml:coverage-standalone.xml --cov-report=term-missing | |
| else | |
| echo "ANTHROPIC_API_KEY not set, skipping standalone tests" | |
| fi | |
| - name: Upload standalone coverage to Codecov | |
| if: always() | |
| uses: codecov/codecov-action@v5 | |
| with: | |
| token: ${{ secrets.CODECOV_TOKEN }} | |
| files: ./coverage-standalone.xml | |
| flags: standalone | |
| name: codecov-standalone | |
| fail_ci_if_error: false | |
| integration-tests: | |
| name: Integration Tests (Real LLM) | |
| runs-on: ubuntu-latest | |
| needs: [check-changes] | |
| # Only run integration tests when: | |
| # 1. Push event (after merge) or manual trigger, not on PRs | |
| # 2. Integration-related files changed (agents, validation, anthropic, or test file itself) | |
| if: | | |
| (github.event_name == 'push' || github.event_name == 'workflow_dispatch') && | |
| (needs.check-changes.outputs.integration_needed == 'true' || github.event_name == 'workflow_dispatch') | |
| steps: | |
| - uses: actions/checkout@v6 | |
| - name: Set up Python | |
| uses: actions/setup-python@v6 | |
| with: | |
| python-version: '3.12' | |
| - name: Install uv | |
| uses: astral-sh/setup-uv@v7 | |
| - name: Install dependencies | |
| run: | | |
| uv pip install --system -e ".[dev]" | |
| uv pip install --system pytest-timeout | |
| - name: Run integration tests only | |
| env: | |
| ANTHROPIC_API_KEY: ${{ secrets.ANTHROPIC_API_KEY }} | |
| ANTHROPIC_BASE_URL: ${{ vars.ANTHROPIC_BASE_URL }} | |
| ANTHROPIC_WORKSPACE_ID: ${{ vars.ANTHROPIC_WORKSPACE_ID }} | |
| run: | | |
| # Only run if the secret is available | |
| if [ -n "$ANTHROPIC_API_KEY" ]; then | |
| echo "Running integration tests with Claude (Claude Platform on AWS)..." | |
| pytest tests/ -v -m integration --timeout=180 --cov=src --cov-report=xml:coverage-integration.xml --cov-report=term-missing | |
| else | |
| echo "ANTHROPIC_API_KEY not set, skipping integration tests" | |
| fi | |
| - name: Upload integration coverage to Codecov | |
| if: always() | |
| uses: codecov/codecov-action@v5 | |
| with: | |
| token: ${{ secrets.CODECOV_TOKEN }} | |
| files: ./coverage-integration.xml | |
| flags: integration | |
| name: codecov-integration | |
| fail_ci_if_error: false | |
| all-tests: | |
| name: All Tests Summary | |
| runs-on: ubuntu-latest | |
| needs: [lint, unit-tests, check-changes, standalone-tests, integration-tests] | |
| # Run even if integration-tests or standalone-tests was skipped | |
| if: always() | |
| steps: | |
| - name: Check lint result | |
| if: needs.lint.result == 'failure' | |
| run: | | |
| echo "Lint check failed" | |
| exit 1 | |
| - name: Check unit tests result | |
| if: needs.unit-tests.result == 'failure' | |
| run: | | |
| echo "Unit tests failed" | |
| exit 1 | |
| - name: Check standalone tests result | |
| # Only fail if standalone tests ran and failed (not if skipped) | |
| if: needs.standalone-tests.result == 'failure' | |
| run: | | |
| echo "Standalone tests failed" | |
| exit 1 | |
| - name: Check integration tests result | |
| # Only fail if integration tests ran and failed (not if skipped) | |
| if: needs.integration-tests.result == 'failure' | |
| run: | | |
| echo "Integration tests failed" | |
| exit 1 | |
| - name: All checks passed | |
| run: | | |
| echo "All required checks passed!" | |
| echo "Lint: ${{ needs.lint.result }}" | |
| echo "Unit tests: ${{ needs.unit-tests.result }}" | |
| echo "Standalone tests: ${{ needs.standalone-tests.result }}" | |
| echo "Integration tests: ${{ needs.integration-tests.result }}" | |
| echo "Integration needed: ${{ needs.check-changes.outputs.integration_needed }}" |