Not sure why the tests are failing. I haven't touched them yet. 🤔

Thanks again for your contribution !

However, adding this to the http_client configuration bothers me a bit, as this option will only be applicable to Meilisearch.
We could instead inject the http_client service directly into the MeilisearchFactory, which would still allow mocking for tests.
Do you see a use case where it would be interesting to pass a custom http_client ?

Hey @AlexandrePetrone,

I was inspired by how the official Meilisearch bundle handles this (ref: meilisearch/meilisearch-symfony#359). Speaking of the configuration, I also think the place is not ideal and share your sentiment.

Let's inject the client directly and make it configurable in the future (if needed). WDYT?

Changes done. Testing this is almost impossible as the created Meilisearch client does not expose any properties which can be asserted. However, creating a client in a test with the passed argument should be enough if it doesn't result in an exception. 😅

Thank you, @daFish . After this last point, I suggest merging. For tests in Symfony 7.3-dev, we should handle that in another PR, as it's not related to this change.

Should be fixed upstream by symfony/symfony#60001.