feat: enforce 100% test coverage in CI

Author: boutetnico

.github/workflows/ci.yml

@@ -31,4 +31,4 @@ jobs:
         run: ruff format --check .
 
       - name: Run tests
-        run: pytest -v
+        run: pytest -v --cov=rdapapi --cov-fail-under=100

pyproject.toml

@@ -32,6 +32,7 @@ dependencies = [
 dev = [
     "pytest>=8.0",
     "pytest-asyncio>=0.24",
+    "pytest-cov>=6.0",
     "respx>=0.22",
     "ruff>=0.8",
 ]

tests/test_async.py

@@ -47,6 +47,78 @@
 }
 
 
+IP_RESPONSE = {
+    "handle": "NET-8-8-8-0-2",
+    "name": "GOGL",
+    "type": "DIRECT ALLOCATION",
+    "start_address": "8.8.8.0",
+    "end_address": "8.8.8.255",
+    "ip_version": "v4",
+    "parent_handle": "NET-8-0-0-0-0",
+    "country": None,
+    "status": ["active"],
+    "dates": {"registered": "2023-12-28T17:24:33-05:00", "expires": None, "updated": None},
+    "entities": {},
+    "cidr": ["8.8.8.0/24"],
+    "remarks": [],
+    "port43": "whois.arin.net",
+    "meta": {
+        "rdap_server": "https://rdap.arin.net/registry/",
+        "raw_rdap_url": "https://rdap.arin.net/registry/ip/8.8.8.8",
+        "cached": False,
+        "cache_expires": "2026-02-24T15:30:00Z",
+    },
+}
+
+NS_RESPONSE = {
+    "ldh_name": "ns1.google.com",
+    "unicode_name": None,
+    "handle": None,
+    "ip_addresses": {"v4": ["216.239.32.10"], "v6": []},
+    "status": [],
+    "dates": {"registered": None, "expires": None, "updated": None},
+    "entities": {},
+    "meta": {
+        "rdap_server": "https://rdap.verisign.com/com/v1/",
+        "raw_rdap_url": "https://rdap.verisign.com/com/v1/nameserver/ns1.google.com",
+        "cached": False,
+        "cache_expires": "2026-02-24T15:30:00Z",
+    },
+}
+
+ENTITY_RESPONSE = {
+    "handle": "GOGL",
+    "name": "Google LLC",
+    "organization": None,
+    "email": None,
+    "phone": None,
+    "address": None,
+    "contact_url": None,
+    "country_code": None,
+    "roles": [],
+    "status": [],
+    "dates": {"registered": "2000-03-30T00:00:00-04:00", "expires": None, "updated": None},
+    "remarks": [],
+    "port43": "whois.arin.net",
+    "public_ids": [],
+    "entities": {},
+    "autnums": [],
+    "networks": [],
+    "meta": {
+        "rdap_server": "https://rdap.arin.net/registry/",
+        "raw_rdap_url": "https://rdap.arin.net/registry/entity/GOGL",
+        "cached": False,
+        "cache_expires": "2026-02-24T15:30:00Z",
+    },
+}
+
+
+@pytest.mark.asyncio
+async def test_async_empty_api_key_raises():
+    with pytest.raises(ValueError, match="non-empty"):
+        AsyncRdapApi("")
+
+
 @pytest.mark.asyncio
 @respx.mock
 async def test_async_domain_lookup():
@@ -190,3 +262,52 @@ async def test_async_bulk_domains_plan_upgrade_required():
             await api.bulk_domains(["google.com"])
 
     assert exc_info.value.error == "plan_upgrade_required"
+
+
+@pytest.mark.asyncio
+@respx.mock
+async def test_async_ip_lookup():
+    respx.get(f"{BASE_URL}/ip/8.8.8.8").mock(return_value=httpx.Response(200, json=IP_RESPONSE))
+
+    async with AsyncRdapApi("test-key", base_url=BASE_URL) as api:
+        result = await api.ip("8.8.8.8")
+
+    assert result.handle == "NET-8-8-8-0-2"
+    assert result.ip_version == "v4"
+
+
+@pytest.mark.asyncio
+@respx.mock
+async def test_async_nameserver_lookup():
+    respx.get(f"{BASE_URL}/nameserver/ns1.google.com").mock(return_value=httpx.Response(200, json=NS_RESPONSE))
+
+    async with AsyncRdapApi("test-key", base_url=BASE_URL) as api:
+        result = await api.nameserver("ns1.google.com")
+
+    assert result.ldh_name == "ns1.google.com"
+
+
+@pytest.mark.asyncio
+@respx.mock
+async def test_async_entity_lookup():
+    respx.get(f"{BASE_URL}/entity/GOGL").mock(return_value=httpx.Response(200, json=ENTITY_RESPONSE))
+
+    async with AsyncRdapApi("test-key", base_url=BASE_URL) as api:
+        result = await api.entity("GOGL")
+
+    assert result.handle == "GOGL"
+    assert result.name == "Google LLC"
+
+
+@pytest.mark.asyncio
+@respx.mock
+async def test_async_bulk_domains_with_follow():
+    import json
+
+    route = respx.post(f"{BASE_URL}/domains/bulk").mock(return_value=httpx.Response(200, json=BULK_RESPONSE))
+
+    async with AsyncRdapApi("test-key", base_url=BASE_URL) as api:
+        await api.bulk_domains(["google.com"], follow=True)
+
+    body = json.loads(route.calls[0].request.content)
+    assert body["follow"] is True

tests/test_client.py

@@ -9,6 +9,7 @@
     NotFoundError,
     RateLimitError,
     RdapApi,
+    RdapApiError,
     SubscriptionRequiredError,
     UpstreamError,
     ValidationError,
@@ -362,6 +363,24 @@ def test_upstream_error():
     api.close()
 
 
+def test_empty_api_key_raises():
+    with pytest.raises(ValueError, match="non-empty"):
+        RdapApi("")
+
+
+@respx.mock
+def test_non_json_error_body():
+    respx.get(f"{BASE_URL}/domain/test.com").mock(return_value=httpx.Response(500, text="Internal Server Error"))
+
+    api = RdapApi("test-key", base_url=BASE_URL)
+    with pytest.raises(RdapApiError) as exc_info:
+        api.domain("test.com")
+
+    assert exc_info.value.status_code == 500
+    assert exc_info.value.error == "unknown_error"
+    api.close()
+
+
 @respx.mock
 def test_auth_header_sent():
     route = respx.get(f"{BASE_URL}/domain/google.com").mock(return_value=httpx.Response(200, json=DOMAIN_RESPONSE))