Skip to content

Commit bd50991

Browse files
phernandezclaude
andcommitted
fix: CI Postgres coverage jobs used wrong pytest marker (-m postgres)
The coverage code paths for Postgres unit and integration jobs filtered with `-m postgres`, but no tests use that marker. Postgres is selected via BASIC_MEMORY_TEST_POSTGRES=1 env var. This caused 0 tests to run on Python 3.12 (the only version with coverage: true). - Postgres unit: remove `-m postgres` (matches `just test-unit-postgres`) - Postgres integration: use `-m "not semantic"` (matches `just test-int-postgres`) Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> Signed-off-by: phernandez <paul@basicmachines.co>
1 parent 8e64caf commit bd50991

1 file changed

Lines changed: 2 additions & 2 deletions

File tree

.github/workflows/test.yml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -217,7 +217,7 @@ jobs:
217217
- name: Run tests with coverage
218218
if: matrix.coverage && github.event_name == 'push' && github.ref == 'refs/heads/main'
219219
run: |
220-
timeout --signal=KILL 600 bash -c 'BASIC_MEMORY_ENV=test BASIC_MEMORY_TEST_POSTGRES=1 uv run coverage run --data-file=.coverage.postgres-unit --source=basic_memory -m pytest -p pytest_mock -v --no-cov -m postgres tests' || test $? -eq 137
220+
timeout --signal=KILL 600 bash -c 'BASIC_MEMORY_ENV=test BASIC_MEMORY_TEST_POSTGRES=1 uv run coverage run --data-file=.coverage.postgres-unit --source=basic_memory -m pytest -p pytest_mock -v --no-cov tests' || test $? -eq 137
221221
222222
- name: Run tests
223223
if: "!(matrix.coverage && github.event_name == 'push' && github.ref == 'refs/heads/main')"
@@ -277,7 +277,7 @@ jobs:
277277
- name: Run tests with coverage
278278
if: matrix.coverage && github.event_name == 'push' && github.ref == 'refs/heads/main'
279279
run: |
280-
timeout --signal=KILL 600 bash -c 'BASIC_MEMORY_ENV=test BASIC_MEMORY_TEST_POSTGRES=1 uv run coverage run --data-file=.coverage.postgres-int --source=basic_memory -m pytest -p pytest_mock -v --no-cov -m postgres test-int' || test $? -eq 137
280+
timeout --signal=KILL 600 bash -c 'BASIC_MEMORY_ENV=test BASIC_MEMORY_TEST_POSTGRES=1 uv run coverage run --data-file=.coverage.postgres-int --source=basic_memory -m pytest -p pytest_mock -v --no-cov -m "not semantic" test-int' || test $? -eq 137
281281
282282
- name: Run tests
283283
if: "!(matrix.coverage && github.event_name == 'push' && github.ref == 'refs/heads/main')"

0 commit comments

Comments
 (0)