Skip to content

Commit

Permalink
TEST: Use less confusing function name for testing CLI with --dry-run (
Browse files Browse the repository at this point in the history
…#424)

* fix: confusing function name in test

* dbg: maximize sphinx verbosity

* fix: add texlive as a dependency

I believe nbsphinx is tripping when calling pandoc.
The notebook has some latex symbols so I'm adding texlive to the
dependencies installation.

I believe this is the only difference between the local testing
environment I've created and CircleCI.

---------

Co-authored-by: Chris Markiewicz <[email protected]>
  • Loading branch information
oesteban and effigies authored Jan 18, 2024
1 parent 048eac9 commit a607eb5
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions sdcflows/cli/tests/test_find_estimators.py
Original file line number Diff line number Diff line change
Expand Up @@ -25,7 +25,7 @@
import pytest
from niworkflows.utils.testing import generate_bids_skeleton

from sdcflows.cli.main import main as find_estimators
from sdcflows.cli.main import main as cli_finder_wrapper
from sdcflows.fieldmaps import clear_registry

OUTPUT = """\
Expand Down Expand Up @@ -131,7 +131,7 @@
("b0field", b0field_config, "pepolar"),
],
)
def test_find_estimators(tmp_path, capsys, test_id, config, estimator_id):
def test_cli_finder_wrapper(tmp_path, capsys, test_id, config, estimator_id):
"""Test the CLI with --dry-run."""
import sdcflows.config as sc

Expand All @@ -141,7 +141,7 @@ def test_find_estimators(tmp_path, capsys, test_id, config, estimator_id):
path = (tmp_path / test_id).absolute()
generate_bids_skeleton(path, config)
with pytest.raises(SystemExit) as wrapped_exit:
find_estimators([str(path), str(tmp_path / "out"), "participant", "--dry-run"])
cli_finder_wrapper([str(path), str(tmp_path / "out"), "participant", "--dry-run"])

assert wrapped_exit.value.code == 0
output = OUTPUT.format(path=path, estimator_id=estimator_id)
Expand Down

0 comments on commit a607eb5

Please sign in to comment.