bugsnag 6.15.0 → 6.16.0

data/spec/session_tracker_spec.rb

@@ -1,156 +0,0 @@
-# encoding: utf-8
-require 'webrick'
-require 'spec_helper'
-require 'json'
-
-describe Bugsnag::SessionTracker do
-  server = nil
-  queue = Queue.new
-
-  before do
-    server = WEBrick::HTTPServer.new :Port => 0, :Logger => WEBrick::Log.new("/dev/null"), :AccessLog => []
-    server.mount_proc '/' do |req, res|
-      headers = []
-      req.each { |header| headers << header }
-      queue << [JSON.parse(req.body), headers]
-      res.status = 202
-      res.body = "OK\n"
-    end
-    Thread.new{ server.start }
-  end
-
-  before(:each) do
-    Bugsnag.instance_variable_set(:@session_tracker, Bugsnag::SessionTracker.new)
-  end
-
-  after do
-    Bugsnag.configure do |conf|
-      conf.auto_capture_sessions = false
-      conf.delivery_method = :synchronous
-    end
-    server.stop
-    queue.clear
-  end
-
-  after(:all) do
-    Bugsnag.instance_variable_set(:@session_tracker, Bugsnag::SessionTracker.new)
-  end
-
-  it 'adds session object to queue' do
-    tracker = Bugsnag::SessionTracker.new
-    tracker.start_session
-    expect(tracker.session_counts.size).to eq(1)
-    time = tracker.session_counts.keys.last
-    count = tracker.session_counts[time]
-
-    expect(count).to eq(1)
-  end
-
-  it 'stores session in thread' do
-    tracker = Bugsnag::SessionTracker.new
-    tracker.start_session
-    session = Thread.current[Bugsnag::SessionTracker::THREAD_SESSION]
-    expect(session.include? :id).to be true
-    expect(session.include? :startedAt).to be true
-    expect(session.include? :events).to be true
-    expect(session[:events].include? :handled).to be true
-    expect(session[:events].include? :unhandled).to be true
-    expect(session[:events][:handled]).to eq(0)
-    expect(session[:events][:unhandled]).to eq(0)
-  end
-
-  it 'gives unique ids to each session' do
-    tracker = Bugsnag::SessionTracker.new
-    tracker.start_session
-    session_one = Thread.current[Bugsnag::SessionTracker::THREAD_SESSION]
-    tracker.start_session
-    session_two = Thread.current[Bugsnag::SessionTracker::THREAD_SESSION]
-    expect(session_one[:id]).to_not eq(session_two[:id])
-  end
-
-  it 'will not create sessions if Configuration.enable_sessions is false' do
-    Bugsnag.configure do |conf|
-      conf.set_endpoints("http://localhost:#{server.config[:Port]}", nil)
-    end
-    expect(Bugsnag.configuration.enable_sessions).to eq(false)
-    expect(Bugsnag.session_tracker.session_counts.size).to eq(0)
-    Bugsnag.start_session
-    expect(Bugsnag.session_tracker.session_counts.size).to eq(0)
-  end
-
-  it 'sends sessions when send_sessions is called' do
-    Bugsnag.configure do |conf|
-      conf.auto_capture_sessions = true
-      conf.delivery_method = :synchronous
-      conf.set_endpoints("http://localhost:#{server.config[:Port]}", "http://localhost:#{server.config[:Port]}")
-    end
-    WebMock.allow_net_connect!
-    Bugsnag.start_session
-    expect(Bugsnag.session_tracker.session_counts.size).to eq(1)
-    Bugsnag.session_tracker.send_sessions
-    expect(Bugsnag.session_tracker.session_counts.size).to eq(0)
-    payload, headers = queue.pop
-    expect(payload.include?("app")).to be true
-    expect(payload.include?("notifier")).to be true
-    expect(payload.include?("device")).to be true
-    expect(payload.include?("sessionCounts")).to be true
-    expect(payload["sessionCounts"].size).to eq(1)
-  end
-
-  it 'sets details from config' do
-    Bugsnag.configure do |conf|
-      conf.auto_capture_sessions = true
-      conf.release_stage = "test_stage"
-      conf.delivery_method = :synchronous
-      conf.set_endpoints("http://localhost:#{server.config[:Port]}", "http://localhost:#{server.config[:Port]}")
-    end
-    WebMock.allow_net_connect!
-    Bugsnag.start_session
-    expect(Bugsnag.session_tracker.session_counts.size).to eq(1)
-    Bugsnag.session_tracker.send_sessions
-    expect(Bugsnag.session_tracker.session_counts.size).to eq(0)
-    payload, headers = queue.pop
-    notifier = payload["notifier"]
-    expect(notifier.include?("name")).to be true
-    expect(notifier["name"]).to eq(Bugsnag::Report::NOTIFIER_NAME)
-    expect(notifier.include?("url")).to be true
-    expect(notifier["url"]).to eq(Bugsnag::Report::NOTIFIER_URL)
-    expect(notifier.include?("version")).to be true
-    expect(notifier["version"]).to eq(Bugsnag::Report::NOTIFIER_VERSION)
-
-    app = payload["app"]
-    expect(app.include?("releaseStage")).to be true
-    expect(app["releaseStage"]).to eq(Bugsnag.configuration.release_stage)
-    expect(app.include?("version")).to be true
-    expect(app["version"]).to eq(Bugsnag.configuration.app_version)
-    expect(app.include?("type")).to be true
-    expect(app["type"]).to eq(Bugsnag.configuration.app_type)
-
-    device = payload["device"]
-    expect(device.include?("hostname")).to be true
-    expect(device["hostname"]).to eq(Bugsnag.configuration.hostname)
-    expect(device["runtimeVersions"]["ruby"]).to eq(Bugsnag.configuration.runtime_versions["ruby"])
-  end
-
-  it 'uses middleware to attach session to notification' do
-    Bugsnag.configure do |conf|
-      conf.auto_capture_sessions = true
-      conf.release_stage = "test_stage"
-    end
-    Bugsnag.start_session
-    Bugsnag.notify(BugsnagTestException.new("It crashed"))
-    expect(Bugsnag).to have_sent_notification{ |payload, headers|
-      event = payload["events"][0]
-      expect(event.include?("session")).to be true
-      session = event["session"]
-      expect(session.include?("id")).to be true
-      expect(session.include?("startedAt")).to be true
-      expect(session.include?("events")).to be true
-      sesevents = session['events']
-      expect(sesevents.include?("unhandled")).to be true
-      expect(sesevents["unhandled"]).to eq(0)
-      expect(sesevents.include?("handled")).to be true
-      expect(sesevents["handled"]).to eq(1)
-    }
-  end
-end

data/spec/spec_helper.rb

@@ -1,99 +0,0 @@
-if ENV['GEMSETS'] and ENV['GEMSETS'].include? "coverage"
-  require 'simplecov'
-  require 'coveralls'
-
-  SimpleCov.formatter = Coveralls::SimpleCov::Formatter
-  SimpleCov.start do
-    add_filter 'spec'
-  end
-end
-
-require 'bugsnag'
-
-require 'tmpdir'
-require 'webmock/rspec'
-require 'rspec/expectations'
-require 'rspec/mocks'
-
-class BugsnagTestException < RuntimeError
-  attr_accessor :skip_bugsnag
-end
-
-def get_event_from_payload(payload)
-  expect(payload["events"].size).to eq(1)
-  payload["events"].first
-end
-
-def get_exception_from_payload(payload)
-  event = get_event_from_payload(payload)
-  expect(event["exceptions"].size).to eq(1)
-  event["exceptions"].last
-end
-
-def get_code_from_payload(payload, index = 0)
-  exception = get_exception_from_payload(payload)
-
-  expect(exception["stacktrace"].size).to be > index
-
-  exception["stacktrace"][index]["code"]
-end
-
-def notify_test_exception(*args)
-  Bugsnag.notify(RuntimeError.new("test message"), *args)
-end
-
-def ruby_version_greater_equal?(version)
-  current_version = RUBY_VERSION.split "."
-  target_version = version.split "."
-  (Integer(current_version[0]) >= Integer(target_version[0])) &&
-    (Integer(current_version[1]) >= Integer(target_version[1])) &&
-    (Integer(current_version[2]) >= Integer(target_version[2]))
-end
-
-RSpec.configure do |config|
-  config.order = "random"
-  config.example_status_persistence_file_path = "#{Dir.tmpdir}/rspec_status"
-
-  config.before(:each) do
-    WebMock.stub_request(:post, "https://notify.bugsnag.com/")
-    WebMock.stub_request(:post, "https://sessions.bugsnag.com/")
-
-    Bugsnag.instance_variable_set(:@configuration, Bugsnag::Configuration.new)
-    Bugsnag.instance_variable_set(:@session_tracker, Bugsnag::SessionTracker.new)
-    Bugsnag.instance_variable_set(:@cleaner, Bugsnag::Cleaner.new(Bugsnag.configuration))
-
-    Bugsnag.configure do |bugsnag|
-      bugsnag.api_key = "c9d60ae4c7e70c4b6c4ebd3e8056d2b8"
-      bugsnag.release_stage = "production"
-      bugsnag.delivery_method = :synchronous
-      # silence logger in tests
-      bugsnag.logger = Logger.new(StringIO.new)
-    end
-  end
-
-  config.after(:each) do
-    Bugsnag.configuration.clear_request_data
-  end
-end
-
-def have_sent_sessions(&matcher)
-  have_requested(:post, "https://sessions.bugsnag.com/").with do |request|
-    if matcher
-      matcher.call([JSON.parse(request.body), request.headers])
-      true
-    else
-      raise "no matcher provided to have_sent_sessions (did you use { })"
-    end
-  end
-end
-
-def have_sent_notification(&matcher)
-  have_requested(:post, "https://notify.bugsnag.com/").with do |request|
-    if matcher
-      matcher.call([JSON.parse(request.body), request.headers])
-      true
-    else
-      raise "no matcher provided to have_sent_notification (did you use { })"
-    end
-  end
-end

data/spec/stacktrace_spec.rb

@@ -1,443 +0,0 @@
-require 'spec_helper'
-
-describe Bugsnag::Stacktrace do
-  context "sending code" do
-    it "includes code in the stack trace" do
-      begin
-        _a = 1
-        _b = 2
-        _c = 3
-        "Test".prepnd "T"
-        _d = 4
-        _e = 5
-        _f = 6
-      rescue Exception => e
-        Bugsnag.notify(e)
-      end
-
-      expect(Bugsnag).to have_sent_notification { |payload, headers|
-        starting_line = __LINE__ - 12
-
-        expect(get_code_from_payload(payload).to_a).to eq({
-          (starting_line + 0).to_s => '        _a = 1',
-          (starting_line + 1).to_s => '        _b = 2',
-          (starting_line + 2).to_s => '        _c = 3',
-          (starting_line + 3).to_s => '        "Test".prepnd "T"',
-          (starting_line + 4).to_s => '        _d = 4',
-          (starting_line + 5).to_s => '        _e = 5',
-          (starting_line + 6).to_s => '        _f = 6'
-        }.to_a)
-      }
-    end
-
-    it "allows you to disable sending code" do
-      Bugsnag.configuration.send_code = false
-
-      notify_test_exception
-
-      expect(Bugsnag).to have_sent_notification { |payload, headers|
-        expect(get_code_from_payload(payload)).to eq(nil)
-      }
-    end
-
-    it 'should send the first 7 lines of the file for exceptions near the top' do
-      load 'spec/fixtures/crashes/start_of_file.rb' rescue Bugsnag.notify $!
-
-      expect(Bugsnag).to have_sent_notification { |payload, headers|
-        expect(get_code_from_payload(payload).to_a).to eq({
-          "1" => "#",
-          "2" => "raise 'hell'",
-          "3" => "#",
-          "4" => "#",
-          "5" => "#",
-          "6" => "#",
-          "7" => "#"
-        }.to_a)
-      }
-    end
-
-    it 'should send the last 7 lines of the file for exceptions near the bottom' do
-      load 'spec/fixtures/crashes/end_of_file.rb' rescue Bugsnag.notify $!
-
-      expect(Bugsnag).to have_sent_notification { |payload, headers|
-        expect(get_code_from_payload(payload)).to eq({
-          "3" => "#",
-          "4" => "#",
-          "5" => "#",
-          "6" => "#",
-          "7" => "#",
-          "8" => "raise 'hell'",
-          "9" => "#"
-        })
-      }
-    end
-
-    it 'should send every line of a very short file' do
-      load 'spec/fixtures/crashes/short_file.rb' rescue Bugsnag.notify $!
-
-      expect(Bugsnag).to have_sent_notification { |payload, headers|
-        expect(get_code_from_payload(payload).to_a).to eq({
-          "1" => "#",
-          "2" => "raise 'hell'",
-          "3" => "#"
-        }.to_a)
-      }
-    end
-
-    it 'should send code for each line in the stacktrace' do
-      load 'spec/fixtures/crashes/functions.rb' rescue Bugsnag.notify $!
-
-      expected_code = [
-        # The topmost frame is centered on where the exception was raised
-        {
-          "11" => "end",
-          "12" => "",
-          "13" => "def xyz",
-          "14" => "  raise 'uh oh'",
-          "15" => "end",
-          "16" => "",
-          "17" => "def abc"
-        },
-        # then we get 'baz' which is where 'xyz' was called
-        {
-          "7" => "end",
-          "8" => "",
-          "9" => "def baz",
-          "10" => "  xyz",
-          "11" => "end",
-          "12" => "",
-          "13" => "def xyz"
-        },
-        # then we get 'bar' which is where 'baz' was called
-        {
-          "3" => "end",
-          "4" => "",
-          "5" => "def bar",
-          "6" => "  baz",
-          "7" => "end",
-          "8" => "",
-          "9" => "def baz"
-        },
-        # then we get 'foo' which is where 'bar' was called - this is the first
-        # 7 lines because the call to 'bar' is on line 2
-        {
-          "1" => "def foo",
-          "2" => "  bar",
-          "3" => "end",
-          "4" => "",
-          "5" => "def bar",
-          "6" => "  baz",
-          "7" => "end"
-        },
-        # finally we get the call to 'foo' - this is the last 7 lines because
-        # the call is on the last line of the file
-        {
-          "23" => "end",
-          "24" => "",
-          "25" => "def abcdefghi",
-          "26" => "  puts 'abcdefghi'",
-          "27" => "end",
-          "28" => "",
-          "29" => "foo"
-        }
-      ]
-
-      expect(Bugsnag).to have_sent_notification { |payload, headers|
-        (0...expected_code.size).each do |index|
-          expect(get_code_from_payload(payload, index).to_a).to eq(expected_code[index].to_a)
-        end
-      }
-    end
-
-    it 'should send code for each line in the stacktrace when split over multiple files' do
-      load 'spec/fixtures/crashes/file1.rb' rescue Bugsnag.notify $!
-
-      expected_code = [
-        {
-          "8" => "  end",
-          "9" => "",
-          "10" => "  def self.baz2",
-          "11" => "    raise 'uh oh'",
-          "12" => "  end",
-          "13" => "",
-          "14" => "  def self.abc2"
-        },
-        {
-          "10" => "  end",
-          "11" => "",
-          "12" => "  def self.baz1",
-          "13" => "    File2.baz2",
-          "14" => "  end",
-          "15" => "",
-          "16" => "  def self.abc1"
-        },
-        {
-          "4" => "  end",
-          "5" => "",
-          "6" => "  def self.bar2",
-          "7" => "    File1.baz1",
-          "8" => "  end",
-          "9" => "",
-          "10" => "  def self.baz2"
-        },
-        {
-          "6" => "  end",
-          "7" => "",
-          "8" => "  def self.bar1",
-          "9" => "    File2.bar2",
-          "10" => "  end",
-          "11" => "",
-          "12" => "  def self.baz1",
-        },
-        {
-          "1" => "module File2",
-          "2" => "  def self.foo2",
-          "3" => "    File1.bar1",
-          "4" => "  end",
-          "5" => "",
-          "6" => "  def self.bar2",
-          "7" => "    File1.baz1"
-        },
-        {
-          "2" => "",
-          "3" => "module File1",
-          "4" => "  def self.foo1",
-          "5" => "    File2.foo2",
-          "6" => "  end",
-          "7" => "",
-          "8" => "  def self.bar1"
-        },
-        {
-          "23" => "",
-          "24" => "  def self.abcdefghi1",
-          "25" => "    puts 'abcdefghi1'",
-          "26" => "  end",
-          "27" => "end",
-          "28" => "",
-          "29" => "File1.foo1"
-        }
-      ]
-
-      expect(Bugsnag).to have_sent_notification { |payload, headers|
-        (0...expected_code.size).each do |index|
-          expect(get_code_from_payload(payload, index).to_a).to eq(expected_code[index].to_a)
-        end
-      }
-    end
-
-    it "can extract code from paths that will be mangled by the project root" do
-      # Set the project root to a nested directory, which will then be stripped
-      # from the file paths in the API call. This ensures that we read the files
-      # based off of the original path, rather than the final file path, e.g.
-      # "spec/fixtures/crashes/file1.rb" will be "file1.rb" in the API call, which
-      # isn't a path that's possible to read
-      project_root = "#{File.dirname(File.dirname(__FILE__))}/spec/fixtures/crashes"
-
-      configuration = Bugsnag::Configuration.new
-      configuration.project_root = project_root
-
-      backtrace = [
-        "spec/fixtures/crashes/file1.rb:13:in `baz1'",
-        "./spec/fixtures/crashes/functions.rb:17:in `abc'",
-        "#{project_root}/file2.rb:19:in `abcdef2'",
-      ]
-
-      stacktrace = Bugsnag::Stacktrace.process(backtrace, configuration)
-
-      expect(stacktrace).to eq([
-        {
-          file: "file1.rb",
-          lineNumber: 13,
-          method: "baz1",
-          inProject: true,
-          code: {
-            10 => "  end",
-            11 => "",
-            12 => "  def self.baz1",
-            13 => "    File2.baz2",
-            14 => "  end",
-            15 => "",
-            16 => "  def self.abc1"
-          }
-        },
-        {
-          file: "functions.rb",
-          lineNumber: 17,
-          method: "abc",
-          inProject: true,
-          code: {
-            14 => "  raise 'uh oh'",
-            15 => "end",
-            16 => "",
-            17 => "def abc",
-            18 => "  puts 'abc'",
-            19 => "end",
-            20 => ""
-          },
-        },
-        {
-          file: "file2.rb",
-          lineNumber: 19,
-          method: "abcdef2",
-          inProject: true,
-          code: {
-            16 => "  end",
-            17 => "",
-            18 => "  def self.abcdef2",
-            19 => "    puts 'abcdef2'",
-            20 => "  end",
-            21 => "",
-            22 => "  def self.abcdefghi2"
-          },
-        },
-      ])
-    end
-  end
-
-  context "file paths" do
-    it "leaves absolute paths alone" do
-      configuration = Bugsnag::Configuration.new
-      configuration.send_code = false
-
-      backtrace = [
-        "/foo/bar/app/models/user.rb:1:in `something'",
-        "/foo/bar/other_vendor/lib/dont.rb:2:in `to_s'",
-        "/foo/bar/vendor/lib/ignore_me.rb:3:in `to_s'",
-        "/foo/bar/.bundle/lib/ignore_me.rb:4:in `to_s'",
-      ]
-
-      stacktrace = Bugsnag::Stacktrace.process(backtrace, configuration)
-
-      expect(stacktrace).to eq([
-        { file: "/foo/bar/app/models/user.rb", lineNumber: 1, method: "something" },
-        { file: "/foo/bar/other_vendor/lib/dont.rb", lineNumber: 2, method: "to_s" },
-        { file: "/foo/bar/vendor/lib/ignore_me.rb", lineNumber: 3, method: "to_s" },
-        { file: "/foo/bar/.bundle/lib/ignore_me.rb", lineNumber: 4, method: "to_s" },
-      ])
-    end
-
-    it "does not modify relative paths if they can't be resolved" do
-      configuration = Bugsnag::Configuration.new
-
-      backtrace = [
-        "./foo/bar/baz.rb:1:in `something'",
-        "../foo.rb:1:in `to_s'",
-        "../xyz.rb:1:in `to_s'",
-        "abc.rb:1:in `defg'",
-      ]
-
-      stacktrace = Bugsnag::Stacktrace.process(backtrace, configuration)
-
-      expect(stacktrace).to eq([
-        { code: nil, file: "./foo/bar/baz.rb", lineNumber: 1, method: "something" },
-        { code: nil, file: "../foo.rb", lineNumber: 1, method: "to_s" },
-        { code: nil, file: "../xyz.rb", lineNumber: 1, method: "to_s" },
-        { code: nil, file: "abc.rb", lineNumber: 1, method: "defg" },
-      ])
-    end
-
-    it "resolves relative paths when the files exist" do
-      configuration = Bugsnag::Configuration.new
-      configuration.send_code = false
-
-      dir = File.dirname(__FILE__)
-
-      backtrace = [
-        "./spec/spec_helper.rb:1:in `something'",
-        "./lib/bugsnag/breadcrumbs/../configuration.rb:100:in `to_s'",
-        "lib/bugsnag.rb:20:in `notify'",
-        "#{dir}/../spec/stacktrace_spec.rb:5:in `something_else'",
-      ]
-
-      stacktrace = Bugsnag::Stacktrace.process(backtrace, configuration)
-
-      expect(stacktrace).to eq([
-        { file: "#{dir}/spec_helper.rb", lineNumber: 1, method: "something" },
-        { file: "#{File.dirname(dir)}/lib/bugsnag/configuration.rb", lineNumber: 100, method: "to_s" },
-        { file: "#{File.dirname(dir)}/lib/bugsnag.rb", lineNumber: 20, method: "notify" },
-        { file: "#{dir}/stacktrace_spec.rb", lineNumber: 5, method: "something_else" },
-      ])
-    end
-
-    it "ignores lines in backtrace that it can't parse" do
-      configuration = Bugsnag::Configuration.new
-      configuration.send_code = false
-
-      backtrace = [
-        "/foo/bar/baz.rb:2:in `to_s'",
-        "this is not formatted correctly :O",
-        "/abc/xyz.rb:4:in `to_s'",
-      ]
-
-      stacktrace = Bugsnag::Stacktrace.process(backtrace, configuration)
-
-      expect(stacktrace).to eq([
-        { file: "/foo/bar/baz.rb", lineNumber: 2, method: "to_s" },
-        { file: "/abc/xyz.rb", lineNumber: 4, method: "to_s" },
-      ])
-    end
-
-    it "trims Gem prefix from paths" do
-      gem_path = Gem.path.first
-
-      # Sanity check that we have a gem path to strip
-      expect(gem_path).not_to be_empty
-
-      configuration = Bugsnag::Configuration.new
-      configuration.send_code = false
-
-      backtrace = [
-        "/foo/bar/baz.rb:2:in `to_s'",
-        "#{gem_path}/abc/xyz.rb:4:in `to_s'",
-        "/not/gem/path/but/has/gem.rb:6:in `to_s'"
-      ]
-
-      stacktrace = Bugsnag::Stacktrace.process(backtrace, configuration)
-
-      expect(stacktrace).to eq([
-        { file: "/foo/bar/baz.rb", lineNumber: 2, method: "to_s" },
-        { file: "abc/xyz.rb", lineNumber: 4, method: "to_s" },
-        { file: "/not/gem/path/but/has/gem.rb", lineNumber: 6, method: "to_s" },
-      ])
-    end
-  end
-
-  context "with configurable vendor_path" do
-    let(:configuration) do
-      configuration = Bugsnag::Configuration.new
-      configuration.project_root = "/foo/bar"
-      configuration
-    end
-
-    let(:backtrace) do
-      [
-        "/foo/bar/app/models/user.rb:1:in `something'",
-        "/foo/bar/other_vendor/lib/dont.rb:1:in `to_s'",
-        "/foo/bar/vendor/lib/ignore_me.rb:1:in `to_s'",
-        "/foo/bar/.bundle/lib/ignore_me.rb:1:in `to_s'",
-      ]
-    end
-
-    def out_project_trace(stacktrace)
-      stacktrace.map do |trace_line|
-        trace_line[:file] unless trace_line[:inProject]
-      end.compact
-    end
-
-    it "marks vendor/ and .bundle/ as out-project by default" do
-      stacktrace = Bugsnag::Stacktrace.process(backtrace, configuration)
-
-      expect(out_project_trace(stacktrace)).to eq([
-        "vendor/lib/ignore_me.rb",
-        ".bundle/lib/ignore_me.rb",
-      ])
-    end
-
-    it "allows vendor_path to be configured and filters out backtrace file paths" do
-      configuration.vendor_path = /other_vendor\//
-      stacktrace = Bugsnag::Stacktrace.process(backtrace, configuration)
-
-      expect(out_project_trace(stacktrace)).to eq(["other_vendor/lib/dont.rb"])
-    end
-  end
-end