steamloop 1.2.1__tar.gz → 1.2.2__tar.gz

{steamloop-1.2.1 → steamloop-1.2.2}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: steamloop
-Version: 1.2.1
+Version: 1.2.2
 Summary: Local control for choochoo based thermostats
 Author-email: "J. Nick Koston" <nick@koston.org>
 License-Expression: Apache-2.0

{steamloop-1.2.1 → steamloop-1.2.2}/pyproject.toml

@@ -4,7 +4,7 @@ requires = [ "setuptools>=77.0.3" ]
 
 [project]
 name = "steamloop"
-version = "1.2.1"
+version = "1.2.2"
 description = "Local control for choochoo based thermostats"
 readme = "README.md"
 license = "Apache-2.0"

{steamloop-1.2.1 → steamloop-1.2.2}/src/steamloop/__init__.py

@@ -1,6 +1,6 @@
 """Local control for thermostat devices over mTLS."""
 
-__version__ = "1.2.1"
+__version__ = "1.2.2"
 
 from .connection import ThermostatConnection, load_pairing, save_pairing
 from .const import DEFAULT_PORT, FanMode, HoldType, ZoneMode

{steamloop-1.2.1 → steamloop-1.2.2}/src/steamloop/cli.py

@@ -86,14 +86,28 @@ def _print_help(active_zone: str) -> None:
     print("  quit                        Disconnect and exit")
 
 
+def _valid_temp(temp: str) -> bool:
+    """Return True if temp parses as a number; print an error otherwise."""
+    try:
+        float(temp)
+    except ValueError:
+        print(f"Invalid temperature: {temp!r}. Expected a number.")
+        return False
+    return True
+
+
 def _cmd_heat(conn: ThermostatConnection, active_zone: str, temp: str) -> None:
     """Handle the heat command."""
+    if not _valid_temp(temp):
+        return
     print(f"Setting heat setpoint to {temp} (zone {active_zone})")
     conn.set_temperature_setpoint(active_zone, heat_setpoint=temp)
 
 
 def _cmd_cool(conn: ThermostatConnection, active_zone: str, temp: str) -> None:
     """Handle the cool command."""
+    if not _valid_temp(temp):
+        return
     print(f"Setting cool setpoint to {temp} (zone {active_zone})")
     conn.set_temperature_setpoint(active_zone, cool_setpoint=temp)
 
@@ -131,10 +145,11 @@ def _handle_command(
         _cmd_cool(conn, active_zone, parts[1])
 
     elif parts[0] == "setpoint" and len(parts) >= 3:
-        print(f"Setting heat={parts[1]}, cool={parts[2]} (zone {active_zone})")
-        conn.set_temperature_setpoint(
-            active_zone, heat_setpoint=parts[1], cool_setpoint=parts[2]
-        )
+        if _valid_temp(parts[1]) and _valid_temp(parts[2]):
+            print(f"Setting heat={parts[1]}, cool={parts[2]} (zone {active_zone})")
+            conn.set_temperature_setpoint(
+                active_zone, heat_setpoint=parts[1], cool_setpoint=parts[2]
+            )
 
     elif parts[0] == "hold" and len(parts) >= 2:
         ht = _HOLD_MAP.get(parts[1])

{steamloop-1.2.1 → steamloop-1.2.2}/src/steamloop.egg-info/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: steamloop
-Version: 1.2.1
+Version: 1.2.2
 Summary: Local control for choochoo based thermostats
 Author-email: "J. Nick Koston" <nick@koston.org>
 License-Expression: Apache-2.0

{steamloop-1.2.1 → steamloop-1.2.2}/tests/test_cli.py

@@ -101,6 +101,31 @@ def test_command_setpoint(capsys: pytest.CaptureFixture[str]) -> None:
     )
 
 
+def test_command_heat_invalid_temp(capsys: pytest.CaptureFixture[str]) -> None:
+    """A non-numeric heat temp is rejected without calling the connection."""
+    conn = _make_mock_conn()
+    result = _handle_command(conn, "heat abc", ["heat", "abc"], "1")
+    assert result == "1"
+    conn.set_temperature_setpoint.assert_not_called()
+    assert "Invalid temperature" in capsys.readouterr().out
+
+
+def test_command_cool_invalid_temp(capsys: pytest.CaptureFixture[str]) -> None:
+    """A non-numeric cool temp is rejected without calling the connection."""
+    conn = _make_mock_conn()
+    _handle_command(conn, "cool hot", ["cool", "hot"], "1")
+    conn.set_temperature_setpoint.assert_not_called()
+    assert "Invalid temperature" in capsys.readouterr().out
+
+
+def test_command_setpoint_invalid_temp(capsys: pytest.CaptureFixture[str]) -> None:
+    """A bad value in setpoint rejects the whole command, no call is made."""
+    conn = _make_mock_conn()
+    _handle_command(conn, "setpoint 68 cold", ["setpoint", "68", "cold"], "1")
+    conn.set_temperature_setpoint.assert_not_called()
+    assert "Invalid temperature" in capsys.readouterr().out
+
+
 def test_command_hold() -> None:
     conn = _make_mock_conn()
     _handle_command(conn, "hold manual", ["hold", "manual"], "1")

{steamloop-1.2.1 → steamloop-1.2.2}/tests/test_dunder_main.py

@@ -7,7 +7,7 @@ import sys
 
 
 def test_can_run_as_python_module() -> None:
-    result = subprocess.run(  # noqa: S603
+    result = subprocess.run(
         [sys.executable, "-m", "steamloop", "--help"],
         check=True,
         capture_output=True,