Reject list-of-non-strings at boundary instead of silently passing through

Live stress test found that `parameter_code=[60, 65]` (ints) was silently
passed to the OGC API, surfacing as a confusing JSONDecodeError when the
server returned an error page. The "list-of-non-str pass-through" clause
in `_get_args` was a defensive shortcut intended for `bbox`/`boundingBox`
(which are `list[float]`), but those params are already covered by
`_NO_NORMALIZE_PARAMS`, making the clause redundant AND bug-silencing.

Now `_normalize_str_iterable` runs for every non-listed list-shaped
param, raising TypeError with the offending element type — same path
that already handles `monitoring_location_id=[..., 12345]`.

Author: thodson-usgs

dataretrieval/waterdata/utils.py

@@ -1329,7 +1329,6 @@ def _get_args(
             k in _NO_NORMALIZE_PARAMS
             or isinstance(v, str)
             or not isinstance(v, Iterable)
-            or (isinstance(v, (list, tuple)) and v and not isinstance(v[0], str))
         ):
             args[k] = v
         else:

tests/waterdata_test.py

@@ -556,3 +556,17 @@ def test_get_daily_parameter_code_as_series(self):
         args_dict = fake.call_args[0][0]
         assert args_dict["parameter_code"] == ["00060", "00010"]
         assert isinstance(args_dict["parameter_code"], list)
+
+    def test_list_of_ints_rejected_at_boundary(self):
+        """List-of-non-strings must be caught client-side, not silently sent.
+
+        Regression: an earlier pass through ``_get_args`` had a
+        ``list-of-non-str`` fast-path that bypassed normalization, so
+        ``parameter_code=[60, 65]`` would reach the OGC API and surface as
+        a confusing JSONDecodeError on the malformed response.
+        """
+        with pytest.raises(TypeError, match="parameter_code elements must be strings"):
+            get_daily(
+                monitoring_location_id="USGS-05427718",
+                parameter_code=[60, 65],
+            )