Validate documents for text/media check when adding to vector store

[ilayaperumalg]

• Since the non-text documents aren't supported, we need a validation to throw exception if non text documents are encountered
• Add tests
  • Add test to QdrantVectorStoreIT with the usecase provided by @devMtn30
  • Add simple vector store test for the validation

Fixes #3609

[dev-jonghoonpark]

I also think this approach is more appropriate. 👍

However, I have a personal question.
Is there a specific reason why testNonTextDocuments test was added in QdrantVectorStoreIT.java?
Was it intended to verify the implementation's behavior? Or was it added because it wasn’t included in PR #3653?

[devMtn30]

Hi @ilayaperumalg, the changes in PR #3687 look excellent! You’ve neatly applied the shared validation logic across all VectorStores and added the automated NPE regression test, so I’m happy to merge it as-is. Thanks again for your thorough review and suggestions! 🙏

[ilayaperumalg]

@dev-jonghoonpark @devMtn30 Thank you for the review!

@dev-jonghoonpark

Is there a specific reason why testNonTextDocuments test was added in QdrantVectorStoreIT.java?
Was it intended to verify the implementation's behavior? Or was it added because it wasn’t included in #3653?

I intended to use one of the vector store ITs and since the report was made from QdrantVectorStore, I ended up choosing one.