"""E2E tests for attachment tools.
Uploads a real file via the Outline REST API (bypassing MCP), then tests
the read-only MCP attachment tools against that uploaded file. The upload
happens once per module via the ``attachment_id`` fixture.
"""
import pytest
from .helpers import (
_create_collection,
_create_document,
_text,
_upload_attachment,
)
pytestmark = [pytest.mark.e2e, pytest.mark.anyio]
@pytest.fixture(scope="module")
def attachment_id(outline_api_key):
"""Upload a test attachment once for all tests in this module."""
return _upload_attachment(outline_api_key)
async def test_list_document_attachments(attachment_id, mcp_session):
"""Create a doc referencing an uploaded attachment and list attachments.
Guards against: list_document_attachments failing to parse attachment
references embedded in document markdown content.
"""
async with mcp_session() as session:
coll_id = await _create_collection(session, "E2E Attach List")
# Create doc whose text references the real attachment
doc_id = await _create_document(
session,
coll_id,
"Attachment List Doc",
f"See [file](/api/attachments.redirect?id={attachment_id})",
)
result = await session.call_tool(
"list_document_attachments",
arguments={"document_id": doc_id},
)
text = _text(result)
assert attachment_id in text
assert "1." in text # at least one attachment listed
async def test_get_attachment_url(attachment_id, mcp_session):
"""Resolve an attachment ID to a signed download URL.
Guards against: get_attachment_url returning an error string instead of
a URL, or returning an empty response when Outline generates a redirect.
"""
async with mcp_session() as session:
result = await session.call_tool(
"get_attachment_url",
arguments={"attachment_id": attachment_id},
)
text = _text(result)
assert text.startswith("http"), f"Expected a URL, got: {text!r}"
async def test_fetch_attachment(attachment_id, mcp_session):
"""Download an attachment and verify the base64-encoded response fields.
Guards against: fetch_attachment omitting Content-Type or Content-Base64
headers, which would break AI image-processing workflows.
"""
async with mcp_session() as session:
result = await session.call_tool(
"fetch_attachment",
arguments={"attachment_id": attachment_id},
)
text = _text(result)
assert "Content-Type:" in text
assert "Content-Base64:" in text
