Skip to content

Commit 52e1c9a

Browse files
committed
Not clear why this didn't fail before...
1 parent 0c2a2bc commit 52e1c9a

File tree

1 file changed

+0
-35
lines changed

1 file changed

+0
-35
lines changed

tests/functional/test_install_cleanup.py

Lines changed: 0 additions & 35 deletions
Original file line numberDiff line numberDiff line change
@@ -1,10 +1,7 @@
1-
import os
21
from os.path import exists
32

43
import pytest
54

6-
from pip._internal.cli.status_codes import PREVIOUS_BUILD_DIR_ERROR
7-
85

96
@pytest.mark.network
107
@pytest.mark.xfail(
@@ -26,38 +23,6 @@ def test_no_clean_option_blocks_cleaning_after_install(script, data):
2623
assert exists(build)
2724

2825

29-
@pytest.mark.network
30-
def test_cleanup_prevented_upon_build_dir_exception(
31-
script,
32-
data,
33-
use_new_resolver,
34-
):
35-
"""
36-
Test no cleanup occurs after a PreviousBuildDirError
37-
"""
38-
build = script.venv_path / 'build'
39-
build_simple = build / 'simple'
40-
os.makedirs(build_simple)
41-
build_simple.joinpath("setup.py").write_text("#")
42-
result = script.pip(
43-
'install', '-f', data.find_links, '--no-index', 'simple',
44-
'--build', build,
45-
expect_error=(not use_new_resolver),
46-
expect_temp=(not use_new_resolver),
47-
expect_stderr=True,
48-
)
49-
50-
assert (
51-
"The -b/--build/--build-dir/--build-directory "
52-
"option is deprecated."
53-
) in result.stderr
54-
55-
if not use_new_resolver:
56-
assert result.returncode == PREVIOUS_BUILD_DIR_ERROR, str(result)
57-
assert "pip can't proceed" in result.stderr, str(result)
58-
assert exists(build_simple), str(result)
59-
60-
6126
@pytest.mark.network
6227
def test_pep517_no_legacy_cleanup(script, data, with_wheel):
6328
"""Test a PEP 517 failed build does not attempt a legacy cleanup"""

0 commit comments

Comments
 (0)