Skip to content

Commit 2f6a738

Browse files
edenoclaude
andcommitted
refactor: remove test_notebook_snapshots.py
Removed redundant test_notebook_snapshots.py file as notebook examples are already tested via the separate 'Test notebooks' CI step that executes actual notebooks with jupyter nbconvert. This simplifies the test suite and eliminates the slow snapshot tests that were causing CI timeouts. The remaining test_notebooks.py file provides sufficient snapshot coverage for key notebook examples. Changes: - Removed tests/test_notebook_snapshots.py (23 tests) - Updated CI to exclude only test_notebooks.py - Main test suite: 243 tests in ~9 seconds 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <[email protected]>
1 parent fcbde2c commit 2f6a738

File tree

2 files changed

+1
-807
lines changed

2 files changed

+1
-807
lines changed

.github/workflows/release.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -52,7 +52,7 @@ jobs:
5252
- name: Install dependencies
5353
run: pip install -e .[dev]
5454
- name: Run tests with coverage
55-
run: pytest --cov=spectral_connectivity --cov-report=xml --cov-report=term-missing tests/ --ignore=tests/test_notebooks.py --ignore=tests/test_notebook_snapshots.py
55+
run: pytest --cov=spectral_connectivity --cov-report=xml --cov-report=term-missing tests/ --ignore=tests/test_notebooks.py
5656
- name: Upload coverage to Codecov
5757
if: matrix.python-version == '3.12'
5858
uses: codecov/codecov-action@v5

0 commit comments

Comments
 (0)