Add network_zone to Locations

Author: LKaemmerling

hcloud/locations/domain.py

@@ -19,6 +19,8 @@ class Location(BaseDomain, DomainIdentityMixin):
            Latitude of the city closest to the location
     :param longitude: float
            Longitude of the city closest to the location
+    :param network_zone: str
+           Name of network zone this location resides in
     """
 
     __slots__ = (
@@ -28,7 +30,8 @@ class Location(BaseDomain, DomainIdentityMixin):
         "country",
         "city",
         "latitude",
-        "longitude"
+        "longitude",
+        "network_zone"
     )
 
     def __init__(
@@ -39,7 +42,8 @@ def __init__(
             country=None,
             city=None,
             latitude=None,
-            longitude=None
+            longitude=None,
+            network_zone=None,
     ):
         self.id = id
         self.name = name
@@ -48,3 +52,4 @@ def __init__(
         self.city = city
         self.latitude = latitude
         self.longitude = longitude
+        self.network_zone = network_zone

tests/unit/locations/conftest.py

@@ -11,7 +11,8 @@ def location_response():
             "country": "DE",
             "city": "Falkenstein",
             "latitude": 50.47612,
-            "longitude": 12.370071
+            "longitude": 12.370071,
+            "network_zone": "eu-central",
         }
     }
 
@@ -28,7 +29,8 @@ def two_locations_response():
                 "country": "DE",
                 "city": "Falkenstein",
                 "latitude": 50.47612,
-                "longitude": 12.370071
+                "longitude": 12.370071,
+                "network_zone": "eu-central",
 
             },
             {
@@ -38,7 +40,8 @@ def two_locations_response():
                 "country": "DE",
                 "city": "Nuremberg",
                 "latitude": 49.452102,
-                "longitude": 11.076665
+                "longitude": 11.076665,
+                "network_zone": "eu-central",
             }
         ]
     }
@@ -56,8 +59,8 @@ def one_locations_response():
                 "country": "DE",
                 "city": "Falkenstein",
                 "latitude": 50.47612,
-                "longitude": 12.370071
-
+                "longitude": 12.370071,
+                "network_zone": "eu-central",
             }
         ]
     }

tests/unit/locations/test_client.py

@@ -17,6 +17,7 @@ def test_get_by_id(self, locations_client, location_response):
         assert location._client is locations_client
         assert location.id == 1
         assert location.name == "fsn1"
+        assert location.network_zone == "eu-central"
 
     @pytest.mark.parametrize("params", [{'name': "fsn1", "page": 1, "per_page": 10}, {}])
     def test_get_list(self, locations_client, two_locations_response, params):
@@ -35,10 +36,12 @@ def test_get_list(self, locations_client, two_locations_response, params):
         assert location1._client is locations_client
         assert location1.id == 1
         assert location1.name == "fsn1"
+        assert location1.network_zone == "eu-central"
 
         assert location2._client is locations_client
         assert location2.id == 2
         assert location2.name == "nbg1"
+        assert location2.network_zone == "eu-central"
 
     @pytest.mark.parametrize("params", [{'name': "fsn1"}, {}])
     def test_get_all(self, locations_client, two_locations_response, params):
@@ -57,10 +60,12 @@ def test_get_all(self, locations_client, two_locations_response, params):
         assert location1._client is locations_client
         assert location1.id == 1
         assert location1.name == "fsn1"
+        assert location1.network_zone == "eu-central"
 
         assert location2._client is locations_client
         assert location2.id == 2
         assert location2.name == "nbg1"
+        assert location2.network_zone == "eu-central"
 
     def test_get_by_name(self, locations_client, one_locations_response):
         locations_client._client.request.return_value = one_locations_response
@@ -73,3 +78,4 @@ def test_get_by_name(self, locations_client, one_locations_response):
         assert location._client is locations_client
         assert location.id == 1
         assert location.name == "fsn1"
+        assert location.network_zone == "eu-central"

tox.ini

@@ -11,6 +11,6 @@ passenv = FAKE_API_ENDPOINT
 deps =
     -r{toxinidir}/requirements/test.txt
 commands =
-    pytest
+    pytest {posargs}