bambuddy/backend/tests/integration/test_pending_upload_display_name.py

"""Integration tests for #1152 follow-up — pending-upload review card name
matches the eventual archive's print_name.

Before this change the review card always showed the raw FTP filename while
the archive's ``print_name`` was resolved from the 3MF metadata title (or, with
the toggle on ``filename``, the stripped stem). That gave users two different
names for the same item — they'd see *Plate_1.gcode* in review and *Some
Creator's Title* in the archive grid.

These tests pin the new contract:
  - ``PendingUploadResponse.display_name`` mirrors what archive_print will
    eventually write to ``PrintArchive.print_name``.
  - The toggle (``virtual_printer_archive_name_source``) flips both views in
    lockstep — never one without the other.
  - Filename normalisation (``Plate_1.gcode.3mf`` → ``Plate_1``) is applied
    consistently regardless of the toggle.
"""

import pytest
from httpx import AsyncClient
from sqlalchemy.ext.asyncio import AsyncSession

from backend.app.models.pending_upload import PendingUpload
from backend.app.models.settings import Settings


async def _set_archive_name_source(db: AsyncSession, value: str) -> None:
    """Write the virtual_printer_archive_name_source setting directly."""
    db.add(Settings(key="virtual_printer_archive_name_source", value=value))
    await db.commit()


async def _seed_pending(
    db: AsyncSession,
    *,
    filename: str,
    metadata_print_name: str | None = None,
) -> int:
    pending = PendingUpload(
        filename=filename,
        file_path=f"/test/pending/{filename}",
        file_size=42,
        source_ip="192.168.1.50",
        status="pending",
        metadata_print_name=metadata_print_name,
    )
    db.add(pending)
    await db.commit()
    await db.refresh(pending)
    return pending.id


class TestDisplayNameResolution:
    """``GET /pending-uploads/`` resolves ``display_name`` to the same value
    ``archive_print`` would store on the eventual ``PrintArchive``."""

    @pytest.mark.asyncio
    @pytest.mark.integration
    async def test_default_toggle_uses_metadata_title_when_present(
        self, async_client: AsyncClient, db_session: AsyncSession
    ):
        """Default toggle is ``metadata`` — review card shows the embedded
        title rather than the FTP filename, matching what the archived
        PrintArchive.print_name will end up being."""
        await _seed_pending(
            db_session,
            filename="Plate_1.gcode.3mf",
            metadata_print_name="Custom Cool Benchy",
        )

        resp = await async_client.get("/api/v1/pending-uploads/")
        assert resp.status_code == 200
        rows = resp.json()
        assert len(rows) == 1
        assert rows[0]["display_name"] == "Custom Cool Benchy"
        # filename is still surfaced separately so the user can see what
        # actually arrived over FTP if they want to (tooltip).
        assert rows[0]["filename"] == "Plate_1.gcode.3mf"

    @pytest.mark.asyncio
    @pytest.mark.integration
    async def test_default_toggle_falls_back_to_stripped_stem_when_no_metadata(
        self, async_client: AsyncClient, db_session: AsyncSession
    ):
        """No embedded title (Bambu Studio's default plate export) — both
        review and archive end up showing the stripped filename stem."""
        await _seed_pending(
            db_session,
            filename="Plate_1.gcode.3mf",
            metadata_print_name=None,
        )

        resp = await async_client.get("/api/v1/pending-uploads/")
        assert resp.status_code == 200
        # ``Plate_1`` — both ``.gcode`` and ``.3mf`` stripped (#1152
        # follow-up: ``Path.stem`` only strips the last suffix and would
        # leave ``Plate_1.gcode``).
        assert resp.json()[0]["display_name"] == "Plate_1"

    @pytest.mark.asyncio
    @pytest.mark.integration
    async def test_filename_toggle_overrides_metadata_title(self, async_client: AsyncClient, db_session: AsyncSession):
        """When the operator opts into ``filename`` (so a user-renamed
        Bambu Studio job surfaces its renamed-at-send filename), the embedded
        creator-baked title is ignored. Review must follow the same toggle
        as the archive."""
        await _set_archive_name_source(db_session, "filename")
        await _seed_pending(
            db_session,
            filename="MyRenamedJob.3mf",
            metadata_print_name="Original Creator Title",
        )

        resp = await async_client.get("/api/v1/pending-uploads/")
        assert resp.status_code == 200
        assert resp.json()[0]["display_name"] == "MyRenamedJob"

    @pytest.mark.asyncio
    @pytest.mark.integration
    async def test_filename_toggle_strips_double_suffix(self, async_client: AsyncClient, db_session: AsyncSession):
        """Filename mode also drops .gcode.3mf — same normalisation as the
        archive's print_name, so the names line up exactly."""
        await _set_archive_name_source(db_session, "filename")
        await _seed_pending(db_session, filename="Plate_4.gcode.3mf", metadata_print_name=None)

        resp = await async_client.get("/api/v1/pending-uploads/")
        assert resp.status_code == 200
        assert resp.json()[0]["display_name"] == "Plate_4"

    @pytest.mark.asyncio
    @pytest.mark.integration
    async def test_get_one_returns_display_name(self, async_client: AsyncClient, db_session: AsyncSession):
        """The single-resource endpoint surfaces display_name too — UIs that
        load a pending upload by id (e.g. detail modals) get the same name."""
        upload_id = await _seed_pending(
            db_session,
            filename="X.gcode.3mf",
            metadata_print_name="Deep Detail Bear",
        )

        resp = await async_client.get(f"/api/v1/pending-uploads/{upload_id}")
        assert resp.status_code == 200
        assert resp.json()["display_name"] == "Deep Detail Bear"

    @pytest.mark.asyncio
    @pytest.mark.integration
    async def test_blank_metadata_title_falls_back_to_stem(self, async_client: AsyncClient, db_session: AsyncSession):
        """A whitespace-only metadata title behaves like absent metadata —
        guards against 3MFs with broken/empty Title fields surfacing as a
        blank review card."""
        await _seed_pending(
            db_session,
            filename="empty-title.gcode.3mf",
            metadata_print_name="   ",
        )

        resp = await async_client.get("/api/v1/pending-uploads/")
        assert resp.status_code == 200
        assert resp.json()[0]["display_name"] == "empty-title"