ruff formatting

Author: julianstirling

src/labthings_fastapi/decorators/__init__.py

@@ -92,6 +92,7 @@ def __get__(self, obj, objtype=None):
         getter=func,
     )
 
+
 def thing_setting(func: Callable) -> SettingDescriptor:
     """Mark a method of a Thing as a Setting.
 

src/labthings_fastapi/descriptors/property.py

@@ -226,7 +226,6 @@ def setter(self, func: Callable) -> Self:
 
 
 class SettingDescriptor(PropertyDescriptor):
-
     @property
     def persistent(self):
         return True
@@ -241,7 +240,7 @@ def get_raw(self, obj):
         Used to get the raw data for saving settings. Any setting that returns
         a non-JSON serialisable class should implement a `raw` flag in its getter
         """
-        #TODO check if has a raw option rather than use a try
+        # TODO check if has a raw option rather than use a try
         if self._getter:
             try:
                 return self._getter(obj, raw=True)

src/labthings_fastapi/server/__init__.py

@@ -81,7 +81,9 @@ def add_thing(self, thing: Thing, path: str):
         # TODO: check for illegal things in `path` - potential security issue.
         settings_folder = os.path.join(self.settings_folder, path.lstrip("/"))
         os.makedirs(settings_folder, exist_ok=True)
-        thing.attach_to_server(self, path, os.path.join(settings_folder, "settings.json"))
+        thing.attach_to_server(
+            self, path, os.path.join(settings_folder, "settings.json")
+        )
 
     @asynccontextmanager
     async def lifespan(self, app: FastAPI):

src/labthings_fastapi/thing.py

@@ -36,6 +36,7 @@
 
 _LOGGER = logging.getLogger(__name__)
 
+
 class Thing:
     """Represents a Thing, as defined by the Web of Things standard.
 
@@ -90,7 +91,9 @@ async def __aexit__(self, exc_t, exc_v, exc_tb):
         if hasattr(self, "__exit__"):
             return await run_sync(self.__exit__, exc_t, exc_v, exc_tb)
 
-    def attach_to_server(self, server: ThingServer, path: str, setting_storage_path: str):
+    def attach_to_server(
+        self, server: ThingServer, path: str, setting_storage_path: str
+    ):
         """Add HTTP handlers to an app for all Interaction Affordances"""
         self.path = path
         self.action_manager: ActionManager = server.action_manager
@@ -119,15 +122,16 @@ def thing_description(request: Request) -> ThingDescription:
         async def websocket(ws: WebSocket):
             await websocket_endpoint(self, ws)
 
-
     _settings: Optional[list[str]] = None
 
     @property
     def settings(self):
         if self._settings is None:
             self._settings = []
             for name, attribute in class_attributes(self):
-                if hasattr(attribute, "property_affordance") and hasattr(attribute, "persistent"):
+                if hasattr(attribute, "property_affordance") and hasattr(
+                    attribute, "persistent"
+                ):
                     self._settings.append(name)
         return self._settings
 
@@ -149,27 +153,31 @@ def load_settings(self, setting_storage_path):
                     setting_dict = json.load(file_obj)
                 setting_attributes = {}
                 for name, attribute in class_attributes(self):
-                    if hasattr(attribute, "property_affordance") and hasattr(attribute, "persistent"):
+                    if hasattr(attribute, "property_affordance") and hasattr(
+                        attribute, "persistent"
+                    ):
                         setting_attributes[name] = attribute
                 for key, value in setting_dict.items():
                     if key in setting_attributes:
-                        setting_attributes[key].set_without_emit(self, value) 
+                        setting_attributes[key].set_without_emit(self, value)
                     else:
                         _LOGGER.warning(
                             "Cannot set %s from persistent storage as %s has no matching setting.",
-                            key, thing_name
+                            key,
+                            thing_name,
                         )
             except (FileNotFoundError, JSONDecodeError, PermissionError):
                 _LOGGER.warning("Error loading settings for %s", thing_name)
         self._setting_storage_path = setting_storage_path
 
-
     def save_settings(self):
         """Save settings to JSON. This is called when a setting is updated with a setter"""
         if self.settings is not None:
             setting_dict = {}
             for name, attribute in class_attributes(self):
-                if hasattr(attribute, "property_affordance") and hasattr(attribute, "persistent"):
+                if hasattr(attribute, "property_affordance") and hasattr(
+                    attribute, "persistent"
+                ):
                     setting_dict[name] = attribute.get_raw(self)
             # Dumpy to string before writing so if this fails the file isn't overwritten
             setting_json = json.dumps(setting_dict, indent=4)