Merge #234

234: Change client#indexes method to retrieve a list of `Index` instance r=curquiza a=nagaroni

# WHY

It would be better to work with a Index instances when listing all indexes rather than the raw response.

# HOW

* Change MeiliSearch::Client#indexes to return a list of MeiliSearch::Index object
* Add raw_indexes to return the raw response from meilisearch server

closes #227 

Co-authored-by: Gabriel Lima <[email protected]>
Co-authored-by: nagaroni <[email protected]>
Co-authored-by: Clémentine Urquizar <[email protected]>

Author: 3 people

lib/meilisearch/client.rb

@@ -6,8 +6,14 @@ module MeiliSearch
   class Client < HTTPRequest
     ### INDEXES
 
+    def raw_indexes
+      http_get('/indexes')
+    end
+
     def indexes
-      http_get '/indexes'
+      raw_indexes.map do |index_hash|
+        index_object(index_hash['uid'], index_hash['primaryKey'])
+      end
     end
 
     # Usage:

spec/meilisearch/client/indexes_spec.rb

@@ -79,10 +79,18 @@
     end.to raise_meilisearch_api_error_with(400, 'invalid_index_uid', 'invalid_request_error')
   end
 
-  it 'gets list of indexes' do
+  it 'gets list of indexes as MeiliSearch::Index' do
+    client.create_index('index')
+
+    response = client.indexes.first
+
+    expect(response).to be_a(MeiliSearch::Index)
+  end
+
+  it 'gets list of raw indexes' do
     ['first_index', 'second_index', 'third_index'].each { |name| client.create_index(name) }
 
-    indexes = client.indexes
+    indexes = client.raw_indexes
 
     expect(indexes).to be_a(Array)
     expect(indexes.length).to eq(3)

spec/support/indexes_helpers.rb

@@ -3,7 +3,7 @@
 module IndexesHelpers
   def clear_all_indexes(client)
     indexes = client.indexes
-    uids = indexes.map { |index| index['uid'] }
+    uids = indexes.map(&:uid)
     uids.each do |uid|
       client.delete_index(uid)
     end