Skip to content

Commit a75009c

Browse files
author
Sylvain MARIE
committed
Better error message when users use THIS_MODULE in cases= instead of module=.
1 parent e27da5d commit a75009c

File tree

2 files changed

+6
-0
lines changed

2 files changed

+6
-0
lines changed

docs/changelog.md

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,9 @@
11
# Changelog
22

3+
### 1.11.3 IN PROGRESS - minor improvements
4+
5+
Better error message when users use `THIS_MODULE` in `cases=` instead of `module=`.
6+
37
### 1.11.2 - Increased tolerance to other plugins + bugfix
48

59
Now when other plugins try to manipulate the fixture closure, warning messages are emitted but no error is raised. Fixed [#55](https://github.com/smarie/python-pytest-cases/issues/55).

pytest_cases/main_params.py

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -269,6 +269,8 @@ def get_all_cases(cases=None, # type: Union[Callable[[Any], Any],
269269
if callable(cases):
270270
# single element
271271
_cases = [case_getter for case_getter in _get_case_getter_s(cases)]
272+
elif cases is THIS_MODULE:
273+
raise ValueError("`THIS_MODULE` should only be used in the `module` argument, not in the `cases` argument")
272274
else:
273275
# already a sequence
274276
_cases = [case_getter for c in cases for case_getter in _get_case_getter_s(c)]

0 commit comments

Comments
 (0)