Skip to content

Commit 6859de0

Browse files
committed
Get the resolver name directly from test CLI
1 parent 07ec301 commit 6859de0

File tree

1 file changed

+9
-14
lines changed

1 file changed

+9
-14
lines changed

tests/conftest.py

Lines changed: 9 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -39,16 +39,11 @@ def pytest_addoption(parser):
3939
help="keep temporary test directories",
4040
)
4141
parser.addoption(
42-
"--new-resolver",
43-
action="store_true",
44-
default=False,
45-
help="use new resolver in tests",
46-
)
47-
parser.addoption(
48-
"--new-resolver-runtests",
49-
action="store_true",
50-
default=False,
51-
help="run the skipped tests for the new resolver",
42+
"--resolver",
43+
action="store",
44+
default="2020-resolver",
45+
choices=["2020-resolver", "legacy"],
46+
help="use given resolver in tests",
5247
)
5348
parser.addoption(
5449
"--use-venv",
@@ -100,17 +95,17 @@ def pytest_collection_modifyitems(config, items):
10095
def resolver_variant(request):
10196
"""Set environment variable to make pip default to the correct resolver.
10297
"""
103-
new_resolver = request.config.getoption("--new-resolver")
98+
resolver = request.config.getoption("--resolver")
99+
100+
# Handle the environment variables for this test.
104101
features = set(os.environ.get("PIP_USE_FEATURE", "").split())
105102
if new_resolver:
106-
retval = "2020-resolver"
107103
features.add("2020-resolver")
108104
else:
109-
retval = "legacy"
110105
features.discard("2020-resolver")
111106

112107
with patch.dict(os.environ, {"PIP_USE_FEATURE": " ".join(features)}):
113-
yield retval
108+
yield resolver
114109

115110

116111
@pytest.fixture(scope='session')

0 commit comments

Comments
 (0)