chore(repo): Linting

Author: devsjc

src/quartz_api/internal/backends/dummydb/client.py

@@ -2,10 +2,9 @@
 # ruff: noqa: S311
 
 import datetime as dt
-from uuid import UUID
 import math
 import random
-from uuid import uuid4
+from uuid import UUID, uuid4
 
 from typing_extensions import override
 

src/quartz_api/internal/backends/quartzdb/client.py

@@ -456,12 +456,12 @@ def check_user_has_access_to_site(
 ) -> None:
     """Checks if a user has access to a site."""
     user = get_user_by_email(session=session, email=email)
-    site_uuids = [str(site.location_uuid) for site in user.location_group.locations]
+    site_uuids = [site.location_uuid for site in user.location_group.locations]
 
     if site_uuid not in site_uuids:
         raise HTTPException(
             status_code=403,
             detail=f"Forbidden. User ({email}) "
-            f"does not have access to this site {str(site_uuid)}. "
-            f"User has access to {site_uuids}",
+            f"does not have access to this site {site_uuid!s}. "
+            f"User has access to {[str(s) for s in site_uuids]}",
         )

src/quartz_api/internal/backends/quartzdb/conftest.py

@@ -2,6 +2,7 @@
 
 import datetime as dt
 import logging
+from collections.abc import Generator
 
 import pytest
 from pvsite_datamodel.read.model import get_or_create_model
@@ -16,7 +17,6 @@
 from sqlalchemy import Engine, create_engine
 from sqlalchemy.orm import Session
 from testcontainers.postgres import PostgresContainer
-from collections.abc import Generator
 
 log = logging.getLogger(__name__)
 

src/quartz_api/internal/backends/quartzdb/test_quartzdb.py

@@ -5,8 +5,8 @@
 import logging
 
 import pytest
-from pvsite_datamodel.sqlmodels import LocationSQL, GenerationSQL
 from fastapi import HTTPException
+from pvsite_datamodel.sqlmodels import GenerationSQL, LocationSQL
 from sqlalchemy import Engine
 from sqlalchemy.orm import Session
 

src/quartz_api/internal/middleware/auth.py

@@ -6,7 +6,6 @@
 import jwt
 from fastapi import Depends, HTTPException, Request
 from fastapi.security import HTTPAuthorizationCredentials, HTTPBearer
-from typing_extensions import override
 
 token_auth_scheme = HTTPBearer()
 

src/quartz_api/internal/models/endpoint_types.py

@@ -2,12 +2,11 @@
 
 import datetime as dt
 from enum import Enum
-from typing import Literal
-from typing import Annotated
+from typing import Annotated, Literal
 from uuid import UUID
 from zoneinfo import ZoneInfo
-from fastapi import Depends
 
+from fastapi import Depends
 from pydantic import BaseModel, Field
 
 

src/quartz_api/internal/service/regions/_csv.py

@@ -8,7 +8,7 @@
 
 
 # NOTE: We should probably make this take a timezone argument.
-# This would mean changing the column names, so I'm leaving it for now,
+# This would mean changing the column names, so I'm leaving it for now,
 # as I don't know whether this would affect clients.
 def format_csv_and_created_time(
     values: list[PredictedPower],

src/quartz_api/internal/service/regions/test_resample.py

@@ -1,5 +1,5 @@
-import unittest
 import datetime as dt
+import unittest
 
 from quartz_api.internal.models import ActualPower