Standardize health method

[alallema]

Checking that method health() return {'status': 'available'} and added isHealthy() method who return boolean value

meilisearch/integration-guides#55

[curquiza]

Thanks for your PR @alallema!

Some points:

• for is_healthy I see you only catch the MeiliSearchApiError. But I don't think this is enough. Most of the time, when the server is not reachable, you will get another error, like CommunicationError (cf

  meilisearch-python/meilisearch/errors.py

  Lines 29 to 33 in 6cf5166

  	class MeiliSearchCommunicationError(MeiliSearchError):
  	"""Error when connecting to MeiliSearch"""
  	def __str__(self):
  	return f'MeiliSearchCommunicationError, {self.message}'

  ).
• for the health() method, you just need to let the error being raised so, no need to catch it to raise it . Just call the route! 🙂

However I just realize the issue I opened in integration guide was done before the change applied on the /health route, so it's not accurate anymore. Should the health() method return true or the body of the health method (status: available)? I'm going to update the issue!

[alallema]

@curquiza you right:

for is_healthy I just check the __validate method and not the send_request, so I think the MeiliSearchTimeoutError is required too. Sorry for forgetting.
for health method I change for a return like it was!

[curquiza]

👍
A small change

[bidoubiwa]

🔥 LGTM ! Thanks for this PR :)

[bidoubiwa]

bors merge

[bors]

Build succeeded:

• integration-tests (3.6)
• integration-tests (3.7)
• integration-tests (3.8)
• integration-tests (3.9)
• pylint