airbrake-ruby 4.8.0 → 4.11.1

data/spec/config/validator_spec.rb

@@ -118,7 +118,7 @@ RSpec.describe Airbrake::Config::Validator do
         promise = described_class.validate(config)
         expect(promise.value).to eq(
           'error' => "the 'environment' option must be configured with a " \
-                     "Symbol (or String), but 'Float' was provided: 1.0"
+                     "Symbol (or String), but 'Float' was provided: 1.0",
         )
       end
     end
@@ -130,7 +130,7 @@ RSpec.describe Airbrake::Config::Validator do
         {
           project_id: valid_id,
           project_key: valid_key,
-          environment: string_inquirer.new('test')
+          environment: string_inquirer.new('test'),
         }
       end
 
@@ -148,14 +148,14 @@ RSpec.describe Airbrake::Config::Validator do
           project_id: valid_id,
           project_key: valid_key,
           environment: 'test',
-          ignore_environments: ['test']
+          ignore_environments: ['test'],
         }
       end
 
       it "returns a rejected promise" do
         promise = described_class.check_notify_ability(config)
         expect(promise.value).to eq(
-          'error' => "current environment 'test' is ignored"
+          'error' => "current environment 'test' is ignored",
         )
       end
     end
@@ -165,7 +165,7 @@ RSpec.describe Airbrake::Config::Validator do
         {
           project_id: valid_id,
           project_key: valid_key,
-          ignore_environments: ['test']
+          ignore_environments: ['test'],
         }
       end
 

data/spec/config_spec.rb

@@ -110,9 +110,7 @@ RSpec.describe Airbrake::Config do
 
   describe "#check_performance_options" do
     it "returns a promise" do
-      resource = Airbrake::Query.new(
-        method: '', route: '', query: '', start_time: Time.now
-      )
+      resource = Airbrake::Query.new(method: '', route: '', query: '', timing: 1)
       expect(subject.check_performance_options(resource))
         .to be_an(Airbrake::Promise)
     end
@@ -122,14 +120,14 @@ RSpec.describe Airbrake::Config do
 
       let(:resource) do
         Airbrake::Request.new(
-          method: 'GET', route: '/foo', status_code: 200, start_time: Time.new
+          method: 'GET', route: '/foo', status_code: 200, timing: 1,
         )
       end
 
       it "returns a rejected promise" do
         promise = subject.check_performance_options(resource)
         expect(promise.value).to eq(
-          'error' => "The Performance Stats feature is disabled"
+          'error' => "The Performance Stats feature is disabled",
         )
       end
     end
@@ -138,15 +136,13 @@ RSpec.describe Airbrake::Config do
       before { subject.query_stats = false }
 
       let(:resource) do
-        Airbrake::Query.new(
-          method: 'GET', route: '/foo', query: '', start_time: Time.new
-        )
+        Airbrake::Query.new(method: 'GET', route: '/foo', query: '', timing: 1)
       end
 
       it "returns a rejected promise" do
         promise = subject.check_performance_options(resource)
         expect(promise.value).to eq(
-          'error' => "The Query Stats feature is disabled"
+          'error' => "The Query Stats feature is disabled",
         )
       end
     end

data/spec/deploy_notifier_spec.rb

@@ -26,7 +26,7 @@ RSpec.describe Airbrake::DeployNotifier do
         expect_any_instance_of(Airbrake::SyncSender).to receive(:send).with(
           { environment: 'barenv' },
           instance_of(Airbrake::Promise),
-          URI('https://api.airbrake.io/api/v4/projects/1/deploys')
+          URI('https://api.airbrake.io/api/v4/projects/1/deploys'),
         )
         subject.notify(environment: 'barenv')
       end
@@ -39,7 +39,7 @@ RSpec.describe Airbrake::DeployNotifier do
         expect_any_instance_of(Airbrake::SyncSender).to receive(:send).with(
           { environment: 'fooenv' },
           instance_of(Airbrake::Promise),
-          URI('https://api.airbrake.io/api/v4/projects/1/deploys')
+          URI('https://api.airbrake.io/api/v4/projects/1/deploys'),
         )
         subject.notify({})
       end

data/spec/filter_chain_spec.rb

@@ -67,7 +67,7 @@ RSpec.describe Airbrake::FilterChain do
 
       foo_filter_mock = double
       expect(foo_filter_mock).to(
-        receive(:name).at_least(:once).and_return('FooFilter')
+        receive(:name).at_least(:once).and_return('FooFilter'),
       )
       subject.delete_filter(foo_filter_mock)
 

data/spec/filters/dependency_filter_spec.rb

@@ -5,7 +5,7 @@ RSpec.describe Airbrake::Filters::DependencyFilter do
     it "attaches loaded dependencies to context/versions/dependencies" do
       subject.call(notice)
       expect(notice[:context][:versions][:dependencies]).to include(
-        'airbrake-ruby' => Airbrake::AIRBRAKE_RUBY_VERSION
+        'airbrake-ruby' => Airbrake::AIRBRAKE_RUBY_VERSION,
       )
     end
   end

data/spec/filters/gem_root_filter_spec.rb

@@ -12,7 +12,7 @@ RSpec.describe Airbrake::Filters::GemRootFilter do
       { file: "/home/kyrylo/code/airbrake/ruby/spec/spec_helper.rb" },
       { file: "#{root1}/gems/rspec-core-3.3.2/lib/rspec/core/configuration.rb" },
       { file: "/opt/rubies/ruby-2.2.2/lib/ruby/2.2.0/rubygems/core_ext/kernel_require.rb" },
-      { file: "#{root2}/gems/rspec-core-3.3.2/exe/rspec" }
+      { file: "#{root2}/gems/rspec-core-3.3.2/exe/rspec" },
     ]
     # rubocop:enable Metrics/LineLength
 
@@ -25,9 +25,9 @@ RSpec.describe Airbrake::Filters::GemRootFilter do
           { file: "/home/kyrylo/code/airbrake/ruby/spec/spec_helper.rb" },
           { file: "/GEM_ROOT/gems/rspec-core-3.3.2/lib/rspec/core/configuration.rb" },
           { file: "/opt/rubies/ruby-2.2.2/lib/ruby/2.2.0/rubygems/core_ext/kernel_require.rb" },
-          { file: "/GEM_ROOT/gems/rspec-core-3.3.2/exe/rspec" }
-        ]
-      )
+          { file: "/GEM_ROOT/gems/rspec-core-3.3.2/exe/rspec" },
+        ],
+      ),
     )
     # rubocop:enable Metrics/LineLength
   end
@@ -35,7 +35,7 @@ RSpec.describe Airbrake::Filters::GemRootFilter do
   it "does not filter file when it is nil" do
     expect(notice[:errors].first[:file]).to be_nil
     expect { subject.call(notice) }.not_to(
-      change { notice[:errors].first[:file] }
+      change { notice[:errors].first[:file] },
     )
   end
 end

data/spec/filters/git_last_checkout_filter_spec.rb

@@ -29,7 +29,7 @@ RSpec.describe Airbrake::Filters::GitLastCheckoutFilter do
 
     it "attaches last checkouted email" do
       expect(notice[:context][:lastCheckout][:email]).to(
-        match(/\A\w+[\w.-]*@\w+\.?\w+?\z/)
+        match(/\A\w+[\w.-]*@\w+\.?\w+?\z/),
       )
     end
 

data/spec/filters/git_repository_filter.rb

@@ -46,7 +46,7 @@ RSpec.describe Airbrake::Filters::GitRepositoryFilter do
     it "attaches context/repository" do
       subject.call(notice)
       expect(notice[:context][:repository]).to eq(
-        'ssh://git@github.com/airbrake/airbrake-ruby.git'
+        'ssh://git@github.com/airbrake/airbrake-ruby.git',
       )
     end
   end

data/spec/filters/git_revision_filter_spec.rb

@@ -44,7 +44,7 @@ RSpec.describe Airbrake::Filters::GitRevisionFilter do
       context "and also when HEAD doesn't start with 'ref: '" do
         before do
           expect(File).to(
-            receive(:read).with('root/dir/.git/HEAD').and_return('refs/foo')
+            receive(:read).with('root/dir/.git/HEAD').and_return('refs/foo'),
           )
         end
 
@@ -57,17 +57,17 @@ RSpec.describe Airbrake::Filters::GitRevisionFilter do
       context "and also when HEAD starts with 'ref: " do
         before do
           expect(File).to(
-            receive(:read).with('root/dir/.git/HEAD').and_return("ref: refs/foo\n")
+            receive(:read).with('root/dir/.git/HEAD').and_return("ref: refs/foo\n"),
           )
         end
 
         context "when the ref exists" do
           before do
             expect(File).to(
-              receive(:exist?).with('root/dir/.git/refs/foo').and_return(true)
+              receive(:exist?).with('root/dir/.git/refs/foo').and_return(true),
             )
             expect(File).to(
-              receive(:read).with('root/dir/.git/refs/foo').and_return("d34db33f\n")
+              receive(:read).with('root/dir/.git/refs/foo').and_return("d34db33f\n"),
             )
           end
 
@@ -80,14 +80,14 @@ RSpec.describe Airbrake::Filters::GitRevisionFilter do
         context "when the ref doesn't exist" do
           before do
             expect(File).to(
-              receive(:exist?).with('root/dir/.git/refs/foo').and_return(false)
+              receive(:exist?).with('root/dir/.git/refs/foo').and_return(false),
             )
           end
 
           context "and when '.git/packed-refs' exists" do
             before do
               expect(File).to(
-                receive(:exist?).with('root/dir/.git/packed-refs').and_return(true)
+                receive(:exist?).with('root/dir/.git/packed-refs').and_return(true),
               )
               expect(File).to(
                 receive(:readlines).with('root/dir/.git/packed-refs').and_return(
@@ -95,9 +95,9 @@ RSpec.describe Airbrake::Filters::GitRevisionFilter do
                     "# pack-refs with: peeled fully-peeled\n",
                     "ccb316eecff79c7528d1ad43e5fa165f7a44d52e refs/tags/v3.0.30\n",
                     "^d358900f73ee5bfd6ca3a592cf23ac6e82df83c1",
-                    "d34db33f refs/foo\n"
-                  ]
-                )
+                    "d34db33f refs/foo\n",
+                  ],
+                ),
               )
             end
 
@@ -110,7 +110,7 @@ RSpec.describe Airbrake::Filters::GitRevisionFilter do
           context "and when '.git/packed-refs' doesn't exist" do
             before do
               expect(File).to(
-                receive(:exist?).with('root/dir/.git/packed-refs').and_return(false)
+                receive(:exist?).with('root/dir/.git/packed-refs').and_return(false),
               )
             end
 

data/spec/filters/keys_blacklist_spec.rb

@@ -19,8 +19,8 @@ RSpec.describe Airbrake::Filters::KeysBlacklist do
       [/\Abon/],
       [
         { bongo: 'bango' },
-        { bongo: '[Filtered]' }
-      ]
+        { bongo: '[Filtered]' },
+      ],
     )
 
     context "and when a key is a hash" do
@@ -40,8 +40,8 @@ RSpec.describe Airbrake::Filters::KeysBlacklist do
       [:bingo],
       [
         { bingo: 'bango' },
-        { bingo: '[Filtered]' }
-      ]
+        { bingo: '[Filtered]' },
+      ],
     )
   end
 
@@ -51,8 +51,8 @@ RSpec.describe Airbrake::Filters::KeysBlacklist do
       ['bingo'],
       [
         { bingo: 'bango' },
-        { bingo: '[Filtered]' }
-      ]
+        { bingo: '[Filtered]' },
+      ],
     )
   end
 
@@ -62,8 +62,8 @@ RSpec.describe Airbrake::Filters::KeysBlacklist do
       ['bingo'],
       [
         { array: [{ bingo: 'bango' }, []] },
-        { array: [{ bingo: '[Filtered]' }, []] }
-      ]
+        { array: [{ bingo: '[Filtered]' }, []] },
+      ],
     )
   end
 
@@ -74,8 +74,8 @@ RSpec.describe Airbrake::Filters::KeysBlacklist do
         [proc { 'bongo' }, :bash],
         [
           { bingo: 'bango', bongo: 'bish', bash: 'bosh' },
-          { bingo: 'bango', bongo: '[Filtered]', bash: '[Filtered]' }
-        ]
+          { bingo: 'bango', bongo: '[Filtered]', bash: '[Filtered]' },
+        ],
       )
     end
 
@@ -85,13 +85,13 @@ RSpec.describe Airbrake::Filters::KeysBlacklist do
         [proc { Object.new }],
         [
           { bingo: 'bango', bongo: 'bish' },
-          { bingo: 'bango', bongo: 'bish' }
-        ]
+          { bingo: 'bango', bongo: 'bish' },
+        ],
       )
 
       it "logs an error" do
         expect(Airbrake::Loggable.instance).to receive(:error).with(
-          /KeysBlacklist is invalid.+patterns: \[#<Object:.+>\]/
+          /KeysBlacklist is invalid.+patterns: \[#<Object:.+>\]/,
         )
         keys_blacklist = described_class.new(patterns)
         keys_blacklist.call(notice)
@@ -104,7 +104,7 @@ RSpec.describe Airbrake::Filters::KeysBlacklist do
       context "and when the filter is called once" do
         it "logs an error" do
           expect(Airbrake::Loggable.instance).to receive(:error).with(
-            /KeysBlacklist is invalid.+patterns: \[#<Proc:.+>\]/
+            /KeysBlacklist is invalid.+patterns: \[#<Proc:.+>\]/,
           )
           keys_blacklist = described_class.new(patterns)
           keys_blacklist.call(notice)
@@ -127,13 +127,13 @@ RSpec.describe Airbrake::Filters::KeysBlacklist do
       [Object.new],
       [
         { bingo: 'bango', bongo: 'bish' },
-        { bingo: 'bango', bongo: 'bish' }
-      ]
+        { bingo: 'bango', bongo: 'bish' },
+      ],
     )
 
     it "logs an error" do
       expect(Airbrake::Loggable.instance).to receive(:error).with(
-        /KeysBlacklist is invalid.+patterns: \[#<Object:.+>\]/
+        /KeysBlacklist is invalid.+patterns: \[#<Object:.+>\]/,
       )
       keys_blacklist = described_class.new(patterns)
       keys_blacklist.call(notice)
@@ -147,8 +147,8 @@ RSpec.describe Airbrake::Filters::KeysBlacklist do
         ['bish'],
         [
           { bongo: { bish: 'bash' } },
-          { bongo: { bish: '[Filtered]' } }
-        ]
+          { bongo: { bish: '[Filtered]' } },
+        ],
       )
     end
 
@@ -161,8 +161,8 @@ RSpec.describe Airbrake::Filters::KeysBlacklist do
         ['bango'],
         [
           bongo,
-          { bingo: { bango: '[Filtered]' } }
-        ]
+          { bingo: { bango: '[Filtered]' } },
+        ],
       )
     end
   end
@@ -177,7 +177,7 @@ RSpec.describe Airbrake::Filters::KeysBlacklist do
 
         subject.call(notice)
         expect(notice[:context][:url]).to(
-          eq 'http://localhost:3000/crash?foo=bar&baz=bongo&bish=[Filtered]&color=%23FFAAFF'
+          eq('http://localhost:3000/crash?foo=bar&baz=bongo&bish=[Filtered]&color=%23FFAAFF'),
         )
       end
     end

data/spec/filters/keys_whitelist_spec.rb

@@ -19,8 +19,8 @@ RSpec.describe Airbrake::Filters::KeysWhitelist do
       [/\Abin/],
       [
         { bingo: 'bango', bongo: 'bish', bash: 'bosh' },
-        { bingo: 'bango', bongo: '[Filtered]', bash: '[Filtered]' }
-      ]
+        { bingo: 'bango', bongo: '[Filtered]', bash: '[Filtered]' },
+      ],
     )
   end
 
@@ -30,8 +30,8 @@ RSpec.describe Airbrake::Filters::KeysWhitelist do
       [:bongo],
       [
         { bongo: 'bish', bash: 'bosh', bbashh: 'bboshh' },
-        { bongo: 'bish', bash: '[Filtered]', bbashh: '[Filtered]' }
-      ]
+        { bongo: 'bish', bash: '[Filtered]', bbashh: '[Filtered]' },
+      ],
     )
   end
 
@@ -41,8 +41,8 @@ RSpec.describe Airbrake::Filters::KeysWhitelist do
       ['bash'],
       [
         { bingo: 'bango', bongo: 'bish', bash: 'bosh' },
-        { bingo: '[Filtered]', bongo: '[Filtered]', bash: 'bosh' }
-      ]
+        { bingo: '[Filtered]', bongo: '[Filtered]', bash: 'bosh' },
+      ],
     )
   end
 
@@ -53,8 +53,8 @@ RSpec.describe Airbrake::Filters::KeysWhitelist do
         [proc { 'bongo' }, :bash],
         [
           { bingo: 'bango', bongo: 'bish', bash: 'bosh' },
-          { bingo: '[Filtered]', bongo: 'bish', bash: 'bosh' }
-        ]
+          { bingo: '[Filtered]', bongo: 'bish', bash: 'bosh' },
+        ],
       )
     end
 
@@ -64,13 +64,13 @@ RSpec.describe Airbrake::Filters::KeysWhitelist do
         [proc { Object.new }],
         [
           { bingo: 'bango', bongo: 'bish', bash: 'bosh' },
-          { bingo: '[Filtered]', bongo: '[Filtered]', bash: '[Filtered]' }
-        ]
+          { bingo: '[Filtered]', bongo: '[Filtered]', bash: '[Filtered]' },
+        ],
       )
 
       it "logs an error" do
         expect(Airbrake::Loggable.instance).to receive(:error).with(
-          /KeysWhitelist is invalid.+patterns: \[#<Object:.+>\]/
+          /KeysWhitelist is invalid.+patterns: \[#<Object:.+>\]/,
         )
         keys_whitelist = described_class.new(patterns)
         keys_whitelist.call(notice)
@@ -83,7 +83,7 @@ RSpec.describe Airbrake::Filters::KeysWhitelist do
       context "and when the filter is called once" do
         it "logs an error" do
           expect(Airbrake::Loggable.instance).to receive(:error).with(
-            /KeysWhitelist is invalid.+patterns: \[#<Proc:.+>\]/
+            /KeysWhitelist is invalid.+patterns: \[#<Proc:.+>\]/,
           )
           keys_whitelist = described_class.new(patterns)
           keys_whitelist.call(notice)
@@ -94,8 +94,8 @@ RSpec.describe Airbrake::Filters::KeysWhitelist do
           [proc { proc { ['bingo'] } }],
           [
             { bingo: 'bango', bongo: 'bish', bash: 'bosh' },
-            { bingo: '[Filtered]', bongo: '[Filtered]', bash: '[Filtered]' }
-          ]
+            { bingo: '[Filtered]', bongo: '[Filtered]', bash: '[Filtered]' },
+          ],
         )
       end
     end
@@ -107,13 +107,13 @@ RSpec.describe Airbrake::Filters::KeysWhitelist do
       [Object.new],
       [
         { bingo: 'bango', bongo: 'bish', bash: 'bosh' },
-        { bingo: '[Filtered]', bongo: '[Filtered]', bash: '[Filtered]' }
-      ]
+        { bingo: '[Filtered]', bongo: '[Filtered]', bash: '[Filtered]' },
+      ],
     )
 
     it "logs an error" do
       expect(Airbrake::Loggable.instance).to receive(:error).with(
-        /KeysWhitelist is invalid.+patterns: \[#<Object:.+>\]/
+        /KeysWhitelist is invalid.+patterns: \[#<Object:.+>\]/,
       )
       keys_whitelist = described_class.new(patterns)
       keys_whitelist.call(notice)
@@ -127,8 +127,8 @@ RSpec.describe Airbrake::Filters::KeysWhitelist do
         %w[bongo bish],
         [
           { bingo: 'bango', bongo: { bish: 'bash' } },
-          { bingo: '[Filtered]', bongo: { bish: 'bash' } }
-        ]
+          { bingo: '[Filtered]', bongo: { bish: 'bash' } },
+        ],
       )
     end
 
@@ -166,7 +166,7 @@ RSpec.describe Airbrake::Filters::KeysWhitelist do
           notice[:context][:url] = 'http://localhost:3000/crash?foo=bar&baz=bongo&bish=bash'
           subject.call(notice)
           expect(notice[:context][:url]).to(
-            eq('http://localhost:3000/crash?foo=[Filtered]&baz=[Filtered]&bish=bash')
+            eq('http://localhost:3000/crash?foo=[Filtered]&baz=[Filtered]&bish=bash'),
           )
         end
       end
@@ -177,7 +177,7 @@ RSpec.describe Airbrake::Filters::KeysWhitelist do
             'http://localhost:3000/cra]]]sh?foo=bar&baz=bongo&bish=bash'
           subject.call(notice)
           expect(notice[:context][:url]).to(
-            eq('http://localhost:3000/cra]]]sh?foo=bar&baz=bongo&bish=bash')
+            eq('http://localhost:3000/cra]]]sh?foo=bar&baz=bongo&bish=bash'),
           )
         end
       end