airbrake-ruby 4.8.0 → 4.10.1

data/spec/query_spec.rb

@@ -2,10 +2,20 @@ RSpec.describe Airbrake::Query do
   describe "#stash" do
     subject do
       described_class.new(
-        method: 'GET', route: '/', query: '', start_time: Time.now
+        method: 'GET', route: '/', query: '', start_time: Time.now,
       )
     end
 
     it { is_expected.to respond_to(:stash) }
   end
+
+  describe "#end_time" do
+    it "is always equal to start_time + 1 second by default" do
+      time = Time.now
+      query = described_class.new(
+        method: 'GET', route: '/', query: '', start_time: time,
+      )
+      expect(query.end_time).to eq(time + 1)
+    end
+  end
 end

data/spec/queue_spec.rb

@@ -0,0 +1,21 @@
+RSpec.describe Airbrake::Queue do
+  subject { described_class.new(queue: 'bananas', error_count: 0) }
+
+  describe "#ignore" do
+    it { is_expected.to respond_to(:ignore!) }
+  end
+
+  describe "#stash" do
+    it { is_expected.to respond_to(:stash) }
+  end
+
+  describe "#end_time" do
+    it "is always equal to start_time + 1 second by default" do
+      time = Time.now
+      queue = described_class.new(
+        queue: 'bananas', error_count: 0, start_time: time,
+      )
+      expect(queue.end_time).to eq(time + 1)
+    end
+  end
+end

data/spec/request_spec.rb

@@ -2,10 +2,20 @@ RSpec.describe Airbrake::Request do
   describe "#stash" do
     subject do
       described_class.new(
-        method: 'GET', route: '/', status_code: 200, start_time: Time.now
+        method: 'GET', route: '/', status_code: 200, start_time: Time.now,
       )
     end
 
     it { is_expected.to respond_to(:stash) }
   end
+
+  describe "#end_time" do
+    it "is always equal to start_time + 1 second by default" do
+      time = Time.now
+      request = described_class.new(
+        method: 'GET', route: '/', status_code: 200, start_time: time,
+      )
+      expect(request.end_time).to eq(time + 1)
+    end
+  end
 end

data/spec/response_spec.rb

@@ -4,7 +4,7 @@ RSpec.describe Airbrake::Response do
       context "when response code is #{code}" do
         it "logs response body" do
           expect(Airbrake::Loggable.instance).to receive(:debug).with(
-            /Airbrake::Response \(#{code}\): {}/
+            /Airbrake::Response \(#{code}\): {}/,
           )
           described_class.parse(OpenStruct.new(code: code, body: '{}'))
         end
@@ -15,10 +15,10 @@ RSpec.describe Airbrake::Response do
       context "when response code is #{code}" do
         it "logs response message" do
           expect(Airbrake::Loggable.instance).to receive(:error).with(
-            /Airbrake: foo/
+            /Airbrake: foo/,
           )
           described_class.parse(
-            OpenStruct.new(code: code, body: '{"message":"foo"}')
+            OpenStruct.new(code: code, body: '{"message":"foo"}'),
           )
         end
       end
@@ -29,7 +29,7 @@ RSpec.describe Airbrake::Response do
 
       it "logs response message" do
         expect(Airbrake::Loggable.instance).to receive(:error).with(
-          /Airbrake: rate limited/
+          /Airbrake: rate limited/,
         )
         described_class.parse(response)
       end
@@ -41,7 +41,7 @@ RSpec.describe Airbrake::Response do
         resp = described_class.parse(response)
         expect(resp).to include(
           'error' => '**Airbrake: rate limited',
-          'rate_limit_reset' => time
+          'rate_limit_reset' => time,
         )
       end
     end
@@ -51,7 +51,7 @@ RSpec.describe Airbrake::Response do
 
       it "logs response body" do
         expect(Airbrake::Loggable.instance).to receive(:error).with(
-          /Airbrake: unexpected code \(500\)\. Body: foo/
+          /Airbrake: unexpected code \(500\)\. Body: foo/,
         )
         described_class.parse(response)
       end
@@ -73,14 +73,14 @@ RSpec.describe Airbrake::Response do
 
       it "logs response body" do
         expect(Airbrake::Loggable.instance).to receive(:error).with(
-          /Airbrake: error while parsing body \(.*unexpected token.*\)\. Body: foo/
+          /Airbrake: error while parsing body \(.*unexpected token.*\)\. Body: foo/,
         )
         described_class.parse(response)
       end
 
       it "returns an error message" do
         expect(described_class.parse(response)['error']).to match(
-          /\A#<JSON::ParserError.+>/
+          /\A#<JSON::ParserError.+>/,
         )
       end
     end

data/spec/spec_helper.rb

@@ -47,7 +47,7 @@ class AirbrakeTestError < RuntimeError
       "/home/kyrylo/.gem/ruby/2.2.2/gems/rspec-core-3.3.2/lib/rspec/core/runner.rb:88:in `run'",
       "/home/kyrylo/.gem/ruby/2.2.2/gems/rspec-core-3.3.2/lib/rspec/core/runner.rb:73:in `run'",
       "/home/kyrylo/.gem/ruby/2.2.2/gems/rspec-core-3.3.2/lib/rspec/core/runner.rb:41:in `invoke'",
-      "/home/kyrylo/.gem/ruby/2.2.2/gems/rspec-core-3.3.2/exe/rspec:4:in `<main>'"
+      "/home/kyrylo/.gem/ruby/2.2.2/gems/rspec-core-3.3.2/exe/rspec:4:in `<main>'",
     ]
     # rubocop:enable Metrics/LineLength
   end
@@ -78,7 +78,7 @@ class JavaAirbrakeTestError < AirbrakeTestError
       "opt.rubies.jruby_minus_9_dot_0_dot_0_dot_0.bin.irb.RUBY$script(/opt/rubies/jruby-9.0.0.0/bin/irb:13)",
       "org.jruby.ir.Compiler$1.load(Compiler.java:111)",
       "org.jruby.Main.run(Main.java:225)",
-      "org.jruby.Main.main(Main.java:197)"
+      "org.jruby.Main.main(Main.java:197)",
     ]
     # rubocop:enable Metrics/LineLength
   end

data/spec/stat_spec.rb

@@ -5,7 +5,7 @@ RSpec.describe Airbrake::Stat do
         'count' => 0,
         'sum' => 0.0,
         'sumsq' => 0.0,
-        'tdigest' => 'AAAAAkA0AAAAAAAAAAAAAA=='
+        'tdigest' => 'AAAAAkA0AAAAAAAAAAAAAA==',
       )
     end
   end
@@ -34,7 +34,7 @@ RSpec.describe Airbrake::Stat do
   describe "#inspect" do
     it "provides custom inspect output" do
       expect(subject.inspect).to eq(
-        '#<struct Airbrake::Stat count=0, sum=0.0, sumsq=0.0>'
+        '#<struct Airbrake::Stat count=0, sum=0.0, sumsq=0.0>',
       )
     end
   end

data/spec/sync_sender_spec.rb

@@ -1,7 +1,7 @@
 RSpec.describe Airbrake::SyncSender do
   before do
     Airbrake::Config.instance = Airbrake::Config.new(
-      project_id: 1, project_key: 'banana'
+      project_id: 1, project_key: 'banana',
     )
   end
 
@@ -17,8 +17,8 @@ RSpec.describe Airbrake::SyncSender do
       subject.send({}, promise)
       expect(
         a_request(:post, endpoint).with(
-          headers: { 'Content-Type' => 'application/json' }
-        )
+          headers: { 'Content-Type' => 'application/json' },
+        ),
       ).to have_been_made.once
     end
 
@@ -27,9 +27,9 @@ RSpec.describe Airbrake::SyncSender do
       expect(
         a_request(:post, endpoint).with(
           headers: {
-            'User-Agent' => %r{airbrake-ruby/\d+\.\d+\.\d+ Ruby/\d+\.\d+\.\d+}
-          }
-        )
+            'User-Agent' => %r{airbrake-ruby/\d+\.\d+\.\d+ Ruby/\d+\.\d+\.\d+},
+          },
+        ),
       ).to have_been_made.once
     end
 
@@ -37,8 +37,8 @@ RSpec.describe Airbrake::SyncSender do
       subject.send({}, promise)
       expect(
         a_request(:post, endpoint).with(
-          headers: { 'Authorization' => 'Bearer banana' }
-        )
+          headers: { 'Authorization' => 'Bearer banana' },
+        ),
       ).to have_been_made.once
     end
 
@@ -47,7 +47,7 @@ RSpec.describe Airbrake::SyncSender do
       allow(subject).to receive(:build_https).and_return(https)
       allow(https).to receive(:request).and_raise(StandardError.new('foo'))
       expect(Airbrake::Loggable.instance).to receive(:error).with(
-        /HTTP error: foo/
+        /HTTP error: foo/,
       )
       expect(subject.send({}, promise)).to be_an(Airbrake::Promise)
       expect(promise.value).to eq('error' => '**Airbrake: HTTP error: foo')
@@ -69,10 +69,10 @@ RSpec.describe Airbrake::SyncSender do
         notice = Airbrake::Notice.new(ex)
 
         expect(Airbrake::Loggable.instance).to receive(:error).with(
-          /data was not sent/
+          /data was not sent/,
         )
         expect(Airbrake::Loggable.instance).to receive(:error).with(
-          /truncation failed/
+          /truncation failed/,
         )
         expect(subject.send(notice, promise)).to be_an(Airbrake::Promise)
         expect(promise.value)
@@ -87,7 +87,7 @@ RSpec.describe Airbrake::SyncSender do
         stub_request(:post, endpoint).to_return(
           status: 429,
           body: '{"message":"IP is rate limited"}',
-          headers: { 'X-RateLimit-Delay' => '1' }
+          headers: { 'X-RateLimit-Delay' => '1' },
         )
       end
 

data/spec/tdigest_spec.rb

@@ -29,7 +29,7 @@ RSpec.describe Airbrake::TDigest do
       new_tdigest = described_class.from_bytes(bytes)
       # Expect some rounding error due to compression
       expect(new_tdigest.percentile(0.9).round(5)).to eq(
-        subject.percentile(0.9).round(5)
+        subject.percentile(0.9).round(5),
       )
       expect(new_tdigest.as_small_bytes).to eq(bytes)
     end
@@ -126,8 +126,8 @@ RSpec.describe Airbrake::TDigest do
           113270270.27027026,
           154459459.45945945,
           123829787.23404256,
-          103191489.36170213
-        ]
+          103191489.36170213,
+        ],
       )
     end
 
@@ -166,13 +166,13 @@ RSpec.describe Airbrake::TDigest do
       it "has the parameters of the left argument (the calling tdigest)" do
         new_tdigest = subject + @other
         expect(new_tdigest.instance_variable_get(:@delta)).to eq(
-          subject.instance_variable_get(:@delta)
+          subject.instance_variable_get(:@delta),
         )
         expect(new_tdigest.instance_variable_get(:@k)).to eq(
-          subject.instance_variable_get(:@k)
+          subject.instance_variable_get(:@k),
         )
         expect(new_tdigest.instance_variable_get(:@cx)).to eq(
-          subject.instance_variable_get(:@cx)
+          subject.instance_variable_get(:@cx),
         )
       end
 

data/spec/thread_pool_spec.rb

@@ -7,7 +7,7 @@ RSpec.describe Airbrake::ThreadPool do
     described_class.new(
       worker_size: worker_size,
       queue_size: queue_size,
-      block: proc { |message| tasks << message }
+      block: proc { |message| tasks << message },
     )
   end
 
@@ -35,7 +35,7 @@ RSpec.describe Airbrake::ThreadPool do
         described_class.new(
           worker_size: 1,
           queue_size: 1,
-          block: proc { |message| tasks << message }
+          block: proc { |message| tasks << message },
         )
       end
 
@@ -98,11 +98,11 @@ RSpec.describe Airbrake::ThreadPool do
     context "when there's some work to do" do
       it "logs how many tasks are left to process" do
         thread_pool = described_class.new(
-          worker_size: 0, queue_size: 2, block: proc {}
+          worker_size: 0, queue_size: 2, block: proc {},
         )
 
         expect(Airbrake::Loggable.instance).to receive(:debug).with(
-          /waiting to process \d+ task\(s\)/
+          /waiting to process \d+ task\(s\)/,
         )
         expect(Airbrake::Loggable.instance).to receive(:debug).with(/closed/)
 
@@ -112,7 +112,7 @@ RSpec.describe Airbrake::ThreadPool do
 
       it "waits until the queue gets empty" do
         thread_pool = described_class.new(
-          worker_size: 1, queue_size: 2, block: proc {}
+          worker_size: 1, queue_size: 2, block: proc {},
         )
 
         10.times { subject << 1 }

data/spec/timed_trace_spec.rb

@@ -117,7 +117,7 @@ RSpec.describe Airbrake::TimedTrace do
       it "returns a Hash with all spans" do
         expect(subject.spans).to match(
           'operation' => be > 0,
-          'another operation' => be > 0
+          'another operation' => be > 0,
         )
       end
     end

data/spec/truncator_spec.rb

@@ -25,7 +25,7 @@ RSpec.describe Airbrake::Truncator do
           banana: multiply_by_2_max_len('a'),
           kiwi: multiply_by_2_max_len('b'),
           strawberry: 'c',
-          shrimp: 'd'
+          shrimp: 'd',
         }.freeze
       end
 
@@ -34,7 +34,7 @@ RSpec.describe Airbrake::Truncator do
         expect(subject).to be_frozen
 
         expect(subject).to eq(
-          banana: 'aaa[Truncated]', kiwi: 'bbb[Truncated]', strawberry: 'c'
+          banana: 'aaa[Truncated]', kiwi: 'bbb[Truncated]', strawberry: 'c',
         )
         expect(subject[:banana]).to be_frozen
         expect(subject[:kiwi]).to be_frozen
@@ -48,7 +48,7 @@ RSpec.describe Airbrake::Truncator do
           multiply_by_2_max_len('a'),
           'b',
           multiply_by_2_max_len('c'),
-          'd'
+          'd',
         ].freeze
       end
 
@@ -69,7 +69,7 @@ RSpec.describe Airbrake::Truncator do
           multiply_by_2_max_len('a'),
           'b',
           multiply_by_2_max_len('c'),
-          'd'
+          'd',
         ]).freeze
       end
 
@@ -78,7 +78,7 @@ RSpec.describe Airbrake::Truncator do
         expect(subject).to be_frozen
 
         expect(subject).to eq(
-          Set.new(['aaa[Truncated]', 'b', 'ccc[Truncated]'])
+          Set.new(['aaa[Truncated]', 'b', 'ccc[Truncated]']),
         )
         expect(subject).to be_frozen
       end
@@ -166,7 +166,7 @@ RSpec.describe Airbrake::Truncator do
 
       it "prevents recursion" do
         expect(subject).to eq(
-          Set.new(['[Circular]', { k: '[Circular]' }, 'aaa[Truncated]'])
+          Set.new(['[Circular]', { k: '[Circular]' }, 'aaa[Truncated]']),
         )
         expect(subject).to be_frozen
       end
@@ -177,13 +177,13 @@ RSpec.describe Airbrake::Truncator do
         {
           a: multiply_by_2_max_len('a'),
           b: multiply_by_2_max_len('b'),
-          c: { d: multiply_by_2_max_len('d'), e: 'e' }
+          c: { d: multiply_by_2_max_len('d'), e: 'e' },
         }
       end
 
       it "truncates the long strings" do
         expect(subject).to eq(
-          a: 'aaa[Truncated]', b: 'bbb[Truncated]', c: { d: 'ddd[Truncated]', e: 'e' }
+          a: 'aaa[Truncated]', b: 'bbb[Truncated]', c: { d: 'ddd[Truncated]', e: 'e' },
         )
         expect(subject).to be_frozen
       end
@@ -218,8 +218,8 @@ RSpec.describe Airbrake::Truncator do
           errors: [
             { file: 'a' },
             { file: 'a' },
-            hashie.new.merge(file: 'bcde')
-          ]
+            hashie.new.merge(file: 'bcde'),
+          ],
         }
       end
 
@@ -228,8 +228,8 @@ RSpec.describe Airbrake::Truncator do
           errors: [
             { file: 'a' },
             { file: 'a' },
-            hashie.new.merge(file: 'bcd[Truncated]')
-          ]
+            hashie.new.merge(file: 'bcd[Truncated]'),
+          ],
         )
         expect(subject).to be_frozen
       end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: airbrake-ruby
 version: !ruby/object:Gem::Version
-  version: 4.8.0
+  version: 4.10.1
 platform: ruby
 authors:
 - Airbrake Technologies, Inc.
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2019-10-23 00:00:00.000000000 Z
+date: 2019-12-12 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rbtree3
@@ -62,10 +62,12 @@ files:
 - lib/airbrake-ruby/filters/sql_filter.rb
 - lib/airbrake-ruby/filters/system_exit_filter.rb
 - lib/airbrake-ruby/filters/thread_filter.rb
+- lib/airbrake-ruby/grouppable.rb
 - lib/airbrake-ruby/hash_keyable.rb
 - lib/airbrake-ruby/ignorable.rb
 - lib/airbrake-ruby/inspectable.rb
 - lib/airbrake-ruby/loggable.rb
+- lib/airbrake-ruby/mergeable.rb
 - lib/airbrake-ruby/monotonic_time.rb
 - lib/airbrake-ruby/nested_exception.rb
 - lib/airbrake-ruby/notice.rb
@@ -74,6 +76,7 @@ files:
 - lib/airbrake-ruby/performance_notifier.rb
 - lib/airbrake-ruby/promise.rb
 - lib/airbrake-ruby/query.rb
+- lib/airbrake-ruby/queue.rb
 - lib/airbrake-ruby/request.rb
 - lib/airbrake-ruby/response.rb
 - lib/airbrake-ruby/stashable.rb
@@ -126,6 +129,7 @@ files:
 - spec/performance_notifier_spec.rb
 - spec/promise_spec.rb
 - spec/query_spec.rb
+- spec/queue_spec.rb
 - spec/request_spec.rb
 - spec/response_spec.rb
 - spec/spec_helper.rb
@@ -203,6 +207,7 @@ test_files:
 - spec/notice_notifier/options_spec.rb
 - spec/filter_chain_spec.rb
 - spec/response_spec.rb
+- spec/queue_spec.rb
 - spec/code_hunk_spec.rb
 - spec/fixtures/notroot.txt
 - spec/fixtures/project_root/long_line.txt