python-openevse-http 0.3.3__py3-none-any.whl → 0.3.5__py3-none-any.whl

openevsehttp/client.py

@@ -6,7 +6,6 @@ import asyncio
 import inspect
 import json
 import logging
-import re
 import threading
 from collections.abc import Callable, Mapping
 from typing import Any
@@ -31,6 +30,7 @@ from .exceptions import (
 from .managers import ManagersMixin
 from .properties import PropertiesMixin
 from .sensors import SensorsMixin
+from .utils import get_awesome_version
 from .websocket import (
     SIGNAL_CONNECTION_STATE,
     STATE_CONNECTED,
@@ -438,41 +438,21 @@ class OpenEVSE(CommandsMixin, ManagersMixin, SensorsMixin, PropertiesMixin):
         """Return bool if minimum version is met."""
         if "version" not in self._config:
             # Throw warning if we can't find the version
-            _LOGGER.warning("Unable to find firmware version.")
+            _LOGGER.debug("Unable to find firmware version.")
             return False
         cutoff = AwesomeVersion(min_version)
-        current = ""
         limit = ""
         if max_version != "":
             limit = AwesomeVersion(max_version)
 
-        firmware_filtered = None
-        firmware_search = re.search(r"\d+\.\d+\.\d+", self._config["version"])
-        if firmware_search:
-            firmware_filtered = firmware_search.group(0)
-
-        if firmware_filtered is None:
-            _LOGGER.warning(
-                "Non-standard versioning string: %s", self._config["version"]
+        current = get_awesome_version(self._config["version"])
+        if current.strategy == "unknown":
+            _LOGGER.debug(
+                "Non-semver firmware version detected: %s",
+                self._config["version"],
             )
-            _LOGGER.debug("Non-semver firmware version detected.")
             return False
 
-        _LOGGER.debug("Detected firmware: %s", self._config["version"])
-        _LOGGER.debug("Filtered firmware: %s", firmware_filtered)
-
-        if "dev" in self._config["version"]:
-            value = self._config["version"]
-            _LOGGER.debug("Stripping 'dev' from version.")
-            value = value.split(".")
-            value = ".".join(value[0:3])
-        elif "master" in self._config["version"]:
-            value = "dev"
-        else:
-            value = firmware_filtered
-
-        current = AwesomeVersion(value)
-
         if limit:
             try:
                 if cutoff <= current < limit:

openevsehttp/commands.py

@@ -12,8 +12,9 @@ from aiohttp.client_exceptions import ContentTypeError, ServerTimeoutError
 from awesomeversion import AwesomeVersion
 from awesomeversion.exceptions import AwesomeVersionCompareException
 
-from .const import MAX_AMPS, MIN_AMPS, RAPI_ERRORS, divert_mode
+from .const import MAX_AMPS, MIN_AMPS, RAPI_ERRORS, SUCCESS_ANSWERS, divert_mode
 from .exceptions import UnknownError, UnsupportedFeature
+from .utils import get_awesome_version
 
 _LOGGER = logging.getLogger(__name__)
 
@@ -67,7 +68,7 @@ class CommandsMixin:
         response = await self.process_request(url=url, method="post", data=data)
         response = self._normalize_response(response)
         msg = response.get("msg") if isinstance(response, Mapping) else None
-        if msg not in ["OK", "done", "no change"]:
+        if msg not in SUCCESS_ANSWERS:
             _LOGGER.error("Problem issuing command: %s", response)
             raise UnknownError
 
@@ -94,11 +95,10 @@ class CommandsMixin:
         response = await self.process_request(url=url, method="post", data=data)
         _LOGGER.debug("divert_mode response: %s", response)
         normalized_response = self._normalize_response(response)
-        if isinstance(normalized_response, dict) and normalized_response.get("msg") in [
-            "OK",
-            "done",
-            "no change",
-        ]:
+        if (
+            isinstance(normalized_response, dict)
+            and normalized_response.get("msg") in SUCCESS_ANSWERS
+        ):
             self._config["divert_enabled"] = mode
         return normalized_response
 
@@ -171,11 +171,10 @@ class CommandsMixin:
             response = await self.process_request(url=url, method="patch")
             response = self._normalize_response(response)
             _LOGGER.debug("Toggle response: %s", response)
-            if not isinstance(response, Mapping) or response.get("msg") not in [
-                "OK",
-                "done",
-                "no change",
-            ]:
+            if (
+                not isinstance(response, Mapping)
+                or response.get("msg") not in SUCCESS_ANSWERS
+            ):
                 _LOGGER.error("Problem toggling override: %s", response)
                 raise RuntimeError(f"Failed to toggle override: {response}")
         else:
@@ -209,7 +208,7 @@ class CommandsMixin:
         response = self._normalize_response(response)
         msg = response.get("msg") if isinstance(response, Mapping) else None
         _LOGGER.debug("Clear override response: %s", msg)
-        if msg not in ["OK", "done", "no change"]:
+        if msg not in SUCCESS_ANSWERS:
             _LOGGER.error("Problem clearing override: %s", response)
             raise RuntimeError(f"Failed to clear override: {response}")
 
@@ -235,11 +234,10 @@ class CommandsMixin:
             _LOGGER.debug("Setting current limit to %s", amps)
             response = await self.set_override(charge_current=amps)
             _LOGGER.debug("Set current response: %s", response)
-            if not isinstance(response, Mapping) or response.get("msg") not in [
-                "OK",
-                "done",
-                "no change",
-            ]:
+            if (
+                not isinstance(response, Mapping)
+                or response.get("msg") not in SUCCESS_ANSWERS
+            ):
                 _LOGGER.error("Problem setting current limit: %s", response)
                 raise UnknownError
 
@@ -274,7 +272,7 @@ class CommandsMixin:
         response = self._normalize_response(response)
         _LOGGER.debug("service response: %s", response)
         msg = response.get("msg") if isinstance(response, Mapping) else None
-        if msg not in ["OK", "done", "no change"]:
+        if msg not in SUCCESS_ANSWERS:
             _LOGGER.error("Problem issuing command: %s", response)
             raise UnknownError
 
@@ -357,29 +355,17 @@ class CommandsMixin:
         """Return the latest firmware version."""
         if "version" not in self._config:
             # Throw warning if we can't find the version
-            _LOGGER.warning("Unable to find firmware version.")
+            _LOGGER.debug("Unable to find firmware version.")
             return None
         base_url = "https://api.github.com/repos/OpenEVSE/"
         url = None
         method = "get"
 
         cutoff = AwesomeVersion("3.0.0")
-        current = ""
-
         _LOGGER.debug("Detected firmware: %s", self._config["version"])
 
-        if "dev" in self._config["version"]:
-            value = self._config["version"]
-            _LOGGER.debug("Stripping 'dev' from version.")
-            value = value.split(".")
-            value = ".".join(value[0:3])
-        elif "master" in self._config["version"]:
-            value = "dev"
-        else:
-            value = self._config["version"]
-
-        _LOGGER.debug("Using version: %s", value)
-        current = AwesomeVersion(value)
+        current = get_awesome_version(self._config["version"])
+        _LOGGER.debug("Using version: %s", current)
 
         try:
             if current >= cutoff:
@@ -387,7 +373,7 @@ class CommandsMixin:
             else:
                 url = f"{base_url}ESP8266_WiFi_v2.x/releases/latest"
         except AwesomeVersionCompareException:
-            _LOGGER.warning("Non-semver firmware version detected.")
+            _LOGGER.debug("Non-semver firmware version detected.")
             return None
 
         try:
@@ -459,7 +445,7 @@ class CommandsMixin:
         response = await self.process_request(url=url, method="post", data=data)
         response = self._normalize_response(response)
         msg = response.get("msg") if isinstance(response, Mapping) else None
-        if msg not in ["OK", "done", "no change"]:
+        if msg not in SUCCESS_ANSWERS:
             _LOGGER.error("Problem issuing command: %s", response)
             raise UnknownError
 
@@ -481,11 +467,7 @@ class CommandsMixin:
             res_lower = response.lower()
             if "divert" in res_lower and "changed" in res_lower:
                 success = True
-        elif isinstance(response, dict) and response.get("msg") in [
-            "OK",
-            "done",
-            "no change",
-        ]:
+        elif isinstance(response, dict) and response.get("msg") in SUCCESS_ANSWERS:
             success = True
 
         if not success:
@@ -508,7 +490,7 @@ class CommandsMixin:
         response = await self.process_request(url=url, method="post", rapi=data)
         response = self._normalize_response(response)
         msg = response.get("msg") if isinstance(response, Mapping) else None
-        if msg not in ["OK", "done", "no change", "Current Shaper state changed"]:
+        if msg not in SUCCESS_ANSWERS and msg != "Current Shaper state changed":
             _LOGGER.error("Problem issuing command: %s", response)
             raise UnknownError
 

openevsehttp/const.py

@@ -60,3 +60,5 @@ RAPI_ERRORS = [
     "RAPI_RESPONSE_BLOCKED",
     "RAPI_RESPONSE_INVALID_COMMAND",
 ]
+
+SUCCESS_ANSWERS = ["OK", "done", "no change", "Created", "Updated", "Deleted"]

openevsehttp/properties.py

@@ -9,6 +9,7 @@ from typing import Any
 
 from .const import MAX_AMPS, MIN_AMPS, states
 from .exceptions import UnsupportedFeature
+from .utils import normalize_version
 
 _LOGGER = logging.getLogger(__name__)
 
@@ -130,10 +131,8 @@ class PropertiesMixin:
     def wifi_firmware(self) -> str | None:
         """Return the ESP firmware version."""
         value = self._config.get("version")
-        if value is not None and "dev" in value:
-            _LOGGER.debug("Stripping 'dev' from version.")
-            value = value.split(".")
-            value = ".".join(value[0:3])
+        if value is not None:
+            value = normalize_version(value)
         return value
 
     @property

openevsehttp/utils.py

@@ -0,0 +1,29 @@
+"""Utility functions for python-openevse-http."""
+
+import logging
+import re
+
+from awesomeversion import AwesomeVersion
+
+_LOGGER = logging.getLogger(__name__)
+
+
+def normalize_version(version: str) -> str:
+    """Normalize the version string to strip 'dev' tag."""
+    if "dev" in version:
+        _LOGGER.debug("Stripping 'dev' from version.")
+        value = version.split(".")
+        return ".".join(value[0:3])
+    return version
+
+
+def get_awesome_version(version: str) -> AwesomeVersion:
+    """Parse and normalize the version string, returning an AwesomeVersion."""
+    if "master" in version:
+        version = "dev"
+    value = normalize_version(version)
+    if "dev" not in version:
+        firmware_search = re.search(r"\d+\.\d+\.\d+", value)
+        if firmware_search:
+            value = firmware_search.group(0)
+    return AwesomeVersion(value)

{python_openevse_http-0.3.3.dist-info → python_openevse_http-0.3.5.dist-info}/METADATA

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: python_openevse_http
-Version: 0.3.3
+Version: 0.3.5
 Summary: Python wrapper for OpenEVSE HTTP API
 Home-page: https://github.com/firstof9/python-openevse-http
 Download-URL: https://github.com/firstof9/python-openevse-http

python_openevse_http-0.3.5.dist-info/RECORD

@@ -0,0 +1,16 @@
+openevsehttp/__init__.py,sha256=I6a1mjOZHYiWb_qfCuDuFLOOncrkkB_7uwybtOIujfY,1165
+openevsehttp/__main__.py,sha256=EHmSdT7GjAVvHQxvLBTjZXsj_V5SB6B2_kpgUAT7mPM,146
+openevsehttp/client.py,sha256=JBAC1jJGdzOabVAqHv8x6EJDVjhaW4t7Iwqn4lDWhwE,17502
+openevsehttp/commands.py,sha256=lANhgVhtJJlQxLwFlMrxk3DrmnY2bXK5h4z3o9o6ZEk,20617
+openevsehttp/const.py,sha256=y-2hGv_PCal_-VCSGC7IIyzQYtfeVdq3MjOhBWIdZvc,1440
+openevsehttp/exceptions.py,sha256=bqz-tHTW1AYJMKcm0s5M6z5tA6XZgjnCiBLW1XrZ_70,672
+openevsehttp/managers.py,sha256=kEX1ZD9u-FY0UEZJsxeFEGBSGzSlkbBc0kmxCiMJtJw,5373
+openevsehttp/properties.py,sha256=QVSyn_5a7vI1b4TdnnToRdw6veVCfnp7a19VYit95hg,17107
+openevsehttp/sensors.py,sha256=sJP2FPnU1Lk5S3VUyFT14JM1nKEBQPsjl-DiZI-pZrs,4673
+openevsehttp/utils.py,sha256=e3HH_jwZgb1iBWJgIoMOM0JPrQNwXyVdOx5vTWOh4T0,858
+openevsehttp/websocket.py,sha256=Mi_WFmlT3-9i6bbHIN6ua09SD8CpIle2vRXB3HyWzmM,10066
+python_openevse_http-0.3.5.dist-info/licenses/LICENSE,sha256=hSB6TOQ7rmwSGb6XzqRjDGMvmUj5_GlacqQin3tegtA,11341
+python_openevse_http-0.3.5.dist-info/METADATA,sha256=3VxDib4FCHGZ3QODauCoWzXQiPggRx_7q8yRvCwa7G8,4363
+python_openevse_http-0.3.5.dist-info/WHEEL,sha256=aeYiig01lYGDzBgS8HxWXOg3uV61G9ijOsup-k9o1sk,91
+python_openevse_http-0.3.5.dist-info/top_level.txt,sha256=u8RUkoEIE33Cjn6gmqiEoVpZ0VZ59WJ3FXBwwOg0CPE,13
+python_openevse_http-0.3.5.dist-info/RECORD,,

{python_openevse_http-0.3.3.dist-info → python_openevse_http-0.3.5.dist-info}/WHEEL

@@ -1,5 +1,5 @@
 Wheel-Version: 1.0
-Generator: setuptools (79.0.1)
+Generator: setuptools (82.0.1)
 Root-Is-Purelib: true
 Tag: py3-none-any
 

python_openevse_http-0.3.3.dist-info/RECORD

@@ -1,15 +0,0 @@
-openevsehttp/__init__.py,sha256=I6a1mjOZHYiWb_qfCuDuFLOOncrkkB_7uwybtOIujfY,1165
-openevsehttp/__main__.py,sha256=EHmSdT7GjAVvHQxvLBTjZXsj_V5SB6B2_kpgUAT7mPM,146
-openevsehttp/client.py,sha256=JYReKIJLbQbymq_o19-oZgxMyxHwZB6Sc-M6unVfMOo,18208
-openevsehttp/commands.py,sha256=yQMsDHTxPJpwzYB9V4LXhzmKIhFF7zFv0NtpxivBn94,21109
-openevsehttp/const.py,sha256=VSc5Xt-KFenft0rT6ql9whCD7J9g_YvjLYU_PX706eE,1360
-openevsehttp/exceptions.py,sha256=bqz-tHTW1AYJMKcm0s5M6z5tA6XZgjnCiBLW1XrZ_70,672
-openevsehttp/managers.py,sha256=kEX1ZD9u-FY0UEZJsxeFEGBSGzSlkbBc0kmxCiMJtJw,5373
-openevsehttp/properties.py,sha256=Lo2p6WfPuhpaEcIZ0OeAxinHSa0G8Sv2wLZ-dPe7rjo,17181
-openevsehttp/sensors.py,sha256=sJP2FPnU1Lk5S3VUyFT14JM1nKEBQPsjl-DiZI-pZrs,4673
-openevsehttp/websocket.py,sha256=Mi_WFmlT3-9i6bbHIN6ua09SD8CpIle2vRXB3HyWzmM,10066
-python_openevse_http-0.3.3.dist-info/licenses/LICENSE,sha256=hSB6TOQ7rmwSGb6XzqRjDGMvmUj5_GlacqQin3tegtA,11341
-python_openevse_http-0.3.3.dist-info/METADATA,sha256=jRmVpNx2qlkUs3Py45EPGjwZHT89vzWJzVB3grOl8Iw,4363
-python_openevse_http-0.3.3.dist-info/WHEEL,sha256=SmOxYU7pzNKBqASvQJ7DjX3XGUF92lrGhMb3R6_iiqI,91
-python_openevse_http-0.3.3.dist-info/top_level.txt,sha256=u8RUkoEIE33Cjn6gmqiEoVpZ0VZ59WJ3FXBwwOg0CPE,13
-python_openevse_http-0.3.3.dist-info/RECORD,,