airbrake 3.0.rc1

data/test/helper.rb

@@ -0,0 +1,251 @@
+require 'test/unit'
+require 'rubygems'
+
+$LOAD_PATH << File.expand_path(File.join(File.dirname(__FILE__), "..", "lib"))
+
+require 'thread'
+
+require "bundler/setup"
+
+require 'shoulda'
+require 'mocha'
+
+require 'action_controller'
+require 'action_controller/test_process'
+require 'active_record'
+require 'active_support'
+require 'nokogiri'
+require 'rack'
+require 'bourne'
+require 'sham_rack'
+
+require "airbrake"
+
+begin require 'redgreen'; rescue LoadError; end
+
+module TestMethods
+  def rescue_action e
+    raise e
+  end
+
+  def do_raise
+    raise "Airbrake"
+  end
+
+  def do_not_raise
+    render :text => "Success"
+  end
+
+  def do_raise_ignored
+    raise ActiveRecord::RecordNotFound.new("404")
+  end
+
+  def do_raise_not_ignored
+    raise ActiveRecord::StatementInvalid.new("Statement invalid")
+  end
+
+  def manual_notify
+    notify_airbrake(Exception.new)
+    render :text => "Success"
+  end
+
+  def manual_notify_ignored
+    notify_airbrake(ActiveRecord::RecordNotFound.new("404"))
+    render :text => "Success"
+  end
+end
+
+class AirbrakeController < ActionController::Base
+  include TestMethods
+end
+
+class Test::Unit::TestCase
+  def request(action = nil, method = :get, user_agent = nil, params = {})
+    @request = ActionController::TestRequest.new
+    @request.action = action ? action.to_s : ""
+
+    if user_agent
+      if @request.respond_to?(:user_agent=)
+        @request.user_agent = user_agent
+      else
+        @request.env["HTTP_USER_AGENT"] = user_agent
+      end
+    end
+    @request.query_parameters = @request.query_parameters.merge(params)
+    @response = ActionController::TestResponse.new
+    @controller.process(@request, @response)
+  end
+
+  # Borrowed from ActiveSupport 2.3.2
+  def assert_difference(expression, difference = 1, message = nil, &block)
+    b = block.send(:binding)
+    exps = Array.wrap(expression)
+    before = exps.map { |e| eval(e, b) }
+
+    yield
+
+    exps.each_with_index do |e, i|
+      error = "#{e.inspect} didn't change by #{difference}"
+      error = "#{message}.\n#{error}" if message
+      assert_equal(before[i] + difference, eval(e, b), error)
+    end
+  end
+
+  def assert_no_difference(expression, message = nil, &block)
+    assert_difference expression, 0, message, &block
+  end
+
+  def stub_sender
+    stub('sender', :send_to_airbrake => nil)
+  end
+
+  def stub_sender!
+    Airbrake.sender = stub_sender
+  end
+
+  def stub_notice
+    stub('notice', :to_xml => 'some yaml', :ignore? => false)
+  end
+
+  def stub_notice!
+     stub_notice.tap do |notice|
+      Airbrake::Notice.stubs(:new => notice)
+    end
+  end
+
+  def create_dummy
+    Airbrake::DummySender.new
+  end
+
+  def reset_config
+    Airbrake.configuration = nil
+    Airbrake.configure do |config|
+      config.api_key = 'abc123'
+    end
+  end
+
+  def clear_backtrace_filters
+    Airbrake.configuration.backtrace_filters.clear
+  end
+
+  def build_exception(opts = {})
+    backtrace = ["airbrake/test/helper.rb:132:in `build_exception'",
+                 "airbrake/test/backtrace.rb:4:in `build_notice_data'",
+                 "/var/lib/gems/1.8/gems/airbrake-2.4.5/rails/init.rb:2:in `send_exception'"]
+    opts = {:backtrace => backtrace}.merge(opts)
+    BacktracedException.new(opts)
+  end
+
+  class BacktracedException < Exception
+    attr_accessor :backtrace
+    def initialize(opts)
+      @backtrace = opts[:backtrace]
+    end
+    def set_backtrace(bt)
+      @backtrace = bt
+    end
+  end
+
+  def build_notice_data(exception = nil)
+    exception ||= build_exception
+    {
+      :api_key       => 'abc123',
+      :error_class   => exception.class.name,
+      :error_message => "#{exception.class.name}: #{exception.message}",
+      :backtrace     => exception.backtrace,
+      :environment   => { 'PATH' => '/bin', 'REQUEST_URI' => '/users/1' },
+      :request       => {
+        :params     => { 'controller' => 'users', 'action' => 'show', 'id' => '1' },
+        :rails_root => '/path/to/application',
+        :url        => "http://test.host/users/1"
+      },
+      :session       => {
+        :key  => '123abc',
+        :data => { 'user_id' => '5', 'flash' => { 'notice' => 'Logged in successfully' } }
+      }
+    }
+  end
+
+  def assert_caught_and_sent
+    assert !Airbrake.sender.collected.empty?
+  end
+
+  def assert_caught_and_not_sent
+    assert Airbrake.sender.collected.empty?
+  end
+
+  def assert_array_starts_with(expected, actual)
+    assert_respond_to actual, :to_ary
+    array = actual.to_ary.reverse
+    expected.reverse.each_with_index do |value, i|
+      assert_equal value, array[i]
+    end
+  end
+
+  def assert_valid_node(document, xpath, content)
+    nodes = document.xpath(xpath)
+    assert nodes.any?{|node| node.content == content },
+           "Expected xpath #{xpath} to have content #{content}, " +
+           "but found #{nodes.map { |n| n.content }} in #{nodes.size} matching nodes." +
+           "Document:\n#{document.to_s}"
+  end
+end
+
+module DefinesConstants
+  def setup
+    @defined_constants = []
+  end
+
+  def teardown
+    @defined_constants.each do |constant|
+      Object.__send__(:remove_const, constant)
+    end
+  end
+
+  def define_constant(name, value)
+    Object.const_set(name, value)
+    @defined_constants << name
+  end
+end
+
+# Also stolen from AS 2.3.2
+class Array
+  # Wraps the object in an Array unless it's an Array.  Converts the
+  # object to an Array using #to_ary if it implements that.
+  def self.wrap(object)
+    case object
+    when nil
+      []
+    when self
+      object
+    else
+      if object.respond_to?(:to_ary)
+        object.to_ary
+      else
+        [object]
+      end
+    end
+  end
+
+end
+
+class CollectingSender
+  attr_reader :collected
+
+  def initialize
+    @collected = []
+  end
+
+  def send_to_airbrake(data)
+    @collected << data
+  end
+end
+
+class FakeLogger
+  def info(*args);  end
+  def debug(*args); end
+  def warn(*args);  end
+  def error(*args); end
+  def fatal(*args); end
+end
+

data/test/javascript_notifier_test.rb

@@ -0,0 +1,52 @@
+require File.dirname(__FILE__) + '/helper'
+require 'airbrake/rails/javascript_notifier'
+require 'ostruct'
+
+class JavascriptNotifierTest < Test::Unit::TestCase
+  module FakeRenderer
+    def javascript_tag(text)
+      "<script>#{text}</script>"
+    end
+    def escape_javascript(text)
+      "ESC#{text}ESC"
+    end
+  end
+
+  class FakeController
+    def self.helper_method(*args)
+    end
+
+    include Airbrake::Rails::JavascriptNotifier
+
+    def action_name
+      "action"
+    end
+
+    def controller_name
+      "controller"
+    end
+
+    def request
+      @request ||= OpenStruct.new
+    end
+
+    def render_to_string(options)
+      context = OpenStruct.new(options[:locals])
+      context.extend(FakeRenderer)
+      context.instance_eval do
+        erb = ERB.new(IO.read(options[:file]))
+        erb.result(binding)
+      end
+    end
+  end
+
+  should "make sure escape_javacript is called on the request.url" do
+    Airbrake.configure do
+    end
+    controller = FakeController.new
+    controller.request.url = "bad_javascript"
+    assert controller.send(:airbrake_javascript_notifier)['"ESCbad_javascriptESC"']
+    assert ! controller.send(:airbrake_javascript_notifier)['"bad_javascript"']
+  end
+end
+

data/test/logger_test.rb

@@ -0,0 +1,85 @@
+require File.dirname(__FILE__) + '/helper'
+
+class LoggerTest < Test::Unit::TestCase
+  def stub_http(response, body = nil)
+    response.stubs(:body => body) if body
+    @http = stub(:post => response,
+                 :read_timeout= => nil,
+                 :open_timeout= => nil,
+                 :use_ssl= => nil)
+    Net::HTTP.stubs(:new).returns(@http)
+  end
+
+  def send_notice
+    Airbrake.sender.send_to_airbrake('data')
+  end
+
+  def stub_verbose_log
+    Airbrake.stubs(:write_verbose_log)
+  end
+
+  def assert_logged(expected)
+    assert_received(Airbrake, :write_verbose_log) do |expect|
+      expect.with {|actual| actual =~ expected }
+    end
+  end
+
+  def assert_not_logged(expected)
+    assert_received(Airbrake, :write_verbose_log) do |expect|
+      expect.with {|actual| actual =~ expected }.never
+    end
+  end
+
+  def configure
+    Airbrake.configure { |config| }
+  end
+
+  should "report that notifier is ready when configured" do
+    stub_verbose_log
+    configure
+    assert_logged /Notifier (.*) ready/
+  end
+
+  should "not report that notifier is ready when internally configured" do
+    stub_verbose_log
+    Airbrake.configure(true) { |config| }
+    assert_not_logged /.*/
+  end
+
+  should "print environment info a successful notification without a body" do
+    reset_config
+    stub_verbose_log
+    stub_http(Net::HTTPSuccess)
+    send_notice
+    assert_logged /Environment Info:/
+    assert_not_logged /Response from Airbrake:/
+  end
+
+  should "print environment info on a failed notification without a body" do
+    reset_config
+    stub_verbose_log
+    stub_http(Net::HTTPError)
+    send_notice
+    assert_logged /Environment Info:/
+    assert_not_logged /Response from Airbrake:/
+  end
+
+  should "print environment info and response on a success with a body" do
+    reset_config
+    stub_verbose_log
+    stub_http(Net::HTTPSuccess, 'test')
+    send_notice
+    assert_logged /Environment Info:/
+    assert_logged /Response from Airbrake:/
+  end
+
+  should "print environment info and response on a failure with a body" do
+    reset_config
+    stub_verbose_log
+    stub_http(Net::HTTPError, 'test')
+    send_notice
+    assert_logged /Environment Info:/
+    assert_logged /Response from Airbrake:/
+  end
+
+end

data/test/notice_test.rb

@@ -0,0 +1,459 @@
+require File.dirname(__FILE__) + '/helper'
+
+class NoticeTest < Test::Unit::TestCase
+
+  include DefinesConstants
+
+  def configure
+    Airbrake::Configuration.new.tap do |config|
+      config.api_key = 'abc123def456'
+    end
+  end
+
+  def build_notice(args = {})
+    configuration = args.delete(:configuration) || configure
+    Airbrake::Notice.new(configuration.merge(args))
+  end
+
+  def stub_request(attrs = {})
+    stub('request', { :parameters  => { 'one' => 'two' },
+                      :protocol    => 'http',
+                      :host        => 'some.host',
+                      :request_uri => '/some/uri',
+                      :session     => { :to_hash => { 'a' => 'b' } },
+                      :env         => { 'three' => 'four' } }.update(attrs))
+  end
+
+  should "set the api key" do
+    api_key = 'key'
+    notice = build_notice(:api_key => api_key)
+    assert_equal api_key, notice.api_key
+  end
+
+  should "accept a project root" do
+    project_root = '/path/to/project'
+    notice = build_notice(:project_root => project_root)
+    assert_equal project_root, notice.project_root
+  end
+
+  should "accept a component" do
+    assert_equal 'users_controller', build_notice(:component => 'users_controller').controller
+  end
+
+  should "alias the component as controller" do
+    assert_equal 'users_controller', build_notice(:controller => 'users_controller').component
+    assert_equal 'users_controller', build_notice(:component => 'users_controller').controller
+  end
+
+  should "accept a action" do
+    assert_equal 'index', build_notice(:action => 'index').action
+  end
+
+  should "accept a url" do
+    url = 'http://some.host/uri'
+    notice = build_notice(:url => url)
+    assert_equal url, notice.url
+  end
+
+  should "set the host name" do
+    notice = build_notice
+    assert_equal hostname, notice.hostname
+  end
+
+  should "accept a backtrace from an exception or hash" do
+    array = ["user.rb:34:in `crazy'"]
+    exception = build_exception
+    exception.set_backtrace array
+    backtrace = Airbrake::Backtrace.parse(array)
+    notice_from_exception = build_notice(:exception => exception)
+
+
+    assert_equal backtrace,
+                 notice_from_exception.backtrace,
+                 "backtrace was not correctly set from an exception"
+
+    notice_from_hash = build_notice(:backtrace => array)
+    assert_equal backtrace,
+                 notice_from_hash.backtrace,
+                 "backtrace was not correctly set from a hash"
+  end
+
+  should "pass its backtrace filters for parsing" do
+    backtrace_array = ['my/file/backtrace:3']
+    exception = build_exception
+    exception.set_backtrace(backtrace_array)
+    Airbrake::Backtrace.expects(:parse).with(backtrace_array, {:filters => 'foo'})
+
+    notice = Airbrake::Notice.new({:exception => exception, :backtrace_filters => 'foo'})
+  end
+
+  should "set the error class from an exception or hash" do
+    assert_accepts_exception_attribute :error_class do |exception|
+      exception.class.name
+    end
+  end
+
+  should "set the error message from an exception or hash" do
+    assert_accepts_exception_attribute :error_message do |exception|
+      "#{exception.class.name}: #{exception.message}"
+    end
+  end
+
+  should "accept parameters from a request or hash" do
+    parameters = { 'one' => 'two' }
+    notice_from_hash = build_notice(:parameters => parameters)
+    assert_equal notice_from_hash.parameters, parameters
+  end
+
+  should "accept session data from a session[:data] hash" do
+    data = { 'one' => 'two' }
+    notice = build_notice(:session => { :data => data })
+    assert_equal data, notice.session_data
+  end
+
+  should "accept session data from a session_data hash" do
+    data = { 'one' => 'two' }
+    notice = build_notice(:session_data => data)
+    assert_equal data, notice.session_data
+  end
+
+  should "accept an environment name" do
+    assert_equal 'development', build_notice(:environment_name => 'development').environment_name
+  end
+
+  should "accept CGI data from a hash" do
+    data = { 'string' => 'value' }
+    notice = build_notice(:cgi_data => data)
+    assert_equal data, notice.cgi_data, "should take CGI data from a hash"
+  end
+
+  should "accept notifier information" do
+    params = { :notifier_name    => 'a name for a notifier',
+               :notifier_version => '1.0.5',
+               :notifier_url     => 'http://notifiers.r.us/download' }
+    notice = build_notice(params)
+    assert_equal params[:notifier_name], notice.notifier_name
+    assert_equal params[:notifier_version], notice.notifier_version
+    assert_equal params[:notifier_url], notice.notifier_url
+  end
+
+  should "set sensible defaults without an exception" do
+    backtrace = Airbrake::Backtrace.parse(build_backtrace_array)
+    notice = build_notice(:backtrace => build_backtrace_array)
+
+    assert_equal 'Notification', notice.error_message
+    assert_array_starts_with backtrace.lines, notice.backtrace.lines
+    assert_equal({}, notice.parameters)
+    assert_equal({}, notice.session_data)
+  end
+
+  should "use the caller as the backtrace for an exception without a backtrace" do
+    filters = Airbrake::Configuration.new.backtrace_filters
+    backtrace = Airbrake::Backtrace.parse(caller, :filters => filters)
+    notice = build_notice(:exception => StandardError.new('error'), :backtrace => nil)
+
+    assert_array_starts_with backtrace.lines, notice.backtrace.lines
+  end
+
+  should "convert unserializable objects to strings" do
+    assert_serializes_hash(:parameters)
+    assert_serializes_hash(:cgi_data)
+    assert_serializes_hash(:session_data)
+  end
+
+  should "filter parameters" do
+    assert_filters_hash(:parameters)
+  end
+
+  should "filter cgi data" do
+    assert_filters_hash(:cgi_data)
+  end
+
+  should "filter session" do
+    assert_filters_hash(:session_data)
+  end
+
+  should "remove rack.request.form_vars" do
+    original = {
+      "rack.request.form_vars" => "story%5Btitle%5D=The+TODO+label",
+      "abc" => "123"
+    }
+
+    notice = build_notice(:cgi_data => original)
+    assert_equal({"abc" => "123"}, notice.cgi_data)
+  end
+
+  context "a Notice turned into XML" do
+    setup do
+      Airbrake.configure do |config|
+        config.api_key = "1234567890"
+      end
+
+      @exception = build_exception
+
+      @notice = build_notice({
+        :notifier_name    => 'a name',
+        :notifier_version => '1.2.3',
+        :notifier_url     => 'http://some.url/path',
+        :exception        => @exception,
+        :controller       => "controller",
+        :action           => "action",
+        :url              => "http://url.com",
+        :parameters       => { "paramskey"     => "paramsvalue",
+                               "nestparentkey" => { "nestkey" => "nestvalue" } },
+        :session_data     => { "sessionkey" => "sessionvalue" },
+        :cgi_data         => { "cgikey" => "cgivalue" },
+        :project_root     => "RAILS_ROOT",
+        :environment_name => "RAILS_ENV"
+      })
+
+      @xml = @notice.to_xml
+
+      @document = Nokogiri::XML::Document.parse(@xml)
+    end
+
+    should "validate against the XML schema" do
+      assert_valid_notice_document @document
+    end
+
+    should "serialize a Notice to XML when sent #to_xml" do
+      assert_valid_node(@document, "//api-key", @notice.api_key)
+
+      assert_valid_node(@document, "//notifier/name",    @notice.notifier_name)
+      assert_valid_node(@document, "//notifier/version", @notice.notifier_version)
+      assert_valid_node(@document, "//notifier/url",     @notice.notifier_url)
+
+      assert_valid_node(@document, "//error/class",   @notice.error_class)
+      assert_valid_node(@document, "//error/message", @notice.error_message)
+
+      assert_valid_node(@document, "//error/backtrace/line/@number", @notice.backtrace.lines.first.number)
+      assert_valid_node(@document, "//error/backtrace/line/@file", @notice.backtrace.lines.first.file)
+      assert_valid_node(@document, "//error/backtrace/line/@method", @notice.backtrace.lines.first.method)
+
+      assert_valid_node(@document, "//request/url",        @notice.url)
+      assert_valid_node(@document, "//request/component", @notice.controller)
+      assert_valid_node(@document, "//request/action",     @notice.action)
+
+      assert_valid_node(@document, "//request/params/var/@key",     "paramskey")
+      assert_valid_node(@document, "//request/params/var",          "paramsvalue")
+      assert_valid_node(@document, "//request/params/var/@key",     "nestparentkey")
+      assert_valid_node(@document, "//request/params/var/var/@key", "nestkey")
+      assert_valid_node(@document, "//request/params/var/var",      "nestvalue")
+      assert_valid_node(@document, "//request/session/var/@key",    "sessionkey")
+      assert_valid_node(@document, "//request/session/var",         "sessionvalue")
+      assert_valid_node(@document, "//request/cgi-data/var/@key",   "cgikey")
+      assert_valid_node(@document, "//request/cgi-data/var",        "cgivalue")
+
+      assert_valid_node(@document, "//server-environment/project-root",     "RAILS_ROOT")
+      assert_valid_node(@document, "//server-environment/environment-name", "RAILS_ENV")
+      assert_valid_node(@document, "//server-environment/hostname", hostname)
+    end
+  end
+
+  should "not send empty request data" do
+    notice = build_notice
+    assert_nil notice.url
+    assert_nil notice.controller
+    assert_nil notice.action
+
+    xml = notice.to_xml
+    document = Nokogiri::XML.parse(xml)
+    assert_nil document.at('//request/url')
+    assert_nil document.at('//request/component')
+    assert_nil document.at('//request/action')
+
+    assert_valid_notice_document document
+  end
+
+  %w(url controller action).each do |var|
+    should "send a request if #{var} is present" do
+      notice = build_notice(var.to_sym => 'value')
+      xml = notice.to_xml
+      document = Nokogiri::XML.parse(xml)
+      assert_not_nil document.at('//request')
+    end
+  end
+
+  %w(parameters cgi_data session_data).each do |var|
+    should "send a request if #{var} is present" do
+      notice = build_notice(var.to_sym => { 'key' => 'value' })
+      xml = notice.to_xml
+      document = Nokogiri::XML.parse(xml)
+      assert_not_nil document.at('//request')
+    end
+  end
+
+  should "not ignore an exception not matching ignore filters" do
+    notice = build_notice(:error_class       => 'ArgumentError',
+                          :ignore            => ['Argument'],
+                          :ignore_by_filters => [lambda { |notice| false }])
+    assert !notice.ignore?
+  end
+
+  should "ignore an exception with a matching error class" do
+    notice = build_notice(:error_class => 'ArgumentError',
+                          :ignore      => [ArgumentError])
+    assert notice.ignore?
+  end
+
+  should "ignore an exception with a matching error class name" do
+    notice = build_notice(:error_class => 'ArgumentError',
+                          :ignore      => ['ArgumentError'])
+    assert notice.ignore?
+  end
+
+  should "ignore an exception with a matching filter" do
+    filter = lambda {|notice| notice.error_class == 'ArgumentError' }
+    notice = build_notice(:error_class       => 'ArgumentError',
+                          :ignore_by_filters => [filter])
+    assert notice.ignore?
+  end
+
+  should "not raise without an ignore list" do
+    notice = build_notice(:ignore => nil, :ignore_by_filters => nil)
+    assert_nothing_raised do
+      notice.ignore?
+    end
+  end
+
+  ignored_error_classes = %w(
+    ActiveRecord::RecordNotFound
+    AbstractController::ActionNotFound
+    ActionController::RoutingError
+    ActionController::InvalidAuthenticityToken
+    CGI::Session::CookieStore::TamperedWithCookie
+    ActionController::UnknownAction
+  )
+
+  ignored_error_classes.each do |ignored_error_class|
+    should "ignore #{ignored_error_class} error by default" do
+      notice = build_notice(:error_class => ignored_error_class)
+      assert notice.ignore?
+    end
+  end
+
+  should "act like a hash" do
+    notice = build_notice(:error_message => 'some message')
+    assert_equal notice.error_message, notice[:error_message]
+  end
+
+  should "return params on notice[:request][:params]" do
+    params = { 'one' => 'two' }
+    notice = build_notice(:parameters => params)
+    assert_equal params, notice[:request][:params]
+  end
+
+  should "ensure #to_hash is called on objects that support it" do
+    assert_nothing_raised do
+      build_notice(:session => { :object => stub(:to_hash => {}) })
+    end
+  end
+
+  should "extract data from a rack environment hash" do
+    url = "https://subdomain.happylane.com:100/test/file.rb?var=value&var2=value2"
+    parameters = { 'var' => 'value', 'var2' => 'value2' }
+    env = Rack::MockRequest.env_for(url)
+
+    notice = build_notice(:rack_env => env)
+
+    assert_equal url, notice.url
+    assert_equal parameters, notice.parameters
+    assert_equal 'GET', notice.cgi_data['REQUEST_METHOD']
+  end
+
+  should "extract data from a rack environment hash with action_dispatch info" do
+    params = { 'controller' => 'users', 'action' => 'index', 'id' => '7' }
+    env = Rack::MockRequest.env_for('/', { 'action_dispatch.request.parameters' => params })
+
+    notice = build_notice(:rack_env => env)
+
+    assert_equal params, notice.parameters
+    assert_equal params['controller'], notice.component
+    assert_equal params['action'], notice.action
+  end
+
+  should "extract session data from a rack environment" do
+    session_data = { 'something' => 'some value' }
+    env = Rack::MockRequest.env_for('/', 'rack.session' => session_data)
+
+    notice = build_notice(:rack_env => env)
+
+    assert_equal session_data, notice.session_data
+  end
+
+  should "prefer passed session data to rack session data" do
+    session_data = { 'something' => 'some value' }
+    env = Rack::MockRequest.env_for('/')
+
+    notice = build_notice(:rack_env => env, :session_data => session_data)
+
+    assert_equal session_data, notice.session_data
+  end
+
+  def assert_accepts_exception_attribute(attribute, args = {}, &block)
+    exception = build_exception
+    block ||= lambda { exception.send(attribute) }
+    value = block.call(exception)
+
+    notice_from_exception = build_notice(args.merge(:exception => exception))
+
+    assert_equal notice_from_exception.send(attribute),
+                 value,
+                 "#{attribute} was not correctly set from an exception"
+
+    notice_from_hash = build_notice(args.merge(attribute => value))
+    assert_equal notice_from_hash.send(attribute),
+                 value,
+                 "#{attribute} was not correctly set from a hash"
+  end
+
+  def assert_serializes_hash(attribute)
+    [File.open(__FILE__), Proc.new { puts "boo!" }, Module.new].each do |object|
+      hash = {
+        :strange_object => object,
+        :sub_hash => {
+          :sub_object => object
+        },
+        :array => [object]
+      }
+      notice = build_notice(attribute => hash)
+      hash = notice.send(attribute)
+      assert_equal object.to_s, hash[:strange_object], "objects should be serialized"
+      assert_kind_of Hash, hash[:sub_hash], "subhashes should be kept"
+      assert_equal object.to_s, hash[:sub_hash][:sub_object], "subhash members should be serialized"
+      assert_kind_of Array, hash[:array], "arrays should be kept"
+      assert_equal object.to_s, hash[:array].first, "array members should be serialized"
+    end
+  end
+
+  def assert_valid_notice_document(document)
+    xsd_path = File.join(File.dirname(__FILE__), "airbrake_2_2.xsd")
+    schema = Nokogiri::XML::Schema.new(IO.read(xsd_path))
+    errors = schema.validate(document)
+    assert errors.empty?, errors.collect{|e| e.message }.join
+  end
+
+  def assert_filters_hash(attribute)
+    filters  = ["abc", :def]
+    original = { 'abc' => "123", 'def' => "456", 'ghi' => "789", 'nested' => { 'abc' => '100' } }
+    filtered = { 'abc'    => "[FILTERED]",
+                 'def'    => "[FILTERED]",
+                 'ghi'    => "789",
+                 'nested' => { 'abc' => '[FILTERED]' } }
+
+    notice = build_notice(:params_filters => filters, attribute => original)
+
+    assert_equal(filtered,
+                 notice.send(attribute))
+  end
+
+  def build_backtrace_array
+    ["app/models/user.rb:13:in `magic'",
+      "app/controllers/users_controller.rb:8:in `index'"]
+  end
+
+  def hostname
+    `hostname`.chomp
+  end
+
+end