test_spoolman_no3mf_remain_fallback.py 17 KB

123456789101112131415161718192021222324252627282930313233343536373839404142434445464748495051525354555657585960616263646566676869707172737475767778798081828384858687888990919293949596979899100101102103104105106107108109110111112113114115116117118119120121122123124125126127128129130131132133134135136137138139140141142143144145146147148149150151152153154155156157158159160161162163164165166167168169170171172173174175176177178179180181182183184185186187188189190191192193194195196197198199200201202203204205206207208209210211212213214215216217218219220221222223224225226227228229230231232233234235236237238239240241242243244245246247248249250251252253254255256257258259260261262263264265266267268269270271272273274275276277278279280281282283284285286287288289290291292293294295296297298299300301302303304305306307308309310311312313314315316317318319320321322323324325326327328329330331332333334335336337338339340341342343344345346347348349350351352353354355356357358359360361362363364365366367368369370371372373374375376377378379380381382383384385386387388389390391392393394395396397398399400401402403404405406407408409410411412413414415416417418419420421422423424425426427428429430431432433434435436
  1. """AMS remain%-delta fallback for the no-3MF Spoolman path (#1820).
  2. When a Bambu print starts without leaving a retrievable .gcode.3mf on the
  3. printer (subtask_name='名称未設定'/'Untitled'), Bambuddy creates a
  4. fallback archive with no 3MF on disk. Before this fix the Spoolman
  5. tracking row was never created, so the print silently didn't decrement
  6. the spool weight. This is the Spoolman mirror of usage_tracker's Path 2
  7. fallback (already in place for the internal-inventory side).
  8. """
  9. from types import SimpleNamespace
  10. from unittest.mock import AsyncMock, MagicMock, patch
  11. import pytest
  12. from backend.app.services.spoolman_tracking import (
  13. _snapshot_tray_remain,
  14. store_print_data,
  15. )
  16. class TestSnapshotTrayRemain:
  17. def test_captures_valid_remain(self):
  18. raw = {
  19. "ams": [
  20. {
  21. "id": 0,
  22. "tray": [
  23. {"id": 0, "tray_uuid": "AAAA", "remain": 75},
  24. {"id": 1, "tray_uuid": "BBBB", "remain": 30},
  25. ],
  26. }
  27. ]
  28. }
  29. snap = _snapshot_tray_remain(raw)
  30. assert snap == {
  31. "0-0": {"remain": 75, "tray_uuid": "AAAA"},
  32. "0-1": {"remain": 30, "tray_uuid": "BBBB"},
  33. }
  34. def test_skips_invalid_remain(self):
  35. """remain=-1 means the AMS hasn't read the spool; a delta would be
  36. meaningless. Skip those slots — usage_tracker does the same
  37. (line 309)."""
  38. raw = {
  39. "ams": [
  40. {
  41. "id": 0,
  42. "tray": [
  43. {"id": 0, "tray_uuid": "AAAA", "remain": 75},
  44. {"id": 1, "tray_uuid": "BBBB", "remain": -1},
  45. {"id": 2, "tray_uuid": "CCCC", "remain": 150},
  46. ],
  47. }
  48. ]
  49. }
  50. snap = _snapshot_tray_remain(raw)
  51. assert set(snap.keys()) == {"0-0"}
  52. def test_captures_vt_tray(self):
  53. """External (VT) spool gets ams_id=255, tray_id=vt_id-254."""
  54. raw = {"ams": [], "vt_tray": [{"id": 254, "tray_uuid": "EEEE", "remain": 50}]}
  55. snap = _snapshot_tray_remain(raw)
  56. assert snap == {"255-0": {"remain": 50, "tray_uuid": "EEEE"}}
  57. def test_empty_when_no_ams_data(self):
  58. assert _snapshot_tray_remain({}) == {}
  59. def test_handles_missing_uuid(self):
  60. raw = {"ams": [{"id": 0, "tray": [{"id": 0, "remain": 80}]}]}
  61. snap = _snapshot_tray_remain(raw)
  62. assert snap == {"0-0": {"remain": 80, "tray_uuid": ""}}
  63. class TestStorePrintDataNo3mf:
  64. """store_print_data must create an ActivePrintSpoolman row even when
  65. no 3MF is available, populating tray_remain_start so report_usage can
  66. write a remain-delta at completion (#1820)."""
  67. @pytest.mark.asyncio
  68. async def test_creates_row_with_snapshot_when_no_3mf(self):
  69. db = AsyncMock()
  70. # No queue lookup for the no-3MF branch — only the DELETE.
  71. delete_result = MagicMock()
  72. db.execute = AsyncMock(side_effect=[delete_result])
  73. db.add = MagicMock()
  74. db.commit = AsyncMock()
  75. printer_manager = MagicMock()
  76. printer_manager.get_status.return_value = SimpleNamespace(
  77. raw_data={
  78. "ams": [
  79. {
  80. "id": 0,
  81. "tray": [
  82. {"id": 0, "tray_uuid": "AAAA", "tag_uid": "11", "tray_type": "PLA", "remain": 80},
  83. {"id": 1, "tray_uuid": "BBBB", "tag_uid": "22", "tray_type": "PLA", "remain": 20},
  84. ],
  85. }
  86. ]
  87. }
  88. )
  89. mock_settings = MagicMock()
  90. mock_path = MagicMock()
  91. mock_path.exists.return_value = False # no 3MF — the #1820 case
  92. mock_settings.base_dir.__truediv__.return_value = mock_path
  93. with (
  94. patch("backend.app.services.spoolman_tracking.app_settings", mock_settings),
  95. patch("backend.app.api.routes.settings.get_setting", AsyncMock(return_value="true")),
  96. ):
  97. await store_print_data(
  98. printer_id=1,
  99. archive_id=42,
  100. file_path="", # fallback-archive file_path
  101. db=db,
  102. printer_manager=printer_manager,
  103. )
  104. db.add.assert_called_once()
  105. tracking = db.add.call_args.args[0]
  106. assert tracking.filament_usage is None
  107. assert tracking.tray_remain_start == {
  108. "0-0": {"remain": 80, "tray_uuid": "AAAA"},
  109. "0-1": {"remain": 20, "tray_uuid": "BBBB"},
  110. }
  111. @pytest.mark.asyncio
  112. async def test_no_row_when_no_3mf_and_no_remain_data(self):
  113. """If the AMS has no slot with valid remain either (e.g. printer
  114. offline at print start), there's nothing to track. Don't create
  115. a row that contributes no value."""
  116. db = AsyncMock()
  117. db.execute = AsyncMock()
  118. db.add = MagicMock()
  119. db.commit = AsyncMock()
  120. printer_manager = MagicMock()
  121. printer_manager.get_status.return_value = SimpleNamespace(
  122. raw_data={"ams": [{"id": 0, "tray": [{"id": 0, "remain": -1}]}]}
  123. )
  124. mock_settings = MagicMock()
  125. mock_path = MagicMock()
  126. mock_path.exists.return_value = False
  127. mock_settings.base_dir.__truediv__.return_value = mock_path
  128. with (
  129. patch("backend.app.services.spoolman_tracking.app_settings", mock_settings),
  130. patch("backend.app.api.routes.settings.get_setting", AsyncMock(return_value="true")),
  131. ):
  132. await store_print_data(
  133. printer_id=1,
  134. archive_id=42,
  135. file_path="",
  136. db=db,
  137. printer_manager=printer_manager,
  138. )
  139. db.add.assert_not_called()
  140. @pytest.mark.asyncio
  141. async def test_3mf_path_also_captures_snapshot(self):
  142. """The remain snapshot is captured ALWAYS, not just for no-3MF.
  143. That lets report_usage fall back per-slot when 3MF coverage is
  144. partial — same shape as usage_tracker.on_print_complete which
  145. runs Path 1 (3MF) and Path 2 (remain delta) for unhandled slots."""
  146. db = AsyncMock()
  147. queue_item = SimpleNamespace(ams_mapping=None, plate_id=None)
  148. queue_result = MagicMock()
  149. queue_result.scalar_one_or_none.return_value = queue_item
  150. delete_result = MagicMock()
  151. db.execute = AsyncMock(side_effect=[queue_result, delete_result])
  152. db.add = MagicMock()
  153. db.commit = AsyncMock()
  154. printer_manager = MagicMock()
  155. printer_manager.get_status.return_value = SimpleNamespace(
  156. raw_data={"ams": [{"id": 0, "tray": [{"id": 0, "tray_uuid": "AAAA", "tray_type": "PLA", "remain": 90}]}]}
  157. )
  158. mock_settings = MagicMock()
  159. mock_path = MagicMock()
  160. mock_path.exists.return_value = True
  161. mock_settings.base_dir.__truediv__.return_value = mock_path
  162. with (
  163. patch("backend.app.services.spoolman_tracking.app_settings", mock_settings),
  164. patch("backend.app.api.routes.settings.get_setting", AsyncMock(return_value="true")),
  165. patch(
  166. "backend.app.utils.threemf_tools.extract_filament_usage_from_3mf",
  167. return_value=[{"slot_id": 1, "used_g": 10.0, "type": "PLA", "color": "#000000"}],
  168. ),
  169. patch("backend.app.utils.threemf_tools.extract_layer_filament_usage_from_3mf", return_value=None),
  170. patch("backend.app.utils.threemf_tools.extract_filament_properties_from_3mf", return_value={}),
  171. ):
  172. await store_print_data(
  173. printer_id=1,
  174. archive_id=42,
  175. file_path="archives/test.3mf",
  176. db=db,
  177. printer_manager=printer_manager,
  178. )
  179. db.add.assert_called_once()
  180. tracking = db.add.call_args.args[0]
  181. assert tracking.filament_usage == [{"slot_id": 1, "used_g": 10.0, "type": "PLA", "color": "#000000"}]
  182. assert tracking.tray_remain_start == {"0-0": {"remain": 90, "tray_uuid": "AAAA"}}
  183. class TestReportUsageRemainDelta:
  184. """report_usage must write a per-slot remain-delta when filament_usage
  185. is missing (no-3MF print), gated on a resolvable Spoolman spool and a
  186. sane current remain%."""
  187. @pytest.mark.asyncio
  188. async def test_remain_delta_writes_to_resolved_spool(self):
  189. """Print started at remain=80% on a 1000g filament, finished at 60%.
  190. Delta = 20% × 1000g = 200g."""
  191. from backend.app.services.spoolman_tracking import report_usage
  192. tracking = SimpleNamespace(
  193. filament_usage=None,
  194. ams_trays={"0": {"tray_uuid": "AAAA", "tag_uid": "11", "tray_type": "PLA"}},
  195. slot_to_tray=None,
  196. tray_remain_start={"0-0": {"remain": 80, "tray_uuid": "AAAA"}},
  197. )
  198. # Mock db.execute().scalar_one_or_none() -> tracking
  199. db = AsyncMock()
  200. select_result = MagicMock()
  201. select_result.scalar_one_or_none.return_value = tracking
  202. db.execute = AsyncMock(return_value=select_result)
  203. db.delete = AsyncMock()
  204. db.commit = AsyncMock()
  205. client = AsyncMock()
  206. client.get_spool = AsyncMock(return_value={"id": 7, "filament": {"weight": 1000.0, "color_hex": "00FF00"}})
  207. client.use_spool = AsyncMock()
  208. printer_manager = MagicMock()
  209. printer_manager.get_status.return_value = SimpleNamespace(
  210. raw_data={"ams": [{"id": 0, "tray": [{"id": 0, "tray_uuid": "AAAA", "remain": 60}]}]}
  211. )
  212. with (
  213. patch("backend.app.services.spoolman_tracking.async_session", lambda: _AsyncCtx(db)),
  214. patch("backend.app.api.routes.settings.get_setting", AsyncMock(return_value="true")),
  215. patch(
  216. "backend.app.services.spoolman_tracking._get_spoolman_client_with_fallback",
  217. AsyncMock(return_value=client),
  218. ),
  219. patch("backend.app.services.spoolman_tracking._get_printer_serial", AsyncMock(return_value="serial")),
  220. patch(
  221. "backend.app.services.spoolman_tracking._resolve_spool_id_via_slot_assignment",
  222. AsyncMock(return_value=7),
  223. ),
  224. patch("backend.app.services.printer_manager.printer_manager", printer_manager),
  225. ):
  226. await report_usage(printer_id=1, archive_id=42)
  227. client.use_spool.assert_awaited_once_with(7, 200.0)
  228. @pytest.mark.asyncio
  229. async def test_remain_delta_skips_swapped_spool(self):
  230. """tray_uuid changed between start and completion → user replaced
  231. the spool mid-print. We don't know how much went to each side; skip
  232. rather than mis-charge."""
  233. from backend.app.services.spoolman_tracking import report_usage
  234. tracking = SimpleNamespace(
  235. filament_usage=None,
  236. ams_trays={"0": {"tray_uuid": "AAAA"}},
  237. slot_to_tray=None,
  238. tray_remain_start={"0-0": {"remain": 80, "tray_uuid": "AAAA"}},
  239. )
  240. db = AsyncMock()
  241. select_result = MagicMock()
  242. select_result.scalar_one_or_none.return_value = tracking
  243. db.execute = AsyncMock(return_value=select_result)
  244. db.delete = AsyncMock()
  245. db.commit = AsyncMock()
  246. client = AsyncMock()
  247. client.use_spool = AsyncMock()
  248. printer_manager = MagicMock()
  249. printer_manager.get_status.return_value = SimpleNamespace(
  250. # tray_uuid changed -> swap detected
  251. raw_data={"ams": [{"id": 0, "tray": [{"id": 0, "tray_uuid": "CCCC", "remain": 50}]}]}
  252. )
  253. with (
  254. patch("backend.app.services.spoolman_tracking.async_session", lambda: _AsyncCtx(db)),
  255. patch("backend.app.api.routes.settings.get_setting", AsyncMock(return_value="true")),
  256. patch(
  257. "backend.app.services.spoolman_tracking._get_spoolman_client_with_fallback",
  258. AsyncMock(return_value=client),
  259. ),
  260. patch("backend.app.services.spoolman_tracking._get_printer_serial", AsyncMock(return_value="serial")),
  261. patch(
  262. "backend.app.services.spoolman_tracking._resolve_spool_id_via_slot_assignment",
  263. AsyncMock(return_value=7),
  264. ),
  265. patch("backend.app.services.printer_manager.printer_manager", printer_manager),
  266. ):
  267. await report_usage(printer_id=1, archive_id=42)
  268. client.use_spool.assert_not_called()
  269. @pytest.mark.asyncio
  270. async def test_remain_delta_skips_slot_handled_by_3mf(self):
  271. """Mixed coverage: 3MF carried slot 1 (=global tray 0). Remain
  272. delta on the same physical slot must not double-charge it."""
  273. from backend.app.services.spoolman_tracking import report_usage
  274. tracking = SimpleNamespace(
  275. filament_usage=[{"slot_id": 1, "used_g": 50.0}],
  276. ams_trays={"0": {"tray_uuid": "AAAA", "tray_type": "PLA"}},
  277. slot_to_tray=None,
  278. tray_remain_start={"0-0": {"remain": 80, "tray_uuid": "AAAA"}},
  279. )
  280. db = AsyncMock()
  281. select_result = MagicMock()
  282. select_result.scalar_one_or_none.return_value = tracking
  283. db.execute = AsyncMock(return_value=select_result)
  284. db.delete = AsyncMock()
  285. db.commit = AsyncMock()
  286. client = AsyncMock()
  287. client.use_spool = AsyncMock()
  288. client.get_spool = AsyncMock()
  289. printer_manager = MagicMock()
  290. printer_manager.get_status.return_value = SimpleNamespace(
  291. raw_data={"ams": [{"id": 0, "tray": [{"id": 0, "tray_uuid": "AAAA", "remain": 60}]}]}
  292. )
  293. # Make the 3MF path resolve to a spool too, so it actually writes.
  294. async def fake_report_slots(_client, items, *args, **kwargs):
  295. for _slot_id, grams in items:
  296. if grams > 0:
  297. await _client.use_spool(99, grams)
  298. return 1
  299. with (
  300. patch("backend.app.services.spoolman_tracking.async_session", lambda: _AsyncCtx(db)),
  301. patch("backend.app.api.routes.settings.get_setting", AsyncMock(return_value="true")),
  302. patch(
  303. "backend.app.services.spoolman_tracking._get_spoolman_client_with_fallback",
  304. AsyncMock(return_value=client),
  305. ),
  306. patch("backend.app.services.spoolman_tracking._get_printer_serial", AsyncMock(return_value="serial")),
  307. patch(
  308. "backend.app.services.spoolman_tracking._report_spool_usage_for_slots",
  309. AsyncMock(side_effect=fake_report_slots),
  310. ),
  311. patch("backend.app.services.printer_manager.printer_manager", printer_manager),
  312. ):
  313. await report_usage(printer_id=1, archive_id=42)
  314. # Only the 3MF path called use_spool. get_spool (remain-delta path)
  315. # was never reached because slot 0 was already in the handled set.
  316. client.use_spool.assert_awaited_once_with(99, 50.0)
  317. client.get_spool.assert_not_called()
  318. class TestPartialUsageRemainDelta:
  319. """cleanup_tracking → _report_partial_usage must also write the
  320. remain-delta for ABORTED no-3MF prints — same shape as the completion
  321. path, otherwise aborts of "Untitled" prints stay silent."""
  322. @pytest.mark.asyncio
  323. async def test_aborted_no_3mf_writes_remain_delta(self):
  324. """Aborted at remain=70% from start of 90% on a 1000g spool.
  325. Delta = 20% × 1000g = 200g — must be written even though no
  326. 3MF / layer data is available."""
  327. from backend.app.services.spoolman_tracking import _report_partial_usage
  328. tracking = SimpleNamespace(
  329. archive_id=99,
  330. filament_usage=None,
  331. layer_usage=None,
  332. filament_properties=None,
  333. ams_trays={"0": {"tray_uuid": "AAAA"}},
  334. slot_to_tray=None,
  335. tray_remain_start={"0-0": {"remain": 90, "tray_uuid": "AAAA"}},
  336. )
  337. client = AsyncMock()
  338. client.get_spool = AsyncMock(return_value={"id": 7, "filament": {"weight": 1000.0}})
  339. client.use_spool = AsyncMock()
  340. printer_manager = MagicMock()
  341. printer_manager.get_status.return_value = SimpleNamespace(
  342. raw_data={"ams": [{"id": 0, "tray": [{"id": 0, "tray_uuid": "AAAA", "remain": 70}]}]},
  343. layer_num=42,
  344. total_layers=100,
  345. )
  346. with (
  347. patch("backend.app.api.routes.settings.get_setting", AsyncMock(return_value="true")),
  348. patch(
  349. "backend.app.services.spoolman_tracking._get_spoolman_client_with_fallback",
  350. AsyncMock(return_value=client),
  351. ),
  352. patch(
  353. "backend.app.services.spoolman_tracking._get_printer_serial",
  354. AsyncMock(return_value="serial"),
  355. ),
  356. patch(
  357. "backend.app.services.spoolman_tracking._resolve_spool_id_via_slot_assignment",
  358. AsyncMock(return_value=7),
  359. ),
  360. patch("backend.app.services.printer_manager.printer_manager", printer_manager),
  361. ):
  362. await _report_partial_usage(printer_id=1, tracking=tracking)
  363. client.use_spool.assert_awaited_once_with(7, 200.0)
  364. class _AsyncCtx:
  365. """Tiny async-context shim returning a pre-built db mock; mirrors
  366. async_session()'s ``async with`` interface."""
  367. def __init__(self, db):
  368. self._db = db
  369. async def __aenter__(self):
  370. return self._db
  371. async def __aexit__(self, *_):
  372. return False