Rubocoping: whitespace and styling

Author: picandocodigo

elasticsearch-model/lib/elasticsearch/model/adapter.rb

@@ -17,7 +17,6 @@
 
 module Elasticsearch
   module Model
-
     # Contains an adapter which provides OxM-specific implementations for common behaviour:
     #
     # * {Adapter::Adapter#records_mixin   Fetching records from the database}
@@ -29,7 +28,6 @@ module Model
     # @see Elasticsearch::Model::Adapter::Mongoid
     #
     module Adapter
-
       # Returns an adapter based on the Ruby class passed
       #
       # @example Create an adapter for an ActiveRecord-based model

elasticsearch-model/lib/elasticsearch/model/importing.rb

@@ -17,14 +17,12 @@
 
 module Elasticsearch
   module Model
-
     # Provides support for easily and efficiently importing large amounts of
     # records from the including class into the index.
     #
     # @see ClassMethods#import
     #
     module Importing
-
       # When included in a model, adds the importing methods.
       #
       # @example Import all records from the `Article` model
@@ -42,13 +40,12 @@ def self.included(base)
       end
 
       module ClassMethods
-
         # Import all model records into the index
         #
         # The method will pick up correct strategy based on the `Importing` module
         # defined in the corresponding adapter.
         #
-        # @param options [Hash] Options passed to the underlying `__find_in_batches`method
+        # @param options [Hash] Options passed to the underlying `__find_in_batches` method
         # @param block  [Proc] Optional block to evaluate for each batch
         #
         # @yield [Hash] Gives the Hash with the Elasticsearch response to the block

elasticsearch-model/lib/elasticsearch/model/proxy.rb

@@ -17,7 +17,6 @@
 
 module Elasticsearch
   module Model
-
     # This module provides a proxy interfacing between the including class and
     # `Elasticsearch::Model`, preventing the pollution of the including class namespace.
     #
@@ -45,17 +44,14 @@ module Model
     #     # => true
     #
     module Proxy
-
       # Define the `__elasticsearch__` class and instance methods in the including class
       # and register a callback for intercepting changes in the model.
       #
       # @note The callback is triggered only when `Elasticsearch::Model` is included in the
       #       module and the functionality is accessible via the proxy.
       #
       def self.included(base)
-
         base.class_eval do
-
           # `ClassMethodsProxy` instance, accessed as `MyModel.__elasticsearch__`
           def self.__elasticsearch__ &block
             @__elasticsearch__ ||= ClassMethodsProxy.new(self)

elasticsearch-model/spec/elasticsearch/model/adapter_spec.rb

@@ -18,7 +18,6 @@
 require 'spec_helper'
 
 describe Elasticsearch::Model::Adapter do
-
   before(:all) do
     class ::DummyAdapterClass; end
     class ::DummyAdapterClassWithAdapter; end
@@ -37,14 +36,12 @@ class ::DummyAdapter
   end
 
   describe '#from_class' do
-
     it 'should return an Adapter instance' do
       expect(Elasticsearch::Model::Adapter.from_class(DummyAdapterClass)).to be_a(Elasticsearch::Model::Adapter::Adapter)
     end
   end
 
   describe 'register' do
-
     before do
       expect(Elasticsearch::Model::Adapter::Adapter).to receive(:register).and_call_original
       Elasticsearch::Model::Adapter.register(:foo, lambda { |c| false })
@@ -55,7 +52,6 @@ class ::DummyAdapter
     end
 
     context 'when a specific adapter class is set' do
-
       before do
         expect(Elasticsearch::Model::Adapter::Adapter).to receive(:register).and_call_original
         Elasticsearch::Model::Adapter::Adapter.register(DummyAdapter,
@@ -73,7 +69,6 @@ class ::DummyAdapter
   end
 
   describe 'default adapter' do
-
     let(:adapter) do
       Elasticsearch::Model::Adapter::Adapter.new(DummyAdapterClass)
     end
@@ -84,11 +79,9 @@ class ::DummyAdapter
   end
 
   describe '#records_mixin' do
-
     before do
       Elasticsearch::Model::Adapter::Adapter.register(DummyAdapter,
                                                       lambda { |c| c == DummyAdapterClassWithAdapter })
-
     end
 
     let(:adapter) do
@@ -101,11 +94,9 @@ class ::DummyAdapter
   end
 
   describe '#callbacks_mixin' do
-
     before do
       Elasticsearch::Model::Adapter::Adapter.register(DummyAdapter,
                                                       lambda { |c| c == DummyAdapterClassWithAdapter })
-
     end
 
     let(:adapter) do
@@ -118,11 +109,9 @@ class ::DummyAdapter
   end
 
   describe '#importing_mixin' do
-
     before do
       Elasticsearch::Model::Adapter::Adapter.register(DummyAdapter,
                                                       lambda { |c| c == DummyAdapterClassWithAdapter })
-
     end
 
     let(:adapter) do

elasticsearch-model/spec/elasticsearch/model/adapters/active_record/import_spec.rb

@@ -19,12 +19,12 @@
 
 describe 'Elasticsearch::Model::Adapter::ActiveRecord Importing' do
   before(:all) do
-    ActiveRecord::Schema.define(:version => 1) do
+    ActiveRecord::Schema.define(version: 1) do
       create_table :import_articles do |t|
         t.string   :title
         t.integer  :views
         t.string   :numeric # For the sake of invalid data sent to Elasticsearch
-        t.datetime :created_at, :default => 'NOW()'
+        t.datetime :created_at, default: 'NOW()'
       end
     end
 

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

@@ -18,18 +18,17 @@
 require 'spec_helper'
 
 describe Elasticsearch::Model::Importing do
-
   before(:all) do
     class DummyImportingModel
     end
 
     module DummyImportingAdapter
       module ImportingMixin
-        def __find_in_batches(options={}, &block)
+        def __find_in_batches( options = {}, &block)
           yield if block_given?
         end
         def __transform
-          lambda {|a|}
+          lambda { |a| }
         end
       end
 
@@ -49,15 +48,13 @@ def importing_mixin
   end
 
   context 'when a model includes the Importing module' do
-
     it 'provides importing methods' do
       expect(DummyImportingModel.respond_to?(:import)).to be(true)
       expect(DummyImportingModel.respond_to?(:__find_in_batches)).to be(true)
     end
   end
 
   describe '#import' do
-
     before do
       allow(DummyImportingModel).to receive(:index_name).and_return('foo')
       allow(DummyImportingModel).to receive(:document_type).and_return('foo')
@@ -75,7 +72,6 @@ def importing_mixin
     end
 
     context 'when no options are provided' do
-
       before do
         expect(DummyImportingModel).to receive(:client).and_return(client)
         allow(DummyImportingModel).to receive(:index_exists?).and_return(true)
@@ -87,7 +83,6 @@ def importing_mixin
     end
 
     context 'when there is an error' do
-
       before do
         expect(DummyImportingModel).to receive(:client).and_return(client)
         allow(DummyImportingModel).to receive(:index_exists?).and_return(true)
@@ -102,14 +97,12 @@ def importing_mixin
       end
 
       context 'when the method is called with the option to return the errors' do
-
         it 'returns the errors' do
           expect(DummyImportingModel.import(return: 'errors')).to eq([{ 'index' => { 'error' => 'FAILED' } }])
         end
       end
 
       context 'when the method is called with a block' do
-
         it 'yields the response to the block' do
           DummyImportingModel.import do |response|
             expect(response['items'].size).to eq(2)
@@ -119,7 +112,6 @@ def importing_mixin
     end
 
     context 'when the index does not exist' do
-
       before do
         allow(DummyImportingModel).to receive(:index_exists?).and_return(false)
       end
@@ -132,7 +124,6 @@ def importing_mixin
     end
 
     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)
@@ -144,7 +135,6 @@ def importing_mixin
     end
 
     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)
@@ -156,7 +146,6 @@ def importing_mixin
     end
 
     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', type: 'foo').and_return(response)
@@ -168,7 +157,6 @@ def importing_mixin
     end
 
     context 'when a different document type is provided' do
-
       before do
         expect(DummyImportingModel).to receive(:client).and_return(client)
         expect(client).to receive(:bulk).with(body: nil, index: 'foo', type: 'my-new-type').and_return(response)
@@ -180,7 +168,6 @@ def importing_mixin
     end
 
     context 'the transform method' do
-
       before do
         expect(DummyImportingModel).to receive(:client).and_return(client)
         expect(DummyImportingModel).to receive(:__transform).and_return(transform)
@@ -197,9 +184,7 @@ def importing_mixin
     end
 
     context 'when a transform is provided as an option' do
-
       context 'when the transform option is not a lambda' do
-
         let(:transform) do
           'not_callable'
         end
@@ -212,7 +197,6 @@ def importing_mixin
       end
 
       context 'when the transform option is a lambda' do
-
         before do
           expect(DummyImportingModel).to receive(:client).and_return(client)
           expect(DummyImportingModel).to receive(:__batch_to_bulk).with(anything, transform)
@@ -229,7 +213,6 @@ def importing_mixin
     end
 
     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', type: 'foo', pipeline: 'my-pipeline').and_return(response)