python-sdk: Support regions/vpc-availability endpoints

Author: mgwoj

linode_api4/groups/region.py

@@ -47,26 +47,6 @@ def availability(self, *filters):
             RegionAvailabilityEntry, *filters, endpoint="/regions/availability"
         )
 
-    def availability_get(self, region_id):
-        """
-        Returns availability data for a specific region.
-
-        API Documentation: https://techdocs.akamai.com/linode-api/reference/get-region-availability
-
-        :param region_id: The ID of the region to retrieve availability for.
-        :type region_id: str
-
-        :returns: A list of availability entries for the specified region.
-        :rtype: list of RegionAvailabilityEntry
-        """
-
-        result = self.client.get(f"/regions/{region_id}/availability")
-
-        if result is None:
-            return []
-
-        return [RegionAvailabilityEntry.from_json(v) for v in result]
-
     def vpc_availability(self, *filters):
         """
         Returns VPC availability data for all regions.
@@ -97,25 +77,3 @@ def vpc_availability(self, *filters):
             *filters,
             endpoint="/regions/vpc-availability",
         )
-
-    def vpc_availability_get(self, region_id):
-        """
-        Returns VPC availability data for a specific region.
-
-        NOTE: IPv6 VPCs may not currently be available to all users.
-
-        API Documentation: https://techdocs.akamai.com/linode-api/reference/get-region-vpc-availability
-
-        :param region_id: The ID of the region to retrieve VPC availability for.
-        :type region_id: str
-
-        :returns: VPC availability data for the specified region.
-        :rtype: RegionVPCAvailability
-        """
-
-        result = self.client.get(f"/regions/{region_id}/vpc-availability")
-
-        if result is None:
-            return None
-
-        return RegionVPCAvailability.from_json(result)

test/integration/models/region/test_region.py

@@ -24,28 +24,6 @@ def test_list_regions_vpc_availability(test_linode_client):
         assert isinstance(entry.available_ipv6_prefix_lengths, list)
 
 
-@pytest.mark.smoke
-def test_get_region_vpc_availability_via_group(test_linode_client):
-    """
-    Test getting VPC availability for a specific region via RegionGroup.
-    """
-    client = test_linode_client
-
-    # Get the first available region
-    regions = client.regions()
-    assert len(regions) > 0
-    test_region_id = regions[0].id
-
-    vpc_avail = client.regions.vpc_availability_get(test_region_id)
-
-    assert vpc_avail is not None
-    assert vpc_avail.region == test_region_id
-    assert vpc_avail.available is not None
-    assert isinstance(vpc_avail.available, bool)
-    assert vpc_avail.available_ipv6_prefix_lengths is not None
-    assert isinstance(vpc_avail.available_ipv6_prefix_lengths, list)
-
-
 @pytest.mark.smoke
 def test_get_region_vpc_availability_via_object(test_linode_client):
     """
@@ -69,30 +47,6 @@ def test_get_region_vpc_availability_via_object(test_linode_client):
     assert isinstance(vpc_avail.available_ipv6_prefix_lengths, list)
 
 
-@pytest.mark.smoke
-def test_get_region_availability_via_group(test_linode_client):
-    """
-    Test getting availability for a specific region via RegionGroup.
-    """
-    client = test_linode_client
-
-    # Get the first available region
-    regions = client.regions()
-    assert len(regions) > 0
-    test_region_id = regions[0].id
-
-    avail_entries = client.regions.availability_get(test_region_id)
-
-    assert len(avail_entries) > 0
-
-    for entry in avail_entries:
-        assert entry.region == test_region_id
-        assert entry.plan is not None
-        assert len(entry.plan) > 0
-        assert entry.available is not None
-        assert isinstance(entry.available, bool)
-
-
 def test_vpc_availability_available_regions(test_linode_client):
     """
     Test that some regions have VPC availability enabled.

test/unit/groups/region_test.py

@@ -23,6 +23,7 @@ def test_list_availability(self):
             assert len(avail_entries) > 0
 
             for entry in avail_entries:
+                assert entry.region is not None
                 assert len(entry.region) > 0
                 assert len(entry.plan) > 0
                 assert entry.available is not None
@@ -72,32 +73,3 @@ def test_list_vpc_availability(self):
                 m.mock.call_args_list[1].args[0]
                 == "//regions/vpc-availability?page=2&page_size=25"
             )
-
-    def test_get_vpc_availability(self):
-        """
-        Tests that VPC availability for a specific region can be retrieved.
-        """
-
-        with self.mock_get("/regions/us-east/vpc-availability") as m:
-            vpc_avail = self.client.regions.vpc_availability_get("us-east")
-
-            assert vpc_avail is not None
-            assert vpc_avail.region == "us-east"
-            assert vpc_avail.available is True
-            assert vpc_avail.available_ipv6_prefix_lengths == [52, 48]
-
-    def test_get_availability(self):
-        """
-        Tests that availability for a specific region can be retrieved.
-        """
-
-        with self.mock_get("/regions/us-east/availability") as m:
-            avail_entries = self.client.regions.availability_get("us-east")
-
-            assert len(avail_entries) > 0
-
-            # Verify first entry to ensure deserialization works
-            first_entry = avail_entries[0]
-            assert first_entry.region == "us-east"
-            assert len(first_entry.plan) > 0
-            assert first_entry.available is not None