[lint] clean errors

pyheatpump/cli.py

@@ -25,9 +25,12 @@ CONTEXT_SETTINGS={
 @click.option('--version', is_flag=True)
 @click.pass_context
 def cli(ctx, version):
+    """
     if version:
         from pyheatpump import version
         return click.echo(pyheatpump.version())
+    """
+    pass
 
 
 @click.option('--host', default=None)
@@ -174,9 +177,9 @@ def supervise(since):
         last_update = get_last_update()
     
     post_packets = [
-        Heatpump(mac_address, last_update, ['Analog']).__dict__(),
-        Heatpump(mac_address, last_update, ['Digital']).__dict__(),
-        Heatpump(mac_address, last_update, ['Integer']).__dict__()
+        Heatpump(mac_address, last_update, ['Analog']).packet,
+        Heatpump(mac_address, last_update, ['Digital']).packet,
+        Heatpump(mac_address, last_update, ['Integer']).packet
     ]
 
     base_url = {

pyheatpump/config.py

@@ -1,11 +1,13 @@
 #!/usr/bin/env python3
 import os
 import re
+import sys
 from datetime import datetime
 from configparser import ConfigParser
 from starlette.requests import Request
 from starlette.routing import Route, Router
 from starlette.responses import PlainTextResponse, JSONResponse
+from starlette.exceptions import HTTPException
 from pprint import pprint
 import uvicorn
 import json

pyheatpump/db.py

@@ -34,7 +34,7 @@ class DB():
         return True
 
 
-    def commit():
+    def commit(self):
         """
             Default mode of connecting is autocommit
         """
@@ -78,7 +78,7 @@ class DB():
                 print(e)
                 return []
             except sqlite3.OperationalError as e:
-                raise Exception('Can\'t find DB file %s'.format(filename)) from e
+                raise Exception(f'Can\'t find DB file {filename}') from e
 
 
 
@@ -93,7 +93,7 @@ class RowClass(object):
         if type(attr) == str:
             q = ("SELECT * FROM {tablename} WHERE ? LIKE '?'", (key, attr))
         elif type(attr) == int:
-            q = ("SELECT * FROM {tablename} WHERE ? LIKE ?", (key, attr))
+            q = ("SELECT * FROM {tablename} WHERE ? = ?", (key, attr))
 
         res = []
         for row in DB.sql(*q):

pyheatpump/models/heatpump.py

@@ -27,9 +27,18 @@ class Heatpump:
             self.var_types = VariableType.getall()
 
     def __str__(self):
-        return str(self.__dict__())
+        res = f'{self.mac_address} ['
+        res += ', '.join([
+            var_type.slabel
+            for var_type in self.var_types
+        ])
+        res += ']'
 
-    def __dict__(self):
+    @property
+    def packet(self):
+        """
+        Returns an object containing all values and the current macAddress
+        """
         r = {'macAddress': self.macformat}
         if not self.last_update:
             # Since beginning

pyheatpump/models/variable_type.py

@@ -2,9 +2,9 @@ from pyheatpump.db import DB, RowClass
 
 
 class VariableType(RowClass):
-    slabel: str = None
-    label: str = None
-    type: str = None
+    slabel: str = '' 
+    label: str = '' 
+    type: str = '' 
     start_address: int = None
     end_address: int = None
 
@@ -28,7 +28,7 @@ class VariableType(RowClass):
             return bool
 
 
-    def select():
+    def select(self):
         try:
             elt = next(super().select('slabel', 'var_type'))
         except StopIteration: