extending tests to consider Mixin filter tests

Author: davidsbatista

integrations/oracle/src/haystack_integrations/document_stores/oracle/document_store.py

@@ -230,6 +230,12 @@ def write_documents(
             and the policy is set to `DuplicatePolicy.FAIL` or `DuplicatePolicy.NONE`.
         :returns: The number of documents written to the document store.
         """
+        if not isinstance(documents, list):
+            msg = "write_documents expects a list of Document objects."
+            raise ValueError(msg)
+        if documents and not isinstance(documents[0], Document):
+            msg = "write_documents expects a list of Document objects."
+            raise ValueError(msg)
         if not documents:
             return 0
         if policy in (DuplicatePolicy.NONE, DuplicatePolicy.FAIL):

integrations/oracle/tests/test_document_store.py

@@ -81,17 +81,11 @@ def test_write_documents(self, document_store: OracleDocumentStore) -> None:
     # test_comparison_not_equal_with_none → IS NOT NULL
     # test_comparison_not_equal           → col != x OR IS NULL
     # test_comparison_not_in              → IS NULL OR NOT IN
-
     @pytest.mark.skip(
         reason="Oracle NULL propagation in NOT(...) cannot match Python 'not (None == x) is True' semantics"
     )
     def test_not_operator(self, document_store, filterable_docs): ...
 
-    # Skipped: input validation not implemented
-
-    @pytest.mark.skip(reason="OracleDocumentStore does not validate input types in write_documents")
-    def test_write_documents_invalid_input(self, document_store): ...
-
     def test_write_documents_none_policy_calls_insert(self, patched_store, mock_pool):
         _, _, cursor = mock_pool
         patched_store.write_documents([self._mock_doc()], policy=DuplicatePolicy.NONE)