Revert test_client.py to main branch state

Remove all ignore() test blocks that were mixed with API tests.
These tests now belong in the dedicated test_warnings.py file.

Author: nlebovits

tests/test_client.py

@@ -23,7 +23,6 @@
     FallbackToPystac,
     MissingLink,
     NoConformsTo,
-    ignore,
     strict,
 )
 
@@ -445,14 +444,6 @@ def test_search_conformance_error(self, api: Client) -> None:
             with pytest.raises(DoesNotConformTo, match="ITEM_SEARCH"):
                 api.search(limit=10, max_items=10, collections="mr-peebles")
 
-        api.clear_conforms_to()
-        with ignore():
-            try:
-                collections = list(api.get_collections())
-                assert isinstance(collections, list)
-            except Exception:
-                pass
-
     def test_no_search_link(self, api: Client) -> None:
         # Remove the search link
         api.remove_links("search")
@@ -464,14 +455,6 @@ def test_no_search_link(self, api: Client) -> None:
             ):
                 api.search(limit=10, max_items=10, collections="naip")
 
-        api.clear_conforms_to()
-        with ignore():
-            try:
-                collections = list(api.get_collections())
-                assert isinstance(collections, list)
-            except Exception:
-                pass
-
     def test_no_conforms_to(self) -> None:
         with open(str(TEST_DATA / "planetary-computer-root.json")) as f:
             data = json.load(f)
@@ -486,14 +469,6 @@ def test_no_conforms_to(self) -> None:
             with pytest.raises(DoesNotConformTo, match="ITEM_SEARCH"):
                 api.search(limit=10, max_items=10, collections="naip")
 
-        api.clear_conforms_to()
-        with ignore():
-            try:
-                collections = list(api.get_collections())
-                assert isinstance(collections, list)
-            except Exception:
-                pass
-
     def test_search(self, api: Client) -> None:
         results = api.search(
             bbox=[-73.21, 43.99, -73.12, 44.05],
@@ -543,14 +518,6 @@ def test_search_conformance_error(self, api: Client) -> None:
             with pytest.raises(DoesNotConformTo, match="COLLECTION_SEARCH"):
                 api.collection_search(limit=10, max_collections=10, q="test")
 
-        api.clear_conforms_to()
-        with ignore():
-            try:
-                collections = list(api.get_collections())
-                assert isinstance(collections, list)
-            except Exception:
-                pass
-
     def test_search_conformance_warning(self) -> None:
         api = Client.from_file(str(TEST_DATA / "planetary-computer-root.json"))
 
@@ -561,14 +528,6 @@ def test_search_conformance_warning(self) -> None:
             with pytest.warns(UserWarning, match="COLLECTION_SEARCH"):
                 api.collection_search(limit=10, max_collections=10, q="test")
 
-        api.clear_conforms_to()
-        with ignore():
-            try:
-                collections = list(api.get_collections())
-                assert isinstance(collections, list)
-            except Exception:
-                pass
-
     @pytest.mark.vcr
     def test_search(self, api: Client) -> None:
         results = api.collection_search(
@@ -737,13 +696,6 @@ def test_no_conforms_to_falls_back_to_pystac(self) -> None:
             with pytest.raises(FallbackToPystac):
                 next(client.get_collections())
 
-        with ignore():
-            try:
-                collections = list(client.get_collections())
-                assert isinstance(collections, list)
-            except Exception:
-                pass
-
     @pytest.mark.vcr
     def test_changing_conforms_to_changes_behavior(self) -> None:
         with pytest.warns(NoConformsTo):