Skip to content

test(storage): align semantic processor fakes with pathlock lease and localized overview heading #3841

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
huangruiteng wants to merge 1 commit into from
Closed

test(storage): align semantic processor fakes with pathlock lease and localized overview heading #3841

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
3 changes: 2 additions & 1 deletion tests/storage/test_semantic_processor_language.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -209,7 +209,8 @@ def test_overview_generation_language_flow(self, lang, file_summaries):
)
assert f"Output Language: {lang}" in prompt
assert "Output in Markdown format" in prompt
assert "Brief Description" in prompt
expected_heading = "简要描述" if lang == "zh-CN" else "Brief Description"
assert expected_heading in prompt
assert "abstract_max_chars" not in prompt

def test_overview_generation_prompt_preserves_repository_hierarchy(self):
Expand Down
1 change: 1 addition & 0 deletions tests/storage/test_semantic_processor_mv_vector_store.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -13,6 +13,7 @@ async def test_mv_canonicalizes_user_shorthand_before_vector_update(monkeypatch)
fs = VikingFS.__new__(VikingFS)
fs._async_agfs = AsyncMock()
fs._async_agfs.stat.return_value = {"isDir": False}
fs._async_agfs.pathlock_acquire_batch.return_value = {"lease_ref": "lease-1"}
fs._collect_uris = AsyncMock(return_value=[])
fs._copy_for_mv = AsyncMock()
fs._update_vector_store_uris = AsyncMock()
Expand Down
4 changes: 2 additions & 2 deletions tests/storage/test_semantic_queue_memory_dedupe.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -125,8 +125,8 @@ def _uri_to_path(self, uri, ctx=None):
del ctx
return f"/fake/{uri.replace('://', '/').strip('/')}"

async def write_file(self, uri, content, ctx=None):
del ctx
async def write_file(self, uri, content, ctx=None, lease_ref=None):
del ctx, lease_ref
self.writes.append((uri, content))


Expand Down