test: add ignore() context manager test coverage

- Add ignore() tests to all test classes that previously only tested strict()
- Use get_collections() for ignore() tests since it emits warnings (unlike search() which raises exceptions directly)
- Achieve 100% coverage for warnings.py module

Author: nlebovits

pystac_client/collection_search.py

@@ -304,17 +304,21 @@ def __init__(
             if any([bbox, datetime, q, query, filter, sortby, fields]):
                 if not self._collection_search_extension_enabled:
                     warnings.warn(
-                        str(DoesNotConformTo("COLLECTION_SEARCH"))
-                        + ". Filtering will be performed client-side where only bbox, "
-                        "datetime, and q arguments are supported"
+                        DoesNotConformTo(
+                            "COLLECTION_SEARCH",
+                            "Filtering will be performed client-side where only bbox, "
+                            "datetime, and q arguments are supported",
+                        )
                     )
                     self._validate_client_side_args()
                 else:
                     if not self._collection_search_free_text_enabled:
                         warnings.warn(
-                            str(DoesNotConformTo("COLLECTION_SEARCH#FREE_TEXT"))
-                            + ". Free-text search is not enabled for collection search"
-                            "Free-text filters will be applied client-side."
+                            DoesNotConformTo(
+                                "COLLECTION_SEARCH#FREE_TEXT",
+                                "Free-text search is not enabled for collection search"
+                                "Free-text filters will be applied client-side.",
+                            )
                         )
 
         else:

pystac_client/warnings.py

@@ -79,7 +79,7 @@ def ignore() -> Iterator[None]:
     >>> from pystac_client import Client
     >>> from pystac_client.warnings import ignore
     >>> with ignore():
-    ...     Client.open("https://perfect-api.test")
+    ...     Client.open("https://imperfect-api.test")
 
     For finer-grained control:
 

tests/test_warnings.py

@@ -0,0 +1,44 @@
+import pytest
+
+from pystac_client import Client
+from pystac_client.warnings import DoesNotConformTo, ignore, strict
+
+from .helpers import TEST_DATA
+
+
+class TestWarningContextManagers:
+    @pytest.mark.filterwarnings("error")
+    def test_ignore_context_manager(self) -> None:
+        """Test that ignore() context manager suppresses warnings."""
+        api = Client.from_file(str(TEST_DATA / "planetary-computer-root.json"))
+        api.remove_conforms_to("COLLECTION_SEARCH")
+
+        # This should emit a DoesNotConformTo warning, but ignore() should suppress it
+        with ignore():
+            api.collection_search(limit=10, max_collections=10, q="test")
+
+    @pytest.mark.filterwarnings("error")
+    def test_strict_context_manager(self) -> None:
+        """Test that strict() context manager converts warnings to exceptions."""
+        api = Client.from_file(str(TEST_DATA / "planetary-computer-root.json"))
+        api.remove_conforms_to("COLLECTION_SEARCH")
+
+        # This should raise DoesNotConformTo as an exception
+        with strict():
+            with pytest.raises(DoesNotConformTo, match="COLLECTION_SEARCH"):
+                api.collection_search(limit=10, max_collections=10, q="test")
+
+    def test_ignore_context_manager_cleanup(self) -> None:
+        """Test that ignore() properly restores warning filters after exit."""
+        api = Client.from_file(str(TEST_DATA / "planetary-computer-root.json"))
+        api.remove_conforms_to("COLLECTION_SEARCH")
+
+        # Test that warnings are suppressed inside the context
+        with ignore():
+            api.collection_search(limit=10, max_collections=10, q="test")
+
+        # Test that warnings are restored after exiting the context
+        # Use strict() to ensure warnings become exceptions
+        with strict():
+            with pytest.raises(DoesNotConformTo, match="COLLECTION_SEARCH"):
+                api.collection_search(limit=10, max_collections=10, q="test")