Merge pull request #288 from tisnik/lcore-381-new-test-minor-refactoring

LCORE-381: new unit test + minor refactoring

Author: tisnik

src/models/requests.py

@@ -267,9 +267,6 @@ def validate_categories(
         if value is None:
             return value
 
-        if not isinstance(value, list):
-            raise ValueError("Categories must be a list")
-
         if len(value) == 0:
             return None  # Convert empty list to None for consistency
 

tests/unit/models/test_requests.py

@@ -122,6 +122,16 @@ def test_get_documents(self) -> None:
         assert documents[1]["content"] == "kind: Pod\n metadata:\n name:    private-reg"
         assert documents[1]["mime_type"] == "application/yaml"
 
+    def test_get_documents_no_attachments(self) -> None:
+        """Test the get_documents method."""
+        attachments = []
+        qr = QueryRequest(
+            query="Tell me about Kubernetes",
+            attachments=attachments,
+        )
+        documents = qr.get_documents()
+        assert len(documents) == 0
+
     def test_validate_provider_and_model(self) -> None:
         """Test the validate_provider_and_model method."""
         qr = QueryRequest(