Skip to content

Commit a04de5a

Browse files
committed
Add FileExistError test case
1 parent a50c442 commit a04de5a

File tree

1 file changed

+30
-8
lines changed

1 file changed

+30
-8
lines changed

tests/test_cli.py

Lines changed: 30 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -174,21 +174,43 @@ def test_copy_examples(case, user_inputs, expected, target, example_cases):
174174
assert not empty_dir, f"Expected nothing, but found: {empty_dir}"
175175

176176

177-
# Test bad inputs to copy_examples
177+
# Test bad inputs to copy_examples on case3
178178
# These include:
179179
# 1) input not found (example or pack)
180180
# 2) mixed good and bad inputs
181-
@pytest.mark.parametrize("case", ["case1", "case2", "case3", "case4", "case5"])
181+
# 3) Path to directory already exists
182182
@pytest.mark.parametrize(
183-
"bad_inputs, expected",
183+
"bad_inputs, expected, path",
184184
[
185-
(["bad_example"], ValueError), # input not found (example or pack)
186-
(["ex1", "bad_example"], ValueError), # mixed good and bad inputs
185+
(
186+
["bad_example"],
187+
ValueError,
188+
None,
189+
), # input not found (example or pack)
190+
(
191+
["ex1", "bad_example"],
192+
ValueError,
193+
None,
194+
), # mixed good ex and bad inputs
195+
(
196+
["packA", "bad_example"],
197+
ValueError,
198+
None,
199+
), # mixed good pack and bad inputs
200+
(
201+
["ex1"],
202+
FileExistsError,
203+
Path("docs/examples/"),
204+
), # path to dir already exists
187205
],
188206
)
189-
def test_copy_examples_bad(bad_inputs, expected, case, example_cases):
190-
case_dir = example_cases / case
207+
def test_copy_examples_bad(bad_inputs, expected, path, example_cases):
208+
case_dir = example_cases / "case3"
191209
pm = PacksManager(root_path=case_dir)
192210
examples_dict = pm.available_examples()
193211
with pytest.raises(expected):
194-
copy_examples(examples_dict, user_input=bad_inputs)
212+
copy_examples(
213+
examples_dict,
214+
user_input=bad_inputs,
215+
target_dir=case_dir / path if path else None,
216+
)

0 commit comments

Comments
 (0)