Force autoformatting with ruff

getlago · Aug 30, 2024 · b47e178 · b47e178
1 parent a6c99c3
commit b47e178
Show file tree

Hide file tree

Showing 68 changed files with 1,981 additions and 1,072 deletions.
diff --git a/.github/workflows/ruff.yml b/.github/workflows/ruff.yml
@@ -12,5 +12,9 @@ jobs:
       - name: Install dependencies
         run: |
           python -m pip install .[test]
+      - name: Validate linter rules with Ruff
+        uses: chartboost/ruff-action@v1
       - name: Validate formatting with Ruff
         uses: chartboost/ruff-action@v1
+        with:
+          args: 'format --check'
diff --git a/lago_python_client/add_ons/clients.py b/lago_python_client/add_ons/clients.py
@@ -1,7 +1,13 @@
 from typing import ClassVar, Type
 
 from ..base_client import BaseClient
-from ..mixins import CreateCommandMixin, DestroyCommandMixin, FindAllCommandMixin, FindCommandMixin, UpdateCommandMixin
+from ..mixins import (
+    CreateCommandMixin,
+    DestroyCommandMixin,
+    FindAllCommandMixin,
+    FindCommandMixin,
+    UpdateCommandMixin,
+)
 from ..models.add_on import AddOnResponse
 
 
@@ -13,6 +19,6 @@ class AddOnClient(
     UpdateCommandMixin[AddOnResponse],
     BaseClient,
 ):
-    API_RESOURCE: ClassVar[str] = 'add_ons'
+    API_RESOURCE: ClassVar[str] = "add_ons"
     RESPONSE_MODEL: ClassVar[Type[AddOnResponse]] = AddOnResponse
-    ROOT_NAME: ClassVar[str] = 'add_on'
+    ROOT_NAME: ClassVar[str] = "add_on"
diff --git a/lago_python_client/billable_metrics/clients.py b/lago_python_client/billable_metrics/clients.py
@@ -1,16 +1,15 @@
 import sys
-from typing import Any, ClassVar, Type, Union
+from typing import ClassVar, Type
 
 from ..base_client import BaseClient
-from ..mixins import CreateCommandMixin, DestroyCommandMixin, FindAllCommandMixin, FindCommandMixin, UpdateCommandMixin
+from ..mixins import (
+    CreateCommandMixin,
+    DestroyCommandMixin,
+    FindAllCommandMixin,
+    FindCommandMixin,
+    UpdateCommandMixin,
+)
 from ..models.billable_metric import BillableMetricResponse
-from ..services.request import make_headers, make_url, send_get_request
-from ..services.response import get_response_data, prepare_index_response, Response
-
-if sys.version_info >= (3, 9):
-    from collections.abc import Mapping
-else:
-    from typing import Mapping
 
 
 class BillableMetricClient(
@@ -21,6 +20,6 @@ class BillableMetricClient(
     UpdateCommandMixin[BillableMetricResponse],
     BaseClient,
 ):
-    API_RESOURCE: ClassVar[str] = 'billable_metrics'
+    API_RESOURCE: ClassVar[str] = "billable_metrics"
     RESPONSE_MODEL: ClassVar[Type[BillableMetricResponse]] = BillableMetricResponse
-    ROOT_NAME: ClassVar[str] = 'billable_metric'
+    ROOT_NAME: ClassVar[str] = "billable_metric"
diff --git a/lago_python_client/client.py b/lago_python_client/client.py
@@ -30,16 +30,18 @@
 
 
 class Client:
-    BASE_URL: Final[str] = 'https://api.getlago.com/'
-    API_PATH: Final[str] = 'api/v1/'
+    BASE_URL: Final[str] = "https://api.getlago.com/"
+    API_PATH: Final[str] = "api/v1/"
 
-    def __init__(self, api_key: str = '', api_url: str = '') -> None:
+    def __init__(self, api_key: str = "", api_url: str = "") -> None:
         self.api_key: str = api_key
         self.api_url: str = api_url
 
     @property
     def base_api_url(self) -> str:
-        return urljoin(self.api_url if self.api_url else Client.BASE_URL, Client.API_PATH)
+        return urljoin(
+            self.api_url if self.api_url else Client.BASE_URL, Client.API_PATH
+        )
 
     @callable_cached_property
     def add_ons(self) -> AddOnClient:

diff --git a/lago_python_client/coupons/clients.py b/lago_python_client/coupons/clients.py
@@ -1,7 +1,13 @@
 from typing import ClassVar, Type
 
 from ..base_client import BaseClient
-from ..mixins import CreateCommandMixin, DestroyCommandMixin, FindAllCommandMixin, FindCommandMixin, UpdateCommandMixin
+from ..mixins import (
+    CreateCommandMixin,
+    DestroyCommandMixin,
+    FindAllCommandMixin,
+    FindCommandMixin,
+    UpdateCommandMixin,
+)
 from ..models.coupon import CouponResponse
 from ..models.applied_coupon import AppliedCouponResponse
 from ..services.request import make_headers, make_url, send_delete_request
@@ -16,21 +22,32 @@ class CouponClient(
     UpdateCommandMixin[CouponResponse],
     BaseClient,
 ):
-    API_RESOURCE: ClassVar[str] = 'coupons'
+    API_RESOURCE: ClassVar[str] = "coupons"
     RESPONSE_MODEL: ClassVar[Type[CouponResponse]] = CouponResponse
-    ROOT_NAME: ClassVar[str] = 'coupon'
+    ROOT_NAME: ClassVar[str] = "coupon"
 
 
-class AppliedCouponClient(CreateCommandMixin[AppliedCouponResponse], FindAllCommandMixin[AppliedCouponResponse], BaseClient):
-    API_RESOURCE: ClassVar[str] = 'applied_coupons'
+class AppliedCouponClient(
+    CreateCommandMixin[AppliedCouponResponse],
+    FindAllCommandMixin[AppliedCouponResponse],
+    BaseClient,
+):
+    API_RESOURCE: ClassVar[str] = "applied_coupons"
     RESPONSE_MODEL: ClassVar[Type[AppliedCouponResponse]] = AppliedCouponResponse
-    ROOT_NAME: ClassVar[str] = 'applied_coupon'
+    ROOT_NAME: ClassVar[str] = "applied_coupon"
 
-    def destroy(self, external_customer_id: str, applied_coupon_id: str) -> AppliedCouponResponse:
+    def destroy(
+        self, external_customer_id: str, applied_coupon_id: str
+    ) -> AppliedCouponResponse:
         api_response: Response = send_delete_request(
             url=make_url(
                 origin=self.base_url,
-                path_parts=('customers', external_customer_id, self.API_RESOURCE, applied_coupon_id),
+                path_parts=(
+                    "customers",
+                    external_customer_id,
+                    self.API_RESOURCE,
+                    applied_coupon_id,
+                ),
             ),
             headers=make_headers(api_key=self.api_key),
         )

diff --git a/lago_python_client/credit_notes/clients.py b/lago_python_client/credit_notes/clients.py
@@ -1,10 +1,24 @@
-from typing import ClassVar, Optional, Type, Union
+from typing import ClassVar, Optional, Type
 
 from ..base_client import BaseClient
-from ..mixins import CreateCommandMixin, FindAllCommandMixin, FindCommandMixin, UpdateCommandMixin
-from ..models.credit_note import CreditNoteResponse, CreditNoteEstimatedResponse, CreditNoteEstimate
+from ..mixins import (
+    CreateCommandMixin,
+    FindAllCommandMixin,
+    FindCommandMixin,
+    UpdateCommandMixin,
+)
+from ..models.credit_note import (
+    CreditNoteResponse,
+    CreditNoteEstimatedResponse,
+    CreditNoteEstimate,
+)
 from ..services.json import to_json
-from ..services.request import make_headers, make_url, send_post_request, send_put_request
+from ..services.request import (
+    make_headers,
+    make_url,
+    send_post_request,
+    send_put_request,
+)
 from ..services.response import get_response_data, prepare_object_response, Response
 
 
@@ -15,16 +29,16 @@ class CreditNoteClient(
     UpdateCommandMixin[CreditNoteResponse],
     BaseClient,
 ):
-    API_RESOURCE: ClassVar[str] = 'credit_notes'
-    ESTIMATE_API_RESOURCE: ClassVar[str] = 'estimated_credit_note'
+    API_RESOURCE: ClassVar[str] = "credit_notes"
+    ESTIMATE_API_RESOURCE: ClassVar[str] = "estimated_credit_note"
     RESPONSE_MODEL: ClassVar[Type[CreditNoteResponse]] = CreditNoteResponse
-    ROOT_NAME: ClassVar[str] = 'credit_note'
+    ROOT_NAME: ClassVar[str] = "credit_note"
 
     def download(self, resource_id: str) -> Optional[CreditNoteResponse]:
         api_response: Response = send_post_request(
             url=make_url(
                 origin=self.base_url,
-                path_parts=(self.API_RESOURCE, resource_id, 'download'),
+                path_parts=(self.API_RESOURCE, resource_id, "download"),
             ),
             headers=make_headers(api_key=self.api_key),
         )
@@ -42,7 +56,7 @@ def void(self, resource_id: str) -> CreditNoteResponse:
         api_response: Response = send_put_request(
             url=make_url(
                 origin=self.base_url,
-                path_parts=(self.API_RESOURCE, resource_id, 'void'),
+                path_parts=(self.API_RESOURCE, resource_id, "void"),
             ),
             headers=make_headers(api_key=self.api_key),
         )
@@ -56,15 +70,19 @@ def estimate(self, input_object: CreditNoteEstimate) -> CreditNoteEstimatedRespo
         api_response: Response = send_post_request(
             url=make_url(
                 origin=self.base_url,
-                path_parts=(self.API_RESOURCE, 'estimate'),
+                path_parts=(self.API_RESOURCE, "estimate"),
+            ),
+            content=to_json(
+                {
+                    self.ROOT_NAME: input_object.dict(),
+                }
             ),
-            content=to_json({
-                self.ROOT_NAME: input_object.dict(),
-            }),
             headers=make_headers(api_key=self.api_key),
         )
 
         return prepare_object_response(
             response_model=CreditNoteEstimatedResponse,
-            data=get_response_data(response=api_response, key=self.ESTIMATE_API_RESOURCE),
+            data=get_response_data(
+                response=api_response, key=self.ESTIMATE_API_RESOURCE
+            ),
         )
diff --git a/lago_python_client/customers/clients.py b/lago_python_client/customers/clients.py
@@ -2,11 +2,26 @@
 from typing import Any, Mapping, ClassVar, Type, Union
 
 from ..base_client import BaseClient
-from ..mixins import CreateCommandMixin, DestroyCommandMixin, FindAllCommandMixin, FindCommandMixin
+from ..mixins import (
+    CreateCommandMixin,
+    DestroyCommandMixin,
+    FindAllCommandMixin,
+    FindCommandMixin,
+)
 from ..models.customer import CustomerResponse
 from ..models.customer_usage import CustomerUsageResponse
-from ..services.request import make_headers, make_url, send_get_request, send_post_request
-from ..services.response import get_response_data, prepare_index_response, prepare_object_response, Response
+from ..services.request import (
+    make_headers,
+    make_url,
+    send_get_request,
+    send_post_request,
+)
+from ..services.response import (
+    get_response_data,
+    prepare_index_response,
+    prepare_object_response,
+    Response,
+)
 
 if sys.version_info >= (3, 9):
     from collections.abc import Mapping
@@ -21,67 +36,81 @@ class CustomerClient(
     FindCommandMixin[CustomerResponse],
     BaseClient,
 ):
-    API_RESOURCE: ClassVar[str] = 'customers'
+    API_RESOURCE: ClassVar[str] = "customers"
     RESPONSE_MODEL: ClassVar[Type[CustomerResponse]] = CustomerResponse
-    ROOT_NAME: ClassVar[str] = 'customer'
+    ROOT_NAME: ClassVar[str] = "customer"
 
-    def current_usage(self, resource_id: str, external_subscription_id: str) -> CustomerUsageResponse:
+    def current_usage(
+        self, resource_id: str, external_subscription_id: str
+    ) -> CustomerUsageResponse:
         api_response: Response = send_get_request(
             url=make_url(
                 origin=self.base_url,
-                path_parts=(self.API_RESOURCE, resource_id, 'current_usage'),
+                path_parts=(self.API_RESOURCE, resource_id, "current_usage"),
                 query_pairs={
-                    'external_subscription_id': external_subscription_id,
+                    "external_subscription_id": external_subscription_id,
                 },
             ),
             headers=make_headers(api_key=self.api_key),
         )
 
         return prepare_object_response(
             response_model=CustomerUsageResponse,
-            data=get_response_data(response=api_response, key='customer_usage'),
+            data=get_response_data(response=api_response, key="customer_usage"),
         )
 
-    def past_usage(self, resource_id: str, external_subscription_id: str, options: Mapping[str, Union[int, str]] = {}) -> Mapping[str, Any]:
+    def past_usage(
+        self,
+        resource_id: str,
+        external_subscription_id: str,
+        options: Mapping[str, Union[int, str]] = {},
+    ) -> Mapping[str, Any]:
         api_response: Response = send_get_request(
             url=make_url(
                 origin=self.base_url,
-                path_parts=(self.API_RESOURCE, resource_id, 'past_usage'),
+                path_parts=(self.API_RESOURCE, resource_id, "past_usage"),
                 query_pairs={
-                    'external_subscription_id': external_subscription_id,
+                    "external_subscription_id": external_subscription_id,
                     **options,
                 },
             ),
             headers=make_headers(api_key=self.api_key),
         )
 
         return prepare_index_response(
-            api_resource='usage_periods',
+            api_resource="usage_periods",
             response_model=CustomerUsageResponse,
             data=get_response_data(response=api_response),
         )
 
-
     def portal_url(self, resource_id: str) -> str:
         api_response: Response = send_get_request(
             url=make_url(
                 origin=self.base_url,
-                path_parts=(self.API_RESOURCE, resource_id, 'portal_url'),
+                path_parts=(self.API_RESOURCE, resource_id, "portal_url"),
             ),
             headers=make_headers(api_key=self.api_key),
         )
 
         response_data = get_response_data(response=api_response, key=self.ROOT_NAME)
-        return response_data.get('portal_url', '') if isinstance(response_data, Mapping) else ''
+        return (
+            response_data.get("portal_url", "")
+            if isinstance(response_data, Mapping)
+            else ""
+        )
 
     def checkout_url(self, resource_id: str) -> str:
         api_response: Response = send_post_request(
             url=make_url(
                 origin=self.base_url,
-                path_parts=(self.API_RESOURCE, resource_id, 'checkout_url'),
+                path_parts=(self.API_RESOURCE, resource_id, "checkout_url"),
             ),
             headers=make_headers(api_key=self.api_key),
         )
 
         response_data = get_response_data(response=api_response, key=self.ROOT_NAME)
-        return response_data.get('checkout_url', '') if isinstance(response_data, Mapping) else ''
+        return (
+            response_data.get("checkout_url", "")
+            if isinstance(response_data, Mapping)
+            else ""
+        )