remove unnecessary tests

Co-authored-by: Jevgeni Kiski <yozik04@gmail.com>

Author: CamW

tests/hardware/evo/test_module_pgm_action.py

@@ -75,81 +75,6 @@ def test_build_trigger_module_pgm_pulse_on():
     assert raw == unhexlify("a4170004000000000004000000000000000000000000c3")
 
 
-def test_build_deactivate_pgm1_steady_off():
-    """Deactivate PGM 1 steady off (off=1): byte 6 = 0x01, checksum = 0xC0"""
-    raw = _build(4, ["off", "release", "release", "release"])
-
-    assert raw[0] == 0xA4
-    assert raw[6] == 0x01   # off
-    assert raw[7] == 0x00
-    assert raw[8] == 0x00
-    assert raw[9] == 0x00
-    assert raw[-1] == _checksum(raw[:-1])
-
-
-def test_build_activate_pgm2_steady_on():
-    """Activate PGM 2 steady on (on=3): byte 7 = 0x03"""
-    raw = _build(4, ["release", "on", "release", "release"])
-
-    assert raw[6] == 0x00
-    assert raw[7] == 0x03   # on
-    assert raw[8] == 0x00
-    assert raw[9] == 0x00
-    assert raw[-1] == _checksum(raw[:-1])
-
-
-def test_build_packet_length_always_23():
-    """Packet length byte is always 23 (0x17) regardless of command."""
-    for pgm_idx in range(4):
-        commands = ["release"] * 4
-        commands[pgm_idx] = "on"
-        raw = _build(4, commands)
-        assert len(raw) == 23
-        assert raw[1] == 0x17
-
-
-def test_build_command_byte_is_0xa4():
-    """First byte is always 0xA4."""
-    raw = _build(4, ["on", "release", "release", "release"])
-    assert raw[0] == 0xA4
-
-
-def test_build_module_address_at_byte3():
-    """Module address is encoded at byte 3."""
-    raw = _build(4, ["on", "release", "release", "release"])
-    assert raw[3] == 0x04
-
-    raw2 = _build(7, ["on", "release", "release", "release"])
-    assert raw2[3] == 0x07
-
-
-def test_build_non_command_bytes_are_zero():
-    """Bytes 2, 4, 5 and the 12 trailing padding bytes are always zero."""
-    raw = _build(4, ["on", "release", "release", "release"])
-    assert raw[2] == 0x00           # _not_used0
-    assert raw[4:6] == b"\x00\x00"  # _not_used1
-    assert raw[10:22] == b"\x00" * 12  # _not_used2
-
-
-def test_build_only_target_pgm_byte_nonzero():
-    """Only the byte for the targeted PGM output is non-zero."""
-    for pgm_idx in range(4):
-        commands = ["release"] * 4
-        commands[pgm_idx] = "on"
-        raw = _build(4, commands)
-        for i, cmd_byte in enumerate(raw[6:10]):
-            if i == pgm_idx:
-                assert cmd_byte == 3  # on
-            else:
-                assert cmd_byte == 0  # release
-
-
-def test_build_checksum_correct():
-    """Checksum is sum of all preceding bytes mod 256."""
-    raw = _build(4, ["on_override", "release", "release", "release"])
-    assert raw[-1] == _checksum(raw[:-1])
-
-
 # ---------------------------------------------------------------------------
 # PerformModulePGMActionResponse — PARSE tests
 # ---------------------------------------------------------------------------