Skip to content

Commit 1c617a5

Browse files
committed
CI: improve pytest parameter readability
1 parent aab0e3e commit 1c617a5

2 files changed

Lines changed: 16 additions & 1 deletion

File tree

conftest.py

Lines changed: 15 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -287,6 +287,21 @@ def pytest_runtest_makereport(item, call):
287287
result.outcome = 'passed'
288288

289289

290+
def pytest_make_parametrize_id(config, val, argname):
291+
"""
292+
Prevents pytest to make obsucre parameter names (param0, param1, ...)
293+
and default to str(val) instead for better log readability.
294+
"""
295+
# First see if it has a name
296+
if hasattr(val, '__name__'):
297+
return val.__name__
298+
# Then try str(val)
299+
try:
300+
return str(val)
301+
except Exception:
302+
return None # Fall back to default behavior
303+
304+
290305
# A list of optimization options/pipelines to be used in testing
291306
# regarding spatial and/or temporal blocking.
292307
opts_tiling = ['advanced',

devito/finite_differences/derivative.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -166,7 +166,7 @@ def _validate_expr(expr):
166166
expr = diffify(expr)
167167
except Exception as e:
168168
d = type(expr)
169-
raise ValueError(f"`expr` must be a `Differentiable` type object not {d}") from e
169+
raise ValueError(f"`expr` must be a `Differentiable` not {d}") from e
170170
return expr
171171

172172
@staticmethod

0 commit comments

Comments
 (0)