1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
149
150
151
152
153
154
155
156
157
158
159
160
161
162
163
164
165
166
167
168
169
170
171
172
173
174
175
176
177
178
179
180
181
182
183
184
185
|
import sys
import pytest # type: ignore[import-not-found]
from helpers import (
PIPX_METADATA_LEGACY_VERSIONS,
app_name,
mock_legacy_venv,
remove_venv_interpreter,
run_pipx_cli,
skip_if_windows,
)
from package_info import PKG
from pipx import paths
def file_or_symlink(filepath):
# Returns True for file or broken symlink or non-broken symlink
# Returns False for no file and no symlink
# filepath.exists() returns True for regular file or non-broken symlink
# filepath.exists() returns False for no regular file or broken symlink
# filepath.is_symlink() returns True for broken or non-broken symlink
return filepath.exists() or filepath.is_symlink()
def test_uninstall(pipx_temp_env):
assert not run_pipx_cli(["install", "pycowsay"])
assert not run_pipx_cli(["uninstall", "pycowsay"])
@skip_if_windows
def test_uninstall_global(pipx_temp_env):
assert not run_pipx_cli(["install", "--global", "pycowsay"])
assert not run_pipx_cli(["uninstall", "--global", "pycowsay"])
# TODO: We can add this test back once a suitable substitute for cloudtoken is found
# def test_uninstall_circular_deps(pipx_temp_env):
# assert not run_pipx_cli(["install", PKG["cloudtoken"]["spec"]])
# assert not run_pipx_cli(["uninstall", "cloudtoken"])
@pytest.mark.parametrize("metadata_version", PIPX_METADATA_LEGACY_VERSIONS)
def test_uninstall_legacy_venv(pipx_temp_env, metadata_version):
executable_path = paths.ctx.bin_dir / app_name("pycowsay")
assert not run_pipx_cli(["install", "pycowsay"])
assert executable_path.exists()
mock_legacy_venv("pycowsay", metadata_version=metadata_version)
assert not run_pipx_cli(["uninstall", "pycowsay"])
assert not file_or_symlink(executable_path)
def test_uninstall_suffix(pipx_temp_env):
name = "pbr"
suffix = "_a"
executable_path = paths.ctx.bin_dir / app_name(f"{name}{suffix}")
assert not run_pipx_cli(["install", PKG[name]["spec"], f"--suffix={suffix}"])
assert executable_path.exists()
assert not run_pipx_cli(["uninstall", f"{name}{suffix}"])
assert not file_or_symlink(executable_path)
def test_uninstall_man_page(pipx_temp_env):
man_page_path = paths.ctx.man_dir / "man6" / "pycowsay.6"
assert not run_pipx_cli(["install", "pycowsay"])
assert man_page_path.exists()
assert not run_pipx_cli(["uninstall", "pycowsay"])
assert not file_or_symlink(man_page_path)
def test_uninstall_injected(pipx_temp_env):
pycowsay_app_paths = [paths.ctx.bin_dir / app for app in PKG["pycowsay"]["apps"]]
pycowsay_man_page_paths = [paths.ctx.man_dir / man_page for man_page in PKG["pycowsay"]["man_pages"]]
pylint_app_paths = [paths.ctx.bin_dir / app for app in PKG["pylint"]["apps"]]
app_paths = pycowsay_app_paths + pylint_app_paths
man_page_paths = pycowsay_man_page_paths
assert not run_pipx_cli(["install", PKG["pycowsay"]["spec"]])
assert not run_pipx_cli(["inject", "--include-apps", "pycowsay", PKG["pylint"]["spec"]])
for app_path in app_paths:
assert app_path.exists()
for man_page_path in man_page_paths:
assert man_page_path.exists()
assert not run_pipx_cli(["uninstall", "pycowsay"])
for app_path in app_paths:
assert not file_or_symlink(app_path)
for man_page_path in man_page_paths:
assert not file_or_symlink(man_page_path)
@pytest.mark.parametrize("metadata_version", ["0.1"])
def test_uninstall_suffix_legacy_venv(pipx_temp_env, metadata_version):
name = "pbr"
# legacy uninstall on Windows only works with "canonical name characters"
# in suffix
suffix = "-a"
executable_path = paths.ctx.bin_dir / app_name(f"{name}{suffix}")
assert not run_pipx_cli(["install", PKG[name]["spec"], f"--suffix={suffix}"])
mock_legacy_venv(f"{name}{suffix}", metadata_version=metadata_version)
assert executable_path.exists()
assert not run_pipx_cli(["uninstall", f"{name}{suffix}"])
assert not file_or_symlink(executable_path)
@pytest.mark.parametrize("metadata_version", PIPX_METADATA_LEGACY_VERSIONS)
def test_uninstall_with_missing_interpreter(pipx_temp_env, metadata_version):
executable_path = paths.ctx.bin_dir / app_name("pycowsay")
assert not run_pipx_cli(["install", "pycowsay"])
assert executable_path.exists()
mock_legacy_venv("pycowsay", metadata_version=metadata_version)
remove_venv_interpreter("pycowsay")
assert not run_pipx_cli(["uninstall", "pycowsay"])
# On Windows we cannot remove app binaries if no metadata and no python
if not (sys.platform.startswith("win") and metadata_version is None):
assert not file_or_symlink(executable_path)
@pytest.mark.parametrize("metadata_version", PIPX_METADATA_LEGACY_VERSIONS)
def test_uninstall_proper_dep_behavior(pipx_temp_env, metadata_version):
# isort is a dependency of pylint. Make sure that uninstalling pylint
# does not also uninstall isort app in LOCAL_BIN_DIR
isort_app_paths = [paths.ctx.bin_dir / app for app in PKG["isort"]["apps"]]
pylint_app_paths = [paths.ctx.bin_dir / app for app in PKG["pylint"]["apps"]]
assert not run_pipx_cli(["install", PKG["pylint"]["spec"]])
assert not run_pipx_cli(["install", PKG["isort"]["spec"]])
mock_legacy_venv("pylint", metadata_version=metadata_version)
mock_legacy_venv("isort", metadata_version=metadata_version)
for pylint_app_path in pylint_app_paths:
assert pylint_app_path.exists()
for isort_app_path in isort_app_paths:
assert isort_app_path.exists()
assert not run_pipx_cli(["uninstall", "pylint"])
for pylint_app_path in pylint_app_paths:
assert not file_or_symlink(pylint_app_path)
# THIS is what we're making sure is true:
for isort_app_path in isort_app_paths:
assert isort_app_path.exists()
@pytest.mark.parametrize("metadata_version", PIPX_METADATA_LEGACY_VERSIONS)
def test_uninstall_proper_dep_behavior_missing_interpreter(pipx_temp_env, metadata_version):
# isort is a dependency of pylint. Make sure that uninstalling pylint
# does not also uninstall isort app in LOCAL_BIN_DIR
isort_app_paths = [paths.ctx.bin_dir / app for app in PKG["isort"]["apps"]]
pylint_app_paths = [paths.ctx.bin_dir / app for app in PKG["pylint"]["apps"]]
assert not run_pipx_cli(["install", PKG["pylint"]["spec"]])
assert not run_pipx_cli(["install", PKG["isort"]["spec"]])
mock_legacy_venv("pylint", metadata_version=metadata_version)
mock_legacy_venv("isort", metadata_version=metadata_version)
remove_venv_interpreter("pylint")
remove_venv_interpreter("isort")
for pylint_app_path in pylint_app_paths:
assert pylint_app_path.exists()
for isort_app_path in isort_app_paths:
assert isort_app_path.exists()
assert not run_pipx_cli(["uninstall", "pylint"])
# Do not check the following on Windows without metadata, we do not
# remove bin dir links by design for missing interpreter in that case
if not (sys.platform.startswith("win") and metadata_version is None):
for pylint_app_path in pylint_app_paths:
assert not file_or_symlink(pylint_app_path)
# THIS is what we're making sure is true:
for isort_app_path in isort_app_paths:
assert isort_app_path.exists()
|
