b4fbd6fe9f
Deploy Site / deploy-vercel (push) Has been skipped
Deploy Site / deploy-docs (push) Has been skipped
Build Skills Index / build-index (push) Has been skipped
CI / Deny unrelated histories (push) Has been skipped
CI / Detect affected areas (push) Successful in 27m35s
CI / OSV scan (push) Failing after 4s
CI / Build&Test Docker image (push) Successful in 9s
CI / Supply-chain scan (push) Has been skipped
CI / Lint Docker scripts (push) Failing after 5m13s
CI / Check contributors (push) Failing after 12m8s
CI / Docs Site (push) Failing after 12m8s
CI / TypeScript (push) Failing after 12m8s
CI / Python lints (push) Failing after 12m9s
CI / Python tests (push) Failing after 12m9s
CI / Check uv.lock (push) Failing after 23m22s
CI / CI timing report (push) Has been cancelled
Build Skills Index / trigger-deploy (push) Has been cancelled
CI / All required checks pass (push) Has been cancelled
121 lines
4.0 KiB
Python
121 lines
4.0 KiB
Python
"""Tests for gateway /compress --preview/--dry-run/--aggressive flags
|
|
(PR #3243 salvage).
|
|
|
|
The preview path must return a report WITHOUT building an agent or
|
|
touching the transcript; --aggressive must return an explanatory
|
|
message rather than being mis-parsed as a focus topic.
|
|
"""
|
|
|
|
from datetime import datetime
|
|
from unittest.mock import MagicMock
|
|
|
|
import pytest
|
|
|
|
from gateway.config import GatewayConfig, Platform, PlatformConfig
|
|
from gateway.platforms.base import MessageEvent
|
|
from gateway.session import SessionEntry, SessionSource, build_session_key
|
|
|
|
|
|
def _make_source() -> SessionSource:
|
|
return SessionSource(
|
|
platform=Platform.TELEGRAM,
|
|
user_id="u1",
|
|
chat_id="c1",
|
|
user_name="tester",
|
|
chat_type="dm",
|
|
)
|
|
|
|
|
|
def _make_event(text: str) -> MessageEvent:
|
|
return MessageEvent(text=text, source=_make_source(), message_id="m1")
|
|
|
|
|
|
def _make_history(n_pairs: int = 3) -> list[dict[str, str]]:
|
|
h: list[dict[str, str]] = []
|
|
for i in range(n_pairs):
|
|
h.append({"role": "user", "content": f"u{i}"})
|
|
h.append({"role": "assistant", "content": f"a{i}"})
|
|
return h
|
|
|
|
|
|
def _make_runner(history: list[dict[str, str]]):
|
|
from gateway.run import GatewayRunner
|
|
|
|
runner = object.__new__(GatewayRunner)
|
|
runner.config = GatewayConfig(
|
|
platforms={Platform.TELEGRAM: PlatformConfig(enabled=True, token="***")}
|
|
)
|
|
session_entry = SessionEntry(
|
|
session_key=build_session_key(_make_source()),
|
|
session_id="sess-1",
|
|
created_at=datetime.now(),
|
|
updated_at=datetime.now(),
|
|
platform=Platform.TELEGRAM,
|
|
chat_type="dm",
|
|
)
|
|
runner.session_store = MagicMock()
|
|
runner.session_store.get_or_create_session.return_value = session_entry
|
|
runner.session_store.load_transcript.return_value = history
|
|
runner.session_store.rewrite_transcript = MagicMock()
|
|
runner.session_store.update_session = MagicMock()
|
|
runner.session_store._save = MagicMock()
|
|
runner._session_db = None
|
|
return runner
|
|
|
|
|
|
@pytest.mark.asyncio
|
|
async def test_preview_reports_without_mutating():
|
|
runner = _make_runner(_make_history(3))
|
|
result = await runner._handle_compress_command(_make_event("/compress --preview"))
|
|
assert "no changes made" in result.lower()
|
|
assert "6 of 6" in result
|
|
runner.session_store.rewrite_transcript.assert_not_called()
|
|
runner.session_store.update_session.assert_not_called()
|
|
|
|
|
|
@pytest.mark.asyncio
|
|
async def test_dry_run_alias_matches_preview():
|
|
runner = _make_runner(_make_history(3))
|
|
result = await runner._handle_compress_command(_make_event("/compress --dry-run"))
|
|
assert "no changes made" in result.lower()
|
|
runner.session_store.rewrite_transcript.assert_not_called()
|
|
|
|
|
|
@pytest.mark.asyncio
|
|
async def test_preview_with_here_boundary():
|
|
runner = _make_runner(_make_history(4))
|
|
result = await runner._handle_compress_command(
|
|
_make_event("/compress --preview here 2")
|
|
)
|
|
assert "last 2 exchange" in result
|
|
assert "4 of 8" in result
|
|
runner.session_store.rewrite_transcript.assert_not_called()
|
|
|
|
|
|
@pytest.mark.asyncio
|
|
async def test_aggressive_returns_unsupported_note_without_mutating():
|
|
runner = _make_runner(_make_history(3))
|
|
result = await runner._handle_compress_command(
|
|
_make_event("/compress --aggressive")
|
|
)
|
|
assert "--aggressive is not supported" in result
|
|
runner.session_store.rewrite_transcript.assert_not_called()
|
|
|
|
|
|
@pytest.mark.asyncio
|
|
async def test_aggressive_dry_run_shows_preview_plus_note():
|
|
runner = _make_runner(_make_history(3))
|
|
result = await runner._handle_compress_command(
|
|
_make_event("/compress --aggressive --dry-run")
|
|
)
|
|
assert "no changes made" in result.lower()
|
|
assert "--aggressive is not supported" in result
|
|
runner.session_store.rewrite_transcript.assert_not_called()
|
|
|
|
|
|
@pytest.mark.asyncio
|
|
async def test_preview_still_requires_enough_history():
|
|
runner = _make_runner(_make_history(1)) # only 2 messages
|
|
result = await runner._handle_compress_command(_make_event("/compress --preview"))
|
|
assert "not enough" in result.lower()
|