Fix filtering message for leading space

src/stpipe/crds_client.py

@@ -7,6 +7,7 @@
 directly in modules other than this crds_client so that dependency order and
 general integration can be managed here.
 """
+import logging
 import re
 
 import crds
@@ -50,10 +51,13 @@ def get_multiple_reference_paths(parameters, reference_file_types, observatory):
 
     log.set_log_time(True)
 
-    def filter_pars_errors(record):
-        return not record.startswith("Error determining best reference for 'pars-")
+    def parsfilter(record):
+        if not record.getMessage().strip().startswith(
+            "Error determining best reference for 'pars-"
+        ):
+            return True
 
-    log.prepend_crds_filter(filter_pars_errors)
+    logging.getLogger("CRDS").addFilter(parsfilter)
 
     return _get_refpaths(parameters, tuple(reference_file_types), observatory)
 

tests/test_crds_client.py

@@ -7,7 +7,7 @@
     "CRDS logs via stderr and pytest can't capture it. "
     "See https://github.com/pytest-dev/pytest/issues/5997"
 )
-def test_pars_log_filtering(caplog):
+def test_pars_log_filtering(capfd):
     # A bogus pars- reffile will raise an exception in CRDS
     with pytest.raises(Exception, match="Error determining best reference"):
         crds_client.get_multiple_reference_paths(
@@ -33,6 +33,7 @@ def test_pars_log_filtering(caplog):
     # https://github.com/pytest-dev/pytest/issues/5997
     # So don't rely on this test passing (currently) to be actually testing what
     # you think it is.
+    captured = capfd.readouterr()
     assert (
-        "Error determining best reference for 'pars-snowblindstep'" not in caplog.text
+        "Error determining best reference for 'pars-crunchyfrogstep'" not in captured.err
     )