Skip to content

Commit d6f999b

Browse files
committed
from warning to ci error (Heberto feedback)
1 parent 9d9e1d6 commit d6f999b

File tree

1 file changed

+6
-2
lines changed

1 file changed

+6
-2
lines changed

neo/test/rawiotest/common_rawio_test.py

Lines changed: 6 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -21,6 +21,7 @@
2121
import unittest
2222
import importlib.util
2323
import warnings
24+
import os
2425

2526
from neo.utils.datasets import download_dataset, get_local_testing_data_folder, default_testing_repo
2627

@@ -34,8 +35,11 @@
3435
else:
3536
HAVE_DATALAD = False
3637
# pytest skip doesn't explain why we are skipping.
37-
# warnings are easy to skip for users that don't want to have to read them.
38-
warnings.warn("datalad failure. Please see installation instructions to run io-testing")
38+
# raise error if in CI to prevent tests from spuriously skipping and appearing
39+
# as passing.
40+
if os.environ.get("GITHUB_ACTIONS") == 'true':
41+
raise RuntimeError("Datalad is required for running the CI.")
42+
3943

4044
# url_for_tests = "https://portal.g-node.org/neo/" #This is the old place
4145
repo_for_test = default_testing_repo

0 commit comments

Comments
 (0)