Skip to content

Commit 6d64bd9

Browse files
committed
rf: move wrapper test to wrapper module
1 parent 142dce0 commit 6d64bd9

File tree

2 files changed

+21
-27
lines changed

2 files changed

+21
-27
lines changed

wrapper/src/tests/__init__.py

Whitespace-only changes.

petprep/cli/tests/test_docker_parser.py renamed to wrapper/src/tests/test_docker_parser.py

Lines changed: 21 additions & 27 deletions
Original file line numberDiff line numberDiff line change
@@ -1,23 +1,19 @@
1-
import importlib
21
import subprocess
32
import sys
4-
from pathlib import Path
53

6-
# Add wrapper package to path
7-
sys.path.insert(0, str(Path(__file__).resolve().parents[3] / 'wrapper' / 'src'))
4+
import petprep_docker.__main__ as ppd
85

96

107
def test_docker_get_parser():
11-
module = importlib.import_module('petprep_docker.__main__')
12-
parser = module.get_parser()
8+
9+
parser = ppd.get_parser()
1310
assert parser is not None
1411

1512

1613
def test_docker_main_help(monkeypatch, capsys):
17-
module = importlib.import_module('petprep_docker.__main__')
18-
monkeypatch.setattr(module, 'check_docker', lambda: 1)
19-
monkeypatch.setattr(module, 'check_image', lambda img: True)
20-
monkeypatch.setattr(module, 'check_memory', lambda img: 16000)
14+
monkeypatch.setattr(ppd, 'check_docker', lambda: 1)
15+
monkeypatch.setattr(ppd, 'check_image', lambda img: True)
16+
monkeypatch.setattr(ppd, 'check_memory', lambda img: 16000)
2117

2218
captured = {}
2319

@@ -31,24 +27,23 @@ def fake_check_output(cmd):
3127
captured['cmd'] = cmd
3228
return b'usage: petprep bids_dir output_dir {participant}\n\noptional arguments:'
3329

34-
monkeypatch.setattr(module.subprocess, 'run', fake_run)
35-
monkeypatch.setattr(module.subprocess, 'check_output', fake_check_output)
36-
monkeypatch.setattr(module, 'merge_help', lambda a, b: 'merged')
30+
monkeypatch.setattr(ppd.subprocess, 'run', fake_run)
31+
monkeypatch.setattr(ppd.subprocess, 'check_output', fake_check_output)
32+
monkeypatch.setattr(ppd, 'merge_help', lambda a, b: 'merged')
3733

3834
sys.argv = ['petprep-docker', '--help']
39-
ret = module.main()
35+
ret = ppd.main()
4036
assert ret == 0
4137
assert captured['cmd'][:3] == ['docker', 'run', '--rm']
4238
assert '-h' in captured['cmd']
4339
assert '-i' in captured['cmd']
44-
assert module.__name__
40+
assert ppd.__name__
4541

4642

4743
def test_docker_main_version(monkeypatch):
48-
module = importlib.import_module('petprep_docker.__main__')
49-
monkeypatch.setattr(module, 'check_docker', lambda: 1)
50-
monkeypatch.setattr(module, 'check_image', lambda img: True)
51-
monkeypatch.setattr(module, 'check_memory', lambda img: 16000)
44+
monkeypatch.setattr(ppd, 'check_docker', lambda: 1)
45+
monkeypatch.setattr(ppd, 'check_image', lambda img: True)
46+
monkeypatch.setattr(ppd, 'check_memory', lambda img: 16000)
5247

5348
calls = []
5449

@@ -58,21 +53,20 @@ def fake_run(cmd, *args, **kwargs):
5853
return subprocess.CompletedProcess(cmd, 0, stdout=b'20.10')
5954
return subprocess.CompletedProcess(cmd, 0)
6055

61-
monkeypatch.setattr(module.subprocess, 'run', fake_run)
56+
monkeypatch.setattr(ppd.subprocess, 'run', fake_run)
6257

6358
sys.argv = ['petprep-docker', '--version']
64-
ret = module.main()
59+
ret = ppd.main()
6560
assert ret == 0
6661
cmd = calls[-1]
6762
assert cmd[:3] == ['docker', 'run', '--rm']
6863
assert '--version' in cmd
6964

7065

7166
def test_docker_command_options(monkeypatch, tmp_path):
72-
module = importlib.import_module('petprep_docker.__main__')
73-
monkeypatch.setattr(module, 'check_docker', lambda: 1)
74-
monkeypatch.setattr(module, 'check_image', lambda img: True)
75-
monkeypatch.setattr(module, 'check_memory', lambda img: 16000)
67+
monkeypatch.setattr(ppd, 'check_docker', lambda: 1)
68+
monkeypatch.setattr(ppd, 'check_image', lambda img: True)
69+
monkeypatch.setattr(ppd, 'check_memory', lambda img: 16000)
7670

7771
bids_dir = tmp_path / 'bids'
7872
out_dir = tmp_path / 'out'
@@ -89,7 +83,7 @@ def fake_run(cmd, *args, **kwargs):
8983
return subprocess.CompletedProcess(cmd, 0, stdout=b'20.10')
9084
return subprocess.CompletedProcess(cmd, 0)
9185

92-
monkeypatch.setattr(module.subprocess, 'run', fake_run)
86+
monkeypatch.setattr(ppd.subprocess, 'run', fake_run)
9387

9488
sys.argv = [
9589
'petprep-docker',
@@ -101,7 +95,7 @@ def fake_run(cmd, *args, **kwargs):
10195
'--output-spaces',
10296
'MNI152Lin',
10397
]
104-
ret = module.main()
98+
ret = ppd.main()
10599
assert ret == 0
106100
cmd = calls[-1]
107101
joined = ' '.join(cmd)

0 commit comments

Comments
 (0)