Merge #1020

1020: Fix tests by passing embedders r=sanders41 a=dureuill

# Pull Request

## Related issue
Fixes #1019 

## What does this PR do?
- Change comment stating the default model for OpenAI
- Add `document_template_max_bytes` to models
- Fix tests



Co-authored-by: Louis Dureuil <[email protected]>

meilisearch/models/index.py

@@ -56,17 +56,19 @@ class ProximityPrecision(str, Enum):
 
 class OpenAiEmbedder(CamelBase):
     source: str = "openAi"
-    model: Optional[str] = None  # Defaults to text-embedding-ada-002
+    model: Optional[str] = None  # Defaults to text-embedding-3-small
     dimensions: Optional[int] = None  # Uses the model default
     api_key: Optional[str] = None  # Can be provided through a CLI option or environment variable
     document_template: Optional[str] = None
+    document_template_max_bytes: Optional[int] = None  # Default to 400
 
 
 class HuggingFaceEmbedder(CamelBase):
     source: str = "huggingFace"
     model: Optional[str] = None  # Defaults to BAAI/bge-base-en-v1.5
     revision: Optional[str] = None
     document_template: Optional[str] = None
+    document_template_max_bytes: Optional[int] = None  # Default to 400
 
 
 class UserProvidedEmbedder(CamelBase):

tests/index/test_index_search_meilisearch.py

@@ -505,7 +505,8 @@ def test_show_ranking_score(index_with_documents):
 @pytest.mark.usefixtures("enable_vector_search")
 def test_vector_search(index_with_documents_and_vectors):
     response = index_with_documents_and_vectors().search(
-        "", opt_params={"vector": [0.1, 0.2], "hybrid": {"semanticRatio": 1.0}}
+        "",
+        opt_params={"vector": [0.1, 0.2], "hybrid": {"semanticRatio": 1.0, "embedder": "default"}},
     )
     assert len(response["hits"]) > 0