xai-review 0.29.0__py3-none-any.whl → 0.31.0__py3-none-any.whl

ai_review/tests/fixtures/clients/gitea.py

@@ -0,0 +1,141 @@
+import pytest
+from pydantic import HttpUrl, SecretStr
+
+from ai_review.clients.gitea.pr.schema.comments import (
+    GiteaPRCommentSchema,
+    GiteaGetPRCommentsResponseSchema,
+    GiteaCreateCommentRequestSchema,
+    GiteaCreateCommentResponseSchema,
+)
+from ai_review.clients.gitea.pr.schema.files import (
+    GiteaGetPRFilesResponseSchema,
+    GiteaPRFileSchema,
+)
+from ai_review.clients.gitea.pr.schema.pull_request import (
+    GiteaGetPRResponseSchema,
+    GiteaBranchSchema,
+)
+from ai_review.clients.gitea.pr.schema.user import GiteaUserSchema
+from ai_review.clients.gitea.pr.types import GiteaPullRequestsHTTPClientProtocol
+from ai_review.config import settings
+from ai_review.libs.config.vcs.base import GiteaVCSConfig
+from ai_review.libs.config.vcs.gitea import GiteaPipelineConfig, GiteaHTTPClientConfig
+from ai_review.libs.constants.vcs_provider import VCSProvider
+from ai_review.services.vcs.gitea.client import GiteaVCSClient
+
+
+class FakeGiteaPullRequestsHTTPClient(GiteaPullRequestsHTTPClientProtocol):
+    def __init__(self):
+        self.calls: list[tuple[str, dict]] = []
+
+    async def get_pull_request(self, owner: str, repo: str, pull_number: str) -> GiteaGetPRResponseSchema:
+        self.calls.append(("get_pull_request", {"owner": owner, "repo": repo, "pull_number": pull_number}))
+        return GiteaGetPRResponseSchema(
+            id=1,
+            number=1,
+            title="Fake Gitea PR",
+            body="This is a fake PR for testing",
+            user=GiteaUserSchema(id=101, login="tester"),
+            base=GiteaBranchSchema(ref="main", sha="abc123"),
+            head=GiteaBranchSchema(ref="feature", sha="def456"),
+        )
+
+    async def get_files(self, owner: str, repo: str, pull_number: str) -> GiteaGetPRFilesResponseSchema:
+        self.calls.append(("get_files", {"owner": owner, "repo": repo, "pull_number": pull_number}))
+        return GiteaGetPRFilesResponseSchema(
+            root=[
+                GiteaPRFileSchema(
+                    sha="abc",
+                    status="modified",
+                    filename="src/main.py",
+                    patch="@@ -1,2 +1,2 @@\n- old\n+ new",
+                ),
+                GiteaPRFileSchema(
+                    sha="def",
+                    status="added",
+                    filename="utils/helper.py",
+                    patch="+ print('Hello')",
+                ),
+            ]
+        )
+
+    async def get_comments(self, owner: str, repo: str, pull_number: str) -> GiteaGetPRCommentsResponseSchema:
+        self.calls.append(("get_comments", {"owner": owner, "repo": repo, "pull_number": pull_number}))
+        return GiteaGetPRCommentsResponseSchema(
+            root=[
+                GiteaPRCommentSchema(
+                    id=1,
+                    body="General comment",
+                    user=GiteaUserSchema(id=201, login="alice"),
+                ),
+                GiteaPRCommentSchema(
+                    id=2,
+                    body="Inline comment",
+                    path="file.py",
+                    line=5,
+                    user=GiteaUserSchema(id=202, login="bob"),
+                ),
+            ]
+        )
+
+    async def create_comment(
+            self,
+            owner: str,
+            repo: str,
+            pull_number: str,
+            request: GiteaCreateCommentRequestSchema
+    ) -> GiteaCreateCommentResponseSchema:
+        self.calls.append(
+            (
+                "create_comment",
+                {"owner": owner, "repo": repo, "pull_number": pull_number, **request.model_dump()},
+            )
+        )
+        return GiteaCreateCommentResponseSchema(id=10, body=request.body)
+
+
+class FakeGiteaHTTPClient:
+    def __init__(self, pull_requests_client: FakeGiteaPullRequestsHTTPClient):
+        self.pr = pull_requests_client
+
+
+@pytest.fixture
+def fake_gitea_pull_requests_http_client() -> FakeGiteaPullRequestsHTTPClient:
+    return FakeGiteaPullRequestsHTTPClient()
+
+
+@pytest.fixture
+def fake_gitea_http_client(
+        fake_gitea_pull_requests_http_client: FakeGiteaPullRequestsHTTPClient
+) -> FakeGiteaHTTPClient:
+    return FakeGiteaHTTPClient(pull_requests_client=fake_gitea_pull_requests_http_client)
+
+
+@pytest.fixture
+def gitea_vcs_client(
+        monkeypatch: pytest.MonkeyPatch,
+        fake_gitea_http_client: FakeGiteaHTTPClient
+) -> GiteaVCSClient:
+    monkeypatch.setattr(
+        "ai_review.services.vcs.gitea.client.get_gitea_http_client",
+        lambda: fake_gitea_http_client,
+    )
+    return GiteaVCSClient()
+
+
+@pytest.fixture
+def gitea_http_client_config(monkeypatch: pytest.MonkeyPatch):
+    fake_config = GiteaVCSConfig(
+        provider=VCSProvider.GITEA,
+        pipeline=GiteaPipelineConfig(
+            repo="repo",
+            owner="owner",
+            pull_number="1",
+        ),
+        http_client=GiteaHTTPClientConfig(
+            timeout=10,
+            api_url=HttpUrl("https://gitea.example.com"),
+            api_token=SecretStr("fake-token"),
+        ),
+    )
+    monkeypatch.setattr(settings, "vcs", fake_config)

ai_review/tests/fixtures/services/review/gateway/{comment.py → review_comment_gateway.py}

@@ -2,7 +2,7 @@ from typing import Any
 
 import pytest
 
-from ai_review.services.review.gateway.comment import ReviewCommentGateway
+from ai_review.services.review.gateway.review_comment_gateway import ReviewCommentGateway
 from ai_review.services.review.gateway.types import ReviewCommentGatewayProtocol
 from ai_review.services.review.internal.inline.schema import InlineCommentSchema, InlineCommentListSchema
 from ai_review.services.review.internal.inline_reply.schema import InlineCommentReplySchema

ai_review/tests/fixtures/services/review/gateway/review_dry_run_comment_gateway.py

@@ -0,0 +1,103 @@
+from typing import Any
+
+import pytest
+
+from ai_review.services.review.gateway.review_dry_run_comment_gateway import ReviewDryRunCommentGateway
+from ai_review.services.review.gateway.types import ReviewCommentGatewayProtocol
+from ai_review.services.review.internal.inline.schema import InlineCommentSchema, InlineCommentListSchema
+from ai_review.services.review.internal.inline_reply.schema import InlineCommentReplySchema
+from ai_review.services.review.internal.summary.schema import SummaryCommentSchema
+from ai_review.services.review.internal.summary_reply.schema import SummaryCommentReplySchema
+from ai_review.services.vcs.types import (
+    UserSchema,
+    ThreadKind,
+    ReviewThreadSchema,
+    ReviewCommentSchema,
+    VCSClientProtocol,
+)
+
+
+class FakeReviewDryRunCommentGateway(ReviewCommentGatewayProtocol):
+    def __init__(self, responses: dict[str, Any] | None = None):
+        self.calls: list[tuple[str, dict]] = []
+
+        fake_user = UserSchema(id="u1", username="tester", name="Tester")
+
+        fake_inline_thread = ReviewThreadSchema(
+            id="t1",
+            kind=ThreadKind.INLINE,
+            file="file.py",
+            line=10,
+            comments=[
+                ReviewCommentSchema(
+                    id="c1",
+                    body="#ai-review-inline some comment",
+                    file="file.py",
+                    line=10,
+                    author=fake_user,
+                ),
+            ],
+        )
+
+        fake_summary_thread = ReviewThreadSchema(
+            id="t2",
+            kind=ThreadKind.SUMMARY,
+            comments=[
+                ReviewCommentSchema(
+                    id="c2",
+                    body="#ai-review-summary summary comment",
+                    author=fake_user,
+                ),
+            ],
+        )
+
+        self.responses = responses or {
+            "get_inline_threads": [fake_inline_thread],
+            "get_summary_threads": [fake_summary_thread],
+            "has_existing_inline_comments": False,
+            "has_existing_summary_comments": False,
+        }
+
+    # --- Методы чтения ---
+    async def get_inline_threads(self) -> list[ReviewThreadSchema]:
+        self.calls.append(("get_inline_threads", {}))
+        return self.responses["get_inline_threads"]
+
+    async def get_summary_threads(self) -> list[ReviewThreadSchema]:
+        self.calls.append(("get_summary_threads", {}))
+        return self.responses["get_summary_threads"]
+
+    async def has_existing_inline_comments(self) -> bool:
+        self.calls.append(("has_existing_inline_comments", {}))
+        return self.responses["has_existing_inline_comments"]
+
+    async def has_existing_summary_comments(self) -> bool:
+        self.calls.append(("has_existing_summary_comments", {}))
+        return self.responses["has_existing_summary_comments"]
+
+    async def process_inline_reply(self, thread_id: str, reply: InlineCommentReplySchema) -> None:
+        self.calls.append(("process_inline_reply", {"thread_id": thread_id, "reply": reply}))
+
+    async def process_summary_reply(self, thread_id: str, reply: SummaryCommentReplySchema) -> None:
+        self.calls.append(("process_summary_reply", {"thread_id": thread_id, "reply": reply}))
+
+    async def process_inline_comment(self, comment: InlineCommentSchema) -> None:
+        self.calls.append(("process_inline_comment", {"comment": comment}))
+
+    async def process_summary_comment(self, comment: SummaryCommentSchema) -> None:
+        self.calls.append(("process_summary_comment", {"comment": comment}))
+
+    async def process_inline_comments(self, comments: InlineCommentListSchema) -> None:
+        self.calls.append(("process_inline_comments", {"comments": comments}))
+        for comment in comments.root:
+            await self.process_inline_comment(comment)
+
+
+@pytest.fixture
+def fake_review_dry_run_comment_gateway() -> FakeReviewDryRunCommentGateway:
+    return FakeReviewDryRunCommentGateway()
+
+
+@pytest.fixture
+def review_dry_run_comment_gateway(fake_vcs_client: VCSClientProtocol) -> ReviewDryRunCommentGateway:
+    return ReviewDryRunCommentGateway(vcs=fake_vcs_client)

ai_review/tests/fixtures/services/review/gateway/review_llm_gateway.py

@@ -0,0 +1,34 @@
+import pytest
+
+from ai_review.services.review.gateway.review_llm_gateway import ReviewLLMGateway
+from ai_review.services.review.gateway.types import ReviewLLMGatewayProtocol
+from ai_review.tests.fixtures.services.artifacts import FakeArtifactsService
+from ai_review.tests.fixtures.services.cost import FakeCostService
+from ai_review.tests.fixtures.services.llm import FakeLLMClient
+
+
+class FakeReviewLLMGateway(ReviewLLMGatewayProtocol):
+    def __init__(self):
+        self.calls: list[tuple[str, dict]] = []
+
+    async def ask(self, prompt: str, prompt_system: str) -> str:
+        self.calls.append(("ask", {"prompt": prompt, "prompt_system": prompt_system}))
+        return "FAKE_LLM_RESPONSE"
+
+
+@pytest.fixture
+def fake_review_llm_gateway() -> FakeReviewLLMGateway:
+    return FakeReviewLLMGateway()
+
+
+@pytest.fixture
+def review_llm_gateway(
+        fake_llm_client: FakeLLMClient,
+        fake_cost_service: FakeCostService,
+        fake_artifacts_service: FakeArtifactsService,
+) -> ReviewLLMGateway:
+    return ReviewLLMGateway(
+        llm=fake_llm_client,
+        cost=fake_cost_service,
+        artifacts=fake_artifacts_service,
+    )

ai_review/tests/suites/clients/gitea/test_client.py

@@ -0,0 +1,14 @@
+import pytest
+from httpx import AsyncClient
+
+from ai_review.clients.gitea.client import get_gitea_http_client, GiteaHTTPClient
+from ai_review.clients.gitea.pr.client import GiteaPullRequestsHTTPClient
+
+
+@pytest.mark.usefixtures("gitea_http_client_config")
+def test_get_gitea_http_client_builds_ok():
+    gitea_http_client = get_gitea_http_client()
+
+    assert isinstance(gitea_http_client, GiteaHTTPClient)
+    assert isinstance(gitea_http_client.pr, GiteaPullRequestsHTTPClient)
+    assert isinstance(gitea_http_client.pr.client, AsyncClient)

ai_review/tests/suites/clients/gitea/test_tools.py

@@ -0,0 +1,26 @@
+from httpx import Response, Request
+
+from ai_review.clients.gitea.tools import gitea_has_next_page
+
+
+def make_response(headers: dict) -> Response:
+    return Response(
+        request=Request("GET", "http://gitea.test"),
+        headers=headers,
+        status_code=200,
+    )
+
+
+def test_gitea_has_next_page_true():
+    resp = make_response({"Link": '<https://gitea.test?page=2>; rel="next"'})
+    assert gitea_has_next_page(resp) is True
+
+
+def test_gitea_has_next_page_false_empty():
+    resp = make_response({"Link": ""})
+    assert gitea_has_next_page(resp) is False
+
+
+def test_gitea_has_next_page_false_missing():
+    resp = make_response({})
+    assert gitea_has_next_page(resp) is False

ai_review/tests/suites/services/review/gateway/{test_comment.py → test_review_comment_gateway.py}

@@ -1,7 +1,7 @@
 import pytest
 
 from ai_review.config import settings
-from ai_review.services.review.gateway.comment import ReviewCommentGateway
+from ai_review.services.review.gateway.review_comment_gateway import ReviewCommentGateway
 from ai_review.services.review.internal.inline.schema import InlineCommentSchema, InlineCommentListSchema
 from ai_review.services.review.internal.inline_reply.schema import InlineCommentReplySchema
 from ai_review.services.review.internal.summary.schema import SummaryCommentSchema

ai_review/tests/suites/services/review/gateway/test_review_dry_run_comment_gateway.py

@@ -0,0 +1,93 @@
+import pytest
+
+from ai_review.services.review.gateway.review_dry_run_comment_gateway import ReviewDryRunCommentGateway
+from ai_review.services.review.internal.inline.schema import InlineCommentSchema, InlineCommentListSchema
+from ai_review.services.review.internal.inline_reply.schema import InlineCommentReplySchema
+from ai_review.services.review.internal.summary.schema import SummaryCommentSchema
+from ai_review.services.review.internal.summary_reply.schema import SummaryCommentReplySchema
+from ai_review.tests.fixtures.services.vcs import FakeVCSClient
+
+
+@pytest.mark.asyncio
+async def test_process_inline_reply_dry_run_logs_and_no_vcs_calls(
+        capsys,
+        fake_vcs_client: FakeVCSClient,
+        review_dry_run_comment_gateway: ReviewDryRunCommentGateway
+):
+    """Dry-run: should log the inline reply but not call VCS."""
+    reply = InlineCommentReplySchema(message="AI reply dry-run")
+    await review_dry_run_comment_gateway.process_inline_reply("t1", reply)
+    output = capsys.readouterr().out
+
+    assert "[dry-run]" in output
+    assert "Would create inline reply" in output
+    assert not any(call[0].startswith("create_") for call in fake_vcs_client.calls)
+
+
+@pytest.mark.asyncio
+async def test_process_summary_reply_dry_run_logs_and_no_vcs_calls(
+        capsys,
+        fake_vcs_client: FakeVCSClient,
+        review_dry_run_comment_gateway: ReviewDryRunCommentGateway
+):
+    """Dry-run: should log the summary reply but not call VCS."""
+    reply = SummaryCommentReplySchema(text="Dry-run summary reply")
+    await review_dry_run_comment_gateway.process_summary_reply("t2", reply)
+    output = capsys.readouterr().out
+
+    assert "[dry-run]" in output
+    assert "Would create summary reply" in output
+    assert not any(call[0].startswith("create_") for call in fake_vcs_client.calls)
+
+
+@pytest.mark.asyncio
+async def test_process_inline_comment_dry_run_logs_and_no_vcs_calls(
+        capsys,
+        fake_vcs_client: FakeVCSClient,
+        review_dry_run_comment_gateway: ReviewDryRunCommentGateway
+):
+    """Dry-run: should log inline comment without creating one."""
+    comment = InlineCommentSchema(file="a.py", line=10, message="Test comment")
+    await review_dry_run_comment_gateway.process_inline_comment(comment)
+    output = capsys.readouterr().out
+
+    assert "[dry-run]" in output
+    assert "Would create inline comment" in output
+    assert "a.py" in output
+    assert not any(call[0].startswith("create_") for call in fake_vcs_client.calls)
+
+
+@pytest.mark.asyncio
+async def test_process_summary_comment_dry_run_logs_and_no_vcs_calls(
+        capsys,
+        fake_vcs_client: FakeVCSClient,
+        review_dry_run_comment_gateway: ReviewDryRunCommentGateway
+):
+    """Dry-run: should log summary comment but not send it."""
+    comment = SummaryCommentSchema(text="Dry-run summary comment")
+    await review_dry_run_comment_gateway.process_summary_comment(comment)
+    output = capsys.readouterr().out
+
+    assert "[dry-run]" in output
+    assert "Would create summary comment" in output
+    assert not any(call[0].startswith("create_") for call in fake_vcs_client.calls)
+
+
+@pytest.mark.asyncio
+async def test_process_inline_comments_iterates_all(
+        capsys,
+        fake_vcs_client: FakeVCSClient,
+        review_dry_run_comment_gateway: ReviewDryRunCommentGateway
+):
+    """Dry-run: should iterate through all inline comments and log each."""
+    comments = InlineCommentListSchema(root=[
+        InlineCommentSchema(file="a.py", line=1, message="C1"),
+        InlineCommentSchema(file="b.py", line=2, message="C2"),
+    ])
+    await review_dry_run_comment_gateway.process_inline_comments(comments)
+    output = capsys.readouterr().out
+
+    assert "[dry-run]" in output
+    assert "a.py" in output
+    assert "b.py" in output
+    assert not any(call[0].startswith("create_") for call in fake_vcs_client.calls)

ai_review/tests/suites/services/review/gateway/{test_llm.py → test_review_llm_gateway.py}

@@ -1,26 +1,12 @@
 import pytest
 
 from ai_review.services.llm.types import ChatResultSchema
-from ai_review.services.review.gateway.llm import ReviewLLMGateway
+from ai_review.services.review.gateway.review_llm_gateway import ReviewLLMGateway
 from ai_review.tests.fixtures.services.artifacts import FakeArtifactsService
 from ai_review.tests.fixtures.services.cost import FakeCostService
 from ai_review.tests.fixtures.services.llm import FakeLLMClient
 
 
-@pytest.fixture
-def review_llm_gateway(
-        fake_llm_client: FakeLLMClient,
-        fake_cost_service: FakeCostService,
-        fake_artifacts_service: FakeArtifactsService,
-) -> ReviewLLMGateway:
-    """Fixture providing ReviewLLMGateway with fake dependencies."""
-    return ReviewLLMGateway(
-        llm=fake_llm_client,
-        cost=fake_cost_service,
-        artifacts=fake_artifacts_service,
-    )
-
-
 @pytest.mark.asyncio
 async def test_ask_happy_path(
         review_llm_gateway: ReviewLLMGateway,

ai_review/tests/suites/services/review/runner/test_context.py

@@ -4,8 +4,8 @@ from ai_review.services.review.runner.context import ContextReviewRunner
 from ai_review.tests.fixtures.services.cost import FakeCostService
 from ai_review.tests.fixtures.services.diff import FakeDiffService
 from ai_review.tests.fixtures.services.prompt import FakePromptService
-from ai_review.tests.fixtures.services.review.gateway.comment import FakeReviewCommentGateway
-from ai_review.tests.fixtures.services.review.gateway.llm import FakeReviewLLMGateway
+from ai_review.tests.fixtures.services.review.gateway.review_comment_gateway import FakeReviewCommentGateway
+from ai_review.tests.fixtures.services.review.gateway.review_llm_gateway import FakeReviewLLMGateway
 from ai_review.tests.fixtures.services.review.internal.inline import FakeInlineCommentService
 from ai_review.tests.fixtures.services.review.internal.policy import FakeReviewPolicyService
 from ai_review.tests.fixtures.services.vcs import FakeVCSClient

ai_review/tests/suites/services/review/runner/test_inline.py

@@ -6,8 +6,8 @@ from ai_review.tests.fixtures.services.cost import FakeCostService
 from ai_review.tests.fixtures.services.diff import FakeDiffService
 from ai_review.tests.fixtures.services.git import FakeGitService
 from ai_review.tests.fixtures.services.prompt import FakePromptService
-from ai_review.tests.fixtures.services.review.gateway.comment import FakeReviewCommentGateway
-from ai_review.tests.fixtures.services.review.gateway.llm import FakeReviewLLMGateway
+from ai_review.tests.fixtures.services.review.gateway.review_comment_gateway import FakeReviewCommentGateway
+from ai_review.tests.fixtures.services.review.gateway.review_llm_gateway import FakeReviewLLMGateway
 from ai_review.tests.fixtures.services.review.internal.inline import FakeInlineCommentService
 from ai_review.tests.fixtures.services.review.internal.policy import FakeReviewPolicyService
 from ai_review.tests.fixtures.services.vcs import FakeVCSClient

ai_review/tests/suites/services/review/runner/test_inline_reply.py

@@ -6,8 +6,8 @@ from ai_review.tests.fixtures.services.cost import FakeCostService
 from ai_review.tests.fixtures.services.diff import FakeDiffService
 from ai_review.tests.fixtures.services.git import FakeGitService
 from ai_review.tests.fixtures.services.prompt import FakePromptService
-from ai_review.tests.fixtures.services.review.gateway.comment import FakeReviewCommentGateway
-from ai_review.tests.fixtures.services.review.gateway.llm import FakeReviewLLMGateway
+from ai_review.tests.fixtures.services.review.gateway.review_comment_gateway import FakeReviewCommentGateway
+from ai_review.tests.fixtures.services.review.gateway.review_llm_gateway import FakeReviewLLMGateway
 from ai_review.tests.fixtures.services.review.internal.inline_reply import FakeInlineCommentReplyService
 from ai_review.tests.fixtures.services.vcs import FakeVCSClient
 

ai_review/tests/suites/services/review/runner/test_summary.py

@@ -5,8 +5,8 @@ from ai_review.services.review.runner.summary import SummaryReviewRunner
 from ai_review.tests.fixtures.services.cost import FakeCostService
 from ai_review.tests.fixtures.services.diff import FakeDiffService
 from ai_review.tests.fixtures.services.prompt import FakePromptService
-from ai_review.tests.fixtures.services.review.gateway.comment import FakeReviewCommentGateway
-from ai_review.tests.fixtures.services.review.gateway.llm import FakeReviewLLMGateway
+from ai_review.tests.fixtures.services.review.gateway.review_comment_gateway import FakeReviewCommentGateway
+from ai_review.tests.fixtures.services.review.gateway.review_llm_gateway import FakeReviewLLMGateway
 from ai_review.tests.fixtures.services.review.internal.policy import FakeReviewPolicyService
 from ai_review.tests.fixtures.services.review.internal.summary import FakeSummaryCommentService
 from ai_review.tests.fixtures.services.vcs import FakeVCSClient

ai_review/tests/suites/services/review/runner/test_summary_reply.py

@@ -5,8 +5,8 @@ from ai_review.services.vcs.types import ReviewInfoSchema, ReviewThreadSchema, R
 from ai_review.tests.fixtures.services.cost import FakeCostService
 from ai_review.tests.fixtures.services.diff import FakeDiffService
 from ai_review.tests.fixtures.services.prompt import FakePromptService
-from ai_review.tests.fixtures.services.review.gateway.comment import FakeReviewCommentGateway
-from ai_review.tests.fixtures.services.review.gateway.llm import FakeReviewLLMGateway
+from ai_review.tests.fixtures.services.review.gateway.review_comment_gateway import FakeReviewCommentGateway
+from ai_review.tests.fixtures.services.review.gateway.review_llm_gateway import FakeReviewLLMGateway
 from ai_review.tests.fixtures.services.review.internal.policy import FakeReviewPolicyService
 from ai_review.tests.fixtures.services.review.internal.summary_reply import FakeSummaryCommentReplyService
 from ai_review.tests.fixtures.services.vcs import FakeVCSClient

ai_review/tests/suites/services/review/test_service.py

@@ -1,6 +1,8 @@
 import pytest
 
 from ai_review.services.llm.types import ChatResultSchema
+from ai_review.services.review.gateway.review_comment_gateway import ReviewCommentGateway
+from ai_review.services.review.gateway.review_dry_run_comment_gateway import ReviewDryRunCommentGateway
 from ai_review.services.review.service import ReviewService
 from ai_review.tests.fixtures.services.cost import FakeCostService
 from ai_review.tests.fixtures.services.review.runner.context import FakeContextReviewRunner
@@ -91,3 +93,19 @@ def test_report_total_cost_no_data(capsys, review_service: ReviewService):
     output = capsys.readouterr().out
 
     assert "No cost data collected" in output
+
+
+def test_review_service_uses_dry_run_comment_gateway(monkeypatch: pytest.MonkeyPatch):
+    """Should use ReviewDryRunCommentGateway when settings.review.dry_run=True."""
+    monkeypatch.setattr("ai_review.config.settings.review.dry_run", True)
+
+    service = ReviewService()
+    assert type(service.review_comment_gateway) is ReviewDryRunCommentGateway
+
+
+def test_review_service_uses_real_comment_gateway(monkeypatch: pytest.MonkeyPatch):
+    """Should use normal ReviewCommentGateway when dry_run=False."""
+    monkeypatch.setattr("ai_review.config.settings.review.dry_run", False)
+
+    service = ReviewService()
+    assert type(service.review_comment_gateway) is ReviewCommentGateway

ai_review/tests/suites/services/vcs/gitea/test_adapter.py

@@ -0,0 +1,52 @@
+from ai_review.clients.gitea.pr.schema.comments import GiteaPRCommentSchema
+from ai_review.clients.gitea.pr.schema.user import GiteaUserSchema
+from ai_review.services.vcs.gitea.adapter import get_review_comment_from_gitea_comment, get_user_from_gitea_user
+from ai_review.services.vcs.types import ReviewCommentSchema, UserSchema
+
+
+def test_get_user_from_gitea_user_maps_fields_correctly():
+    user = GiteaUserSchema(id=42, login="tester")
+    result = get_user_from_gitea_user(user)
+
+    assert isinstance(result, UserSchema)
+    assert result.id == 42
+    assert result.username == "tester"
+    assert result.name == "tester"
+
+
+def test_get_user_from_gitea_user_handles_none():
+    result = get_user_from_gitea_user(None)
+    assert result.id is None
+    assert result.username == ""
+    assert result.name == ""
+
+
+def test_get_review_comment_from_gitea_comment_maps_all_fields():
+    comment = GiteaPRCommentSchema(
+        id=10,
+        body="Inline comment",
+        path="src/main.py",
+        line=15,
+        user=GiteaUserSchema(id=1, login="dev"),
+    )
+
+    result = get_review_comment_from_gitea_comment(comment)
+
+    assert isinstance(result, ReviewCommentSchema)
+    assert result.id == 10
+    assert result.body == "Inline comment"
+    assert result.file == "src/main.py"
+    assert result.line == 15
+    assert result.thread_id == 10
+    assert isinstance(result.author, UserSchema)
+    assert result.author.username == "dev"
+
+
+def test_get_review_comment_handles_missing_user_and_body():
+    comment = GiteaPRCommentSchema(id=11, body="", path=None, line=None, user=None)
+
+    result = get_review_comment_from_gitea_comment(comment)
+    assert result.body == ""
+    assert result.author.username == ""
+    assert result.file is None
+    assert result.line is None