Repository navigation
Expand file tree
/
Copy pathtest_cli_clean.py
More file actions
254 lines (212 loc) · 8.11 KB
/
Copy pathtest_cli_clean.py
File metadata and controls
254 lines (212 loc) · 8.11 KB
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
186
187
188
189
190
191
192
193
194
195
196
197
198
199
200
201
202
203
204
205
206
207
208
209
210
211
212
213
214
215
216
217
218
219
220
221
222
223
224
225
226
227
228
229
230
231
232
233
234
235
236
237
238
239
240
241
242
243
244
245
246
247
248
249
250
251
252
253
254
from __future__ import annotations
import json
from collections.abc import Generator
from pathlib import Path
from unittest.mock import MagicMock, patch
import pytest
from typer.testing import CliRunner
from codebase_rag import constants as cs
from codebase_rag.cli import app
from codebase_rag.config import CgrignorePatterns
runner = CliRunner()
@pytest.fixture
def mock_memgraph_connect() -> Generator[MagicMock, None, None]:
with patch("codebase_rag.cli.connect_memgraph") as mock_connect:
mock_ingestor = MagicMock()
mock_connect.return_value.__enter__ = MagicMock(return_value=mock_ingestor)
mock_connect.return_value.__exit__ = MagicMock(return_value=False)
yield mock_connect
def _get_ingestor(mock_connect: MagicMock) -> MagicMock:
return mock_connect.return_value.__enter__.return_value
class TestCleanWithoutUpdateGraph:
def test_clean_alone_wipes_database(
self,
mock_memgraph_connect: MagicMock,
tmp_path: Path,
) -> None:
result = runner.invoke(
app,
["start", "--clean", "--repo-path", str(tmp_path)],
)
assert result.exit_code == 0, result.output
ingestor = _get_ingestor(mock_memgraph_connect)
ingestor.clean_database.assert_called_once()
def test_clean_alone_purges_vector_store(
self,
mock_memgraph_connect: MagicMock,
tmp_path: Path,
) -> None:
# Stale vectors keyed by recycled Memgraph node ids crowd out live
# hits and can map onto unrelated nodes after a clean rebuild.
with patch("codebase_rag.cli.clear_all_embeddings") as clear:
result = runner.invoke(
app,
["start", "--clean", "--repo-path", str(tmp_path)],
)
assert result.exit_code == 0, result.output
clear.assert_called_once()
def test_clean_alone_surfaces_vector_purge_failure(
self,
mock_memgraph_connect: MagicMock,
tmp_path: Path,
) -> None:
# A silently failed purge would report a clean database while stale
# vectors keep resolving to unrelated nodes after the rebuild.
with patch(
"codebase_rag.cli.clear_all_embeddings",
side_effect=RuntimeError("purge failed"),
):
result = runner.invoke(
app,
["start", "--clean", "--repo-path", str(tmp_path)],
)
assert result.exit_code != 0
def test_clean_alone_deletes_hash_cache(
self,
mock_memgraph_connect: MagicMock,
tmp_path: Path,
) -> None:
cache_path = tmp_path / cs.HASH_CACHE_FILENAME
cache_path.write_text(json.dumps({"file.py": "abc123"}))
result = runner.invoke(
app,
["start", "--clean", "--repo-path", str(tmp_path)],
)
assert result.exit_code == 0, result.output
assert not cache_path.exists()
def test_clean_alone_no_cache_file_still_succeeds(
self,
mock_memgraph_connect: MagicMock,
tmp_path: Path,
) -> None:
cache_path = tmp_path / cs.HASH_CACHE_FILENAME
assert not cache_path.exists()
result = runner.invoke(
app,
["start", "--clean", "--repo-path", str(tmp_path)],
)
assert result.exit_code == 0, result.output
def test_clean_alone_does_not_invoke_graph_updater(
self,
mock_memgraph_connect: MagicMock,
tmp_path: Path,
) -> None:
with patch("codebase_rag.cli.GraphUpdater") as mock_updater:
result = runner.invoke(
app,
["start", "--clean", "--repo-path", str(tmp_path)],
)
assert result.exit_code == 0, result.output
mock_updater.assert_not_called()
def test_clean_alone_skips_model_validation(
self,
mock_memgraph_connect: MagicMock,
tmp_path: Path,
) -> None:
with patch("codebase_rag.cli._update_and_validate_models") as mock_validate:
result = runner.invoke(
app,
["start", "--clean", "--repo-path", str(tmp_path)],
)
assert result.exit_code == 0, result.output
mock_validate.assert_not_called()
def test_clean_alone_shows_clean_done_message(
self,
mock_memgraph_connect: MagicMock,
tmp_path: Path,
) -> None:
result = runner.invoke(
app,
["start", "--clean", "--repo-path", str(tmp_path)],
)
assert result.exit_code == 0
assert cs.CLI_MSG_CLEAN_DONE in result.output
class TestCleanWithUpdateGraph:
@patch("codebase_rag.cli.GraphUpdater")
@patch("codebase_rag.cli.load_parsers", return_value=({}, {}))
@patch("codebase_rag.cli.load_ignore_patterns")
def test_clean_with_update_deletes_hash_cache(
self,
mock_cgrignore: MagicMock,
mock_load_parsers: MagicMock,
mock_graph_updater: MagicMock,
mock_memgraph_connect: MagicMock,
tmp_path: Path,
) -> None:
mock_cgrignore.return_value = CgrignorePatterns(
exclude=frozenset(), unignore=frozenset()
)
cache_path = tmp_path / cs.HASH_CACHE_FILENAME
cache_path.write_text(json.dumps({"file.py": "abc123"}))
result = runner.invoke(
app,
["start", "--clean", "--update-graph", "--repo-path", str(tmp_path)],
)
assert result.exit_code == 0, result.output
assert not cache_path.exists()
@patch("codebase_rag.cli.GraphUpdater")
@patch("codebase_rag.cli.load_parsers", return_value=({}, {}))
@patch("codebase_rag.cli.load_ignore_patterns")
def test_clean_with_update_calls_clean_database(
self,
mock_cgrignore: MagicMock,
mock_load_parsers: MagicMock,
mock_graph_updater: MagicMock,
mock_memgraph_connect: MagicMock,
tmp_path: Path,
) -> None:
mock_cgrignore.return_value = CgrignorePatterns(
exclude=frozenset(), unignore=frozenset()
)
result = runner.invoke(
app,
["start", "--clean", "--update-graph", "--repo-path", str(tmp_path)],
)
assert result.exit_code == 0, result.output
ingestor = _get_ingestor(mock_memgraph_connect)
ingestor.clean_database.assert_called_once()
@patch("codebase_rag.cli.GraphUpdater")
@patch("codebase_rag.cli.load_parsers", return_value=({}, {}))
@patch("codebase_rag.cli.load_ignore_patterns")
def test_clean_with_update_purges_vector_store(
self,
mock_cgrignore: MagicMock,
mock_load_parsers: MagicMock,
mock_graph_updater: MagicMock,
mock_memgraph_connect: MagicMock,
tmp_path: Path,
) -> None:
mock_cgrignore.return_value = CgrignorePatterns(
exclude=frozenset(), unignore=frozenset()
)
with patch("codebase_rag.cli.clear_all_embeddings") as clear:
result = runner.invoke(
app,
["start", "--clean", "--update-graph", "--repo-path", str(tmp_path)],
)
assert result.exit_code == 0, result.output
clear.assert_called_once()
@patch("codebase_rag.cli.GraphUpdater")
@patch("codebase_rag.cli.load_parsers", return_value=({}, {}))
@patch("codebase_rag.cli.load_ignore_patterns")
def test_update_without_clean_preserves_hash_cache(
self,
mock_cgrignore: MagicMock,
mock_load_parsers: MagicMock,
mock_graph_updater: MagicMock,
mock_memgraph_connect: MagicMock,
tmp_path: Path,
) -> None:
mock_cgrignore.return_value = CgrignorePatterns(
exclude=frozenset(), unignore=frozenset()
)
cache_path = tmp_path / cs.HASH_CACHE_FILENAME
cache_data = {"file.py": "abc123"}
cache_path.write_text(json.dumps(cache_data))
result = runner.invoke(
app,
["start", "--update-graph", "--repo-path", str(tmp_path)],
)
assert result.exit_code == 0, result.output
assert cache_path.exists()
assert json.loads(cache_path.read_text()) == cache_data