refactor: allow differences between api object properties and slots (#421)

Allow differences between the classes slots and the API object
properties.

Author: jooola

hcloud/actions/domain.py

@@ -31,7 +31,7 @@ class Action(BaseDomain):
     STATUS_ERROR = "error"
     """Action Status error"""
 
-    __slots__ = (
+    __api_properties__ = (
         "id",
         "command",
         "status",
@@ -41,6 +41,7 @@ class Action(BaseDomain):
         "started",
         "finished",
     )
+    __slots__ = __api_properties__
 
     def __init__(
         self,

hcloud/certificates/domain.py

@@ -31,7 +31,7 @@ class Certificate(BaseDomain, DomainIdentityMixin):
     :param status: ManagedCertificateStatus Current status of a type managed Certificate, always none for type uploaded Certificates
     """
 
-    __slots__ = (
+    __api_properties__ = (
         "id",
         "name",
         "certificate",
@@ -44,6 +44,8 @@ class Certificate(BaseDomain, DomainIdentityMixin):
         "type",
         "status",
     )
+    __slots__ = __api_properties__
+
     TYPE_UPLOADED = "uploaded"
     TYPE_MANAGED = "managed"
 
@@ -119,7 +121,8 @@ class CreateManagedCertificateResponse(BaseDomain):
            Shows the progress of the certificate creation
     """
 
-    __slots__ = ("certificate", "action")
+    __api_properties__ = ("certificate", "action")
+    __slots__ = __api_properties__
 
     def __init__(
         self,

hcloud/core/domain.py

@@ -2,23 +2,22 @@
 
 
 class BaseDomain:
-    __slots__ = ()
+    __api_properties__: tuple
 
     @classmethod
     def from_dict(cls, data: dict):  # type: ignore[no-untyped-def]
         """
         Build the domain object from the data dict.
         """
-        supported_data = {k: v for k, v in data.items() if k in cls.__slots__}
+        supported_data = {k: v for k, v in data.items() if k in cls.__api_properties__}
         return cls(**supported_data)
 
     def __repr__(self) -> str:
-        kwargs = [f"{key}={getattr(self, key)!r}" for key in self.__slots__]  # type: ignore[var-annotated]
+        kwargs = [f"{key}={getattr(self, key)!r}" for key in self.__api_properties__]  # type: ignore[var-annotated]
         return f"{self.__class__.__qualname__}({', '.join(kwargs)})"
 
 
 class DomainIdentityMixin:
-    __slots__ = ()
 
     id: int | None
     name: str | None
@@ -54,14 +53,15 @@ def has_id_or_name(self, id_or_name: int | str) -> bool:
 
 
 class Pagination(BaseDomain):
-    __slots__ = (
+    __api_properties__ = (
         "page",
         "per_page",
         "previous_page",
         "next_page",
         "last_page",
         "total_entries",
     )
+    __slots__ = __api_properties__
 
     def __init__(
         self,
@@ -81,7 +81,8 @@ def __init__(
 
 
 class Meta(BaseDomain):
-    __slots__ = ("pagination",)
+    __api_properties__ = ("pagination",)
+    __slots__ = __api_properties__
 
     def __init__(self, pagination: Pagination | None = None):
         self.pagination = pagination

hcloud/datacenters/domain.py

@@ -19,7 +19,8 @@ class Datacenter(BaseDomain, DomainIdentityMixin):
     :param server_types: :class:`DatacenterServerTypes <hcloud.datacenters.domain.DatacenterServerTypes>`
     """
 
-    __slots__ = ("id", "name", "description", "location", "server_types")
+    __api_properties__ = ("id", "name", "description", "location", "server_types")
+    __slots__ = __api_properties__
 
     def __init__(
         self,
@@ -47,7 +48,8 @@ class DatacenterServerTypes(BaseDomain):
            All available for migration (change type) server types for this datacenter
     """
 
-    __slots__ = ("available", "supported", "available_for_migration")
+    __api_properties__ = ("available", "supported", "available_for_migration")
+    __slots__ = __api_properties__
 
     def __init__(
         self,

hcloud/deprecation/domain.py

@@ -17,10 +17,11 @@ class DeprecationInfo(BaseDomain):
            new servers with this image after the mentioned date.
     """
 
-    __slots__ = (
+    __api_properties__ = (
         "announced",
         "unavailable_after",
     )
+    __slots__ = __api_properties__
 
     def __init__(
         self,

hcloud/firewalls/domain.py

@@ -29,7 +29,8 @@ class Firewall(BaseDomain, DomainIdentityMixin):
            Point in time when the image was created
     """
 
-    __slots__ = ("id", "name", "labels", "rules", "applied_to", "created")
+    __api_properties__ = ("id", "name", "labels", "rules", "applied_to", "created")
+    __slots__ = __api_properties__
 
     def __init__(
         self,
@@ -66,14 +67,15 @@ class FirewallRule(BaseDomain):
            Short description of the firewall rule
     """
 
-    __slots__ = (
+    __api_properties__ = (
         "direction",
         "port",
         "protocol",
         "source_ips",
         "destination_ips",
         "description",
     )
+    __slots__ = __api_properties__
 
     DIRECTION_IN = "in"
     """Firewall Rule Direction In"""
@@ -138,7 +140,8 @@ class FirewallResource(BaseDomain):
            applied to.
     """
 
-    __slots__ = ("type", "server", "label_selector", "applied_to_resources")
+    __api_properties__ = ("type", "server", "label_selector", "applied_to_resources")
+    __slots__ = __api_properties__
 
     TYPE_SERVER = "server"
     """Firewall Used By Type Server"""
@@ -177,7 +180,8 @@ class FirewallResourceAppliedToResources(BaseDomain):
     :param server: Server the Firewall is applied to
     """
 
-    __slots__ = ("type", "server")
+    __api_properties__ = ("type", "server")
+    __slots__ = __api_properties__
 
     def __init__(
         self,
@@ -207,7 +211,8 @@ class CreateFirewallResponse(BaseDomain):
            The Action which shows the progress of the Firewall Creation
     """
 
-    __slots__ = ("firewall", "actions")
+    __api_properties__ = ("firewall", "actions")
+    __slots__ = __api_properties__
 
     def __init__(
         self,

hcloud/floating_ips/domain.py

@@ -42,7 +42,7 @@ class FloatingIP(BaseDomain, DomainIdentityMixin):
            Name of the Floating IP
     """
 
-    __slots__ = (
+    __api_properties__ = (
         "id",
         "type",
         "description",
@@ -56,6 +56,7 @@ class FloatingIP(BaseDomain, DomainIdentityMixin):
         "name",
         "created",
     )
+    __slots__ = __api_properties__
 
     def __init__(
         self,
@@ -95,7 +96,8 @@ class CreateFloatingIPResponse(BaseDomain):
            The Action which shows the progress of the Floating IP Creation
     """
 
-    __slots__ = ("floating_ip", "action")
+    __api_properties__ = ("floating_ip", "action")
+    __slots__ = __api_properties__
 
     def __init__(
         self,

hcloud/images/domain.py

@@ -51,7 +51,7 @@ class Image(BaseDomain, DomainIdentityMixin):
             User-defined labels (key-value pairs)
     """
 
-    __slots__ = (
+    __api_properties__ = (
         "id",
         "name",
         "type",
@@ -70,6 +70,7 @@ class Image(BaseDomain, DomainIdentityMixin):
         "created",
         "deprecated",
     )
+    __slots__ = __api_properties__
 
     # pylint: disable=too-many-locals
     def __init__(
@@ -120,7 +121,8 @@ class CreateImageResponse(BaseDomain):
            The Action which shows the progress of the Floating IP Creation
     """
 
-    __slots__ = ("action", "image")
+    __api_properties__ = ("action", "image")
+    __slots__ = __api_properties__
 
     def __init__(
         self,

hcloud/isos/domain.py

@@ -27,14 +27,15 @@ class Iso(BaseDomain, DomainIdentityMixin):
         deprecated. If it has a value, it is considered deprecated.
     """
 
-    __slots__ = (
+    __api_properties__ = (
         "id",
         "name",
         "type",
         "architecture",
         "description",
         "deprecation",
     )
+    __slots__ = __api_properties__
 
     def __init__(
         self,

hcloud/load_balancer_types/domain.py

@@ -25,7 +25,7 @@ class LoadBalancerType(BaseDomain, DomainIdentityMixin):
 
     """
 
-    __slots__ = (
+    __api_properties__ = (
         "id",
         "name",
         "description",
@@ -35,6 +35,7 @@ class LoadBalancerType(BaseDomain, DomainIdentityMixin):
         "max_assigned_certificates",
         "prices",
     )
+    __slots__ = __api_properties__
 
     def __init__(
         self,