airbrake_tools 0.0.4 → 0.0.5

data/Gemfile.lock

@@ -1,7 +1,7 @@
 PATH
   remote: .
   specs:
-    airbrake_tools (0.0.4)
+    airbrake_tools (0.0.5)
 
 GEM
   remote: http://rubygems.org/

data/airbrake_tools.gemspec

@@ -9,4 +9,5 @@ Gem::Specification.new name, AirbrakeTools::VERSION do |s|
   s.homepage = "http://github.com/jcheatham/#{name}"
   s.files = `git ls-files`.split("\n")
   s.license = "MIT"
+  s.executables = ["airbrake-tools"]
 end

data/lib/airbrake_tools/version.rb

@@ -1,3 +1,3 @@
 module AirbrakeTools
-  VERSION = "0.0.4"
+  VERSION = "0.0.5"
 end

data/lib/airbrake_tools.rb

@@ -4,6 +4,7 @@ require "airbrake-api"
 
 module AirbrakeTools
   DEFAULT_HOT_PAGES = 1
+  DEFAULT_NEW_PAGES = 1
   DEFAULT_SUMMARY_PAGES = 5
   DEFAULT_COMPARE_DEPTH = 7
   DEFAULT_ENVIRONMENT = "production"
@@ -23,37 +24,31 @@ module AirbrakeTools
 
       case ARGV[2]
       when "hot"
-        errors = hot(options)
-        print_errors(errors)
+        print_errors(hot(options))
       when "list"
         list(options)
       when "summary"
         summary(ARGV[3] || raise("Need error id"), options)
+      when "new"
+        print_errors(new(options))
       else
-        raise "Unknown command try hot/list/summary"
+        raise "Unknown command #{ARGV[2].inspect} try hot/new/list/summary"
       end
       return 0
     end
 
     def hot(options = {})
-      pages = (options[:pages] || DEFAULT_HOT_PAGES).to_i
-      errors = []
-      pages.times do |i|
-        errors.concat(AirbrakeAPI.errors(:page => i+1) || [])
-      end
-      select_env!(errors, options)
+      errors = errors_with_notices({:pages => DEFAULT_HOT_PAGES}.merge(options))
+      errors.sort_by{|e,n,f| f }.reverse
+    end
 
-      errors = Parallel.map(errors, :in_threads => 10) do |error|
-        begin
-          notices = AirbrakeAPI.notices(error.id, :pages => 1, :raw => true).compact
-          print "."
-          [error, notices, frequency(notices)]
-        rescue Faraday::Error::ParsingError
-          $stderr.puts "Ignoring #{hot_summary(error)}, got 500 from http://#{AirbrakeAPI.account}.airbrake.io/errors/#{error.id}"
-        end
-      end.compact
+    def new(options = {})
+      errors = errors_with_notices({:pages => DEFAULT_NEW_PAGES}.merge(options))
+      errors.sort_by{|e,n,f| e.created_at }.reverse
+    end
 
-      errors.sort_by{|e,n,f| f }.reverse
+    def errors_with_notices(options)
+      add_notices_to_pages(errors_from_pages(options))
     end
 
     def list(options)
@@ -77,7 +72,11 @@ module AirbrakeTools
       puts "last day:      #{sparkline(notices, :slots => 24, :interval => 60 * 60)}"
 
       backtraces = notices.compact.select{|n| n.backtrace }.group_by do |notice|
-        notice.backtrace.first[1][0..compare_depth]
+        if notice.backtrace.is_a?(String) # no backtrace recorded...
+          []
+        else
+          notice.backtrace.first[1][0..compare_depth]
+        end
       end
 
       backtraces.sort_by{|_,notices| notices.size }.reverse.each_with_index do |(backtrace, notices), index|
@@ -90,6 +89,27 @@ module AirbrakeTools
 
     private
 
+    def add_notices_to_pages(errors)
+      Parallel.map(errors, :in_threads => 10) do |error|
+        begin
+          notices = AirbrakeAPI.notices(error.id, :pages => 1, :raw => true).compact
+          print "."
+          [error, notices, frequency(notices)]
+        rescue Faraday::Error::ParsingError
+          $stderr.puts "Ignoring #{hot_summary(error)}, got 500 from http://#{AirbrakeAPI.account}.airbrake.io/errors/#{error.id}"
+        end
+      end.compact
+    end
+
+    def errors_from_pages(options)
+      errors = []
+      options[:pages].times do |i|
+        errors.concat(AirbrakeAPI.errors(:page => i+1) || [])
+      end
+      select_env!(errors, options)
+      errors
+    end
+
     def select_env!(errors, options)
       errors.select!{|e| e.rails_env == options[:env] || DEFAULT_ENVIRONMENT }
     end
@@ -100,13 +120,12 @@ module AirbrakeTools
       end
     end
 
+    # we only have a limited sample size, so we do not know how many errors occurred in total
     def frequency(notices)
-      hour = 60 * 60
-      sum_of_ages = notices.map { |n| Time.now - n.created_at }.inject(&:+)
-      average_age = sum_of_ages / notices.size
-      time_to_error = average_age / notices.size
-      rate = 1 / time_to_error
-      (rate * hour).round(1)
+      return 0 if notices.empty?
+      range = Time.now.to_f - notices.map{ |n| n.created_at.to_f }.min
+      errors_per_second = notices.size / range
+      (errors_per_second * 60 * 60).round(2) # errors_per_hour
     end
 
     def hot_summary(error)
@@ -129,8 +148,8 @@ module AirbrakeTools
 
             Options:
         BANNER
-        opts.on("-c NUM", "--compare-depth NUM", Integer, "How deep to compare backtraces in summary (default: #{DEFAULT_COMPARE_DEPTH})") {|s| options[:env] = s }
-        opts.on("-p NUM", "--pages NUM", Integer, "How maybe pages to iterate over (default: hot:#{DEFAULT_HOT_PAGES} summary:#{DEFAULT_SUMMARY_PAGES})") {|s| options[:env] = s }
+        opts.on("-c NUM", "--compare-depth NUM", Integer, "How deep to compare backtraces in summary (default: #{DEFAULT_COMPARE_DEPTH})") {|s| options[:compare_depth] = s }
+        opts.on("-p NUM", "--pages NUM", Integer, "How maybe pages to iterate over (default: hot:#{DEFAULT_HOT_PAGES} new:#{DEFAULT_NEW_PAGES} summary:#{DEFAULT_SUMMARY_PAGES})") {|s| options[:pages] = s }
         opts.on("-e ENV", "--environment ENV", String, "Only show errors from this environment (default: #{DEFAULT_ENVIRONMENT})") {|s| options[:env] = s }
         opts.on("-h", "--help", "Show this.") { puts opts; exit }
         opts.on("-v", "--version", "Show Version"){ puts "airbrake-tools #{VERSION}"; exit }

data/spec/airbrake_tools_spec.rb

@@ -1,4 +1,5 @@
 require 'yaml'
+require 'airbrake_tools'
 
 ROOT = File.expand_path('../../', __FILE__)
 
@@ -64,5 +65,45 @@ describe "airbrake-tools" do
       output.should include("last 2 hours: ")
     end
   end
+
+  describe "newest" do
+    it "kinda works" do
+      output = airbrake_tools("#{config["subdomain"]} #{config["auth_token"]} new")
+      output.should =~ /#\d+\s+\d+\.\d+\/hour\s+total:\d+/
+    end
+  end
+
+  describe ".extract_options" do
+    it "finds nothing" do
+      AirbrakeTools.send(:extract_options, []).should == {}
+    end
+
+    it "finds pages" do
+      AirbrakeTools.send(:extract_options, ["--pages", "1"]).should == {:pages => 1}
+    end
+
+    it "finds env" do
+      AirbrakeTools.send(:extract_options, ["--environment", "xx"]).should == {:env => "xx"}
+    end
+
+    it "finds compare-depth" do
+      AirbrakeTools.send(:extract_options, ["--compare-depth", "1"]).should == {:compare_depth => 1}
+    end
+  end
+
+  describe ".frequency" do
+    it "calculates for 0" do
+      AirbrakeTools.send(:frequency, []).should == 0
+    end
+
+    it "calculates for 1" do
+      AirbrakeTools.send(:frequency, [stub(:created_at => Time.now - (60*60))]).should == 1
+    end
+
+    it "calculates for n" do
+      # 3 per minute => 180/hour
+      AirbrakeTools.send(:frequency, [stub(:created_at => Time.now-60), stub(:created_at => Time.now-40), stub(:created_at => Time.now-20)]).should == 180
+    end
+  end
 end
 

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: airbrake_tools
 version: !ruby/object:Gem::Version
-  version: 0.0.4
+  version: 0.0.5
   prerelease: 
 platform: ruby
 authors:
@@ -9,11 +9,12 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2012-12-20 00:00:00.000000000 Z
+date: 2012-12-27 00:00:00.000000000 Z
 dependencies: []
 description: 
 email: coaxis@gmail.com
-executables: []
+executables:
+- airbrake-tools
 extensions: []
 extra_rdoc_files: []
 files: