mirror of
https://github.com/Sea-Haven-Industries/open-swe.git
synced 2026-10-02 13:13:16 +00:00
Adam Moussa
ae1f883b4c
Applies the plan's C5 step: git mv every test per the domain-reorg
move-map (movemap-m50.txt) into tests/{agent,analyzer,auth,dashboard,
github,middleware,models,reviewer,sandbox,slack,tools,webhooks}/, plus
the 13 fork-only placements from the scoping report §2c (Atlassian
webhook tests -> tests/webhooks/, test_atlassian_connect.py and
test_auth_error_leak.py -> tests/auth/, jira/confluence util tests ->
tests/tools/, test_repo_binding_isolation.py -> tests/sandbox/,
bot-identity/autofix tests -> tests/github/).
Path-only move: the only content edits are parents[1] -> parents[2]
fixes in test_e2b_integration.py and test_daytona_integration.py,
required because their __file__-relative ROOT path gained one more
directory level in the move.
Monkeypatch retargets for these files were already completed in C4;
none remained outstanding here.
27 lines
840 B
Python
27 lines
840 B
Python
from __future__ import annotations
|
|
|
|
from agent.review.style_collector import (
|
|
ReviewSample,
|
|
ReviewStyleSamples,
|
|
format_samples_for_analyzer,
|
|
)
|
|
|
|
|
|
def test_format_samples_for_analyzer_groups_by_reviewer() -> None:
|
|
samples = ReviewStyleSamples(
|
|
full_name="acme/widget",
|
|
owner="acme",
|
|
name="widget",
|
|
top_reviewers=["alice", "bob"],
|
|
samples=[
|
|
ReviewSample(1, "alice", "review", "Missing nil check on line 42.", state="COMMENTED"),
|
|
ReviewSample(2, "bob", "inline", "Race in cache update path.", path="cache.go"),
|
|
],
|
|
prs_scanned=5,
|
|
reviews_scanned=10,
|
|
)
|
|
text = format_samples_for_analyzer(samples)
|
|
assert "acme/widget" in text
|
|
assert "## Reviewer: @alice" in text
|
|
assert "PR #1" in text
|
|
assert "Race in cache" in text
|