refactoring

Author: mietzen

.gitignore

@@ -6,5 +6,6 @@ __pycache__
 *.egg-info
 build
 dist
-local_test.py
 venv
+.pytest_cache
+testing.ipynb

README.md

@@ -79,6 +79,6 @@ except FritzAdvancedThermostatError as err:
     print(err)
 ```
 
-## Contribute
+## Credits
 
-Contributions are always welcome, just open a PR, specially if you find a way to obtain the thermostat data without selenium!
+Thanks to [Argelbargel](https://github.com/Argelbargel) from the openHab community for [showing me a way](https://community.openhab.org/t/groovy-script-rule-to-update-temperature-offsets-of-avm-fritz-dect-301-302-based-on-external-temperature-sensors/139917) to obtain the thermostat data without selenium.

fritz_advanced_thermostat/fritz_advanced_thermostat.py fritz_advanced_thermostat.py

@@ -50,12 +50,25 @@
 import urllib3
 from packaging import version
 
-from .errors import (
-    FritzAdvancedThermostatCompatibilityError,
-    FritzAdvancedThermostatConnectionError,
-    FritzAdvancedThermostatExecutionError,
-    FritzAdvancedThermostatKeyError,
-)
+
+class FritzAdvancedThermostatExecutionError(Exception):
+    """Unknown error while executing"""
+
+
+class FritzAdvancedThermostatCompatibilityError(Exception):
+    """Fritz!BOX is not compatible with this module"""
+
+
+class FritzAdvancedThermostatKeyError(KeyError):
+    """Error while obtaining a key from the Fritz!BOX"""
+
+
+class FritzAdvancedThermostatConnectionError(ConnectionError):
+    """Error while connecting to the Fritz!BOX"""
+
+
+FritzAdvancedThermostatError = (FritzAdvancedThermostatExecutionError, FritzAdvancedThermostatCompatibilityError, FritzAdvancedThermostatKeyError, FritzAdvancedThermostatConnectionError)
+
 
 # Silence annoying urllib3 Unverified HTTPS warnings, even so if we have checked verify ssl false in requests
 urllib3.disable_warnings(urllib3.exceptions.InsecureRequestWarning)

fritz_advanced_thermostat/__init__.py

@@ -1,6 +1,7 @@
 import sys
+from os import environ, path
+
 from setuptools import setup
-from os import path, environ
 
 setup_file_dir = path.abspath(path.dirname(__file__))
 
@@ -20,23 +21,23 @@
 
 setup(
     name="fritz-advanced-thermostat",
-    version=environ.get('VERSION'),
+    version=environ.get("VERSION"),
     description="A library for setting FRITZ!DECT thermostat values (e.g. offset, holidays, timer), that can't be set via AHA requests.",
     long_description=long_description,
-    long_description_content_type='text/markdown',
+    long_description_content_type="text/markdown",
     url="https://github.com/mietzen/python-fritz-advanced-thermostat",
     author="Nils Stein",
     author_email="github.nstein@mailbox.org",
     license="MIT",
     classifiers=[
-        'Development Status :: 4 - Beta',
-        'License :: OSI Approved :: MIT License',
-        'Programming Language :: Python :: 3.9',
-        'Programming Language :: Python :: 3.10',
-        'Programming Language :: Python :: 3.11',
+        "Development Status :: 4 - Beta",
+        "License :: OSI Approved :: MIT License",
+        "Programming Language :: Python :: 3.9",
+        "Programming Language :: Python :: 3.10",
+        "Programming Language :: Python :: 3.11",
         "Operating System :: OS Independent",
     ],
     keywords="fritzbox smarthome avm thermostat",
     packages=["fritz_advanced_thermostat"],
-    install_requires=requirements.split('\n')
+    install_requires=requirements.split("\n"),
 )