Skip to content
Snippets Groups Projects
Commit 99afaf60 authored by Chris Lamb's avatar Chris Lamb :eyes:
Browse files

Avoid parens-in-parens in test skipping messages.

parent e52eef5a
No related branches found
No related tags found
No related merge requests found
Pipeline #665907 failed
......@@ -111,7 +111,7 @@ def skip_if_tool_version_is(tool, actual_ver, target_ver, vcls=Version):
actual_ver = actual_ver()
return skipif(
vcls(str(actual_ver)) == vcls(str(target_ver)),
reason="requires {} != {} ({} detected)".format(
reason="requires {} != {}; {} detected".format(
tool, target_ver, actual_ver
),
tools=(tool,),
......@@ -125,7 +125,7 @@ def skip_unless_tool_is_at_least(tool, actual_ver, min_ver, vcls=Version):
actual_ver = actual_ver()
return skipif(
vcls(str(actual_ver)) < vcls(str(min_ver)),
reason="{} >= {} ({} detected)".format(
reason="{} >= {}; {} detected".format(
reason(tool, force_include=True), min_ver, actual_ver
),
tools=(tool,),
......@@ -139,7 +139,7 @@ def skip_unless_tool_is_at_most(tool, actual_ver, max_ver, vcls=Version):
actual_ver = actual_ver()
return skipif(
vcls(str(actual_ver)) > vcls(str(max_ver)),
reason="{} <= {} ({} detected)".format(
reason="{} <= {}; {} detected".format(
reason(tool, force_include=True), max_ver, actual_ver
),
tools=(tool,),
......@@ -156,7 +156,7 @@ def skip_unless_tool_is_between(
return skipif(
(vcls(str(actual_ver)) < vcls(str(min_ver)))
or (vcls(str(actual_ver)) > vcls(str(max_ver))),
reason="{} min {} >= {} ({} detected)".format(
reason="{} min {} >= {}; {} detected".format(
reason(tool, force_include=True), min_ver, max_ver, actual_ver
),
tools=(tool,),
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment