treewide: fixes to make pylint happier

Right now the CI fails and that makes me and my PRs unhappy.

Signed-off-by: Paul Spooren <mail@aparcar.org>

Author: aparcar

labgrid/driver/mqtt.py

@@ -43,6 +43,8 @@ def _on_message(self, client, userdata, msg):
             status = True
         elif msg.payload == b'OFF':
             status = False
+        else:
+            raise ValueError(f"Unknown status: {msg.payload}. Must be 'ON' or 'OFF'")
         self._status = status
 
     def _on_connect(self, client, userdata, flags, reason_code, properties):

labgrid/remote/client.py

@@ -1212,6 +1212,9 @@ def tmc_channel(self):
             data = drv.get_channel_info(channel)
         elif action == 'values':
             data = drv.get_channel_values(channel)
+        else:
+            raise ValueError(f"unknown action {action}")
+
         for k, v in sorted(data.items()):
             print(f"{k:<16s} {str(v):<10s}")
 
@@ -1323,6 +1326,9 @@ async def export(self, place, target):
             data = "\n".join(lines) + "\n"
         elif self.args.format is ExportFormat.JSON:
             data = json.dumps(exported)
+        else:
+            raise NotImplementedError(f"unsupported format {self.args.format}")
+
         if self.args.filename == "-":
             sys.stdout.write(data)
         else:

labgrid/util/yaml.py

@@ -84,6 +84,9 @@ def resolve_templates(data, mapping):
         items = enumerate(data)
     elif isinstance(data, dict):
         items = data.items()
+    else:
+        raise TypeError(f"Expected list or dict, got {type(data)}")
+
     for k, val in items:
         if isinstance(val, Template):
             try: