test: prune tautological / redundant tests

Follow-up to DOI-USGS#334 (mocking the live NWIS tests). Once mocked, two NWIS classes
became tautological — they assert the columns/shape of the fixture they feed,
exercising no production logic the mocked-out parse doesn't already cover:

- nwis_test.py: delete TestSiteseriesCatalogOutput (DOI-USGS#34 — RDB column parsing is
  covered by rdb_test.py + TestReadRdb, and the seriesCatalogOutput param is not
  exercised because the mock returns the fixture regardless) and TestTZ (DOI-USGS#60 —
  now identical to test_iv_service_answer: same fixture, same datetime-index
  assertion). Drop the orphaned nwis_site_seriescatalog.txt fixture and the
  now-unused imports.
- waterdata_filters_test.py: delete test_construct_filter_passthrough — `filter`
  is forwarded verbatim, so it is subsumed by the parametrized
  test_construct_filter_on_all_ogc_services (8 services + filter-lang) and the
  long-filter fan-out tests.

A 4-agent review of the full suite (17 files) found the rest well-curated; these
are the only defensible deletions.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01Sjb14HkwuCydKSKMsaXsgd

Author: thodson-usgs

tests/data/nwis_site_seriescatalog.txt

@@ -1,4 +1,3 @@
-import datetime
 import json
 import re
 import warnings
@@ -14,14 +13,12 @@
     _read_rdb,
     get_discharge_measurements,
     get_gwlevels,
-    get_info,
     get_iv,
     get_pmcodes,
     get_qwdata,
     get_record,
     get_water_use,
     preformat_peaks_response,
-    what_sites,
 )
 
 START_DATE = "2018-01-24"
@@ -30,9 +27,8 @@
 DATETIME_COL = "datetime"
 SITENO_COL = "site_no"
 
-# Legacy NWIS endpoints these tests mock — this module makes no live calls.
+# Legacy NWIS site endpoint these tests mock — this module makes no live calls.
 _SITE_RE = re.compile(r"^https://waterservices\.usgs\.gov/nwis/site(\?.*)?$")
-_IV_RE = re.compile(r"^https://waterservices\.usgs\.gov/nwis/iv(\?.*)?$")
 
 
 def _load_mock_json(file_name):
@@ -223,86 +219,6 @@ def test_get_record_defunct_service_water_use(self):
             get_record(service="water_use")
 
 
-class TestTZ:
-    """Tests relating to GitHub Issue #60 — merging IV results across sites
-    yields a proper datetime index. Mocked against fixture responses."""
-
-    def _mock(self, httpx_mock):
-        _mock_site(httpx_mock)
-        httpx_mock.add_response(
-            method="GET", url=_IV_RE, json=_load_mock_json("nwis_iv_mock.json")
-        )
-
-    def test_multiple_tz_01(self, httpx_mock):
-        """Issue #60 - merging IV across sites yields a datetime index."""
-        self._mock(httpx_mock)
-        sites, _ = what_sites(stateCd="MD")
-        iv, _ = get_iv(sites=sites.site_no.values[:25].tolist())
-        assert "datetime" in iv.index.names
-        assert isinstance(iv.index[0][1], datetime.datetime)
-
-    def test_multiple_tz_02(self, httpx_mock):
-        """Issue #60 - the same-tz path also yields a datetime index."""
-        self._mock(httpx_mock)
-        sites, _ = what_sites(stateCd="MD")
-        iv, _ = get_iv(sites=sites.site_no.values[:20].tolist())
-        assert "datetime" in iv.index.names
-        assert isinstance(iv.index[0][1], datetime.datetime)
-
-
-class TestSiteseriesCatalogOutput:
-    """Tests relating to GitHub Issue #34 — ``seriesCatalogOutput`` adds the
-    data-inventory columns (begin_date / end_date / count_nu). Mocked against
-    fixture responses (the chosen fixture, not the request param, decides which
-    columns come back)."""
-
-    _SERIESCATALOG = "nwis_site_seriescatalog.txt"
-
-    def test_seriesCatalogOutput_get_record(self, httpx_mock):
-        """seriesCatalogOutput=True with get_record exposes inventory columns."""
-        _mock_site(httpx_mock, self._SERIESCATALOG)
-        data = get_record(
-            huc="20", parameterCd="00060", service="site", seriesCatalogOutput="True"
-        )
-        assert "begin_date" in data.columns
-        assert "end_date" in data.columns
-        assert "count_nu" in data.columns
-
-    def test_seriesCatalogOutput_get_info(self, httpx_mock):
-        """seriesCatalogOutput=TRUE with get_info exposes inventory columns."""
-        _mock_site(httpx_mock, self._SERIESCATALOG)
-        data, _ = get_info(huc="20", parameterCd="00060", seriesCatalogOutput="TRUE")
-        assert "begin_date" in data.columns
-        assert "end_date" in data.columns
-        assert "count_nu" in data.columns
-
-    def test_seriesCatalogOutput_bool(self, httpx_mock):
-        """A boolean seriesCatalogOutput is accepted and exposes inventory cols."""
-        _mock_site(httpx_mock, self._SERIESCATALOG)
-        data, _ = get_info(huc="20", parameterCd="00060", seriesCatalogOutput=True)
-        assert "begin_date" in data.columns
-        assert "end_date" in data.columns
-        assert "count_nu" in data.columns
-
-    def test_expandedrdb_get_record(self, httpx_mock):
-        """The default expanded-rdb format omits the inventory columns."""
-        _mock_site(httpx_mock)
-        data = get_record(
-            huc="20", parameterCd="00060", service="site", seriesCatalogOutput="False"
-        )
-        assert "begin_date" not in data.columns
-        assert "end_date" not in data.columns
-        assert "count_nu" not in data.columns
-
-    def test_expandedrdb_get_info(self, httpx_mock):
-        """get_info default omits the inventory columns."""
-        _mock_site(httpx_mock)
-        data, _ = get_info(huc="20", parameterCd="00060")
-        assert "begin_date" not in data.columns
-        assert "end_date" not in data.columns
-        assert "count_nu" not in data.columns
-
-
 def test_empty_timeseries(httpx_mock):
     """Test based on empty case from GitHub Issue #26."""
     sites = "011277906"

tests/nwis_test.py

@@ -41,22 +41,6 @@ def test_quote_cql_str_doubles_embedded_quotes():
     assert _quote_cql_str("a'b'c") == "a''b''c"
 
 
-def test_construct_filter_passthrough():
-    """`filter` is forwarded verbatim as a query parameter."""
-    expr = (
-        "(time >= '2023-01-06T16:00:00Z' AND time <= '2023-01-06T18:00:00Z') "
-        "OR (time >= '2023-01-10T18:00:00Z' AND time <= '2023-01-10T20:00:00Z')"
-    )
-    req = _construct_api_requests(
-        service="continuous",
-        monitoring_location_id="USGS-07374525",
-        parameter_code="72255",
-        filter=expr,
-    )
-    qs = _query_params(req)
-    assert qs["filter"] == [expr]
-
-
 def test_construct_filter_lang_hyphenated():
     """The Python kwarg `filter_lang` is sent as URL key `filter-lang`."""
     req = _construct_api_requests(