Code Style changes

Author: picandocodigo

elasticsearch-model/Rakefile

@@ -56,6 +56,8 @@ namespace :test do
       puts '-' * 80
     end
   end
+
+  task unit: :all
 end
 
 # ----- Documentation tasks ---------------------------------------------------

elasticsearch-model/spec/elasticsearch/model/importing_spec.rb

@@ -24,7 +24,7 @@ class DummyImportingModel
 
     module DummyImportingAdapter
       module ImportingMixin
-        def __find_in_batches( options = {}, &block)
+        def __find_in_batches(options = {}, &block)
           yield if block_given?
         end
         def __transform
@@ -125,7 +125,7 @@ def importing_mixin
     context 'when the method is called with the force option' do
       before do
         expect(DummyImportingModel).to receive(:create_index!).with(force: true, index: 'foo').and_return(true)
-        expect(DummyImportingModel).to receive(:__find_in_batches).with(foo: 'bar').and_return(true)
+        expect(DummyImportingModel).to receive(:__find_in_batches).with({ foo: 'bar' }).and_return(true)
       end
 
       it 'deletes and creates the index' do
@@ -136,7 +136,7 @@ def importing_mixin
     context 'when the method is called with the refresh option' do
       before do
         expect(DummyImportingModel).to receive(:refresh_index!).with(index: 'foo').and_return(true)
-        expect(DummyImportingModel).to receive(:__find_in_batches).with(foo: 'bar').and_return(true)
+        expect(DummyImportingModel).to receive(:__find_in_batches).with({ foo: 'bar' }).and_return(true)
       end
 
       it 'refreshes the index' do
@@ -147,7 +147,7 @@ def importing_mixin
     context 'when a different index name is provided' do
       before do
         expect(DummyImportingModel).to receive(:client).and_return(client)
-        expect(client).to receive(:bulk).with(body: nil, index: 'my-new-index').and_return(response)
+        expect(client).to receive(:bulk).with({ body: nil, index: 'my-new-index' }).and_return(response)
       end
 
       it 'uses the alternate index name' do
@@ -203,7 +203,7 @@ def importing_mixin
     context 'when a pipeline is provided as an options' do
       before do
         expect(DummyImportingModel).to receive(:client).and_return(client)
-        expect(client).to receive(:bulk).with(body: nil, index: 'foo', pipeline: 'my-pipeline').and_return(response)
+        expect(client).to receive(:bulk).with({ body: nil, index: 'foo', pipeline: 'my-pipeline' }).and_return(response)
       end
 
       it 'uses the pipeline option' do

elasticsearch-model/spec/elasticsearch/model/proxy_spec.rb

@@ -122,10 +122,9 @@ def changes_to_save
       expect(model).to eq(model_target)
       expect(duplicate).to eq(duplicate_target)
     end
-  end
 
-  it 'forwards keyword arguments to target methods' do
-    expect(DummyProxyModel.new.__elasticsearch__.keyword_method(foo: 'bar')).to eq('bar')
+    it 'forwards keyword arguments to target methods' do
+      expect(DummyProxyModel.new.__elasticsearch__.keyword_method(foo: 'bar')).to eq('bar')
+    end
   end
-
 end

elasticsearch-model/spec/elasticsearch/model/response/aggregations_spec.rb

@@ -33,9 +33,10 @@ def self.index_name;    'foo'; end
     {
         'aggregations' => {
             'foo' => {'bar' => 10 },
-            'price' => { 'doc_count' => 123,
-                         'min' => { 'value' => 1.0},
-                         'max' => { 'value' => 99 }
+            'price' => {
+              'doc_count' => 123,
+              'min' => { 'value' => 1.0},
+              'max' => { 'value' => 99 }
             }
         }
     }

elasticsearch-model/spec/elasticsearch/model/searching_search_request_spec.rb

@@ -43,7 +43,7 @@ def self.index_name;    'foo'; end
     context 'when the search definition is a simple query' do
 
       before do
-        expect(client).to receive(:search).with(index: 'foo', q: 'foo').and_return({})
+        expect(client).to receive(:search).with({ index: 'foo', q: 'foo' }).and_return({})
       end
 
       let(:search) do
@@ -58,7 +58,7 @@ def self.index_name;    'foo'; end
     context 'when the search definition is a hash' do
 
       before do
-        expect(client).to receive(:search).with(index: 'foo', body: { foo: 'bar' }).and_return({})
+        expect(client).to receive(:search).with({ index: 'foo', body: { foo: 'bar' } }).and_return({})
       end
 
       let(:search) do
@@ -73,7 +73,7 @@ def self.index_name;    'foo'; end
     context 'when the search definition is a json string' do
 
       before do
-        expect(client).to receive(:search).with(index: 'foo', body: '{"foo":"bar"}').and_return({})
+        expect(client).to receive(:search).with({ index: 'foo', body: '{"foo":"bar"}' }).and_return({})
       end
 
       let(:search) do
@@ -98,7 +98,7 @@ def to_hash; {foo: 'bar'}; end
       end
 
       before do
-        expect(client).to receive(:search).with(index: 'foo', body: {foo: 'bar'}).and_return({})
+        expect(client).to receive(:search).with({ index: 'foo', body: {foo: 'bar'} }).and_return({})
       end
 
       let(:search) do
@@ -113,7 +113,7 @@ def to_hash; {foo: 'bar'}; end
     context 'when extra options are specified' do
 
       before do
-        expect(client).to receive(:search).with(index: 'foo', q: 'foo', size: 15).and_return({})
+        expect(client).to receive(:search).with({ index: 'foo', q: 'foo', size: 15 }).and_return({})
       end
 
       let(:search) do