Add validation for document_id to prevent nil or empty values

[Trehana]

Pull Request

Related issue

Fixes #556

What does this PR do?

Add validation for document_id to prevent nil or empty values

PR checklist

Please check if your PR fulfills the following requirements:

• Does this PR fix an existing issue, or have you listed the changes applied in the PR description (and why they are needed)?
• Have you read the contributing guidelines?
• Have you made sure that the title is accurate and descriptive of the changes?

Thank you so much for contributing to Meilisearch!

[nicolasconnault]

This fixes a critical issue in Meilisearch, thank you!

[curquiza]

@Trehana

For the PR
But the tests fail. Can fix them?

[curquiza]

@Trehana the tests still don't pass 😢

[Trehana]

@Trehana the tests still don't pass 😢

i have added a fix, @curquiza

[curquiza]

@Trehana the tests still do not pass
You can run them locally by following our CONTRIBUTING.md: https://github.com/meilisearch/meilisearch-ruby/blob/main/CONTRIBUTING.md

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 100.00%. Comparing base (3557ff5) to head (57030fb).
Report is 6 commits behind head on main.

Additional details and impacted files

@@            Coverage Diff            @@
##              main      #557   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files           10        10           
  Lines          677       684    +7     
=========================================
+ Hits           677       684    +7     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[brunoocasali]

Hi @Trehana the PR looks good!

I just need you to run the rubocop command: bundle exec rubocop --autp-gen-config
and commit the changes.
After that I will be able to merge and release :)

[brunoocasali]

bors merge

Thanks a lot for your contribution!

[meili-bors]

This PR was included in a batch that successfully built, but then failed to merge into main. It will not be retried.

Additional information:

{"message":"1 review requesting changes and 1 approving review by reviewers with write access.","documentation_url":"https://docs.github.com/articles/about-protected-branches","status":"422"}