test(waterdata): Merge redundant RequestExceedsQuota tests

``test_request_exceeds_quota_carries_call_handle`` used the same
fetch fixture as ``test_request_exceeds_quota_after_first_chunk``;
folded the ``.call`` assertions into the existing test and dropped
the duplicate.

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>

Author: thodson-usgs

tests/waterdata_chunking_test.py

@@ -421,7 +421,9 @@ def test_request_exceeds_quota_after_first_chunk():
     """Plan totals 4 sub-requests. The first response reports
     ``x-ratelimit-remaining=1`` — only 2 sub-requests fit total
     (the one just issued + 1 more). The wrapper must raise
-    ``RequestExceedsQuota`` *before* issuing chunk 2."""
+    ``RequestExceedsQuota`` *before* issuing chunk 2, and the
+    exception must carry a ``.call`` handle so the first chunk's
+    already-fetched data is recoverable."""
     calls: list[dict] = []
 
     def fetch(args):
@@ -438,6 +440,11 @@ def fetch(args):
     assert err.available == 2  # remaining=1 + the chunk we just spent
     assert err.deficit == 2
     assert len(calls) == 1, "only the first chunk should have been issued"
+    # The originating ChunkedCall is exposed on .call so the first
+    # chunk's already-fetched data is recoverable.
+    assert err.call is not None
+    assert err.call.completed_chunks == 1
+    assert not err.call.partial_frame.empty
 
 
 def test_request_exceeds_quota_message_reports_deficit():
@@ -1262,22 +1269,3 @@ def fetch(args):
     # .call carries the in-flight call so the user can recover.
     assert err.call is not None
     assert err.call.completed_chunks == 3
-
-
-def test_request_exceeds_quota_carries_call_handle():
-    """Regression: ``RequestExceedsQuota`` was a bare ``ValueError``
-    with no ``.call`` attribute, so the first chunk's already-fetched
-    data was unrecoverable. The exception now carries the originating
-    ``ChunkedCall`` for symmetry with ``ChunkInterrupted``."""
-
-    def fetch(args):
-        return pd.DataFrame({"sites": list(args["sites"])}), _quota_response(1)
-
-    decorated = multi_value_chunked(build_request=_fake_build, url_limit=240)(fetch)
-    with pytest.raises(RequestExceedsQuota) as excinfo:
-        decorated({"sites": ["S1" * 10, "S2" * 10, "S3" * 10, "S4" * 10]})
-    err = excinfo.value
-    assert err.call is not None
-    # The already-fetched first chunk is accessible via .call.partial_frame.
-    assert not err.call.partial_frame.empty
-    assert err.call.completed_chunks == 1