Skip to content

Commit 4cbb311

Browse files
committed
Remove --use-pep517 option test
This option is now always on, so need to test it.
1 parent 768be06 commit 4cbb311

File tree

1 file changed

+0
-58
lines changed

1 file changed

+0
-58
lines changed

tests/unit/test_options.py

Lines changed: 0 additions & 58 deletions
Original file line numberDiff line numberDiff line change
@@ -198,64 +198,6 @@ def test_cache_dir__PIP_NO_CACHE_DIR_invalid__with_no_cache_dir(
198198
main(["--no-cache-dir", "fake"])
199199

200200

201-
class TestUsePEP517Options:
202-
"""
203-
Test options related to using --use-pep517.
204-
"""
205-
206-
def parse_args(self, args: list[str]) -> Values:
207-
# We use DownloadCommand since that is one of the few Command
208-
# classes with the use_pep517 options.
209-
command = create_command("download")
210-
options, args = command.parse_args(args)
211-
212-
return options
213-
214-
def test_no_option(self) -> None:
215-
"""
216-
Test passing no option.
217-
"""
218-
options = self.parse_args([])
219-
assert options.use_pep517 is None
220-
221-
def test_use_pep517(self) -> None:
222-
"""
223-
Test passing --use-pep517.
224-
"""
225-
options = self.parse_args(["--use-pep517"])
226-
assert options.use_pep517 is True
227-
228-
def test_PIP_USE_PEP517_true(self, monkeypatch: pytest.MonkeyPatch) -> None:
229-
"""
230-
Test setting PIP_USE_PEP517 to "true".
231-
"""
232-
monkeypatch.setenv("PIP_USE_PEP517", "true")
233-
options = self.parse_args([])
234-
# This is an int rather than a boolean because strtobool() in pip's
235-
# configuration code returns an int.
236-
assert options.use_pep517 == 1
237-
238-
def test_PIP_USE_PEP517_false(self, monkeypatch: pytest.MonkeyPatch) -> None:
239-
"""
240-
Test setting PIP_USE_PEP517 to "false".
241-
"""
242-
monkeypatch.setenv("PIP_USE_PEP517", "false")
243-
options = self.parse_args([])
244-
# This is an int rather than a boolean because strtobool() in pip's
245-
# configuration code returns an int.
246-
assert options.use_pep517 == 0
247-
248-
def test_use_pep517_and_PIP_USE_PEP517_false(
249-
self, monkeypatch: pytest.MonkeyPatch
250-
) -> None:
251-
"""
252-
Test passing --use-pep517 and setting PIP_USE_PEP517 to "false".
253-
"""
254-
monkeypatch.setenv("PIP_USE_PEP517", "false")
255-
options = self.parse_args(["--use-pep517"])
256-
assert options.use_pep517 is True
257-
258-
259201
class TestOptionsInterspersed(AddFakeCommandMixin):
260202
def test_general_option_after_subcommand(self) -> None:
261203
# FakeCommand intentionally returns the wrong type.

0 commit comments

Comments
 (0)