party_fouls 1.5.6 → 1.6.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
-SHA1:
-  metadata.gz: 24bb7632ea8be8c278cd4583225f4f7d5dfbf4fb
-  data.tar.gz: ad2655ec1e1f1b26d46355941b2a77f0ab7b2a19
+SHA256:
+  metadata.gz: 021176d3d1af86241e608570bf3333d36c3e08c3c0bfd8c9781bf34061271791
+  data.tar.gz: 30fcfd0e6840fc862cc16b919a0474ffc6124ec311c844fd90ecb7abd88d2ae8
 SHA512:
-  metadata.gz: 0eb1eaea0c9d6a0beed7ae74c16cee3e485daeb01cb41f9fb11f0b66069d641b95d59bd9b5746db2eeb0392bc323459a2210e26aa9ed08849b50c7180e0a5670
-  data.tar.gz: da6149b6f020abd7f5f9e3f1f5d2a63a531b90b933b40932ce62bba6f87e0841045b069d79b5cca5a93206e22b766281dc6a1bd1fb0624003932866e5a7ca33a
+  metadata.gz: 639dab7820cd7b7570fdff7caca5fd8cad33012cde7b7623e127b64b9f42e6fe0dd934e0140b92f09b9df48f9971e5e1418bd3dfe188cc859649fd2adf66545e
+  data.tar.gz: 0d83042df6f96da24b01e23e363b8aa61e93d07b94841f2e50a3a6ce0a4778c8360c5a23d87c23e26cab1d00e5d2c34534e5ce5a797df4e0d19e74de7dab7fe9

data/README.md

@@ -4,10 +4,9 @@ Rails exceptions automatically opened as issues on GitHub
 
 # This is a fork, released as the `party_fouls` gem
 
-## Looking for help? ##
-
-If it is a bug [please open an issue on
-GitHub](https://github.com/dockyard/party_foul/issues).
+```ruby
+gem 'party_fouls', require: 'party_foul'
+```
 
 ## About ##
 
@@ -37,12 +36,6 @@ for the installation below. If you use your own account you will
 not receive emails when issues are created, updated, reopened, etc...
 because all of the work is done as your account.
 
-In your Gemfile add the following:
-
-```ruby
-gem 'party_foul'
-```
-
 ### Rails ###
 If you are using Rails you can run the install generator.
 
@@ -242,23 +235,15 @@ end
 
 This will pass the worker class name and queue as well as all worker-related parameters off to PartyFoul before passing on the exception.
 
-## Authors ##
+## Authors (original) ##
 
 * [Brian Cardarella](http://twitter.com/bcardarella)
 * [Dan McClain](http://twitter.com/_danmcclain)
 
-[We are very thankful for the many contributors](https://github.com/dockyard/party_foul/graphs/contributors)
-
 ## Versioning ##
 
 This gem follows [Semantic Versioning](http://semver.org)
 
-## Want to help? ##
-
-Please do! We are always looking to improve this gem. Please see our
-[Contribution Guidelines](https://github.com/dockyard/party_foul/blob/master/CONTRIBUTING.md)
-on how to properly submit issues and pull requests.
-
 ## Legal ##
 
 [DockYard](http://dockyard.com), LLC © 2013

data/Rakefile

@@ -1,4 +1,6 @@
 #!/usr/bin/env rake
+# frozen_string_literal: true
+
 begin
   require 'bundler/setup'
 rescue LoadError

data/lib/generators/party_foul/install_generator.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'rails/generators'
 require 'net/http'
 

data/lib/generators/party_foul/templates/party_foul.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 PartyFoul.configure do |config|
   # The collection of exceptions PartyFoul should not be allowed to handle
   # The constants here *must* be represented as strings

data/lib/party_foul.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'octokit'
 
 module PartyFoul

data/lib/party_foul/exception_handler.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 class PartyFoul::ExceptionHandler
   attr_accessor :rendered_issue
 

data/lib/party_foul/issue_renderers.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module PartyFoul::IssueRenderers; end
 require 'party_foul/issue_renderers/base'
 require 'party_foul/issue_renderers/rack'

data/lib/party_foul/issue_renderers/base.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'cgi'
 
 class PartyFoul::IssueRenderers::Base
@@ -15,13 +17,11 @@ class PartyFoul::IssueRenderers::Base
   #
   # @return [String]
   def title
-    _title = if PartyFoul.title_prefix
+    if PartyFoul.title_prefix
       "[#{PartyFoul.title_prefix}] #{masked_title}"
     else
       masked_title
-    end
-
-    _title[0..255]
+    end[0..255]
   end
 
   # Renders the issue body
@@ -134,7 +134,7 @@ BODY
       else
         value = CGI.escapeHTML(value.to_s)
       end
-      rows << "<tr><th>#{key}</th><td>#{value}</td></tr>"
+      rows += "<tr><th>#{key}</th><td>#{value}</td></tr>"
     end
   end
 

data/lib/party_foul/issue_renderers/rack.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 class PartyFoul::IssueRenderers::Rack < PartyFoul::IssueRenderers::Base
     
   def request

data/lib/party_foul/issue_renderers/rackless.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'party_foul/issue_renderers/base'
 
 class PartyFoul::IssueRenderers::Rackless < PartyFoul::IssueRenderers::Base

data/lib/party_foul/issue_renderers/rails.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 class PartyFoul::IssueRenderers::Rails < PartyFoul::IssueRenderers::Rack
   # Rails params hash. Filtered parms are respected.
   #

data/lib/party_foul/middleware.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module PartyFoul
   class Middleware
     def initialize(app)

data/lib/party_foul/processors.rb

@@ -1,2 +1,4 @@
+# frozen_string_literal: true
+
 module PartyFoul::Processors; end
 require 'party_foul/processors/sync'

data/lib/party_foul/processors/base.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 class PartyFoul::Processors::Base
   # Passes the exception and rack env data to the ExceptionHandler and
   # runs everything synchronously. This base class method must be

data/lib/party_foul/processors/delayed_job.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'party_foul/processors/base'
 
 class PartyFoul::Processors::DelayedJob < PartyFoul::Processors::Base

data/lib/party_foul/processors/resque.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'party_foul/processors/base'
 
 class PartyFoul::Processors::Resque < PartyFoul::Processors::Base

data/lib/party_foul/processors/sidekiq.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'party_foul/processors/base'
 
 class PartyFoul::Processors::Sidekiq < PartyFoul::Processors::Base

data/lib/party_foul/processors/sync.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'party_foul/processors/base'
 
 class PartyFoul::Processors::Sync < PartyFoul::Processors::Base

data/lib/party_foul/rackless_exception_handler.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 class PartyFoul::RacklessExceptionHandler < PartyFoul::ExceptionHandler
   # This handler will pass the exception and working environment from Rack off to a processor.
   # The default PartyFoul processor will work synchronously. Processor adapters can be written

data/lib/party_foul/version.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module PartyFoul
-  VERSION = '1.5.6'.freeze
+  VERSION = '1.6.0'
 end

metadata

@@ -1,15 +1,16 @@
 --- !ruby/object:Gem::Specification
 name: party_fouls
 version: !ruby/object:Gem::Version
-  version: 1.5.6
+  version: 1.6.0
 platform: ruby
 authors:
 - Brian Cardarella
 - Dan McClain
+- Tee Parham
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2017-02-24 00:00:00.000000000 Z
+date: 2018-02-23 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: octokit
@@ -31,42 +32,42 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '4.0'
+        version: '5.0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '4.0'
+        version: '5.0'
 - !ruby/object:Gem::Dependency
   name: activesupport
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '4.0'
+        version: '5.0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '4.0'
+        version: '5.0'
 - !ruby/object:Gem::Dependency
   name: railties
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '4.0'
+        version: '5.0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '4.0'
+        version: '5.0'
 - !ruby/object:Gem::Dependency
   name: minitest
   requirement: !ruby/object:Gem::Requirement
@@ -141,6 +142,7 @@ description: Auto-submit Rails exceptions as new issues on GitHub
 email:
 - bcardarella@gmail.com
 - rubygems@danmcclain.net
+- parhameter@gmail.com
 executables: []
 extensions: []
 extra_rdoc_files: []
@@ -165,17 +167,6 @@ files:
 - lib/party_foul/processors/sync.rb
 - lib/party_foul/rackless_exception_handler.rb
 - lib/party_foul/version.rb
-- test/generator_test.rb
-- test/party_foul/configure_test.rb
-- test/party_foul/exception_handler_test.rb
-- test/party_foul/issue_renderers/base_test.rb
-- test/party_foul/issue_renderers/rack_test.rb
-- test/party_foul/issue_renderers/rackless_test.rb
-- test/party_foul/issue_renderers/rails_test.rb
-- test/party_foul/middleware_test.rb
-- test/party_foul/rackless_exception_handler_test.rb
-- test/test_helper.rb
-- test/tmp/config/initializers/party_foul.rb
 homepage: https://github.com/neighborland/party_foul
 licenses:
 - MIT
@@ -188,7 +179,7 @@ required_ruby_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
-      version: 2.1.0
+      version: 2.3.3
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
@@ -196,19 +187,8 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
 requirements: []
 rubyforge_project: 
-rubygems_version: 2.6.10
+rubygems_version: 2.7.6
 signing_key: 
 specification_version: 4
 summary: Auto-submit Rails exceptions as new issues on GitHub
-test_files:
-- test/generator_test.rb
-- test/party_foul/configure_test.rb
-- test/party_foul/exception_handler_test.rb
-- test/party_foul/issue_renderers/base_test.rb
-- test/party_foul/issue_renderers/rack_test.rb
-- test/party_foul/issue_renderers/rackless_test.rb
-- test/party_foul/issue_renderers/rails_test.rb
-- test/party_foul/middleware_test.rb
-- test/party_foul/rackless_exception_handler_test.rb
-- test/test_helper.rb
-- test/tmp/config/initializers/party_foul.rb
+test_files: []

data/test/generator_test.rb

@@ -1,26 +0,0 @@
-require 'test_helper'
-require 'rails/generators/test_case'
-require 'generators/party_foul/install_generator'
-
-class PartyFoul::GeneratorTest < Rails::Generators::TestCase
-  destination File.expand_path('../tmp', __FILE__)
-  tests PartyFoul::InstallGenerator
-
-  test 'it copies the initializer' do
-    owner = 'test_owner'
-    repo  = 'test_repo'
-    octokit = mock('Octokit::Client')
-    octokit.expects(:create_authorization).with(scopes: ['repo'], note: 'PartyFoul test_owner/test_repo', note_url: 'http://example.com/test_owner/test_repo').returns(sawyer_resource({token: 'test_token'}))
-    Octokit::Client.stubs(:new).with(:login => 'test_login', :password => 'test_password', :api_endpoint => 'http://api.example.com').returns(octokit)
-    ::Readline.stubs(:readline).returns('test_login').then.returns('test_password').then.returns(owner).then.returns(repo).then.returns('http://api.example.com').then.returns('http://example.com').then.returns('')
-    run_generator
-
-    assert_file 'config/initializers/party_foul.rb' do |initializer|
-      assert_match(/config\.api_endpoint\s+=\s'http:\/\/api\.example\.com'/, initializer)
-      assert_match(/config\.web_url\s+=\s'http:\/\/example\.com'/, initializer)
-      assert_match(/config\.owner\s+=\s'test_owner'/, initializer)
-      assert_match(/config\.repo\s+=\s'test_repo'/, initializer)
-      assert_match(/config\.oauth_token\s+=\s'test_token'/, initializer)
-    end
-  end
-end

data/test/party_foul/configure_test.rb

@@ -1,37 +0,0 @@
-require 'test_helper'
-
-describe 'Party Foul Confg' do
-
-  after do
-    clean_up_party
-  end
-
-  it 'sets the proper config variables' do
-    PartyFoul.configure do |config|
-      config.blacklisted_exceptions = ['StandardError']
-      config.oauth_token        = 'test_token'
-      config.web_url            = 'http://example.com'
-      config.api_endpoint       = 'http://api.example.com'
-      config.owner              = 'test_owner'
-      config.repo               = 'test_repo'
-      config.branch             = 'master'
-      config.comment_limit      = 10
-    end
-
-    PartyFoul.blacklisted_exceptions.must_equal ['StandardError']
-    PartyFoul.github.must_be_instance_of Octokit::Client
-    PartyFoul.github.access_token.must_equal 'test_token'
-    PartyFoul.github.api_endpoint.must_equal 'http://api.example.com/'
-    PartyFoul.owner.must_equal 'test_owner'
-    PartyFoul.repo.must_equal 'test_repo'
-    PartyFoul.repo_path.must_equal 'test_owner/test_repo'
-    PartyFoul.repo_url.must_equal 'http://example.com/test_owner/test_repo'
-    PartyFoul.branch.must_equal 'master'
-    PartyFoul.comment_limit.must_equal 10
-  end
-
-  it 'has default values' do
-    PartyFoul.web_url.must_equal 'https://github.com'
-    PartyFoul.branch.must_equal  'master'
-  end
-end

data/test/party_foul/exception_handler_test.rb

@@ -1,205 +0,0 @@
-require 'test_helper'
-
-describe 'Party Foul Exception Handler' do
-  before do
-    PartyFoul.configure do |config|
-      config.oauth_token = 'abcdefg1234567890'
-      config.owner       = 'test_owner'
-      config.repo        = 'test_repo'
-    end
-
-    PartyFoul.stubs(:branch).returns('deploy')
-    PartyFoul::IssueRenderers::Rails.any_instance.stubs(:title).returns('Test Title')
-    PartyFoul::IssueRenderers::Rails.any_instance.stubs(:fingerprint).returns('test_fingerprint')
-  end
-
-  context 'when error is new' do
-    it 'will open a new error on GitHub' do
-      PartyFoul::IssueRenderers::Rails.any_instance.stubs(:body).returns('Test Body')
-      PartyFoul::IssueRenderers::Rails.any_instance.stubs(:comment).returns('Test Comment')
-      PartyFoul.github.expects(:search_issues).with('test_fingerprint repo:test_owner/test_repo state:open').returns(no_search_results)
-      PartyFoul.github.expects(:search_issues).with('test_fingerprint repo:test_owner/test_repo state:closed').returns(no_search_results)
-      PartyFoul.github.expects(:create_issue).with('test_owner/test_repo', 'Test Title', 'Test Body', labels: ['bug']).returns( {number: 1} )
-      PartyFoul.github.expects(:references).with('test_owner/test_repo', 'heads/deploy').returns( sawyer_resource({object: {sha: 'abcdefg1234567890'}}) )
-      PartyFoul.github.expects(:add_comment).with('test_owner/test_repo', 1, 'Test Comment')
-      PartyFoul::ExceptionHandler.new(nil, {}).run
-    end
-
-    context 'when additional labels are configured' do
-      before do
-        PartyFoul.configure do |config|
-          config.additional_labels = ['custom', 'label']
-        end
-      end
-      after do
-        clean_up_party
-      end
-      it 'will open a new error on GitHub with the additional labels' do
-        PartyFoul::IssueRenderers::Rails.any_instance.stubs(:body).returns('Test Body')
-        PartyFoul::IssueRenderers::Rails.any_instance.stubs(:comment).returns('Test Comment')
-        PartyFoul.github.expects(:search_issues).with('test_fingerprint repo:test_owner/test_repo state:open').returns(no_search_results)
-        PartyFoul.github.expects(:search_issues).with('test_fingerprint repo:test_owner/test_repo state:closed').returns(no_search_results)
-        PartyFoul.github.expects(:create_issue).with('test_owner/test_repo', 'Test Title', 'Test Body', :labels => ['bug', 'custom', 'label']).returns( { number: 1 } )
-        PartyFoul.github.expects(:references).with('test_owner/test_repo', 'heads/deploy').returns( sawyer_resource({object: {sha: 'abcdefg1234567890'}}) )
-        PartyFoul.github.expects(:add_comment).with('test_owner/test_repo', 1, 'Test Comment')
-        PartyFoul::ExceptionHandler.new(nil, {}).run
-      end
-    end
-
-    context 'when a proc for additional labels are configured' do
-      before do
-        PartyFoul.configure do |config|
-          config.additional_labels = Proc.new do |exception, env|
-            if env[:http_host] =~ /beta\./
-              ['beta']
-            elsif exception.message =~ /Database/
-              ['database_error']
-            end
-          end
-        end
-        PartyFoul::IssueRenderers::Rails.any_instance.stubs(:body).returns('Test Body')
-        PartyFoul::IssueRenderers::Rails.any_instance.stubs(:comment).returns('Test Comment')
-        PartyFoul.github.expects(:search_issues).with('test_fingerprint repo:test_owner/test_repo state:open').returns(no_search_results)
-        PartyFoul.github.expects(:search_issues).with('test_fingerprint repo:test_owner/test_repo state:closed').returns(no_search_results)
-        PartyFoul.github.expects(:references).with('test_owner/test_repo', 'heads/deploy').returns( sawyer_resource({object: {sha: 'abcdefg1234567890'}}) )
-        PartyFoul.github.expects(:add_comment).with('test_owner/test_repo', 1, 'Test Comment')
-      end
-      after do
-        clean_up_party
-      end
-
-      it 'will open a new error on GitHub with the default labels if no additional labels are returned from the proc' do
-        PartyFoul.github.expects(:create_issue).with('test_owner/test_repo', 'Test Title', 'Test Body', :labels => ['bug']).returns({ number: 1 })
-        PartyFoul::ExceptionHandler.new(stub(:message => ''), {}).run
-      end
-
-      it 'will open a new error on GitHub with the additional labels based on the exception message' do
-        PartyFoul.github.expects(:create_issue).with('test_owner/test_repo', 'Test Title', 'Test Body', :labels => ['bug', 'database_error']).returns({ number: 1 })
-        PartyFoul::ExceptionHandler.new(stub(:message => 'Database'), {}).run
-      end
-
-      it 'will open a new error on GitHub with the additional labels based on the env' do
-        PartyFoul.github.expects(:create_issue).with('test_owner/test_repo', 'Test Title', 'Test Body', :labels => ['bug', 'beta']).returns({ number: 1 })
-        PartyFoul::ExceptionHandler.new(stub(:message => ''), {:http_host => 'beta.example.com'}).run
-      end
-    end
-  end
-
-  context 'when error is not new' do
-    before do
-      PartyFoul::IssueRenderers::Rails.any_instance.stubs(:update_body).returns('New Body')
-      PartyFoul::IssueRenderers::Rails.any_instance.stubs(:comment).returns('Test Comment')
-    end
-
-    context 'and open' do
-      before do
-        PartyFoul.github.expects(:search_issues).with('test_fingerprint repo:test_owner/test_repo state:open').returns(
-          sawyer_resource({total_count: 1, incomplete_results: false, items:[{title: 'Test Title', body: 'Test Body', state: 'open', number: 1}]}) )
-        PartyFoul.github.expects(:update_issue).with('test_owner/test_repo', 1, 'Test Title', 'New Body', state: 'open')
-        PartyFoul.github.expects(:references).with('test_owner/test_repo', 'heads/deploy').returns(
-          sawyer_resource({object: {sha: 'abcdefg1234567890'}}) )
-      end
-
-      it 'will update the issue' do
-        PartyFoul.github.expects(:add_comment).with('test_owner/test_repo', 1, 'Test Comment')
-        PartyFoul::ExceptionHandler.new(nil, {}).run
-      end
-
-      it "doesn't post a comment if the limit has been met" do
-        PartyFoul.comment_limit = 10
-        PartyFoul::ExceptionHandler.any_instance.expects(:occurrence_count).returns(10)
-        PartyFoul.github.expects(:add_comment).never
-        PartyFoul::ExceptionHandler.new(nil, {}).run
-      end
-    end
-
-    context 'and closed' do
-      it 'will update the count on the body and re-open the issue' do
-        PartyFoul.github.expects(:search_issues).with('test_fingerprint repo:test_owner/test_repo state:open').returns(no_search_results)
-        PartyFoul.github.expects(:search_issues).with('test_fingerprint repo:test_owner/test_repo state:closed').returns(
-          sawyer_resource({total_count: 1, incomplete_results: false, items:[{title: 'Test Title', body: 'Test Body', state: 'closed', number: 1, labels: [{url: 'https://api.github.com/repos/test_owner/test_repo/labels/staging', name: 'staging', color: 'f29513'}]}]}) )
-        PartyFoul.github.expects(:update_issue).with('test_owner/test_repo', 1, 'Test Title', 'New Body', state: 'open', labels: ['bug', 'regression', 'staging'])
-        PartyFoul.github.expects(:add_comment).with('test_owner/test_repo', 1, 'Test Comment')
-        PartyFoul.github.expects(:references).with('test_owner/test_repo', 'heads/deploy').returns(sawyer_resource({object: {sha: 'abcdefg1234567890'}}))
-        PartyFoul::ExceptionHandler.new(nil, {}).run
-      end
-    end
-  end
-
-  context 'when issue is marked as "wontfix"' do
-    it 'does nothing' do
-      PartyFoul::IssueRenderers::Rails.any_instance.stubs(:body).returns('Test Body')
-      PartyFoul.github.expects(:search_issues).with('test_fingerprint repo:test_owner/test_repo state:open').returns(no_search_results)
-      PartyFoul.github.expects(:search_issues).with('test_fingerprint repo:test_owner/test_repo state:closed').returns(
-        sawyer_resource({total_count: 1, incomplete_results: false, items:[{title: 'Test Title', body: 'Test Body', state: 'closed', number: 1, labels: [{url: 'https://api.github.com/repos/test_owner/test_repo/labels/wontfix', name: 'wontfix', color: 'f29513'}]}]}) )
-      PartyFoul.github.expects(:create_issue).never
-      PartyFoul.github.expects(:update_issue).never
-      PartyFoul.github.expects(:add_comment).never
-      PartyFoul.github.expects(:references).never
-      PartyFoul::ExceptionHandler.new(nil, {}).run
-    end
-  end
-
-  describe '#occurrence_count' do
-    before do
-      @handler = PartyFoul::ExceptionHandler.new(nil, {})
-    end
-
-    it "returns the count" do
-      @handler.send(:occurrence_count, "<th>Count</th><td>1</td>").must_equal 1
-    end
-
-    it "returns 0 if no count is found" do
-      @handler.send(:occurrence_count, "Unexpected Body").must_equal 0
-    end
-  end
-
-  describe '#comment_limit_met?' do
-    before do
-      @handler = PartyFoul::ExceptionHandler.new(nil, {})
-    end
-
-    context "with no limit" do
-      it "returns false when there is no limit" do
-        PartyFoul.configure do |config|
-          config.comment_limit = nil
-        end
-        @handler.send(:comment_limit_met?, "").must_equal false
-      end
-    end
-
-    context "with a limit" do
-      before do
-        PartyFoul.configure do |config|
-          config.comment_limit = 10
-        end
-      end
-
-      it "returns false when there is a limit that has not been hit" do
-        @handler.stubs(:occurrence_count).returns(1)
-        @handler.send(:comment_limit_met?, "").must_equal false
-      end
-
-      it "returns true if the limit has been hit" do
-        @handler.stubs(:occurrence_count).returns(10)
-        @handler.send(:comment_limit_met?, "").must_equal true
-      end
-    end
-  end
-
-  describe '.clean_env' do
-    context 'when env contains file reference' do
-      before do
-        @env = { 
-          'action_dispatch.request.parameters' => {
-            :@tempfile => File.open(__FILE__)
-          }
-        }
-      end
-
-      it "marshal dump shouldn't reject values when they contain File" do
-        PartyFoul::ExceptionHandler.send(:clean_env, @env)['action_dispatch.request.parameters'].must_be_instance_of Hash
-      end
-    end
-  end
-end

data/test/party_foul/issue_renderers/base_test.rb

@@ -1,210 +0,0 @@
-require 'test_helper'
-require 'active_support/core_ext/object/blank'
-require 'action_dispatch/http/parameter_filter'
-
-describe 'Party Foul Issue Renderer Base' do
-  before do
-    Time.stubs(:now).returns(Time.new(1970, 1, 1, 0, 0, 0, '-05:00'))
-  end
-
-  after do
-    clean_up_party
-  end
-
-  describe '#body' do
-    describe 'updating issue body' do
-      before do
-        @rendered_issue = PartyFoul::IssueRenderers::Base.new(nil, nil)
-        @rendered_issue.stubs(:exception).returns('Test Exception')
-        @rendered_issue.stubs(:fingerprint).returns('abcdefg1234567890')
-        @rendered_issue.stubs(:stack_trace)
-      end
-
-      it 'updates count and timestamp' do
-        body = <<-BODY.gsub(/\n/, '')
-<table>
-<tr><th>Exception</th><td>Test Exception</td></tr>
-<tr><th>Last Occurrence</th><td>January 01, 1970 00:00:00 -0500</td></tr>
-<tr><th>Count</th><td>1</td></tr>
-</table>
-
-## Stack Trace
-<pre></pre>
-Fingerprint: `abcdefg1234567890`
-    BODY
-
-        Time.stubs(:now).returns(Time.new(1985, 10, 25, 1, 22, 0, '-05:00'))
-
-        expected_body = <<-BODY.gsub(/\n/, '')
-<table>
-<tr><th>Exception</th><td>Test Exception</td></tr>
-<tr><th>Last Occurrence</th><td>October 25, 1985 01:22:00 -0500</td></tr>
-<tr><th>Count</th><td>2</td></tr>
-</table>
-
-## Stack Trace
-<pre></pre>
-Fingerprint: `abcdefg1234567890`
-    BODY
-
-        @rendered_issue.update_body(body).must_equal expected_body
-      end
-    end
-
-    describe 'empty body' do
-      before do
-        @rendered_issue = PartyFoul::IssueRenderers::Base.new(nil, nil)
-        @rendered_issue.stubs(:exception).returns('Test Exception')
-        @rendered_issue.stubs(:fingerprint).returns('abcdefg1234567890')
-        @rendered_issue.stubs(:stack_trace)
-      end
-
-      it 'resets body' do
-        expected_body = <<-BODY
-<table><tr><th>Exception</th><td>Test Exception</td></tr><tr><th>Last Occurrence</th><td>January 01, 1970 00:00:00 -0500</td></tr><tr><th>Count</th><td>1</td></tr></table>
-
-## Stack Trace
-<pre></pre>
-Fingerprint: `abcdefg1234567890`
-    BODY
-        @rendered_issue.update_body(nil).must_equal expected_body
-      end
-    end
-  end
-
-  describe '#build_table_from_hash' do
-    it 'builds an HTML table from a hash' do
-      rendered_issue = PartyFoul::IssueRenderers::Base.new(nil, nil)
-      hash = { 'Value 1' => 'abc', 'Value 2' => { 'Value A' => 123, 'Value B' => 456 } }
-      expected = '<table><tr><th>Value 1</th><td>abc</td></tr><tr><th>Value 2</th><td><table><tr><th>Value A</th><td>123</td></tr><tr><th>Value B</th><td>456</td></tr></table></td></tr></table>'
-      rendered_issue.build_table_from_hash(hash).must_equal expected
-    end
-
-    it 'escapes HTML entities' do
-      rendered_issue = PartyFoul::IssueRenderers::Base.new(nil, nil)
-      hash = { 'Value 1' => 'Error in #<Foo>' }
-      expected = '<table><tr><th>Value 1</th><td>Error in #&lt;Foo&gt;</td></tr></table>'
-      rendered_issue.build_table_from_hash(hash).must_equal expected
-    end
-  end
-
-  describe '#fingerprint' do
-    it 'SHA1s the title' do
-      rendered_issue = PartyFoul::IssueRenderers::Base.new(nil, nil)
-      rendered_issue.stubs(:title).returns('abcdefg1234567890')
-      rendered_issue.fingerprint.must_equal Digest::SHA1.hexdigest(rendered_issue.title)
-    end
-  end
-
-  describe '#occurred_at' do
-    it 'memoizes the time' do
-      rendered_issue = PartyFoul::IssueRenderers::Base.new(nil, nil)
-      expected = rendered_issue.occurred_at
-      Time.stubs(:now).returns(Time.new(1970, 1, 1, 0, 0, 1, '-05:00'))
-      rendered_issue.occurred_at.must_equal expected
-    end
-  end
-
-  describe '#title' do
-    context 'when no title prefix is configured' do
-      before do
-        PartyFoul.configure do |config|
-          config.title_prefix = nil
-        end
-      end
-
-      it 'masks the object ids in the raw_title' do
-        rendered_issue = PartyFoul::IssueRenderers::Base.new(nil, nil)
-        rendered_issue.stubs(:raw_title).returns('Error for #<ClassName:0xabcdefg1234567>')
-        rendered_issue.title.must_equal 'Error for #<ClassName:0xXXXXXX>'
-      end
-
-      it 'masks the extended object ids in the raw_title' do
-        rendered_issue = PartyFoul::IssueRenderers::Base.new(nil, nil)
-        rendered_issue.stubs(:raw_title).returns('Error for #<#<ClassName:0x007fbddbdcd340>:0x007fbddf6be0a0>')
-        rendered_issue.title.must_equal 'Error for #<#<ClassName:0xXXXXXX>:0xXXXXXX>'
-      end
-    end
-
-    context 'when a custom title prefix is configured' do
-      before do
-        PartyFoul.configure do |config|
-          config.title_prefix = 'PRODUCTION'
-        end
-      end
-
-      it 'adds a custom prefix' do
-        rendered_issue = PartyFoul::IssueRenderers::Base.new(nil, nil)
-        rendered_issue.stubs(:raw_title).returns('Error')
-        rendered_issue.title.must_equal '[PRODUCTION] Error'
-      end
-
-      it 'generated unique fingerprints' do
-        rendered_issue = PartyFoul::IssueRenderers::Base.new(nil, nil)
-        rendered_issue.stubs(:raw_title).returns('Error')
-        fingerprint = rendered_issue.fingerprint
-
-        PartyFoul.configure do |config|
-          config.title_prefix = 'STAGING'
-        end
-
-        fingerprint.wont_equal rendered_issue.fingerprint
-      end
-    end
-
-    context 'title length limit' do
-      before do
-        PartyFoul.configure do |config|
-          config.title_prefix = 'PRODUCTION'
-        end
-      end
-
-      it 'at most 256 chars' do
-        rendered_issue = PartyFoul::IssueRenderers::Base.new(nil, nil)
-        rendered_issue.stubs(:raw_title).returns('F'*300)
-        rendered_issue.title.size.must_equal 256
-      end
-    end
-  end
-
-  describe '#stack_trace' do
-    # it 'returns the stack trace' do
-      # exception = mock do
-        # stubs backtrace: ['/path/to/gems/gem-name/lib/some/file.rb:123 in `method`']
-      # end
-      # rendered_issue = PartyFoul::IssueRenderers::Base.new(exception, nil)
-      # rendered_issue.stack_trace.must_equal exception.backtrace.first
-    # end
-
-    it 'formats the stack trace with shortened bundle paths' do
-      exception = mock do
-        stubs backtrace: ["#{Bundler.bundle_path}/some_gem/lib/some/file.rb:123 in `method`"]
-      end
-
-      rendered_issue = PartyFoul::IssueRenderers::Base.new(exception, nil)
-      rendered_issue.stack_trace.must_equal '[bundle].../some_gem/lib/some/file.rb:123 in `method`'
-    end
-
-    it 'formats the stack trace with link to shortened application path' do
-      clean_up_party
-      exception = mock do
-        stubs backtrace: ['/path/to/app/lib/some/file.rb:123 in `method`']
-      end
-      rendered_issue = PartyFoul::IssueRenderers::Base.new(exception, nil)
-      rendered_issue.stubs app_root: '/path/to/app'
-      rendered_issue.stack_trace.must_match "<a href='https://github.com///blob//lib/some/file.rb#L123'>[app].../lib/some/file.rb:123 in `method`</a>"
-    end
-
-    it 'does not link to bundled resources on Heroku' do
-      Bundler.stub(:bundle_path, '/app/vendor/bundle/ruby/2.0.0') do
-        exception = mock do
-          stubs backtrace: ["#{Bundler.bundle_path}/some_gem/lib/some/file.rb:123 in `method`"]
-        end
-
-        rendered_issue = PartyFoul::IssueRenderers::Base.new(exception, nil)
-        rendered_issue.stubs app_root: '/app'
-        rendered_issue.stack_trace.must_equal '[bundle].../some_gem/lib/some/file.rb:123 in `method`'
-      end
-    end
-  end
-end

data/test/party_foul/issue_renderers/rack_test.rb

@@ -1,80 +0,0 @@
-require 'test_helper'
-
-describe 'Rack Issue Renderer' do
-  describe '#params' do
-    before do
-      @issue_renderer = PartyFoul::IssueRenderers::Rack.new(nil, ::Rack::MockRequest.env_for("/?status=ok"))
-    end
-
-    it 'returns the value of the query param' do
-      @issue_renderer.params['status'].must_equal 'ok'
-    end
-  end
-  
-  describe '#raw_title' do
-    before do
-      @exception = Exception.new('message')
-    end
-
-    it 'constructs the title with the class and instance method' do
-      @issue_renderer = PartyFoul::IssueRenderers::Rack.new(@exception, {})
-      @issue_renderer.send(:raw_title).must_equal %{(Exception) "message"}
-    end
-  end
-  
-  describe '#request' do
-    it 'builds a rack request' do
-      issue_renderer = PartyFoul::IssueRenderers::Rack.new(nil, {})
-      assert issue_renderer.request.is_a?(::Rack::Request)
-    end
-  end  
-  
-  describe '#ip_address_geolocator' do
-    # we are delegating to rack, see
-    # https://github.com/rack/rack/blob/master/test/spec_request.rb    
-    
-    it 'returns the link to geolocation IP address when REMOTE_ADDR is set' do
-      issue_renderer = PartyFoul::IssueRenderers::Rack.new(nil, { 'REMOTE_ADDR' => '1.2.3.4' })
-      ip_addr = "<a href='http://ipinfo.io/1.2.3.4'>1.2.3.4</a>"
-      issue_renderer.ip_address_locator.must_equal ip_addr
-    end
-    
-    it 'return the link to geolocation IP address when HTTP_X_FORWARDED_FOR is set' do
-      issue_renderer = PartyFoul::IssueRenderers::Rack.new(nil, { 'HTTP_X_FORWARDED_FOR' => '10.0.0.1, 10.0.0.1, 3.4.5.6' })
-      ip_addr = "<a href='http://ipinfo.io/3.4.5.6'>3.4.5.6</a>"
-      issue_renderer.ip_address_locator.must_equal ip_addr
-    end
-  end
-  
-  describe '#url' do
-    it 'returns the method and uri' do
-      issue_renderer = PartyFoul::IssueRenderers::Rack.new(nil, { 'REQUEST_METHOD' => 'GET', 'REQUEST_URI' => '/something' })
-      issue_renderer.url.must_equal '[GET] /something'
-    end    
-  end  
-  
-  describe '#session' do
-    it 'returns the session' do
-      issue_renderer = PartyFoul::IssueRenderers::Rack.new(nil, { 'rack.session' => 'abc:123' })
-      issue_renderer.session.must_equal 'abc:123'
-    end    
-  end
-  
-  describe '#http_headers' do
-    it 'returns http headers' do
-      issue_renderer = PartyFoul::IssueRenderers::Rack.new(nil, 
-        { 
-          'HTTP_VERSION' => 'version',
-          'HTTP_USER_AGENT' => 'user agent',
-          'HTTP_ACCEPT_ENCODING' => 'accept encoding',
-          'HTTP_ACCEPT' => 'accept' 
-        })
-        
-      issue_renderer.http_headers[:Version].must_equal 'version'
-      issue_renderer.http_headers['User Agent'].must_equal 'user agent'
-      issue_renderer.http_headers['Accept Encoding'].must_equal 'accept encoding'
-      issue_renderer.http_headers[:Accept].must_equal 'accept'
-    end    
-  end
-      
-end

data/test/party_foul/issue_renderers/rackless_test.rb

@@ -1,29 +0,0 @@
-require 'test_helper'
-
-describe 'Rackless Issue Renderer' do
-  before do
-    @env = { :params => { :val1 => '1', :val2 => '2' }, :class => 'Worker', :method => 'perform' }
-  end
-
-  describe '#params' do
-    before do
-      @rendered_issue = PartyFoul::IssueRenderers::Rackless.new(nil, @env)
-    end
-
-    it 'returns the parameters' do
-      @rendered_issue.params[:val1].must_equal '1'
-      @rendered_issue.params[:val2].must_equal '2'
-    end
-  end
-
-  describe '#raw_title' do
-    before do
-      @exception = Exception.new('message')
-      @rendered_issue = PartyFoul::IssueRenderers::Rackless.new(@exception, @env)
-    end
-
-    it 'constructs the title with the controller and action' do
-      @rendered_issue.send(:raw_title).must_equal %{Worker#perform (Exception) "message"}
-    end
-  end
-end

data/test/party_foul/issue_renderers/rails_test.rb

@@ -1,83 +0,0 @@
-require 'test_helper'
-
-describe 'Rails Issue Renderer' do
-  describe '#params' do
-    let(:request_parameters) { { 'status' => 'ok', 'password' => 'test' } }
-    before do
-      @rendered_issue = PartyFoul::IssueRenderers::Rails.new(nil, {'action_dispatch.parameter_filter' => ['password'], 'action_dispatch.request.parameters' => request_parameters, 'QUERY_STRING' => { 'status' => 'fail' } })
-    end
-
-    it 'returns ok' do
-      @rendered_issue.params['status'].must_equal 'ok'
-      @rendered_issue.params['password'].must_equal '[FILTERED]'
-    end
-
-    context 'without request parameters' do
-      let(:request_parameters) { nil }
-
-      it 'returns empty hash' do
-        @rendered_issue.params.must_equal({})
-      end
-    end
-  end
-
-  describe '#occurred_at' do
-    it 'uses current when possible' do
-      Time.stubs(:now).returns(Time.new(1970, 1, 1, 0, 0, 1, '-08:00'))
-      Time.stubs(:current).returns(Time.new(1970, 1, 1, 0, 0, 1, '-05:00'))
-      current_as_string = Time.current.strftime('%B %d, %Y %H:%M:%S %z')
-      rendered_issue = PartyFoul::IssueRenderers::Rails.new(nil, nil)
-      rendered_issue.occurred_at.must_equal current_as_string
-    end
-  end
-
-  describe '#session' do
-    let(:params) { {'action_dispatch.parameter_filter' => ['password'], 'rack.session' => { 'status' => 'ok', 'password' => 'test' }, 'QUERY_STRING' => { 'status' => 'fail' } } }
-
-    before do
-      @rendered_issue = PartyFoul::IssueRenderers::Rails.new(nil, params)
-    end
-
-    it 'returns ok' do
-      @rendered_issue.session['status'].must_equal 'ok'
-      @rendered_issue.session['password'].must_equal '[FILTERED]'
-    end
-
-    context "without session" do
-
-      let(:params) { {'action_dispatch.parameter_filter' => ['password'], 'QUERY_STRING' => { 'status' => 'fail' } } }
-
-      it 'returns empty hash' do
-        @rendered_issue.session.must_be_empty
-      end
-    end
-
-
-  end
-
-  describe '#raw_title' do
-    let(:request_parameters) { { 'controller' => 'landing', 'action' => 'index' } }
-    before do
-      @exception = Exception.new('message')
-      controller_instance = mock('Controller')
-      controller_instance.stubs(:class).returns('LandingController')
-      env = {
-        'action_dispatch.request.parameters' => request_parameters,
-        'action_controller.instance' => controller_instance
-      }
-      @rendered_issue = PartyFoul::IssueRenderers::Rails.new(@exception, env)
-    end
-
-    it 'constructs the title with the controller and action' do
-      @rendered_issue.send(:raw_title).must_equal %{LandingController#index (Exception) "message"}
-    end
-
-    context 'without request parameters' do
-      let(:request_parameters) { nil }
-
-      it 'leaves action blank' do
-        @rendered_issue.send(:raw_title).must_equal %{LandingController# (Exception) "message"}
-      end
-    end
-  end
-end

data/test/party_foul/middleware_test.rb

@@ -1,48 +0,0 @@
-require 'test_helper'
-
-describe 'Party Foul Middleware' do
-  include Rack::Test::Methods
-
-  after do
-    clean_up_party
-  end
-
-  def error_to_raise
-    Exception
-  end
-
-  def app
-    _self = self
-    Rack::Builder.new {
-      map '/' do
-        use PartyFoul::Middleware
-        run lambda { |env| raise _self.error_to_raise }
-      end
-    }
-  end
-
-  context 'handling an exception' do
-    it 'does not handle exception' do
-      PartyFoul::ExceptionHandler.stubs(:handle)
-      PartyFoul::ExceptionHandler.expects(:handle)
-      lambda {
-        get '/'
-      }.must_raise(Exception)
-    end
-  end
-
-  context 'filtering based upon exception' do
-    before do
-      PartyFoul.blacklisted_exceptions = ['StandardError']
-      self.stubs(:error_to_raise).returns(StandardError)
-    end
-
-    it 'does not handle exception' do
-      PartyFoul::ExceptionHandler.stubs(:handle)
-      PartyFoul::ExceptionHandler.expects(:handle).never
-      lambda {
-        get '/'
-      }.must_raise(StandardError)
-    end
-  end
-end

data/test/party_foul/rackless_exception_handler_test.rb

@@ -1,33 +0,0 @@
-require 'test_helper'
-
-describe 'Party Foul Rackless Exception Handler' do
-  before do
-    PartyFoul.configure do |config|
-      config.oauth_token = 'abcdefg1234567890'
-      config.owner       = 'test_owner'
-      config.repo        = 'test_repo'
-    end
-  end
-
-  describe '#handle' do
-    it 'will call run on itself' do
-      PartyFoul::RacklessExceptionHandler.any_instance.expects(:run)
-      PartyFoul::RacklessExceptionHandler.handle(nil, {})
-    end
-
-    it 'should pass parameters required by the renderer to the initializer' do
-      params = {:class => Object, :method => :to_s, :params => {:one => "One", :two => "Two"}}
-      expected_exception_handler = PartyFoul::RacklessExceptionHandler.new(nil, params)
-      expected_exception_handler.expects(:run)
-      PartyFoul::RacklessExceptionHandler.expects(:new).with(nil, params).returns(expected_exception_handler)
-      PartyFoul::RacklessExceptionHandler.handle(nil, params)      
-    end
-  end
-
-  describe '#initialize' do
-    it 'should use PartyFoul::IssueRenderers::Rackless for rendering issues' do
-      issue_renderer = PartyFoul::RacklessExceptionHandler.new(nil, {}).rendered_issue
-      assert_kind_of(PartyFoul::IssueRenderers::Rackless, issue_renderer)
-    end
-  end
-end

data/test/test_helper.rb

@@ -1,42 +0,0 @@
-require 'bundler/setup'
-
-if defined?(M)
-  require 'minitest/spec'
-else
-  require 'minitest/autorun'
-end
-begin
-  require 'byebug'
-rescue LoadError
-end
-require 'rack/test'
-require 'mocha/setup'
-require 'active_support'
-require 'party_foul'
-
-ActiveSupport.test_order = :random
-
-class MiniTest::Spec
-  class << self
-    alias :context :describe
-  end
-end
-
-module MiniTest::Expectations
-  infect_an_assertion :assert_received, :must_have_received
-end
-
-def clean_up_party
-  %w{github oauth_token api_endpoint owner repo blacklisted_exceptions processor web_url branch additional_labels comment_limit}.each do |attr|
-    PartyFoul.send("#{attr}=", nil)
-  end
-end
-
-def sawyer_resource(attrs)
-  agent = Sawyer::Agent.new(PartyFoul.api_endpoint)
-  Sawyer::Resource.new(agent, attrs)
-end
-
-def no_search_results
-  sawyer_resource(total_count: 0, incomplete_results: false, items: [])
-end

data/test/tmp/config/initializers/party_foul.rb

@@ -1,39 +0,0 @@
-PartyFoul.configure do |config|
-  # The collection of exceptions PartyFoul should not be allowed to handle
-  # The constants here *must* be represented as strings
-  config.blacklisted_exceptions = ['ActiveRecord::RecordNotFound', 'ActionController::RoutingError']
-
-  # The OAuth token for the account that is opening the issues on GitHub
-  config.oauth_token            = 'test_token'
-
-  # The API endpoint for GitHub. Unless you are hosting a private
-  # instance of Enterprise GitHub you do not need to include this
-  config.api_endpoint           = 'http://api.example.com'
-
-  # The Web URL for GitHub. Unless you are hosting a private
-  # instance of Enterprise GitHub you do not need to include this
-  config.web_url                = 'http://example.com'
-
-  # The organization or user that owns the target repository
-  config.owner                  = 'test_owner'
-
-  # The repository for this application
-  config.repo                   = 'test_repo'
-
-  # The branch for your deployed code
-  # config.branch               = 'master'
-
-  # Additional labels to add to issues created
-  # config.additional_labels    = ['production']
-  # or
-  # config.additional_labels    = Proc.new do |exception, env|
-  #   []
-  # end
-
-  # Limit the number of comments per issue
-  # config.comment_limit        = 10
-
-  # Setting your title prefix can help with
-  # distinguishing the issue between environments
-  # config.title_prefix         = Rails.env
-end