Merge pull request #121 from roidrage/fix-rubocop-offenses

Resolves outstanding rubocop offenses

.rubocop.yml

@@ -8,8 +8,5 @@ AllCops:
   Exclude:
     - 'Guardfile'
 
-Style/RegexpLiteral:
-  MaxSlashes: 2
-
 Metrics/AbcSize:
   Enabled: false

lib/lograge.rb

@@ -12,6 +12,7 @@
 require 'active_support/core_ext/string/inflections'
 require 'active_support/ordered_options'
 
+# rubocop:disable ModuleLength
 module Lograge
   module_function
 

spec/formatters/logstash_spec.rb

@@ -28,7 +28,7 @@
   end
 
   it "includes the 'path' key/value" do
-    expect(subject).to match(/"path":"\/"/)
+    expect(subject).to match(%r{"path":"/"})
   end
 
   it "includes the 'controller' key/value" do

spec/lograge_logsubscriber_spec.rb

@@ -150,7 +150,7 @@
 
       it 'adds the location to the log line' do
         subscriber.process_action(event)
-        expect(log_output.string).to match(/location=http:\/\/www.example.com/)
+        expect(log_output.string).to match(%r{location=http://www.example.com})
       end
 
       it 'removes the thread local variable' do

spec/lograge_spec.rb

@@ -43,12 +43,12 @@
     context 'when keep_original_rails_log is true' do
       let(:app_config) do
         double(config:
-          ActiveSupport::OrderedOptions.new.tap do |config|
-            config.action_dispatch = double(rack_cache: false)
-            config.lograge = ActiveSupport::OrderedOptions.new
-            config.lograge.keep_original_rails_log = true
-          end
-        )
+                ActiveSupport::OrderedOptions.new.tap do |config|
+                  config.action_dispatch = double(rack_cache: false)
+                  config.lograge = ActiveSupport::OrderedOptions.new
+                  config.lograge.keep_original_rails_log = true
+                end
+              )
       end
 
       it "does not remove Rails' subscribers" do
@@ -62,12 +62,12 @@
     subject { -> { Lograge.setup(app_config) } }
     let(:app_config) do
       double(config:
-        ActiveSupport::OrderedOptions.new.tap do |config|
-          config.action_dispatch = config_option
-          config.lograge = ActiveSupport::OrderedOptions.new
-          config.lograge.keep_original_rails_log = true
-        end
-      )
+              ActiveSupport::OrderedOptions.new.tap do |config|
+                config.action_dispatch = config_option
+                config.lograge = ActiveSupport::OrderedOptions.new
+                config.lograge.keep_original_rails_log = true
+              end
+            )
     end
     let(:config_option) { double(rack_cache: rack_cache) }
 
@@ -99,12 +99,12 @@
   describe 'deprecated log_format interpreter' do
     let(:app_config) do
       double(config:
-        ActiveSupport::OrderedOptions.new.tap do |config|
-          config.action_dispatch = double(rack_cache: false)
-          config.lograge = ActiveSupport::OrderedOptions.new
-          config.lograge.log_format = format
-        end
-      )
+              ActiveSupport::OrderedOptions.new.tap do |config|
+                config.action_dispatch = double(rack_cache: false)
+                config.lograge = ActiveSupport::OrderedOptions.new
+                config.lograge.log_format = format
+              end
+            )
     end
     before { ActiveSupport::Deprecation.silence { Lograge.setup(app_config) } }
     subject { Lograge.formatter }