particle-os/debian-forge
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

test/depsolve: use strings for pytest.skip()

Browse source 
The pytest.skip() argument must be a string.  Calling it with an
exception produces an error.
This commit is contained in:
Achilleas Koutsou 2024-08-15 17:04:50 +02:00 • committed by Brian C. Lane
parent ef16cbf04f
commit d1dbcc20cf
1 changed files with 3 additions and 3 deletions
Download patch file Download diff file Expand all files Collapse all files

6
tools/test/test_depsolve.py
View file

@ -1250,7 +1250,7 @@ def test_depsolve(tmp_path, repo_servers, dnf_config, detect_fn, test_case):
try: try:
detect_fn() detect_fn()
except RuntimeError as e: except RuntimeError as e:
pytest.skip(e) pytest.skip(str(e))
# pylint: disable=fixme # pylint: disable=fixme
# TODO: remove this once dnf5 implementation is fixed # TODO: remove this once dnf5 implementation is fixed
@ -1314,7 +1314,7 @@ def test_dump(tmp_path, repo_servers, dnf_config, detect_fn, test_case):
try: try:
detect_fn() detect_fn()
except RuntimeError as e: except RuntimeError as e:
pytest.skip(e) pytest.skip(str(e))
repo_servers_copy = repo_servers.copy() repo_servers_copy = repo_servers.copy()
if "additional_servers" in test_case: if "additional_servers" in test_case:
@ -1361,7 +1361,7 @@ def test_search(tmp_path, repo_servers, dnf_config, detect_fn, test_case):
try: try:
detect_fn() detect_fn()
except RuntimeError as e: except RuntimeError as e:
pytest.skip(e) pytest.skip(str(e))
repo_servers_copy = repo_servers.copy() repo_servers_copy = repo_servers.copy()
if "additional_servers" in test_case: if "additional_servers" in test_case: