hockeybrake 0.0.5 → 0.0.6

checksums.yaml

@@ -0,0 +1,7 @@
+---
+SHA1:
+  metadata.gz: b4d5daa3c3f6ae1971e40f2b91ae4aa0303de410
+  data.tar.gz: 102820519f0775ea5a1fa34ed2eb45988e869e57
+SHA512:
+  metadata.gz: 1231e4ab3a10bbc75051b0c13daf8c132c660cd11f12c5017839b108bdaef9771f5abfe6252b2ea0c27547e1ab3057006596446162bfbd4f183ca65bf3b597e2
+  data.tar.gz: b7cc2795b5fd985076f4d20283dd49fd5fa9170034a52a27fb3c239ceb6185b13547df137ce44ee5268c199f2759c5345124f1cd1407a398435bdb6983f9a35f

data/.gitignore

@@ -10,3 +10,4 @@ ehthumbs.db
 Thumbs.db
 .gem
 .idea/
+Gemfile.lock

data/.travis.yml

@@ -0,0 +1,12 @@
+language: ruby
+rvm:
+  - 1.9.2
+  - 1.9.3
+
+gemfile:
+  - gemfiles/airbrake-3.1.2
+  - gemfiles/airbrake-3.1.3
+  - gemfiles/airbrake-3.1.4
+  - gemfiles/airbrake-3.1.5
+  - gemfiles/airbrake-3.1.6
+  - gemfiles/airbrake-3.1.16

data/Gemfile

@@ -1,5 +1,18 @@
 source "http://rubygems.org"
 
+gem 'rake'
+gem 'rspec'
+gem 'shoulda'
+gem 'rails'
+
+gem 'airbrake'
+
+group :test do
+  gem "guard-rspec"
+  gem "terminal-notifier-guard"
+end
+
 # Specify your gem's dependencies in hockeybrake.gemspec
 gemspec
 
+

data/Guardfile

@@ -0,0 +1,24 @@
+# A sample Guardfile
+# More info at https://github.com/guard/guard#readme
+
+guard 'rspec' do
+  watch(%r{^spec/.+_spec\.rb$})
+  watch(%r{^lib/(.+)\.rb$})     { |m| "spec/lib/#{m[1]}_spec.rb" }
+  watch('spec/spec_helper.rb')  { "spec" }
+
+  # Rails example
+  watch(%r{^app/(.+)\.rb$})                           { |m| "spec/#{m[1]}_spec.rb" }
+  watch(%r{^app/(.*)(\.erb|\.haml)$})                 { |m| "spec/#{m[1]}#{m[2]}_spec.rb" }
+  watch(%r{^app/controllers/(.+)_(controller)\.rb$})  { |m| ["spec/routing/#{m[1]}_routing_spec.rb", "spec/#{m[2]}s/#{m[1]}_#{m[2]}_spec.rb", "spec/acceptance/#{m[1]}_spec.rb"] }
+  watch(%r{^spec/support/(.+)\.rb$})                  { "spec" }
+  watch('config/routes.rb')                           { "spec/routing" }
+  watch('app/controllers/application_controller.rb')  { "spec/controllers" }
+
+  # Capybara request specs
+  watch(%r{^app/views/(.+)/.*\.(erb|haml)$})          { |m| "spec/requests/#{m[1]}_spec.rb" }
+
+  # Turnip features and steps
+  watch(%r{^spec/acceptance/(.+)\.feature$})
+  watch(%r{^spec/acceptance/steps/(.+)_steps\.rb$})   { |m| Dir[File.join("**/#{m[1]}.feature")][0] || 'spec/acceptance' }
+end
+

data/README.md

@@ -2,6 +2,8 @@
 
 An extension for the amazing airbrake gem which routes crash reports to HockeyApp
 
+[![Build Status](https://secure.travis-ci.org/dei79/hockeybrake.png)](http://travis-ci.org/dei79/hockeybrake)
+
 ## Installation 
 
 Add this to your Gemfile
@@ -44,6 +46,7 @@ Resque comes with out of the box support for Airbrake. This gem is compatible wi
  
 * Fork the project
 * Fix the issue
+* Add specs
 * Create pull request on github
 
 ## More Information

data/Rakefile

@@ -0,0 +1,11 @@
+require "rake"
+require "bundler/gem_tasks"
+require "rspec/core/rake_task"
+Dir["lib/tasks/*.rake"].each { |rake| load rake }
+
+RSpec::Core::RakeTask.new(:spec) do |spec|
+  spec.pattern = 'spec/**/*_spec.rb'
+  spec.rspec_opts = ['--backtrace']
+end
+
+task :default => 'spec'

data/gemfiles/airbrake-3.1.16

@@ -0,0 +1,18 @@
+source "http://rubygems.org"
+
+gem 'rake'
+gem 'rspec'
+gem 'shoulda'
+gem 'rails'
+
+gem 'airbrake', '=3.1.16'
+
+group :test do
+  gem "guard-rspec"
+  gem "terminal-notifier-guard"
+end
+
+# Specify your gem's dependencies in hockeybrake.gemspec
+gemspec :path => '../'
+
+

data/gemfiles/airbrake-3.1.2

@@ -0,0 +1,18 @@
+source "http://rubygems.org"
+
+gem 'rake'
+gem 'rspec'
+gem 'shoulda'
+gem 'rails'
+
+gem 'airbrake', '=3.1.2'
+
+group :test do
+  gem "guard-rspec"
+  gem "terminal-notifier-guard"
+end
+
+# Specify your gem's dependencies in hockeybrake.gemspec
+gemspec :path => '../'
+
+

data/gemfiles/airbrake-3.1.3

@@ -0,0 +1,18 @@
+source "http://rubygems.org"
+
+gem 'rake'
+gem 'rspec'
+gem 'shoulda'
+gem 'rails'
+
+gem 'airbrake', '=3.1.3'
+
+group :test do
+  gem "guard-rspec"
+  gem "terminal-notifier-guard"
+end
+
+# Specify your gem's dependencies in hockeybrake.gemspec
+gemspec :path => '../'
+
+

data/gemfiles/airbrake-3.1.4

@@ -0,0 +1,18 @@
+source "http://rubygems.org"
+
+gem 'rake'
+gem 'rspec'
+gem 'shoulda'
+gem 'rails'
+
+gem 'airbrake', '=3.1.4'
+
+group :test do
+  gem "guard-rspec"
+  gem "terminal-notifier-guard"
+end
+
+# Specify your gem's dependencies in hockeybrake.gemspec
+gemspec :path => '../'
+
+

data/gemfiles/airbrake-3.1.5

@@ -0,0 +1,18 @@
+source "http://rubygems.org"
+
+gem 'rake'
+gem 'rspec'
+gem 'shoulda'
+gem 'rails'
+
+gem 'airbrake', '=3.1.5'
+
+group :test do
+  gem "guard-rspec"
+  gem "terminal-notifier-guard"
+end
+
+# Specify your gem's dependencies in hockeybrake.gemspec
+gemspec :path => '../'
+
+

data/gemfiles/airbrake-3.1.6

@@ -0,0 +1,18 @@
+source "http://rubygems.org"
+
+gem 'rake'
+gem 'rspec'
+gem 'shoulda'
+gem 'rails'
+
+gem 'airbrake', '=3.1.6'
+
+group :test do
+  gem "guard-rspec"
+  gem "terminal-notifier-guard"
+end
+
+# Specify your gem's dependencies in hockeybrake.gemspec
+gemspec :path => '../'
+
+

data/hockeybrake.gemspec

@@ -18,4 +18,5 @@ Gem::Specification.new do |s|
 
   s.add_runtime_dependency(%q<airbrake>, [">= 3.1.2"])
   s.add_runtime_dependency(%q<multipart-post>, [">= 1.1.5"])
+  s.add_runtime_dependency(%q<activesupport>)
 end

data/lib/hockeybrake/hockey_log.rb

@@ -1,3 +1,5 @@
+require 'active_support/core_ext'
+
 #
 # This class converts the given Airbrake XML data to the log format of the HockeyApp server
 #
@@ -6,45 +8,55 @@ module HockeyBrake
 
     #
     # Generates a string which can be sent to the hockey service
-    #
-    def self.generate(data)
-      # the output
-      output = ""
-
-      # generate our time string
-      dtstr = Time.now.strftime("%a %b %d %H:%M:%S %Z %Y")
-
-      # write the header so that we have something to send
-      output += "Package: #{HockeyBrake.configuration.app_bundle_id}\n"
-      output += "Version: #{HockeyBrake.configuration.app_version}\n"
-      output += "Date: #{dtstr}\n"
-
-      # add the optional values if possible
+    def self.generate_safe(data)
       begin
-        output += "Android: #{RUBY_PLATFORM}\n"
-        output += "Model: Ruby #{RUBY_VERSION} Rails #{Rails.version}\n"
-      rescue
-        # nothing to do
+        output = HockeyLog.generate(data)
+      rescue HockeyBrake::HockeyLogException => e
+        output += e.message
       end
 
-      # add the empty line
-      output += "\n"
+      # go ahead
+      output
+    end
 
-      # parse the XML and convert them to the HockeyApp format
+    #
+    # Generates a string which can be sent to the hockey service
+    def self.generate(data)
       begin
+        # the output
+        output = ""
+
+        # generate our time string
+        dtstr = Time.now.strftime("%a %b %d %H:%M:%S %Z %Y")
+
+        # write the header so that we have something to send
+        output += "Package: #{HockeyBrake.configuration.app_bundle_id}\n"
+        output += "Version: #{HockeyBrake.configuration.app_version}\n"
+        output += "Date: #{dtstr}\n"
+
+        # add the optional values if possible
+        begin
+          output += "Android: #{RUBY_PLATFORM}\n"
+          output += "Model: Ruby #{RUBY_VERSION} Rails #{Rails.version}\n"
+        rescue
+          # nothing to do
+        end
+
+        # add the empty line
+        output += "\n"
+
+        # parse the XML and convert them to the HockeyApp format
         if ( data.is_a?(String))
           output += generate_from_xml(data)
         else
           output += generate_from_notice(data)
         end
+
+        # return the output
+        output
       rescue Exception => e
-        # nothing to do
-        output += "An exception was thrown during handling of the exception from the HockeyBrake injector\n"
-        output += "Exception: #{e.message}"
+        raise HockeyLogException.new(e)
       end
-
-      # return the output
-      output
     end
 
     def self.generate_from_notice(data)
@@ -56,7 +68,12 @@ module HockeyBrake
       # generate the call stacke
       data.backtrace.lines.each do |line|
         class_name =   File.basename(line.file, ".rb").classify
-        output += "    at #{class_name}##{line.method}(#{line.file}:#{line.number})\n"
+
+        begin
+          output += "    at #{class_name}##{line.method}(#{line.file}:#{line.number})\n"
+        rescue
+          output += "    at #{class_name}##{line.method_name}(#{line.file}:#{line.number})\n"
+        end
       end
 
       # emit

data/lib/hockeybrake/hockey_log_exception.rb

@@ -0,0 +1,14 @@
+module HockeyBrake
+  class HockeyLogException < Exception
+    attr_accessor(:innerexception)
+
+    def initialize(exception)
+      @innerexception = exception
+    end
+
+    def message()
+      "An exception was thrown during handling of the exception from the HockeyBrake injector\n" +
+      "Exception: #{@innerexception.message}"
+    end
+  end
+end

data/lib/hockeybrake/hockey_sender.rb

@@ -17,7 +17,7 @@ module HockeyBrake
     def send_to_airbrake(data)
 
       # generate the log
-      logstr = HockeyLog.generate(data)
+      logstr = HockeyLog.generate_safe(data)
 
       # generate the stirng io
       logio = StringIO.new(logstr)
@@ -28,7 +28,7 @@ module HockeyBrake
       # send the request
       response = begin
 
-        # build the request
+                   # build the request
         req = Net::HTTP::Post::Multipart.new( url.path, "log" => UploadIO.new(logio, 'application/octet-stream', "log.txt") )
 
         # start the upload
@@ -37,15 +37,15 @@ module HockeyBrake
         end
 
       rescue *HTTP_ERRORS => e
-        log :error, "Unable to contact the HockeyApp server. HTTP Error=#{e}"
+        log_internal  :level => :error, :message => "Unable to contact the HockeyApp server. HTTP Error=#{e}"
         nil
       end
 
       case response
         when Net::HTTPSuccess then
-          log :info, "Success: #{response.class}", response
+          log_internal  :level => :info, :message => "Success: #{response.class}", :response => response
         else
-          log :error, "Failure: #{response.class}", response
+          log_internal  :level => :error, :message => "Failure: #{response.class}", :response => response
       end
 
       if response && response.respond_to?(:body)
@@ -53,8 +53,14 @@ module HockeyBrake
         error_id[1] if error_id
       end
     rescue Exception => e
-      log :error, "[HockeyBrake::HockeySender#send_to_airbrake] Cannot send notification. Error: #{e.class} - #{e.message}\nBacktrace:\n#{e.backtrace.join("\n\t")}"
+      log_internal  :level => :error,  :message => "[HockeyBrake::HockeySender#send_to_airbrake] Cannot send notification. Error: #{e.class} - #{e.message}\nBacktrace:\n#{e.backtrace.join("\n\t")}"
       nil
     end
+
+    def log_internal(options = {})
+      log options
+    rescue
+      log options[:level], options[:message], options[:response]
+    end
   end
 end

data/lib/hockeybrake/version.rb

@@ -1,4 +1,4 @@
 module HockeyBrake
-  VERSION = "0.0.5"
+  VERSION = "0.0.6"
 end
 

data/lib/hockeybrake.rb

@@ -1,4 +1,5 @@
 require 'hockeybrake/configuration'
+require 'hockeybrake/hockey_log_exception'
 require 'hockeybrake/hockey_log'
 require 'hockeybrake/hockey_sender'
 require 'hockeybrake/hockey_sender_injector'
@@ -23,8 +24,8 @@ module HockeyBrake
 
           Resque::Failure::Multiple.classes = [Resque::Failure::Redis, Resque::Failure::Airbrake]
           Resque::Failure.backend = Resque::Failure::Multiple
-        rescue
-          # nothing to do
+        rescue LoadError
+          # No resquem it's ok
         end
 
       end

data/spec/hockeybrake/configuration_spec.rb

@@ -0,0 +1,25 @@
+require 'spec_helper'
+
+describe 'Hockeybrake configuration' do
+
+  it 'accepts basic settings' do
+    HockeyBrake.configure do |config|
+      config.app_bundle_id= "com.test.app"
+      config.app_id="secret"
+      config.app_version="test"
+    end
+
+    HockeyBrake.configuration.app_bundle_id.should eq('com.test.app')
+    HockeyBrake.configuration.app_id.should eq('secret')
+    HockeyBrake.configuration.app_version.should eq('test')
+  end
+
+  it 'accepts resque specific settings' do
+    HockeyBrake.configure do |config|
+      config.no_resque_handler = true
+    end
+
+    HockeyBrake.configuration.no_resque_handler.should be_true
+  end
+
+end

data/spec/hockeybrake/hockeylog_spec.rb

@@ -0,0 +1,26 @@
+require 'spec_helper'
+require 'spec_hockeybrake_configuration'
+
+describe 'HockeyLog Exception Tracing' do
+  it 'trace standard exception' do
+    begin
+      raise "This is a sample exception"
+    rescue
+      notice = Airbrake.send(:build_notice_for, $!)
+      HockeyBrake::HockeyLog.generate(notice)
+    end
+  end
+
+  it 'raise specific exception when trace failed' do
+    expect {
+      HockeyBrake::HockeyLog.generate("Not a good exception")
+    }.to raise_exception(HockeyBrake::HockeyLogException) do |error|
+      error.message.should start_with("An exception was thrown during handling of the exception from the HockeyBrake injector")
+    end
+  end
+end
+
+
+
+
+

data/spec/spec_helper.rb

@@ -0,0 +1,6 @@
+ENV['RAILS_ENV'] = 'test'
+
+require 'bundler'
+Bundler.require(:default)
+
+require 'shoulda'

data/spec/spec_hockeybrake_configuration.rb

@@ -0,0 +1,9 @@
+Airbrake.configure do |config|
+  config.development_environments.delete("test")
+end
+
+HockeyBrake.configure do |config|
+  config.app_bundle_id= "com.test.app"
+  config.app_id="secret"
+  config.app_version="test"
+end

metadata

@@ -1,48 +1,57 @@
 --- !ruby/object:Gem::Specification
 name: hockeybrake
 version: !ruby/object:Gem::Version
-  version: 0.0.5
-  prerelease: 
+  version: 0.0.6
 platform: ruby
 authors:
 - Dirk Eisenberg
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2012-12-06 00:00:00.000000000 Z
+date: 2014-03-16 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: airbrake
   requirement: !ruby/object:Gem::Requirement
-    none: false
     requirements:
-    - - ! '>='
+    - - '>='
       - !ruby/object:Gem::Version
         version: 3.1.2
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
-    none: false
     requirements:
-    - - ! '>='
+    - - '>='
       - !ruby/object:Gem::Version
         version: 3.1.2
 - !ruby/object:Gem::Dependency
   name: multipart-post
   requirement: !ruby/object:Gem::Requirement
-    none: false
     requirements:
-    - - ! '>='
+    - - '>='
       - !ruby/object:Gem::Version
         version: 1.1.5
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
-    none: false
     requirements:
-    - - ! '>='
+    - - '>='
       - !ruby/object:Gem::Version
         version: 1.1.5
+- !ruby/object:Gem::Dependency
+  name: activesupport
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - '>='
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - '>='
+      - !ruby/object:Gem::Version
+        version: '0'
 description: An extension for the amazing airbrake gem which routes crash reports
   to HockeyApp
 email:
@@ -52,41 +61,58 @@ extensions: []
 extra_rdoc_files: []
 files:
 - .gitignore
+- .travis.yml
 - Gemfile
-- Gemfile.lock
+- Guardfile
 - LICENSE.txt
 - README.md
+- Rakefile
+- gemfiles/airbrake-3.1.16
+- gemfiles/airbrake-3.1.2
+- gemfiles/airbrake-3.1.3
+- gemfiles/airbrake-3.1.4
+- gemfiles/airbrake-3.1.5
+- gemfiles/airbrake-3.1.6
 - hockeybrake.gemspec
 - lib/generators/hockeybrake/hockeybrake_generator.rb
 - lib/generators/hockeybrake/templates/initializer.rb
 - lib/hockeybrake.rb
 - lib/hockeybrake/configuration.rb
 - lib/hockeybrake/hockey_log.rb
+- lib/hockeybrake/hockey_log_exception.rb
 - lib/hockeybrake/hockey_sender.rb
 - lib/hockeybrake/hockey_sender_injector.rb
 - lib/hockeybrake/version.rb
+- spec/hockeybrake/configuration_spec.rb
+- spec/hockeybrake/hockeylog_spec.rb
+- spec/spec_helper.rb
+- spec/spec_hockeybrake_configuration.rb
 homepage: http://rubygems.org/gems/hockeybrake
 licenses: []
+metadata: {}
 post_install_message: 
 rdoc_options: []
 require_paths:
 - lib
 required_ruby_version: !ruby/object:Gem::Requirement
-  none: false
   requirements:
-  - - ! '>='
+  - - '>='
     - !ruby/object:Gem::Version
       version: '0'
 required_rubygems_version: !ruby/object:Gem::Requirement
-  none: false
   requirements:
-  - - ! '>='
+  - - '>='
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
 rubyforge_project: 
-rubygems_version: 1.8.23
+rubygems_version: 2.1.11
 signing_key: 
-specification_version: 3
+specification_version: 4
 summary: An extension for the amazing airbrake gem which routes crash reports to HockeyApp
-test_files: []
+test_files:
+- spec/hockeybrake/configuration_spec.rb
+- spec/hockeybrake/hockeylog_spec.rb
+- spec/spec_helper.rb
+- spec/spec_hockeybrake_configuration.rb
+has_rdoc: 

data/Gemfile.lock

@@ -1,28 +0,0 @@
-PATH
-  remote: .
-  specs:
-    hockeybrake (0.0.4)
-      airbrake (>= 3.1.2)
-      multipart-post (>= 1.1.5)
-
-GEM
-  remote: http://rubygems.org/
-  specs:
-    airbrake (3.1.6)
-      builder
-      girl_friday
-    builder (3.0.4)
-    connection_pool (0.9.2)
-    girl_friday (0.11.1)
-      connection_pool (~> 0.9.0)
-      rubinius-actor
-    multipart-post (1.1.5)
-    rubinius-actor (0.0.2)
-      rubinius-core-api
-    rubinius-core-api (0.0.1)
-
-PLATFORMS
-  ruby
-
-DEPENDENCIES
-  hockeybrake!