fix: address pytest warnings without global filters (#192)

Co-authored-by: claude[bot] <41898282+claude[bot]@users.noreply.github.com>
Co-authored-by: William Easton <strawgate@users.noreply.github.com>

Author: strawgate

key-value/key-value-aio/src/key_value/aio/stores/vault/store.py

@@ -122,7 +122,7 @@ async def _delete_managed_entry(self, *, key: str, collection: str) -> bool:
         try:
             # Vault doesnt tell us if the delete was successful so we read the entry and if it exists before we call delete,
             # we count it as a deletion.
-            entry = self._kv_v2.read_secret(path=combo_key, mount_point=self._mount_point)  # pyright: ignore[reportUnknownMemberType,reportUnknownVariableType]
+            entry = self._kv_v2.read_secret(path=combo_key, mount_point=self._mount_point, raise_on_deleted_version=True)  # pyright: ignore[reportUnknownMemberType,reportUnknownVariableType]
             self._kv_v2.delete_metadata_and_all_versions(path=combo_key, mount_point=self._mount_point)  # pyright: ignore[reportUnknownMemberType]
         except InvalidPath:
             return False

key-value/key-value-aio/tests/stores/elasticsearch/test_elasticsearch.py

@@ -148,6 +148,7 @@ async def test_put_put_two_indices(self, store: ElasticsearchStore, es_client: A
 
 
 @pytest.mark.skipif(should_skip_docker_tests(), reason="Docker is not running")
+@pytest.mark.filterwarnings("ignore:A configured store is unstable and may change in a backwards incompatible way. Use at your own risk.")
 class TestElasticsearchStoreNativeMode(BaseTestElasticsearchStore):
     """Test Elasticsearch store in native mode (i.e. it stores flattened objects)"""
 
@@ -207,6 +208,7 @@ async def test_migration_from_non_native_mode(self, store: ElasticsearchStore, e
 
 
 @pytest.mark.skipif(should_skip_docker_tests(), reason="Docker is not running")
+@pytest.mark.filterwarnings("ignore:A configured store is unstable and may change in a backwards incompatible way. Use at your own risk.")
 class TestElasticsearchStoreNonNativeMode(BaseTestElasticsearchStore):
     """Test Elasticsearch store in non-native mode (i.e. it stores stringified JSON values)"""
 

key-value/key-value-sync/src/key_value/sync/code_gen/stores/vault/store.py

@@ -119,7 +119,7 @@ def _delete_managed_entry(self, *, key: str, collection: str) -> bool:
         try:
             # Vault doesnt tell us if the delete was successful so we read the entry and if it exists before we call delete,
             # we count it as a deletion.
-            entry = self._kv_v2.read_secret(path=combo_key, mount_point=self._mount_point)  # pyright: ignore[reportUnknownMemberType,reportUnknownVariableType]
+            entry = self._kv_v2.read_secret(path=combo_key, mount_point=self._mount_point, raise_on_deleted_version=True)  # pyright: ignore[reportUnknownMemberType,reportUnknownVariableType]
             self._kv_v2.delete_metadata_and_all_versions(path=combo_key, mount_point=self._mount_point)  # pyright: ignore[reportUnknownMemberType]
         except InvalidPath:
             return False

key-value/key-value-sync/tests/code_gen/stores/elasticsearch/test_elasticsearch.py

@@ -141,6 +141,7 @@ def test_put_put_two_indices(self, store: ElasticsearchStore, es_client: Elastic
 
 
 @pytest.mark.skipif(should_skip_docker_tests(), reason="Docker is not running")
+@pytest.mark.filterwarnings("ignore:A configured store is unstable and may change in a backwards incompatible way. Use at your own risk.")
 class TestElasticsearchStoreNativeMode(BaseTestElasticsearchStore):
     """Test Elasticsearch store in native mode (i.e. it stores flattened objects)"""
 
@@ -189,6 +190,7 @@ def test_migration_from_non_native_mode(self, store: ElasticsearchStore, es_clie
 
 
 @pytest.mark.skipif(should_skip_docker_tests(), reason="Docker is not running")
+@pytest.mark.filterwarnings("ignore:A configured store is unstable and may change in a backwards incompatible way. Use at your own risk.")
 class TestElasticsearchStoreNonNativeMode(BaseTestElasticsearchStore):
     """Test Elasticsearch store in non-native mode (i.e. it stores stringified JSON values)"""