Skip to content

Commit c197799

Browse files
committed
Simplify code
1 parent 87d8e50 commit c197799

File tree

1 file changed

+18
-14
lines changed

1 file changed

+18
-14
lines changed

pytest_bdd/scenario.py

Lines changed: 18 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -36,21 +36,25 @@ def find_argumented_step_fixture_name(name, type_, fixturemanager, request=None)
3636
for fixturename, fixturedefs in list(fixturemanager._arg2fixturedefs.items()):
3737
for fixturedef in fixturedefs:
3838
parser = getattr(fixturedef.func, "parser", None)
39-
match = parser.is_matching(name) if parser else None
40-
if match:
41-
converters = getattr(fixturedef.func, "converters", {})
42-
for arg, value in parser.parse_arguments(name).items():
43-
if arg in converters:
44-
value = converters[arg](value)
45-
if request:
46-
inject_fixture(request, arg, value)
47-
parser_name = get_step_fixture_name(parser.name, type_)
39+
if parser is None:
40+
continue
41+
match = parser.is_matching(name)
42+
if not match:
43+
continue
44+
45+
converters = getattr(fixturedef.func, "converters", {})
46+
for arg, value in parser.parse_arguments(name).items():
47+
if arg in converters:
48+
value = converters[arg](value)
4849
if request:
49-
try:
50-
request.getfixturevalue(parser_name)
51-
except pytest_fixtures.FixtureLookupError:
52-
continue
53-
return parser_name
50+
inject_fixture(request, arg, value)
51+
parser_name = get_step_fixture_name(parser.name, type_)
52+
if request:
53+
try:
54+
request.getfixturevalue(parser_name)
55+
except pytest_fixtures.FixtureLookupError:
56+
continue
57+
return parser_name
5458

5559

5660
def _find_step_function(request, step, scenario):

0 commit comments

Comments
 (0)