airbrake-ruby 6.1.0-java → 6.2.0-java

data/spec/filter_chain_spec.rb

@@ -1,124 +0,0 @@
-RSpec.describe Airbrake::FilterChain do
-  subject(:filter_chain) { described_class.new }
-
-  let(:notice) { Airbrake::Notice.new(AirbrakeTestError.new) }
-
-  describe "#refine" do
-    let(:filter) do
-      Class.new do
-        attr_reader :weight
-
-        def initialize(weight)
-          @weight = weight
-        end
-
-        def call(notice)
-          notice[:params][:bingo] << @weight
-        end
-      end
-    end
-
-    it "executes filters from heaviest to lightest" do
-      notice[:params][:bingo] = []
-
-      (0...3).reverse_each { |i| filter_chain.add_filter(filter.new(i)) }
-      filter_chain.refine(notice)
-
-      expect(notice[:params][:bingo]).to eq([2, 1, 0])
-    end
-
-    it "stops execution once a notice was ignored" do
-      f2 = filter.new(2)
-      allow(f2).to receive(:call)
-
-      f1 = proc { |notice| notice.ignore! }
-
-      f0 = filter.new(-1)
-      allow(f0).to receive(:call)
-
-      [f2, f1, f0].each { |f| filter_chain.add_filter(f) }
-
-      filter_chain.refine(notice)
-
-      expect(f2).to have_received(:call)
-      expect(f0).not_to have_received(:call)
-    end
-  end
-
-  describe "#delete_filter" do
-    let(:filter) do
-      Class.new do
-        class << self
-          def name
-            'FooFilter'
-          end
-        end
-
-        def initialize(foo)
-          @foo = foo
-        end
-
-        def call(notice)
-          notice[:params][:foo] << @foo
-        end
-      end
-    end
-
-    it "deletes a class filter" do
-      notice[:params][:foo] = []
-
-      f1 = filter.new(1)
-      filter_chain.add_filter(f1)
-
-      foo_filter_mock = double
-      allow(foo_filter_mock).to receive(:name).at_least(:once).and_return('FooFilter')
-      filter_chain.delete_filter(foo_filter_mock)
-
-      expect(foo_filter_mock).to have_received(:name)
-
-      f2 = filter.new(2)
-      filter_chain.add_filter(f2)
-
-      filter_chain.refine(notice)
-      expect(notice[:params][:foo]).to eq([2])
-    end
-  end
-
-  describe "#inspect" do
-    it "returns a string representation of an empty FilterChain" do
-      expect(filter_chain.inspect).to eq('[]')
-    end
-
-    it "returns a string representation of a non-empty FilterChain" do
-      filter_chain.add_filter(proc {})
-      expect(filter_chain.inspect).to eq('[Proc]')
-    end
-  end
-
-  describe "#includes?" do
-    context "when a custom filter class is included in the filter chain" do
-      it "returns true" do
-        klass = Class.new
-
-        filter_chain.add_filter(klass.new)
-        expect(filter_chain.includes?(klass)).to be(true)
-      end
-    end
-
-    context "when Proc filter class is included in the filter chain" do
-      it "returns true" do
-        filter_chain.add_filter(proc {})
-        expect(filter_chain.includes?(Proc)).to be(true)
-      end
-    end
-
-    context "when filter class is NOT included in the filter chain" do
-      it "returns false" do
-        klass = Class.new
-
-        filter_chain.add_filter(proc {})
-        expect(filter_chain.includes?(klass)).to be(false)
-      end
-    end
-  end
-end

data/spec/filters/context_filter_spec.rb

@@ -1,32 +0,0 @@
-RSpec.describe Airbrake::Filters::ContextFilter do
-  let(:notice) { Airbrake::Notice.new(AirbrakeTestError.new) }
-
-  context "when the current context is empty" do
-    it "doesn't merge anything with params" do
-      described_class.new.call(notice)
-      expect(notice[:params]).to be_empty
-    end
-  end
-
-  context "when the current context has some data" do
-    it "merges the data with params" do
-      Airbrake.merge_context(apples: 'oranges')
-      described_class.new.call(notice)
-      expect(notice[:params]).to eq(airbrake_context: { apples: 'oranges' })
-    end
-
-    it "clears the data from the current context" do
-      context = { apples: 'oranges' }
-      Airbrake.merge_context(context)
-      described_class.new.call(notice)
-      expect(Airbrake::Context.current).to be_empty
-    end
-
-    it "does not mutate the provided context object" do
-      context = { apples: 'oranges' }
-      Airbrake.merge_context(context)
-      described_class.new.call(notice)
-      expect(context).to match(apples: 'oranges')
-    end
-  end
-end

data/spec/filters/dependency_filter_spec.rb

@@ -1,14 +0,0 @@
-RSpec.describe Airbrake::Filters::DependencyFilter do
-  subject(:dependency_filter) { described_class.new }
-
-  let(:notice) { Airbrake::Notice.new(AirbrakeTestError.new) }
-
-  describe "#call" do
-    it "attaches loaded dependencies to context/versions/dependencies" do
-      dependency_filter.call(notice)
-      expect(notice[:context][:versions][:dependencies]).to include(
-        'airbrake-ruby' => Airbrake::AIRBRAKE_RUBY_VERSION,
-      )
-    end
-  end
-end

data/spec/filters/exception_attributes_filter_spec.rb

@@ -1,52 +0,0 @@
-RSpec.describe Airbrake::Filters::ExceptionAttributesFilter do
-  subject(:exception_attributes_filter) { described_class.new }
-
-  describe "#call" do
-    let(:notice) { Airbrake::Notice.new(ex) }
-
-    context "when #to_airbrake returns a non-Hash object" do
-      let(:ex) do
-        Class.new(AirbrakeTestError) do
-          def to_airbrake
-            Object.new
-          end
-        end.new
-      end
-
-      it "doesn't raise" do
-        expect { exception_attributes_filter.call(notice) }.not_to raise_error
-        expect(notice[:params]).to be_empty
-      end
-    end
-
-    context "when #to_airbrake errors out" do
-      let(:ex) do
-        Class.new(AirbrakeTestError) do
-          def to_airbrake
-            1 / 0
-          end
-        end.new
-      end
-
-      it "doesn't raise" do
-        expect { exception_attributes_filter.call(notice) }.not_to raise_error
-        expect(notice[:params]).to be_empty
-      end
-    end
-
-    context "when #to_airbrake returns a hash" do
-      let(:ex) do
-        Class.new(AirbrakeTestError) do
-          def to_airbrake
-            { params: { foo: '1' } }
-          end
-        end.new
-      end
-
-      it "merges parameters with the notice" do
-        exception_attributes_filter.call(notice)
-        expect(notice[:params]).to eq(foo: '1')
-      end
-    end
-  end
-end

data/spec/filters/gem_root_filter_spec.rb

@@ -1,44 +0,0 @@
-RSpec.describe Airbrake::Filters::GemRootFilter do
-  subject(:gem_root_filter) { described_class.new }
-
-  let(:notice) { Airbrake::Notice.new(AirbrakeTestError.new) }
-  let(:root1) { '/my/gem/root' }
-  let(:root2) { '/my/other/gem/root' }
-
-  before { Gem.path << root1 << root2 }
-
-  after { 2.times { Gem.path.pop } }
-
-  it "replaces gem root in the backtrace with a label" do
-    # rubocop:disable Layout/LineLength
-    notice[:errors].first[:backtrace] = [
-      { 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" },
-    ]
-    # rubocop:enable Layout/LineLength
-
-    gem_root_filter.call(notice)
-
-    # rubocop:disable Layout/LineLength
-    expect(notice[:errors].first[:backtrace]).to(
-      eq(
-        [
-          { 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" },
-        ],
-      ),
-    )
-    # rubocop:enable Layout/LineLength
-  end
-
-  it "does not filter file when it is nil" do
-    expect(notice[:errors].first[:file]).to be_nil
-    expect { gem_root_filter.call(notice) }.not_to(
-      change { notice[:errors].first[:file] },
-    )
-  end
-end

data/spec/filters/git_last_checkout_filter_spec.rb

@@ -1,61 +0,0 @@
-RSpec.describe Airbrake::Filters::GitLastCheckoutFilter do
-  subject(:git_last_checkout_filter) { described_class.new('.') }
-
-  let(:notice) { Airbrake::Notice.new(AirbrakeTestError.new) }
-
-  context "when context/lastCheckout is defined" do
-    it "doesn't attach anything to context/lastCheckout" do
-      notice[:context][:lastCheckout] = '123'
-      git_last_checkout_filter.call(notice)
-      expect(notice[:context][:lastCheckout]).to eq('123')
-    end
-  end
-
-  context "when .git directory doesn't exist" do
-    subject(:git_last_checkout_without_git_dir_filter) { described_class.new('root/dir') }
-
-    it "doesn't attach anything to context/lastCheckout" do
-      git_last_checkout_without_git_dir_filter.call(notice)
-      expect(notice[:context][:lastCheckout]).to be_nil
-    end
-  end
-
-  context "when .git directory exists" do
-    context "when AIRBRAKE_DEPLOY_USERNAME env variable is set" do
-      before { ENV['AIRBRAKE_DEPLOY_USERNAME'] = 'deployer' }
-
-      it "attaches username from the environment" do
-        git_last_checkout_filter.call(notice)
-        expect(notice[:context][:lastCheckout][:username]).to eq('deployer')
-      end
-    end
-
-    context "when AIRBRAKE_DEPLOY_USERNAME env variable is NOT set" do
-      before { ENV['AIRBRAKE_DEPLOY_USERNAME'] = nil }
-
-      it "attaches last checkouted username" do
-        git_last_checkout_filter.call(notice)
-        username = notice[:context][:lastCheckout][:username]
-        expect(username).not_to be_empty
-        expect(username).not_to be_nil
-      end
-    end
-
-    it "attaches last checkouted email" do
-      git_last_checkout_filter.call(notice)
-      expect(notice[:context][:lastCheckout][:email]).to(match(/\A\w+@[\w\-.]+\z/))
-    end
-
-    it "attaches last checkouted revision" do
-      git_last_checkout_filter.call(notice)
-      expect(notice[:context][:lastCheckout][:revision]).not_to be_empty
-      expect(notice[:context][:lastCheckout][:revision].size).to eq(40)
-    end
-
-    it "attaches last checkouted time" do
-      git_last_checkout_filter.call(notice)
-      expect(notice[:context][:lastCheckout][:time]).not_to be_empty
-      expect(notice[:context][:lastCheckout][:time].size).to eq(25)
-    end
-  end
-end

data/spec/filters/git_repository_filter_spec.rb

@@ -1,72 +0,0 @@
-RSpec.describe Airbrake::Filters::GitRepositoryFilter do
-  subject(:git_repository_filter) { described_class.new('.') }
-
-  let(:notice) { Airbrake::Notice.new(AirbrakeTestError.new) }
-
-  describe "#initialize" do
-    it "parses standard git version" do
-      allow_any_instance_of(Kernel)
-        .to receive(:`).and_return('git version 2.18.0')
-      expect { git_repository_filter }.not_to raise_error
-    end
-
-    it "parses release candidate git version" do
-      allow_any_instance_of(Kernel)
-        .to receive(:`).and_return('git version 2.21.0-rc0')
-      expect { git_repository_filter }.not_to raise_error
-    end
-
-    it "parses git version with brackets" do
-      allow_any_instance_of(Kernel)
-        .to receive(:`).and_return('git version 2.17.2 (Apple Git-113)')
-      expect { git_repository_filter }.not_to raise_error
-    end
-
-    context "when Errno::EAGAIN is raised when detecting git version" do
-      it "doesn't attach anything to context/repository" do
-        allow_any_instance_of(Kernel).to receive(:`).and_raise(Errno::EAGAIN)
-        git_repository_filter.call(notice)
-        expect(notice[:context][:repository]).to be_nil
-      end
-    end
-  end
-
-  context "when context/repository is defined" do
-    it "doesn't attach anything to context/repository" do
-      notice[:context][:repository] = 'git@github.com:kyrylo/test.git'
-      git_repository_filter.call(notice)
-      expect(notice[:context][:repository]).to eq('git@github.com:kyrylo/test.git')
-    end
-  end
-
-  context "when .git directory doesn't exist" do
-    subject(:git_repository_filter) { described_class.new('root/dir') }
-
-    it "doesn't attach anything to context/repository" do
-      git_repository_filter.call(notice)
-      expect(notice[:context][:repository]).to be_nil
-    end
-  end
-
-  context "when .git directory exists" do
-    it "attaches context/repository" do
-      git_repository_filter.call(notice)
-      expect(notice[:context][:repository]).to match(
-        'github.com/airbrake/airbrake-ruby',
-      )
-    end
-  end
-
-  context "when git is not in PATH" do
-    let!(:path) { ENV['PATH'] }
-
-    before { ENV['PATH'] = '' }
-
-    after { ENV['PATH'] = path }
-
-    it "does not attach context/repository" do
-      git_repository_filter.call(notice)
-      expect(notice[:context][:repository]).to be_nil
-    end
-  end
-end

data/spec/filters/git_revision_filter_spec.rb

@@ -1,126 +0,0 @@
-RSpec.describe Airbrake::Filters::GitRevisionFilter do
-  subject(:git_revision_filter) { described_class.new('root/dir') }
-
-  # 'let!', not 'let' to make sure Notice doesn't call File.exist? with
-  # unexpected arguments.
-  let!(:notice) { Airbrake::Notice.new(AirbrakeTestError.new) }
-
-  context "when context/revision is defined" do
-    it "doesn't attach anything to context/revision" do
-      notice[:context][:revision] = '1.2.3'
-      git_revision_filter.call(notice)
-      expect(notice[:context][:revision]).to eq('1.2.3')
-    end
-  end
-
-  context "when .git directory doesn't exist" do
-    it "doesn't attach anything to context/revision" do
-      git_revision_filter.call(notice)
-      expect(notice[:context][:revision]).to be_nil
-    end
-  end
-
-  context "when .git directory exists" do
-    before do
-      allow(File).to receive(:exist?).with('root/dir/.git').and_return(true)
-    end
-
-    context "and when HEAD doesn't exist" do
-      before do
-        allow(File).to receive(:exist?).with('root/dir/.git/HEAD').and_return(false)
-      end
-
-      it "doesn't attach anything to context/revision" do
-        git_revision_filter.call(notice)
-        expect(notice[:context][:revision]).to be_nil
-      end
-    end
-
-    context "and when HEAD exists" do
-      before do
-        allow(File).to receive(:exist?).with('root/dir/.git/HEAD').and_return(true)
-      end
-
-      context "and also when HEAD doesn't start with 'ref: '" do
-        before do
-          allow(File).to(
-            receive(:read).with('root/dir/.git/HEAD').and_return('refs/foo'),
-          )
-        end
-
-        it "attaches the content of HEAD to context/revision" do
-          git_revision_filter.call(notice)
-          expect(notice[:context][:revision]).to eq('refs/foo')
-        end
-      end
-
-      context "and also when HEAD starts with 'ref: '" do
-        before do
-          allow(File).to(
-            receive(:read).with('root/dir/.git/HEAD').and_return("ref: refs/foo\n"),
-          )
-        end
-
-        context "when the ref exists" do
-          before do
-            allow(File).to(
-              receive(:exist?).with('root/dir/.git/refs/foo').and_return(true),
-            )
-            allow(File).to(
-              receive(:read).with('root/dir/.git/refs/foo').and_return("d34db33f\n"),
-            )
-          end
-
-          it "attaches the revision from the ref to context/revision" do
-            git_revision_filter.call(notice)
-            expect(notice[:context][:revision]).to eq('d34db33f')
-          end
-        end
-
-        context "when the ref doesn't exist" do
-          before do
-            allow(File).to(
-              receive(:exist?).with('root/dir/.git/refs/foo').and_return(false),
-            )
-          end
-
-          context "and when '.git/packed-refs' exists" do
-            before do
-              allow(File).to(
-                receive(:exist?).with('root/dir/.git/packed-refs').and_return(true),
-              )
-              allow(File).to(
-                receive(:readlines).with('root/dir/.git/packed-refs').and_return(
-                  [
-                    "# pack-refs with: peeled fully-peeled\n",
-                    "ccb316eecff79c7528d1ad43e5fa165f7a44d52e refs/tags/v3.0.30\n",
-                    "^d358900f73ee5bfd6ca3a592cf23ac6e82df83c1",
-                    "d34db33f refs/foo\n",
-                  ],
-                ),
-              )
-            end
-
-            it "attaches the revision from 'packed-refs' to context/revision" do
-              git_revision_filter.call(notice)
-              expect(notice[:context][:revision]).to eq('d34db33f')
-            end
-          end
-
-          context "and when '.git/packed-refs' doesn't exist" do
-            before do
-              allow(File).to(
-                receive(:exist?).with('root/dir/.git/packed-refs').and_return(false),
-              )
-            end
-
-            it "attaches the content of HEAD to context/revision" do
-              git_revision_filter.call(notice)
-              expect(notice[:context][:revision]).to eq('refs/foo')
-            end
-          end
-        end
-      end
-    end
-  end
-end