"""A plate printed entirely from the external spool must dispatch use_ams=False (#3087).
The reporter's P1S sat at "Heatbed preheating" for ten and a half minutes and
then paused with 07FF_8012, "Failed to get AMS mapping table". The plate was one
filament, mapped by hand to the external spool, out of a seven-filament
MakerWorld project -- so the mapping was ``[-1, -1, -1, -1, -1, -1, 254]`` and
the command went out as ``use_ams: true`` with a flat mapping of nothing but
-1 (254 is deliberately not sent raw: the firmware reads it as AMS tray 0).
The decision belongs here rather than in the MQTT command builder. Down there a
-1 is either padding for a filament this plate does not print -- BambuStudio's
own convention, and what the other six entries are -- or a slot that never
resolved, which must never be redirected to the spool holder (#2589). The two
are the same byte. Only the plate's own filament list tells them apart, and
``extract_filament_requirements`` already drops anything with ``used_g <= 0``,
so it names exactly the slots that are printed.
"""
from __future__ import annotations
import json
import zipfile
from contextlib import ExitStack
from pathlib import Path
from types import SimpleNamespace
from unittest.mock import AsyncMock, MagicMock, patch
import pytest
from sqlalchemy.ext.asyncio import async_sessionmaker, create_async_engine
import backend.app.models # noqa: F401 - populate Base.metadata
import backend.app.services.print_scheduler as scheduler_module
from backend.app.core.database import Base
from backend.app.models.archive import PrintArchive
from backend.app.models.print_queue import PrintQueueItem
from backend.app.models.printer import Printer
from backend.app.models.settings import Settings # noqa: F401 - registers the table
from backend.app.services.print_scheduler import PrintScheduler
from backend.tests._fixtures.background_tasks import discarding_spawn_patch
pytestmark = pytest.mark.integration
# The reporter's plate: filament 7 of a seven-filament project, and it is the
# only one this plate consumes. slice_info.config lists a plate's filaments by
# their project-wide id, which is why the mapping is seven long.
_PLATE_4_ONE_FILAMENT = ''
def _write_3mf(path: Path, plate_index: int = 4, filaments: str = _PLATE_4_ONE_FILAMENT) -> None:
path.parent.mkdir(parents=True, exist_ok=True)
with zipfile.ZipFile(path, "w") as zf:
zf.writestr(
"Metadata/slice_info.config",
f'{filaments}',
)
def _write_3mf_without_slice_info(path: Path) -> None:
path.parent.mkdir(parents=True, exist_ok=True)
with zipfile.ZipFile(path, "w") as zf:
zf.writestr("3D/3dmodel.model", "")
@pytest.fixture
async def dispatch_case(tmp_path):
engine = create_async_engine("sqlite+aiosqlite:///:memory:")
async with engine.begin() as conn:
await conn.run_sync(Base.metadata.create_all)
session_maker = async_sessionmaker(engine, expire_on_commit=False)
base_dir = tmp_path / "external-spool"
async def _build(
mapping, *, use_ams=True, plate_id=4, filaments=_PLATE_4_ONE_FILAMENT, slice_info=True, model="P1S"
):
archive_rel = Path("archives") / f"plate-{plate_id}-{abs(hash(str(mapping))) % 10**6}.gcode.3mf"
if slice_info:
_write_3mf(base_dir / archive_rel, plate_index=plate_id, filaments=filaments)
else:
_write_3mf_without_slice_info(base_dir / archive_rel)
async with session_maker() as db:
printer = Printer(
name="P1S",
serial_number=f"01P{abs(hash(str(mapping))) % 10**9}",
ip_address="127.0.0.1",
access_code="access-code",
model=model,
)
db.add(printer)
await db.flush()
archive = PrintArchive(
printer_id=printer.id,
filename=archive_rel.name,
file_path=str(archive_rel),
file_size=(base_dir / archive_rel).stat().st_size,
status="completed",
)
db.add(archive)
await db.flush()
item = PrintQueueItem(
printer_id=printer.id,
archive_id=archive.id,
plate_id=plate_id,
status="pending",
use_ams=use_ams,
ams_mapping=json.dumps(mapping) if mapping is not None else None,
)
db.add(item)
await db.commit()
return SimpleNamespace(item_id=item.id, printer_id=printer.id)
try:
yield SimpleNamespace(session_maker=session_maker, base_dir=base_dir, build=_build)
finally:
await engine.dispose()
async def _dispatch(ctx, ids, status=None):
scheduler = PrintScheduler()
start_print = MagicMock(return_value=True)
status = status or SimpleNamespace(state="IDLE", nozzle_rack=None, raw_data={}, nozzles=[])
with ExitStack() as stack:
for patcher in (
patch.object(scheduler_module, "async_session", ctx.session_maker),
patch.object(scheduler_module.settings, "base_dir", ctx.base_dir),
patch("backend.app.services.print_scheduler.printer_manager.is_connected", MagicMock(return_value=True)),
patch("backend.app.services.print_scheduler.printer_manager.get_status", MagicMock(return_value=status)),
patch("backend.app.services.print_scheduler.printer_manager.start_print", start_print),
patch("backend.app.services.print_scheduler.printer_manager.set_awaiting_plate_clear", MagicMock()),
patch("backend.app.services.print_scheduler.delete_file_async", AsyncMock(return_value=True)),
patch("backend.app.services.print_scheduler.upload_file_async", AsyncMock(return_value=True)),
patch(
"backend.app.services.print_scheduler.get_ftp_retry_settings",
AsyncMock(return_value=(False, 3, 2.0, 30.0)),
),
patch("backend.app.services.print_scheduler.cache_3mf_download", MagicMock()),
discarding_spawn_patch(),
patch("backend.app.services.notification_service.notification_service.on_queue_job_started", AsyncMock()),
patch("backend.app.services.notification_service.notification_service.on_queue_job_failed", AsyncMock()),
patch("backend.app.services.mqtt_relay.mqtt_relay.on_queue_job_started", AsyncMock()),
patch.object(scheduler, "_propagate_owner_to_printer_manager", AsyncMock()),
patch.object(scheduler, "_power_off_if_needed", AsyncMock()),
patch.object(scheduler, "_preheat_and_soak", AsyncMock()),
):
stack.enter_context(patcher)
await scheduler._dispatch_one(ids.item_id)
assert start_print.call_count == 1, "the print command was never sent"
return start_print.call_args
class TestThePlateThatOnlyPrintsFromTheSpoolHolder:
async def test_the_reporters_mapping_dispatches_without_the_ams(self, dispatch_case):
"""[-1]*6 + [254] on a plate whose only printed filament is #7."""
ids = await dispatch_case.build([-1, -1, -1, -1, -1, -1, 254])
call = await _dispatch(dispatch_case, ids)
assert call.kwargs["use_ams"] is False
# The mapping itself still goes out untouched — the builder is what
# turns 254 into -1 plus ams_mapping2, and none of that changes.
assert call.kwargs["ams_mapping"] == [-1, -1, -1, -1, -1, -1, 254]
async def test_the_main_nozzle_sentinel_counts_too(self, dispatch_case):
ids = await dispatch_case.build([-1, -1, -1, -1, -1, -1, 255])
call = await _dispatch(dispatch_case, ids)
assert call.kwargs["use_ams"] is False
async def test_an_unpadded_single_filament_plate_is_unaffected(self, dispatch_case):
"""[254] already worked: the MQTT command builder downgrades an
all-external mapping by itself. The scheduler now reaches the same
answer one layer earlier, so the two agree rather than one undoing the
other — this pins that they do."""
ids = await dispatch_case.build([254], filaments='', plate_id=1)
call = await _dispatch(dispatch_case, ids)
assert call.kwargs["use_ams"] is False
class TestWhatMustNotChange:
async def test_a_consumed_slot_that_never_resolved_still_goes_out_with_the_ams(self, dispatch_case):
"""The #2589 contract, and the reason this lives in the scheduler.
Filaments 1 and 7 are both printed; 7 is on the spool holder and 1
resolved to nothing. Redirecting the plate to the external spool would
print filament 1 in the wrong material without saying so. use_ams stays
true and the firmware rejects the print, exactly as before.
"""
ids = await dispatch_case.build(
[-1, -1, -1, -1, -1, -1, 254],
filaments='' + _PLATE_4_ONE_FILAMENT,
)
call = await _dispatch(dispatch_case, ids)
assert call.kwargs["use_ams"] is True
async def test_a_plate_mixing_an_ams_tray_with_the_spool_holder_keeps_the_ams(self, dispatch_case):
ids = await dispatch_case.build(
[5, -1, -1, -1, -1, -1, 254],
filaments='' + _PLATE_4_ONE_FILAMENT,
)
call = await _dispatch(dispatch_case, ids)
assert call.kwargs["use_ams"] is True
async def test_a_plate_printed_from_ams_trays_is_untouched(self, dispatch_case):
ids = await dispatch_case.build([-1, -1, -1, -1, -1, -1, 5])
call = await _dispatch(dispatch_case, ids)
assert call.kwargs["use_ams"] is True
async def test_use_ams_false_is_never_promoted_here(self, dispatch_case):
"""Promotion is the builder's job (#2595) and stays there."""
ids = await dispatch_case.build([-1, -1, -1, -1, -1, -1, 5], use_ams=False)
call = await _dispatch(dispatch_case, ids)
assert call.kwargs["use_ams"] is False
async def test_a_3mf_with_no_filament_list_falls_back_to_the_stored_flag(self, dispatch_case):
"""No evidence, no decision — the same convention as #2771."""
ids = await dispatch_case.build([-1, -1, -1, -1, -1, -1, 254], slice_info=False)
call = await _dispatch(dispatch_case, ids)
assert call.kwargs["use_ams"] is True
async def test_a_plate_the_file_does_not_describe_falls_back(self, dispatch_case):
"""The item says plate 4; the file only describes plate 1."""
ids = await dispatch_case.build([-1, -1, -1, -1, -1, -1, 254], plate_id=4)
# Rewrite the archive's 3MF so its only plate is index 1.
async with dispatch_case.session_maker() as db:
archive = (await db.get(PrintQueueItem, ids.item_id)).archive_id
path = dispatch_case.base_dir / (await db.get(PrintArchive, archive)).file_path
_write_3mf(path, plate_index=1)
call = await _dispatch(dispatch_case, ids)
assert call.kwargs["use_ams"] is True
async def test_an_item_with_no_mapping_at_all_is_untouched(self, dispatch_case):
ids = await dispatch_case.build(None)
call = await _dispatch(dispatch_case, ids)
assert call.kwargs["use_ams"] is True
class TestDualNozzleIsNotOursToRewrite:
"""On a two-extruder printer use_ams is which nozzle to feed, not whether to
use the AMS — H2D Pro firmware reads it as an extruder index. The MQTT
command builder skips its own reconcile for exactly that reason, and this
must skip it too, or a perfectly normal dual external-spool print gets its
routing rewritten."""
async def test_a_dual_nozzle_model_keeps_its_flag(self, dispatch_case):
ids = await dispatch_case.build([-1, -1, -1, -1, -1, -1, 254], model="H2D")
call = await _dispatch(dispatch_case, ids)
assert call.kwargs["use_ams"] is True
async def test_both_external_feeds_on_a_dual_nozzle_are_left_alone(self, dispatch_case):
"""254 is the deputy feed and 255 the main one — an ordinary H2D print
with a spool on each side, and the one this would have broken."""
ids = await dispatch_case.build(
[254, -1, -1, -1, -1, -1, 255],
filaments='' + _PLATE_4_ONE_FILAMENT,
model="H2D",
)
call = await _dispatch(dispatch_case, ids)
assert call.kwargs["use_ams"] is True
async def test_live_telemetry_can_veto_a_single_nozzle_model_name(self, dispatch_case):
"""A model string we do not recognise as dual is not the last word: two
external feeds is something only a two-extruder printer reports."""
ids = await dispatch_case.build([-1, -1, -1, -1, -1, -1, 254], model="Something New")
status = SimpleNamespace(
state="IDLE",
nozzle_rack=None,
nozzles=[],
raw_data={"vt_tray": [{"id": "254"}, {"id": "255"}]},
)
call = await _dispatch(dispatch_case, ids, status=status)
assert call.kwargs["use_ams"] is True
async def test_a_second_nozzle_reporting_a_diameter_vetoes_it_too(self, dispatch_case):
ids = await dispatch_case.build([-1, -1, -1, -1, -1, -1, 254], model="Something New")
status = SimpleNamespace(
state="IDLE",
nozzle_rack=None,
nozzles=[SimpleNamespace(nozzle_diameter="0.4"), SimpleNamespace(nozzle_diameter="0.4")],
raw_data={},
)
call = await _dispatch(dispatch_case, ids, status=status)
assert call.kwargs["use_ams"] is True
async def test_h2s_is_single_nozzle_and_still_gets_the_fix(self, dispatch_case):
"""H2S shares the H2 serial prefix and firmware quirks but has one
extruder — the #1386 distinction, which must survive here."""
ids = await dispatch_case.build([-1, -1, -1, -1, -1, -1, 254], model="H2S")
call = await _dispatch(dispatch_case, ids)
assert call.kwargs["use_ams"] is False