aiobmsble 0.2.1__tar.gz → 0.2.3__tar.gz

{aiobmsble-0.2.1/aiobmsble.egg-info → aiobmsble-0.2.3}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: aiobmsble
-Version: 0.2.1
+Version: 0.2.3
 Summary: Asynchronous Python library to query battery management systems via Bluetooth Low Energy.
 Author: Patrick Loschmidt
 Maintainer: Patrick Loschmidt
@@ -19,7 +19,7 @@ Requires-Python: >=3.12
 Description-Content-Type: text/markdown
 License-File: LICENSE
 Requires-Dist: bleak~=1.1.0
-Requires-Dist: bleak-retry-connector~=4.0.2
+Requires-Dist: bleak-retry-connector>=4.0.2
 Requires-Dist: asyncio
 Requires-Dist: logging
 Requires-Dist: statistics

{aiobmsble-0.2.1 → aiobmsble-0.2.3}/aiobmsble/__init__.py

@@ -1,4 +1,8 @@
-"""Package for battery management systems (BMS) via Bluetooth LE."""
+"""Package for battery management systems (BMS) via Bluetooth LE (aiobmsble).
+
+Project: aiobmsble, https://pypi.org/p/aiobmsble/
+License: Apache-2.0, http://www.apache.org/licenses/
+"""
 
 from collections.abc import Callable
 from enum import IntEnum

{aiobmsble-0.2.1 → aiobmsble-0.2.3}/aiobmsble/__main__.py

@@ -1,4 +1,8 @@
-"""Example function for package usage."""
+"""Example script for aiobmsble package usage.
+
+Project: aiobmsble, https://pypi.org/p/aiobmsble/
+License: Apache-2.0, http://www.apache.org/licenses/
+"""
 
 import argparse
 import asyncio

{aiobmsble-0.2.1 → aiobmsble-0.2.3}/aiobmsble/basebms.py

@@ -1,4 +1,8 @@
-"""Base class defintion for battery management systems (BMS)."""
+"""Base class defintion for battery management systems (BMS).
+
+Project: aiobmsble, https://pypi.org/p/aiobmsble/
+License: Apache-2.0, http://www.apache.org/licenses/
+"""
 
 from abc import ABC, abstractmethod
 import asyncio
@@ -79,6 +83,11 @@ class BaseBMS(ABC):
         self._data: bytearray = bytearray()
         self._data_event: Final[asyncio.Event] = asyncio.Event()
 
+    @classmethod
+    def get_bms_module(cls) -> str:
+        """Return BMS module name, e.g. aiobmsble.bms.dummy_bms."""
+        return cls.__module__
+
     @staticmethod
     @abstractmethod
     def matcher_dict_list() -> list[MatcherPattern]:

aiobmsble-0.2.3/aiobmsble/bms/__init__.py

@@ -0,0 +1,5 @@
+"""Package for battery management systems (BMS) plugins.
+
+Project: aiobmsble, https://pypi.org/p/aiobmsble/
+License: Apache-2.0, http://www.apache.org/licenses/
+"""

{aiobmsble-0.2.1 → aiobmsble-0.2.3}/aiobmsble/bms/abc_bms.py

@@ -1,4 +1,8 @@
-"""Module to support ABC BMS."""
+"""Module to support ABC BMS.
+
+Project: aiobmsble, https://pypi.org/p/aiobmsble/
+License: Apache-2.0, http://www.apache.org/licenses/
+"""
 
 import contextlib
 from typing import Final

{aiobmsble-0.2.1 → aiobmsble-0.2.3}/aiobmsble/bms/ant_bms.py

@@ -1,4 +1,8 @@
-"""Module to support ANT BMS."""
+"""Module to support ANT BMS.
+
+Project: aiobmsble, https://pypi.org/p/aiobmsble/
+License: Apache-2.0, http://www.apache.org/licenses/
+"""
 
 from typing import Final
 

{aiobmsble-0.2.1 → aiobmsble-0.2.3}/aiobmsble/bms/braunpwr_bms.py

@@ -1,4 +1,8 @@
-"""Module to support Braun Power BMS."""
+"""Module to support Braun Power BMS.
+
+Project: aiobmsble, https://pypi.org/p/aiobmsble/
+License: Apache-2.0, http://www.apache.org/licenses/
+"""
 
 from typing import Final
 

{aiobmsble-0.2.1 → aiobmsble-0.2.3}/aiobmsble/bms/cbtpwr_bms.py

@@ -1,4 +1,8 @@
-"""Module to support CBT Power Smart BMS."""
+"""Module to support CBT Power Smart BMS.
+
+Project: aiobmsble, https://pypi.org/p/aiobmsble/
+License: Apache-2.0, http://www.apache.org/licenses/
+"""
 
 from typing import Final
 

{aiobmsble-0.2.1 → aiobmsble-0.2.3}/aiobmsble/bms/cbtpwr_vb_bms.py

@@ -1,4 +1,8 @@
-"""Module to support CBT Power VB series BMS."""
+"""Module to support CBT Power VB series BMS.
+
+Project: aiobmsble, https://pypi.org/p/aiobmsble/
+License: Apache-2.0, http://www.apache.org/licenses/
+"""
 
 from string import hexdigits
 from typing import Final

{aiobmsble-0.2.1 → aiobmsble-0.2.3}/aiobmsble/bms/daly_bms.py

@@ -1,4 +1,8 @@
-"""Module to support Daly Smart BMS."""
+"""Module to support Daly Smart BMS.
+
+Project: aiobmsble, https://pypi.org/p/aiobmsble/
+License: Apache-2.0, http://www.apache.org/licenses/
+"""
 
 from typing import Final
 

{aiobmsble-0.2.1 → aiobmsble-0.2.3}/aiobmsble/bms/dpwrcore_bms.py

@@ -1,4 +1,8 @@
-"""Module to support D-powercore Smart BMS."""
+"""Module to support D-powercore Smart BMS.
+
+Project: aiobmsble, https://pypi.org/p/aiobmsble/
+License: Apache-2.0, http://www.apache.org/licenses/
+"""
 
 from enum import IntEnum
 from string import hexdigits

{aiobmsble-0.2.1 → aiobmsble-0.2.3}/aiobmsble/bms/dummy_bms.py

@@ -1,4 +1,8 @@
-"""Module to support Dummy BMS."""
+"""Module to support Dummy BMS.
+
+Project: aiobmsble, https://pypi.org/p/aiobmsble/
+License: Apache-2.0, http://www.apache.org/licenses/
+"""
 
 from bleak.backends.characteristic import BleakGATTCharacteristic
 from bleak.backends.device import BLEDevice

{aiobmsble-0.2.1 → aiobmsble-0.2.3}/aiobmsble/bms/ecoworthy_bms.py

@@ -1,4 +1,8 @@
-"""Module to support ECO-WORTHY BMS."""
+"""Module to support ECO-WORTHY BMS.
+
+Project: aiobmsble, https://pypi.org/p/aiobmsble/
+License: Apache-2.0, http://www.apache.org/licenses/
+"""
 
 import asyncio
 from typing import Final

{aiobmsble-0.2.1 → aiobmsble-0.2.3}/aiobmsble/bms/ective_bms.py

@@ -1,4 +1,8 @@
-"""Module to support Ective BMS."""
+"""Module to support Ective BMS.
+
+Project: aiobmsble, https://pypi.org/p/aiobmsble/
+License: Apache-2.0, http://www.apache.org/licenses/
+"""
 
 import asyncio
 from string import hexdigits

{aiobmsble-0.2.1 → aiobmsble-0.2.3}/aiobmsble/bms/ej_bms.py

@@ -1,4 +1,8 @@
-"""Module to support E&J Technology BMS."""
+"""Module to support E&J Technology BMS.
+
+Project: aiobmsble, https://pypi.org/p/aiobmsble/
+License: Apache-2.0, http://www.apache.org/licenses/
+"""
 
 from enum import IntEnum
 from string import hexdigits

{aiobmsble-0.2.1 → aiobmsble-0.2.3}/aiobmsble/bms/felicity_bms.py

@@ -1,4 +1,8 @@
-"""Module to support Felicity BMS."""
+"""Module to support Felicity BMS.
+
+Project: aiobmsble, https://pypi.org/p/aiobmsble/
+License: Apache-2.0, http://www.apache.org/licenses/
+"""
 
 from collections.abc import Callable
 from json import JSONDecodeError, loads

{aiobmsble-0.2.1 → aiobmsble-0.2.3}/aiobmsble/bms/jbd_bms.py

@@ -1,4 +1,8 @@
-"""Module to support JBD Smart BMS."""
+"""Module to support JBD Smart BMS.
+
+Project: aiobmsble, https://pypi.org/p/aiobmsble/
+License: Apache-2.0, http://www.apache.org/licenses/
+"""
 
 from typing import Final
 

{aiobmsble-0.2.1 → aiobmsble-0.2.3}/aiobmsble/bms/jikong_bms.py

@@ -1,4 +1,8 @@
-"""Module to support Jikong Smart BMS."""
+"""Module to support Jikong Smart BMS.
+
+Project: aiobmsble, https://pypi.org/p/aiobmsble/
+License: Apache-2.0, http://www.apache.org/licenses/
+"""
 
 import asyncio
 from typing import Final

{aiobmsble-0.2.1 → aiobmsble-0.2.3}/aiobmsble/bms/neey_bms.py

@@ -1,4 +1,8 @@
-"""Module to support Neey Smart BMS."""
+"""Module to support Neey Smart BMS.
+
+Project: aiobmsble, https://pypi.org/p/aiobmsble/
+License: Apache-2.0, http://www.apache.org/licenses/
+"""
 
 from collections.abc import Callable
 from struct import unpack_from

{aiobmsble-0.2.1 → aiobmsble-0.2.3}/aiobmsble/bms/ogt_bms.py

@@ -1,4 +1,8 @@
-"""Module to support Offgridtec Smart Pro BMS."""
+"""Module to support Offgridtec Smart Pro BMS.
+
+Project: aiobmsble, https://pypi.org/p/aiobmsble/
+License: Apache-2.0, http://www.apache.org/licenses/
+"""
 
 from collections.abc import Callable
 from string import digits, hexdigits

{aiobmsble-0.2.1 → aiobmsble-0.2.3}/aiobmsble/bms/pro_bms.py

@@ -1,4 +1,8 @@
-"""Module to support Pro BMS."""
+"""Module to support Pro BMS.
+
+Project: aiobmsble, https://pypi.org/p/aiobmsble/
+License: Apache-2.0, http://www.apache.org/licenses/
+"""
 
 import asyncio
 from typing import Final

{aiobmsble-0.2.1 → aiobmsble-0.2.3}/aiobmsble/bms/redodo_bms.py

@@ -1,4 +1,8 @@
-"""Module to support Redodo BMS."""
+"""Module to support Redodo BMS.
+
+Project: aiobmsble, https://pypi.org/p/aiobmsble/
+License: Apache-2.0, http://www.apache.org/licenses/
+"""
 
 from typing import Final
 

{aiobmsble-0.2.1 → aiobmsble-0.2.3}/aiobmsble/bms/renogy_bms.py

@@ -1,4 +1,7 @@
-"""Module to support Renogy BMS."""
+"""Module to support Renogy BMS.
+
+Project: aiobmsble, https://pypi.org/p/aiobmsble/
+"""
 
 from typing import Final
 

{aiobmsble-0.2.1 → aiobmsble-0.2.3}/aiobmsble/bms/renogy_pro_bms.py

@@ -1,4 +1,8 @@
-"""Module to support Renogy Pro BMS."""
+"""Module to support Renogy Pro BMS.
+
+Project: aiobmsble, https://pypi.org/p/aiobmsble/
+License: Apache-2.0, http://www.apache.org/licenses/
+"""
 
 from bleak.backends.characteristic import BleakGATTCharacteristic
 from bleak.backends.device import BLEDevice

{aiobmsble-0.2.1 → aiobmsble-0.2.3}/aiobmsble/bms/roypow_bms.py

@@ -1,4 +1,8 @@
-"""Module to support RoyPow BMS."""
+"""Module to support RoyPow BMS.
+
+Project: aiobmsble, https://pypi.org/p/aiobmsble/
+License: Apache-2.0, http://www.apache.org/licenses/
+"""
 
 from typing import Final
 

{aiobmsble-0.2.1 → aiobmsble-0.2.3}/aiobmsble/bms/seplos_bms.py

@@ -1,4 +1,8 @@
-"""Module to support Seplos V3 Smart BMS."""
+"""Module to support Seplos V3 Smart BMS.
+
+Project: aiobmsble, https://pypi.org/p/aiobmsble/
+License: Apache-2.0, http://www.apache.org/licenses/
+"""
 
 from collections.abc import Callable
 from typing import Any, Final

{aiobmsble-0.2.1 → aiobmsble-0.2.3}/aiobmsble/bms/seplos_v2_bms.py

@@ -1,4 +1,8 @@
-"""Module to support Seplos v2 BMS."""
+"""Module to support Seplos v2 BMS.
+
+Project: aiobmsble, https://pypi.org/p/aiobmsble/
+License: Apache-2.0, http://www.apache.org/licenses/
+"""
 
 from typing import Final
 

{aiobmsble-0.2.1 → aiobmsble-0.2.3}/aiobmsble/bms/tdt_bms.py

@@ -1,4 +1,8 @@
-"""Module to support TDT BMS."""
+"""Module to support TDT BMS.
+
+Project: aiobmsble, https://pypi.org/p/aiobmsble/
+License: Apache-2.0, http://www.apache.org/licenses/
+"""
 
 from typing import Final
 

{aiobmsble-0.2.1 → aiobmsble-0.2.3}/aiobmsble/bms/tianpwr_bms.py

@@ -1,4 +1,8 @@
-"""Module to support TianPwr BMS."""
+"""Module to support TianPwr BMS.
+
+Project: aiobmsble, https://pypi.org/p/aiobmsble/
+License: Apache-2.0, http://www.apache.org/licenses/
+"""
 
 from typing import Final
 

{aiobmsble-0.2.1 → aiobmsble-0.2.3}/aiobmsble/utils.py

@@ -1,4 +1,8 @@
-"""Utilitiy/Support functions for aiobmsble."""
+"""Utilitiy/Support functions for aiobmsble.
+
+Project: aiobmsble, https://pypi.org/p/aiobmsble/
+License: Apache-2.0, http://www.apache.org/licenses/
+"""
 
 from fnmatch import translate
 from functools import lru_cache
@@ -6,11 +10,15 @@ import importlib
 import pkgutil
 import re
 from types import ModuleType
+from typing import Final
 
 from bleak.backends.scanner import AdvertisementData
 
 from aiobmsble import MatcherPattern
 from aiobmsble.basebms import BaseBMS
+import aiobmsble.bms
+
+_MODULE_POSTFIX: Final[str] = "_bms"
 
 
 def _advertisement_matches(
@@ -78,8 +86,8 @@ def load_bms_plugins() -> set[ModuleType]:
     """
     return {
         importlib.import_module(f"aiobmsble.bms.{module_name}")
-        for _, module_name, _ in pkgutil.iter_modules(["aiobmsble/bms"])
-        if module_name.endswith("_bms")
+        for _, module_name, _ in pkgutil.iter_modules(aiobmsble.bms.__path__)
+        if module_name.endswith(_MODULE_POSTFIX)
     }
 
 
@@ -87,14 +95,16 @@ def bms_cls(name: str) -> type[BaseBMS] | None:
     """Return the BMS class that is defined by the name argument.
 
     Args:
-        name (str): The name of the BMS type
+        name (str): The name of the BMS type (filename of the module)
 
     Returns:
         type[BaseBMS] | None: If the BMS class defined by name is found, None otherwise.
 
     """
+    if not name.endswith(_MODULE_POSTFIX):
+        return None
     try:
-        bms_module: ModuleType = importlib.import_module(f"aiobmsble.bms.{name}_bms")
+        bms_module: ModuleType = importlib.import_module(f"aiobmsble.bms.{name}")
     except ModuleNotFoundError:
         return None
     return bms_module.BMS

{aiobmsble-0.2.1 → aiobmsble-0.2.3/aiobmsble.egg-info}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: aiobmsble
-Version: 0.2.1
+Version: 0.2.3
 Summary: Asynchronous Python library to query battery management systems via Bluetooth Low Energy.
 Author: Patrick Loschmidt
 Maintainer: Patrick Loschmidt
@@ -19,7 +19,7 @@ Requires-Python: >=3.12
 Description-Content-Type: text/markdown
 License-File: LICENSE
 Requires-Dist: bleak~=1.1.0
-Requires-Dist: bleak-retry-connector~=4.0.2
+Requires-Dist: bleak-retry-connector>=4.0.2
 Requires-Dist: asyncio
 Requires-Dist: logging
 Requires-Dist: statistics

{aiobmsble-0.2.1 → aiobmsble-0.2.3}/aiobmsble.egg-info/requires.txt

@@ -1,5 +1,5 @@
 bleak~=1.1.0
-bleak-retry-connector~=4.0.2
+bleak-retry-connector>=4.0.2
 asyncio
 logging
 statistics

{aiobmsble-0.2.1 → aiobmsble-0.2.3}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "setuptools.build_meta"
 
 [project]
 name = "aiobmsble"
-version = "0.2.1"
+version = "0.2.3"
 requires-python = ">=3.12"
 description = "Asynchronous Python library to query battery management systems via Bluetooth Low Energy."
 readme = "README.md"
@@ -28,7 +28,7 @@ classifiers = [
 ]
 dependencies = [
   "bleak~=1.1.0",
-  "bleak-retry-connector~=4.0.2",
+  "bleak-retry-connector>=4.0.2",
   "asyncio",
   "logging",
   "statistics",
@@ -59,62 +59,58 @@ testpaths = [
 asyncio_mode = "auto"
 asyncio_default_fixture_loop_scope = "function"
 
-# ruff settings from HA 2025.2.2
+# ruff settings from HA 2025.9.0
 [tool.ruff]
-required-version = ">=0.9.1"
+required-version = "~=0.12.1"
 
 [tool.ruff.lint]
 select = [
-    "A001", # Variable {name} is shadowing a Python builtin
-    "ASYNC210", # Async functions should not call blocking HTTP methods
-    "ASYNC220", # Async functions should not create subprocesses with blocking methods
-    "ASYNC221", # Async functions should not run processes with blocking methods
-    "ASYNC222", # Async functions should not wait on processes with blocking methods
-    "ASYNC230", # Async functions should not open files with blocking methods like open
-    "ASYNC251", # Async functions should not call time.sleep
-    "B002", # Python does not support the unary prefix increment
-    "B005", # Using .strip() with multi-character strings is misleading
-    "B007", # Loop control variable {name} not used within loop body
-    "B014", # Exception handler with duplicate exception
-    "B015", # Pointless comparison. Did you mean to assign a value? Otherwise, prepend assert or remove it.
-    "B017", # pytest.raises(BaseException) should be considered evil
-    "B018", # Found useless attribute access. Either assign it to a variable or remove it.
-    "B023", # Function definition does not bind loop variable {name}
-    "B024", # `{name}` is an abstract base class, but it has no abstract methods or properties
-    "B026", # Star-arg unpacking after a keyword argument is strongly discouraged
-    "B032", # Possible unintentional type annotation (using :). Did you mean to assign (using =)?
-    "B035", # Dictionary comprehension uses static key
-    "B904", # Use raise from to specify exception cause
-    "B905", # zip() without an explicit strict= parameter
+    "A001",   # Variable {name} is shadowing a Python builtin
+    "ASYNC",  # flake8-async
+    "B002",   # Python does not support the unary prefix increment
+    "B005",   # Using .strip() with multi-character strings is misleading
+    "B007",   # Loop control variable {name} not used within loop body
+#    "B009",   # Do not call getattr with a constant attribute value. It is not any safer than normal property access.
+    "B014",   # Exception handler with duplicate exception
+    "B015",   # Pointless comparison. Did you mean to assign a value? Otherwise, prepend assert or remove it.
+    "B017",   # pytest.raises(BaseException) should be considered evil
+    "B018",   # Found useless attribute access. Either assign it to a variable or remove it.
+    "B023",   # Function definition does not bind loop variable {name}
+    "B024",   # `{name}` is an abstract base class, but it has no abstract methods or properties
+    "B026",   # Star-arg unpacking after a keyword argument is strongly discouraged
+    "B032",   # Possible unintentional type annotation (using :). Did you mean to assign (using =)?
+    "B035",   # Dictionary comprehension uses static key
+    "B904",   # Use raise from to specify exception cause
+    "B905",   # zip() without an explicit strict= parameter
     "BLE",
-    "C", # complexity
+    "C",      # complexity
     "COM818", # Trailing comma on bare tuple prohibited
-    "D", # docstrings
+    "D",      # docstrings
     "DTZ003", # Use datetime.now(tz=) instead of datetime.utcnow()
     "DTZ004", # Use datetime.fromtimestamp(ts, tz=) instead of datetime.utcfromtimestamp(ts)
-    "E", # pycodestyle
-    "F", # pyflakes/autoflake
-    "F541", # f-string without any placeholders
-    "FLY", # flynt
-    "FURB", # refurb
-    "G", # flake8-logging-format
-    "I", # isort
-    "INP", # flake8-no-pep420
-    "ISC", # flake8-implicit-str-concat
+    "E",      # pycodestyle
+    "F",      # pyflakes/autoflake
+    "F541",   # f-string without any placeholders
+    "FLY",    # flynt
+    "FURB",   # refurb
+    "G",      # flake8-logging-format
+    "I",      # isort
+    "INP",    # flake8-no-pep420
+    "ISC",    # flake8-implicit-str-concat
     "ICN001", # import concentions; {name} should be imported as {asname}
-    "LOG", # flake8-logging
-    "N804", # First argument of a class method should be named cls
-    "N805", # First argument of a method should be named self
-    "N815", # Variable {name} in class scope should not be mixedCase
-    "PERF", # Perflint
-    "PGH", # pygrep-hooks
-    "PIE", # flake8-pie
-    "PL", # pylint
-    "PT", # flake8-pytest-style
-    "PTH", # flake8-pathlib
-    "PYI", # flake8-pyi
-    "RET", # flake8-return
-    "RSE", # flake8-raise
+    "LOG",    # flake8-logging
+    "N804",   # First argument of a class method should be named cls
+    "N805",   # First argument of a method should be named self
+    "N815",   # Variable {name} in class scope should not be mixedCase
+    "PERF",   # Perflint
+    "PGH",    # pygrep-hooks
+    "PIE",    # flake8-pie
+    "PL",     # pylint
+    "PT",     # flake8-pytest-style
+    "PTH",    # flake8-pathlib
+    "PYI",    # flake8-pyi
+    "RET",    # flake8-return
+    "RSE",    # flake8-raise
     "RUF005", # Consider iterable unpacking instead of concatenation
     "RUF006", # Store a reference to the return value of asyncio.create_task
     "RUF007", # Prefer itertools.pairwise() over zip() when iterating over successive pairs
@@ -138,44 +134,46 @@ select = [
     "RUF100", # Unused `noqa` directive
     "RUF101", # noqa directives that use redirected rule codes
     "RUF200", # Failed to parse pyproject.toml: {message}
-    "S102", # Use of exec detected
-    "S103", # bad-file-permissions
-    "S108", # hardcoded-temp-file
-    "S306", # suspicious-mktemp-usage
-    "S307", # suspicious-eval-usage
-    "S313", # suspicious-xmlc-element-tree-usage
-    "S314", # suspicious-xml-element-tree-usage
-    "S315", # suspicious-xml-expat-reader-usage
-    "S316", # suspicious-xml-expat-builder-usage
-    "S317", # suspicious-xml-sax-usage
-    "S318", # suspicious-xml-mini-dom-usage
-    "S319", # suspicious-xml-pull-dom-usage
-    "S601", # paramiko-call
-    "S602", # subprocess-popen-with-shell-equals-true
-    "S604", # call-with-shell-equals-true
-    "S608", # hardcoded-sql-expression
-    "S609", # unix-command-wildcard-injection
-    "SIM", # flake8-simplify
-    "SLF", # flake8-self
-    "SLOT", # flake8-slots
-    "T100", # Trace found: {name} used
-    "T20", # flake8-print
-    "TC", # flake8-type-checking
-    "TID", # Tidy imports
-    "TRY", # tryceratops
-    "UP", # pyupgrade
-    "UP031", # Use format specifiers instead of percent format
-    "UP032", # Use f-string instead of `format` call
-    "W", # pycodestyle
+    "S102",   # Use of exec detected
+    "S103",   # bad-file-permissions
+    "S108",   # hardcoded-temp-file
+    "S306",   # suspicious-mktemp-usage
+    "S307",   # suspicious-eval-usage
+    "S313",   # suspicious-xmlc-element-tree-usage
+    "S314",   # suspicious-xml-element-tree-usage
+    "S315",   # suspicious-xml-expat-reader-usage
+    "S316",   # suspicious-xml-expat-builder-usage
+    "S317",   # suspicious-xml-sax-usage
+    "S318",   # suspicious-xml-mini-dom-usage
+    "S319",   # suspicious-xml-pull-dom-usage
+    "S601",   # paramiko-call
+    "S602",   # subprocess-popen-with-shell-equals-true
+    "S604",   # call-with-shell-equals-true
+    "S608",   # hardcoded-sql-expression
+    "S609",   # unix-command-wildcard-injection
+    "SIM",    # flake8-simplify
+    "SLF",    # flake8-self
+    "SLOT",   # flake8-slots
+    "T100",   # Trace found: {name} used
+    "T20",    # flake8-print
+    "TC",     # flake8-type-checking
+    "TID",    # Tidy imports
+    "TRY",    # tryceratops
+    "UP",     # pyupgrade
+    "UP031",  # Use format specifiers instead of percent format
+    "UP032",  # Use f-string instead of `format` call
+    "W",      # pycodestyle
 ]
 
 ignore = [
-    "D202", # No blank lines allowed after function docstring
-    "D203", # 1 blank line required before class docstring
-    "D213", # Multi-line docstring summary should start at the second line
-    "D406", # Section name should end with a newline
-    "D407", # Section name underlining
-    "E501", # line too long
+    "ASYNC109", # Async function definition with a `timeout` parameter Use `asyncio.timeout` instead
+    "ASYNC110", # Use `asyncio.Event` instead of awaiting `asyncio.sleep` in a `while` loop
+    "D202",     # No blank lines allowed after function docstring
+    "D203",     # 1 blank line required before class docstring
+    "D213",     # Multi-line docstring summary should start at the second line
+    "D406",     # Section name should end with a newline
+    "D407",     # Section name underlining
+    "E501",     # line too long
 
 #    "PLC1901", # {existing} can be simplified to {replacement} as an empty string is falsey; too many false positives
 #    "PLR0911", # Too many return statements ({returns} > {max_returns})
@@ -183,27 +181,33 @@ ignore = [
     "PLR0913", # Too many arguments to function call ({c_args} > {max_args})
 #    "PLR0915", # Too many statements ({statements} > {max_statements})
     "PLR2004", # Magic value used in comparison, consider replacing {value} with a constant variable
+    "PLW1641", # __eq__ without __hash__
 #    "PLW2901", # Outer {outer_kind} variable {name} overwritten by inner {inner_kind} target
-#    "PT011", # pytest.raises({exception}) is too broad, set the `match` parameter or use a more specific exception
-    "PT018", # Assertion should be broken down into multiple parts
-#    "RUF001", # String contains ambiguous unicode character.
-#    "RUF002", # Docstring contains ambiguous unicode character.
-#    "RUF003", # Comment contains ambiguous unicode character.
-#    "RUF015", # Prefer next(...) over single element slice
-    "SIM102", # Use a single if statement instead of nested if statements
-#    "SIM103", # Return the condition {condition} directly
-#    "SIM108", # Use ternary operator {contents} instead of if-else-block
-#    "SIM115", # Use context handler for opening files
+#    "PT011",   # pytest.raises({exception}) is too broad, set the `match` parameter or use a more specific exception
+    "PT018",   # Assertion should be broken down into multiple parts
+#    "RUF001",  # String contains ambiguous unicode character.
+#    "RUF002",  # Docstring contains ambiguous unicode character.
+#    "RUF003",  # Comment contains ambiguous unicode character.
+#    "RUF015",  # Prefer next(...) over single element slice
+    "SIM102",  # Use a single if statement instead of nested if statements
+#    "SIM103",  # Return the condition {condition} directly
+#    "SIM108",  # Use ternary operator {contents} instead of if-else-block
+#    "SIM115",  # Use context handler for opening files
 
     # Moving imports into type-checking blocks can mess with pytest.patch()
     "TC001", # Move application import {} into a type-checking block
     "TC002", # Move third-party import {} into a type-checking block
     "TC003", # Move standard library import {} into a type-checking block
+    # Quotes for typing.cast generally not necessary, only for performance critical paths
+    "TC006", # Add quotes to type expression in typing.cast()
 
     "TRY003", # Avoid specifying long messages outside the exception class
     "TRY400", # Use `logging.exception` instead of `logging.error`
     # Ignored due to performance: https://github.com/charliermarsh/ruff/issues/2923
-    "UP038", # Use `X | Y` in `isinstance` call instead of `(X, Y)`
+#    "UP038", # Use `X | Y` in `isinstance` call instead of `(X, Y)`
+#    "UP046", # Non PEP 695 generic class
+#    "UP047", # Non PEP 696 generic function
+#    "UP049", # Avoid private type parameter names
 
     # May conflict with the formatter, https://docs.astral.sh/ruff/formatter/#conflicting-lint-rules
     "W191",
@@ -215,12 +219,12 @@ ignore = [
     "Q",
     "COM812",
     "COM819",
-    "ISC001",
 
     # Disabled because ruff does not understand type of __all__ generated by a function
-    "PLE0605"
+    "PLE0605",
 ]
 
+
 [tool.ruff.lint.isort]
 force-sort-within-sections = true
 known-first-party = [

{aiobmsble-0.2.1 → aiobmsble-0.2.3}/tests/test_basebms.py

@@ -14,6 +14,7 @@ import pytest
 
 from aiobmsble import BMSdp, BMSsample, MatcherPattern
 from aiobmsble.basebms import BaseBMS, crc8, crc_modbus, crc_sum, crc_xmodem, lrc_modbus
+from aiobmsble.bms.dummy_bms import BMS as DummyBMS
 
 from .bluetooth import generate_ble_device
 from .conftest import MockBleakClient
@@ -315,6 +316,12 @@ async def test_wr_mode_reset(
     assert bms._inv_wr_mode is None
 
 
+def test_get_bms_module() -> None:
+    """Check that basebms and dummy_bms return correct module name."""
+    assert BaseBMS.get_bms_module() == "aiobmsble.basebms"
+    assert DummyBMS.get_bms_module() == "aiobmsble.bms.dummy_bms"
+
+
 async def test_no_notify(
     patch_bleak_client: Callable[..., None], caplog: pytest.LogCaptureFixture
 ) -> None:

{aiobmsble-0.2.1 → aiobmsble-0.2.3}/tests/test_utils.py

@@ -55,15 +55,14 @@ def test_bms_identify(plugin: ModuleType) -> None:
 def test_bms_cls(plugin: ModuleType) -> None:
     """Test that a BMS class is correctly returned from its name."""
     # strip _bms to get only type
-    bms_type: str = getattr(plugin, "__name__", "").rsplit(".", 1)[-1][:-4]
+    bms_type: str = getattr(plugin, "__name__", "").rsplit(".", 1)[-1]
     bms_class: type[BaseBMS] | None = bms_cls(bms_type)
     assert bms_class == plugin.BMS
 
 
-def test_bms_cls_none() -> None:
-    """Test that a BMS class is correctly returned from its name."""
-    # strip _bms to get only type
-    bms_type: str = "unvailable_bms"
+@pytest.mark.parametrize("bms_type", ["unavailable_bms", "ignore_me"])
+def test_bms_cls_none(bms_type: str) -> None:
+    """Test that a BMS class is None when name is not correct."""
     bms_class: type[BaseBMS] | None = bms_cls(bms_type)
     assert bms_class is None
 

aiobmsble-0.2.1/aiobmsble/bms/__init__.py

@@ -1 +0,0 @@
-"""Package for battery management systems (BMS) plugins."""