refactor: do not use isoparse in expected values

Author: jooola

tests/unit/core/test_domain.py

@@ -1,7 +1,8 @@
 from __future__ import annotations
 
+from datetime import datetime
+
 import pytest
-from dateutil.parser import isoparse
 
 from hcloud.core import BaseDomain, DomainIdentityMixin, Meta, Pagination
 
@@ -99,7 +100,7 @@ class ActionDomain(BaseDomain, DomainIdentityMixin):
     def __init__(self, id, name="name1", started=None):
         self.id = id
         self.name = name
-        self.started = isoparse(started) if started else None
+        self.started = self._parse_datetime(started)
 
 
 class SomeOtherDomain(BaseDomain):
@@ -133,7 +134,7 @@ class TestBaseDomain:
                 {
                     "id": 4,
                     "name": "name-name3",
-                    "started": isoparse("2016-01-30T23:50+00:00"),
+                    "started": datetime.fromisoformat("2016-01-30T23:50+00:00"),
                 },
             ),
         ],

tests/unit/networks/test_client.py

@@ -1,9 +1,9 @@
 from __future__ import annotations
 
+from datetime import datetime, timezone
 from unittest import mock
 
 import pytest
-from dateutil.parser import isoparse
 
 from hcloud import Client
 from hcloud.networks import (
@@ -44,7 +44,9 @@ def test_init(self, network_response):
         )
 
         assert bound_network.id == 1
-        assert bound_network.created == isoparse("2016-01-30T23:50:11+00:00")
+        assert bound_network.created == datetime(
+            2026, 1, 30, 23, 50, 11, tzinfo=timezone.utc
+        )
         assert bound_network.name == "mynet"
         assert bound_network.ip_range == "10.0.0.0/16"
         assert bound_network.protection["delete"] is False

tests/unit/storage_box_types/test_client.py

@@ -2,10 +2,10 @@
 
 from __future__ import annotations
 
+from datetime import datetime, timezone
 from unittest import mock
 
 import pytest
-from dateutil.parser import isoparse
 
 from hcloud import Client
 from hcloud.storage_box_types import (
@@ -58,9 +58,11 @@ def test_get_by_id(
                 "setup_fee": {"gross": "0.0000", "net": "0.0000"},
             }
         ]
-        assert result.deprecation.announced == isoparse("2023-06-01T00:00:00+00:00")
-        assert result.deprecation.unavailable_after == isoparse(
-            "2023-09-01T00:00:00+00:00"
+        assert result.deprecation.announced == datetime(
+            2023, 6, 1, 0, 0, 0, tzinfo=timezone.utc
+        )
+        assert result.deprecation.unavailable_after == datetime(
+            2023, 9, 1, 0, 0, 0, tzinfo=timezone.utc
         )
 
     @pytest.mark.parametrize(

tests/unit/storage_boxes/test_client.py

@@ -2,10 +2,10 @@
 
 from __future__ import annotations
 
+from datetime import datetime, timezone
 from unittest import mock
 
 import pytest
-from dateutil.parser import isoparse
 
 from hcloud import Client
 from hcloud.locations import Location
@@ -122,7 +122,7 @@ def test_init(self, bound_model: BoundStorageBox, resource_client):
         assert o.stats.size_data == 2102612983808
         assert o.stats.size_snapshots == 239623733248
         assert o.status == "active"
-        assert o.created == isoparse("2025-01-30T23:55:00Z")
+        assert o.created == datetime(2025, 1, 30, 23, 50, 00, tzinfo=timezone.utc)
 
 
 class TestBoundStorageBoxSnapshot(BoundModelTestCase):
@@ -156,7 +156,7 @@ def test_init(self, bound_model: BoundStorageBoxSnapshot, resource_client):
         assert o.labels == {"key": "value"}
         assert o.stats.size == 394957594
         assert o.stats.size_filesystem == 3949572745
-        assert o.created == isoparse("2025-11-10T19:16:57Z")
+        assert o.created == datetime(2025, 11, 10, 19, 16, 57, tzinfo=timezone.utc)
 
     def test_reload(
         self,
@@ -232,7 +232,7 @@ def test_init(self, bound_model: BoundStorageBoxSubaccount, resource_client):
         assert o.access_settings.webdav_enabled is False
         assert o.access_settings.readonly is False
         assert o.labels == {"key": "value"}
-        assert o.created == isoparse("2025-11-10T19:18:57Z")
+        assert o.created == datetime(2025, 11, 10, 19, 18, 57, tzinfo=timezone.utc)
 
     def test_reload(
         self,

tests/unit/volumes/test_client.py

@@ -1,9 +1,9 @@
 from __future__ import annotations
 
+from datetime import datetime, timezone
 from unittest import mock
 
 import pytest
-from dateutil.parser import isoparse
 
 from hcloud import Client
 from hcloud.locations import BoundLocation, Location
@@ -37,7 +37,9 @@ def test_bound_volume_init(self, volume_response):
         )
 
         assert bound_volume.id == 1
-        assert bound_volume.created == isoparse("2016-01-30T23:50:11+00:00")
+        assert bound_volume.created == datetime(
+            2016, 1, 30, 23, 50, 11, tzinfo=timezone.utc
+        )
         assert bound_volume.name == "database-storage"
         assert isinstance(bound_volume.server, BoundServer)
         assert bound_volume.server.id == 12

tests/unit/zones/test_client.py

@@ -2,10 +2,10 @@
 
 from __future__ import annotations
 
+from datetime import datetime, timezone
 from unittest import mock
 
 import pytest
-from dateutil.parser import isoparse
 
 from hcloud import Client
 from hcloud.zones import (
@@ -73,7 +73,7 @@ def test_get_using_id(
         )
 
         assert_bound_zone1(result, resource_client)
-        assert result.created == isoparse("2016-01-30T23:55:00+00:00")
+        assert result.created == datetime(2016, 1, 30, 23, 55, 00, tzinfo=timezone.utc)
         assert result.mode == "primary"
         assert result.ttl == 10800
         assert result.protection == {"delete": False}
@@ -89,8 +89,8 @@ def test_get_using_id(
         assert (
             result.authoritative_nameservers.delegated[0] == "hydrogen.ns.hetzner.com."
         )
-        assert result.authoritative_nameservers.delegation_last_check == isoparse(
-            "2016-01-30T23:55:00+00:00"
+        assert result.authoritative_nameservers.delegation_last_check == datetime(
+            2016, 1, 30, 23, 55, 0, tzinfo=timezone.utc
         )
         assert result.authoritative_nameservers.delegation_status == "valid"
 
@@ -935,7 +935,7 @@ def test_init(self, resource_client: ZonesClient, bound_model: BoundZone):
 
         assert o.id == 42
         assert o.name == "example1.com"
-        assert o.created == isoparse("2016-01-30T23:55:00+00:00")
+        assert o.created == datetime(2016, 1, 30, 23, 55, 0, tzinfo=timezone.utc)
         assert o.mode == "primary"
         assert o.ttl == 10800
         assert o.protection == {"delete": False}
@@ -960,8 +960,8 @@ def test_init(self, resource_client: ZonesClient, bound_model: BoundZone):
             "oxygen.ns.hetzner.com.",
             "helium.ns.hetzner.de.",
         ]
-        assert o.authoritative_nameservers.delegation_last_check == isoparse(
-            "2016-01-30T23:55:00+00:00"
+        assert o.authoritative_nameservers.delegation_last_check == datetime(
+            2016, 1, 30, 23, 55, 0, tzinfo=timezone.utc
         )
         assert o.authoritative_nameservers.delegation_status == "valid"