#556 -Add validation for document_id to prevent nil or empty values

lib/meilisearch/index.rb

@@ -220,6 +220,10 @@ def delete_documents!(documents_ids)
     alias delete_multiple_documents! delete_documents!
 
     def delete_document(document_id)
+      if document_id.nil? || document_id.empty?
+        raise MeiliSearch::Error::InvalidDocumentId, 'document_id cannot be empty or nil'
+      end
+
       encode_document = URI.encode_www_form_component(document_id)
       response = http_delete "/indexes/#{@uid}/documents/#{encode_document}"
 

spec/meilisearch/index/documents_spec.rb

@@ -393,6 +393,10 @@
     describe '#delete_document' do
       before { index.add_documents(documents).await }
 
+      it 'if the document id is nil, it raises an error' do
+        expect { index.delete_document(nil).await }.to raise_error(MeiliSearch::ApiError)
+      end
+
       it 'deletes one document from index' do
         id = 456
         index.delete_document(id).await