Skip to content

Commit e773de1

Browse files
karlwaldmanclaude
andcommitted
fix(ci): Ignore integration/contract test dirs in CI
conftest.py in those dirs imports python-dotenv (not in dev deps). pytest collects conftest even with -m exclusion. Use --ignore instead. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
1 parent 70f6594 commit e773de1

File tree

2 files changed

+2
-2
lines changed

2 files changed

+2
-2
lines changed

.github/workflows/publish.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -29,7 +29,7 @@ jobs:
2929
continue-on-error: true
3030

3131
- name: Run unit tests
32-
run: pytest tests/ -m 'not integration and not contract' --cov=oilpriceapi -v
32+
run: pytest tests/ --ignore=tests/integration --ignore=tests/contract -m 'not slow' --cov=oilpriceapi -v
3333

3434
publish:
3535
name: Publish to PyPI

.github/workflows/test.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -37,7 +37,7 @@ jobs:
3737
continue-on-error: true
3838

3939
- name: Run unit tests
40-
run: pytest tests/ -m 'not integration and not contract' --cov=oilpriceapi --cov-report=xml -v
40+
run: pytest tests/ --ignore=tests/integration --ignore=tests/contract -m 'not slow' --cov=oilpriceapi --cov-report=xml -v
4141

4242
- name: Upload coverage
4343
if: matrix.python-version == '3.12'

0 commit comments

Comments
 (0)