mirror of
https://github.com/openglow-org/forgefirm.git
synced 2026-09-27 16:51:12 -07:00
Leave the lens to the cloud client in cloud mode
cloud.verdict-hold printed to completion and then failed its hand-back on head/z_mode=0. That job's header carried ZSmd 0, gfhardware's set_mode_from_puls turns that into z_mode 0, and z_mode 0 is full step (lenshome.c): the cloud client had set the lens the way the pulse file it was playing asked. Nothing was left behind - the machine did what the job said. The baseline already skips nine attributes in cloud mode, for the reason written above the list: the cloud client sets its own values for them from every pulse header, and forcing the GRBL values under it would be the baseline configuring another controller's machine. head/z_current and head/z_mode are the same thing and were not on the list. They are now, in cloud mode only; in GRBL mode the pair is still checked against 1 and 1, which is the half of this an exemption could quietly break, so a test holds it. Host-proven: 89 baseline unit tests.
This commit is contained in:
@@ -400,6 +400,23 @@ class TransientNotLeftoverTests(BaselineTests):
|
||||
self.assertEqual(items[0].found, "1014")
|
||||
|
||||
|
||||
class CloudOwnedSysfsTests(unittest.TestCase):
|
||||
"""In cloud mode the cloud client configures the machine from the
|
||||
pulse header it is playing, the lens included: z_mode comes from
|
||||
ZSmd through gfhardware's set_mode_from_puls. Handing the GRBL values
|
||||
back under it would be the baseline configuring another controller's
|
||||
machine. In GRBL mode the pair is checked as before."""
|
||||
|
||||
def test_the_z_pair_is_the_cloud_clients(self):
|
||||
for attr in ("head/z_current", "head/z_mode"):
|
||||
self.assertIn(attr, baseline.GRBL_CONTROLLER_SYSFS)
|
||||
|
||||
def test_the_z_pair_is_still_checked_in_grbl_mode(self):
|
||||
fixed = dict(baseline.fixed_sysfs())
|
||||
self.assertEqual(fixed.get("head/z_current"), "1")
|
||||
self.assertEqual(fixed.get("head/z_mode"), "1")
|
||||
|
||||
|
||||
class PositionDeadbandTests(unittest.TestCase):
|
||||
"""The counters count steps and a controller's own return lands within
|
||||
a few hundredths of a millimeter, not on the step: a difference that
|
||||
|
||||
Reference in New Issue
Block a user