| 123456789101112131415161718192021222324252627282930313233343536373839404142434445464748495051525354555657585960616263646566676869707172737475767778798081828384858687888990919293949596979899100101102103104105106107108109110111112113114115116117118119120121122123124125126127128129130131132133134135136137138139140141142143144145146147148149150151152153154155156157158159160161162163164165166167168169170171172173174175176177178179180181182183184185186187188189190191192193194195196197198199200201202203204205206 |
- """Tests for _extract_filament_data_from_mqtt (#1533).
- The fallback PrintArchive path in main.py fires when the source 3MF can't
- be downloaded from the printer at print start — common on P1S / A1 / P2S
- firmwares that lock the file during printing. Before this fix the
- fallback archive had every filament field NULL even though the MQTT
- print-start payload already carried the AMS state and the slicer's
- slot-per-print-filament mapping. The helper extracts a comma-separated
- ``filament_type`` / ``filament_color`` from that payload so the inventory
- views can at least show what's loaded, and operators planning AMS
- expansion can count filaments per print.
- """
- import pytest
- from backend.app.main import _extract_filament_data_from_mqtt
- def _ams_unit(unit_id: int, trays: list[dict]) -> dict:
- return {"id": unit_id, "tray": trays}
- def _tray(tray_id: int, ttype: str | None, color: str | None) -> dict:
- out: dict = {"id": tray_id}
- if ttype is not None:
- out["tray_type"] = ttype
- if color is not None:
- out["tray_color"] = color
- return out
- class TestExtractFilamentDataFromMqtt:
- def test_empty_payload_returns_empty_dict(self):
- assert _extract_filament_data_from_mqtt({}) == {}
- assert _extract_filament_data_from_mqtt({"ams": None}) == {}
- assert _extract_filament_data_from_mqtt({"ams": {}}) == {}
- assert _extract_filament_data_from_mqtt({"ams": {"ams": []}}) == {}
- def test_no_loaded_slots_returns_empty(self):
- """All slots empty (no tray_type) → nothing to report."""
- data = {
- "ams": {
- "ams": [
- _ams_unit(0, [_tray(i, "", "") for i in range(4)]),
- ],
- }
- }
- assert _extract_filament_data_from_mqtt(data) == {}
- def test_no_mapping_lists_all_loaded_slots_sorted(self):
- data = {
- "ams": {
- "ams": [
- _ams_unit(
- 0,
- [
- _tray(0, "PLA", "FF0000"),
- _tray(1, "PETG", "00FF00"),
- _tray(2, "", ""), # Empty slot — skipped.
- _tray(3, "ABS", "0000ff"),
- ],
- ),
- ],
- }
- }
- result = _extract_filament_data_from_mqtt(data)
- # Order is by ascending global tray id, colors uppercased.
- assert result == {"filament_type": "PLA,PETG,ABS", "filament_color": "FF0000,00FF00,0000FF"}
- def test_ams_mapping_narrows_to_used_slots(self):
- """The slicer's slot-per-print-filament mapping wins — only used
- slots contribute, in the slicer's order (which is the order the
- print materially consumes them)."""
- data = {
- "ams": {
- "ams": [
- _ams_unit(
- 0,
- [
- _tray(0, "PLA", "FF0000"),
- _tray(1, "PETG", "00FF00"),
- _tray(2, "ABS", "0000FF"),
- _tray(3, "TPU", "FFFF00"),
- ],
- ),
- ],
- }
- }
- # Print uses slots 3 then 0 then 1 (slot 2 untouched, no entry).
- result = _extract_filament_data_from_mqtt(data, ams_mapping=[3, 0, 1])
- assert result == {"filament_type": "TPU,PLA,PETG", "filament_color": "FFFF00,FF0000,00FF00"}
- def test_ams_mapping_with_vt_tray_sentinels_filtered_out(self):
- """ams_mapping entries equal to -1 represent the VT tray (external
- spool feed). We have no AMS tray data for them — they must be
- skipped, not treated as global tray id 0."""
- data = {
- "ams": {
- "ams": [
- _ams_unit(
- 0,
- [
- _tray(0, "PLA", "FF0000"),
- _tray(1, "PETG", "00FF00"),
- ],
- ),
- ],
- }
- }
- result = _extract_filament_data_from_mqtt(data, ams_mapping=[-1, 0, 1])
- assert result == {"filament_type": "PLA,PETG", "filament_color": "FF0000,00FF00"}
- def test_dual_ams_global_ids_use_unit4_offset(self):
- """A dual-AMS rig has unit 0 → trays 0-3, unit 1 → trays 4-7.
- ``ams_mapping=4`` must resolve to unit 1, tray 0 — not unit 0."""
- data = {
- "ams": {
- "ams": [
- _ams_unit(0, [_tray(0, "PLA", "FF0000")]),
- _ams_unit(1, [_tray(0, "PETG-CF", "112233")]),
- ],
- }
- }
- result = _extract_filament_data_from_mqtt(data, ams_mapping=[4, 0])
- assert result == {"filament_type": "PETG-CF,PLA", "filament_color": "112233,FF0000"}
- def test_mapping_pointing_at_unknown_slot_falls_through_to_known_only(self):
- data = {
- "ams": {
- "ams": [
- _ams_unit(0, [_tray(0, "PLA", "FF0000")]),
- ],
- }
- }
- # Slot 7 isn't in our AMS — entry skipped, only slot 0 remains.
- result = _extract_filament_data_from_mqtt(data, ams_mapping=[7, 0])
- assert result == {"filament_type": "PLA", "filament_color": "FF0000"}
- def test_mapping_entirely_unknown_returns_empty(self):
- """If every mapped slot is unknown the helper returns {} rather
- than silently misreporting from the all-slots fallback — the
- slicer was explicit about which slots to use."""
- data = {
- "ams": {
- "ams": [
- _ams_unit(0, [_tray(0, "PLA", "FF0000")]),
- ],
- }
- }
- assert _extract_filament_data_from_mqtt(data, ams_mapping=[5, 6]) == {}
- def test_color_truncation_at_column_limit(self):
- """filament_color column is VARCHAR(200); long multi-color prints
- must not exceed it."""
- # 16 trays of 6-char colors + 15 commas = 96+15 = 111 chars. Safe.
- # Construct an oversized synthetic case with many distinct colors.
- trays = [_tray(i, "PLA", f"{i:06X}") for i in range(4)]
- data = {"ams": {"ams": [_ams_unit(u, trays) for u in range(8)]}}
- result = _extract_filament_data_from_mqtt(data)
- assert "filament_color" in result
- assert len(result["filament_color"]) <= 200
- def test_type_truncation_at_column_limit(self):
- """filament_type column is VARCHAR(50). Many filaments must truncate."""
- # 16 PETG-CF entries: 7 chars × 16 + 15 commas = 127 chars.
- trays = [_tray(i, "PETG-CF", "AABBCC") for i in range(4)]
- data = {"ams": {"ams": [_ams_unit(u, trays) for u in range(4)]}}
- result = _extract_filament_data_from_mqtt(data)
- assert "filament_type" in result
- assert len(result["filament_type"]) <= 50
- def test_color_missing_only_emits_type(self):
- """A tray with type but blank color still contributes to filament_type."""
- data = {
- "ams": {
- "ams": [
- _ams_unit(0, [_tray(0, "PLA", "")]),
- ],
- }
- }
- result = _extract_filament_data_from_mqtt(data)
- assert result == {"filament_type": "PLA"}
- # filament_color absent — not empty string.
- assert "filament_color" not in result
- def test_malformed_unit_skipped_without_crash(self):
- """Defensive: unexpected MQTT shapes (non-dict in ams list, missing
- id, string tray.id) must not raise. The fallback-archive write
- runs in a hot path during print start — anything that throws here
- would bubble up and break the print log entirely."""
- data = {
- "ams": {
- "ams": [
- "garbage",
- {"id": "not-an-int", "tray": []},
- _ams_unit(0, [_tray(0, "PLA", "FF0000"), {"id": "x", "tray_type": "PETG"}]),
- ],
- }
- }
- result = _extract_filament_data_from_mqtt(data)
- # Only the well-formed entry contributes; no exception.
- assert result.get("filament_type") == "PLA"
- @pytest.mark.parametrize("data", [None, {}, {"ams": "weird-string"}])
- def test_garbage_top_level_is_empty(self, data):
- assert _extract_filament_data_from_mqtt(data or {}) == {}
|