mirror of
https://github.com/Sea-Haven-Industries/open-swe.git
synced 2026-09-30 12:43: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.
78 lines
2.5 KiB
Python
78 lines
2.5 KiB
Python
from __future__ import annotations
|
|
|
|
import importlib
|
|
from typing import Any
|
|
|
|
import pytest
|
|
|
|
slack_reaction_tool = importlib.import_module("agent.tools.slack_add_reaction")
|
|
|
|
|
|
def _config() -> dict[str, Any]:
|
|
return {
|
|
"configurable": {
|
|
"slack_thread": {
|
|
"channel_id": "C1",
|
|
"thread_ts": "1.0",
|
|
"triggering_event_ts": "1.1",
|
|
}
|
|
}
|
|
}
|
|
|
|
|
|
async def test_slack_add_reaction_defaults_to_triggering_event(
|
|
monkeypatch: pytest.MonkeyPatch,
|
|
) -> None:
|
|
captured: dict[str, str] = {}
|
|
|
|
async def fake_add_slack_reaction(
|
|
channel_id: str, message_ts: str, emoji: str = "eyes"
|
|
) -> bool:
|
|
captured.update({"channel_id": channel_id, "message_ts": message_ts, "emoji": emoji})
|
|
return True
|
|
|
|
monkeypatch.setattr(slack_reaction_tool, "get_config", _config)
|
|
monkeypatch.setattr(slack_reaction_tool, "add_slack_reaction", fake_add_slack_reaction)
|
|
|
|
result = await slack_reaction_tool.slack_add_reaction()
|
|
|
|
assert result == {"success": True}
|
|
assert captured == {"channel_id": "C1", "message_ts": "1.1", "emoji": "eyes"}
|
|
|
|
|
|
async def test_slack_add_reaction_accepts_explicit_message_and_normalizes_emoji(
|
|
monkeypatch: pytest.MonkeyPatch,
|
|
) -> None:
|
|
captured: dict[str, str] = {}
|
|
|
|
async def fake_add_slack_reaction(
|
|
channel_id: str, message_ts: str, emoji: str = "eyes"
|
|
) -> bool:
|
|
captured.update({"channel_id": channel_id, "message_ts": message_ts, "emoji": emoji})
|
|
return True
|
|
|
|
monkeypatch.setattr(slack_reaction_tool, "get_config", _config)
|
|
monkeypatch.setattr(slack_reaction_tool, "add_slack_reaction", fake_add_slack_reaction)
|
|
|
|
result = await slack_reaction_tool.slack_add_reaction(
|
|
emoji=":white_check_mark:", message_ts="1.2"
|
|
)
|
|
|
|
assert result == {"success": True}
|
|
assert captured == {"channel_id": "C1", "message_ts": "1.2", "emoji": "white_check_mark"}
|
|
|
|
|
|
async def test_slack_add_reaction_requires_slack_channel(monkeypatch: pytest.MonkeyPatch) -> None:
|
|
monkeypatch.setattr(slack_reaction_tool, "get_config", lambda: {"configurable": {}})
|
|
|
|
result = await slack_reaction_tool.slack_add_reaction()
|
|
|
|
assert result == {"success": False, "error": "Missing slack_thread.channel_id in config"}
|
|
|
|
|
|
async def test_slack_add_reaction_rejects_empty_emoji(monkeypatch: pytest.MonkeyPatch) -> None:
|
|
monkeypatch.setattr(slack_reaction_tool, "get_config", _config)
|
|
|
|
result = await slack_reaction_tool.slack_add_reaction(emoji="::")
|
|
|
|
assert result == {"success": False, "error": "emoji is required"}
|