Skip to content

Commit

Permalink
another fix to custom dataloder test and github action
Browse files Browse the repository at this point in the history
  • Loading branch information
ori-kron-wis committed Aug 12, 2024
1 parent 8cf3517 commit ba5a028
Show file tree
Hide file tree
Showing 3 changed files with 4 additions and 4 deletions.
2 changes: 1 addition & 1 deletion .github/workflows/test_linux.yml
Original file line number Diff line number Diff line change
Expand Up @@ -53,7 +53,7 @@ jobs:
DISPLAY: :42
COLUMNS: 120
run: |
coverage run -m pytest -v --color=yes -m "not custom.dataloader"
coverage run -m pytest -v --color=yes -m "not custom_dataloader"
coverage report
- uses: codecov/codecov-action@v4
Expand Down
4 changes: 2 additions & 2 deletions tests/conftest.py
Original file line number Diff line number Diff line change
Expand Up @@ -72,8 +72,8 @@ def pytest_collection_modifyitems(config, items):
if not run_internet and ("internet" in item.keywords):
item.add_marker(skip_internet)

# All tests marked with `pytest.mark.custom.dataloader` get skipped unless
# `--custom.dataloader-tests` passed
# All tests marked with `pytest.mark.custom_dataloader` get skipped unless
# `--custom_dataloader-tests` passed
if not run_custom_dataloader and ("dataloader" in item.keywords):
item.add_marker(skip_custom_dataloader)

Expand Down
2 changes: 1 addition & 1 deletion tests/dataloaders/test_custom_dataloader.py
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,7 @@
from scvi.data import synthetic_iid


@pytest.mark.custom.dataloader
@pytest.mark.custom_dataloader
def test_custom_dataloader(save_path):
# local bracnh with fix only for this test
import sys
Expand Down

0 comments on commit ba5a028

Please sign in to comment.