right_hook 0.5.1 → 0.5.2

data/lib/right_hook/app.rb

@@ -16,6 +16,8 @@ module RightHook
       halt 404, "Unknown event type" unless Event::KNOWN_TYPES.include?(event_type)
       halt 501, "Event type not implemented" unless respond_to?("on_#{event_type}")
 
+      return 200 if request.env['HTTP_X_GITHUB_EVENT'] == Event::PING
+
       require_valid_signature(content, owner, repo_name, event_type)
 
       json = JSON.parse(params['payload'])
@@ -39,6 +41,7 @@ module RightHook
     end
 
     private
+
     def require_valid_signature(content, owner, repo_name, event_type)
       s = secret(owner, repo_name, event_type)
       expected_signature = OpenSSL::HMAC.hexdigest(OpenSSL::Digest::Digest.new('sha1'), s, content)

data/lib/right_hook/event.rb

@@ -18,6 +18,7 @@ module RightHook
     ISSUE = 'issue'.freeze
     PULL_REQUEST = 'pull_request'.freeze
     ISSUE_COMMENT = 'issue_comment'.freeze
+    PING = 'ping'.freeze
 
     KNOWN_TYPES = [
       ISSUE,

data/lib/right_hook/version.rb

@@ -1,3 +1,3 @@
 module RightHook
-  VERSION = "0.5.1"
+  VERSION = "0.5.2"
 end

data/spec/app_spec.rb

@@ -1,7 +1,7 @@
 require 'spec_helper'
 
 describe RightHook::App do
-  class BareApp < RightHook::App
+  class BareApp < RightHook::DebugApp
     def secret(owner, repo_name, event_type)
       'secret'
     end
@@ -25,7 +25,7 @@ describe RightHook::App do
     expect { RightHook::App.new!.secret('owner', 'repo', 'issue') }.to raise_error(NotImplementedError)
   end
 
-  describe 'the status of a request' do
+  describe 'when an app implements the hook method' do
     class WeirdApp < BareApp
       def on_issue(owner, repo_name, action, issue_json)
         302
@@ -34,11 +34,25 @@ describe RightHook::App do
 
     let(:app) { WeirdApp }
 
-    it 'is 200 regardless of what the hook method returns' do
-      payload =  '{}'
-      WeirdApp.any_instance.should_receive(:on_issue).and_call_original
+    let(:payload) { '{}' }
+
+    it 'responds with a 200 regardless of what the hook method returns' do
+      expect_any_instance_of(WeirdApp).to receive(:on_issue).and_call_original
+
       post_with_signature(path: '/hook/mark-rushakoff/right_hook/issue', payload: payload, secret: 'secret')
       expect(last_response.status).to eq(200)
     end
+
+    it 'does not call the hook method when a ping event is received' do
+      expect_any_instance_of(WeirdApp).not_to receive(:on_issue)
+
+      post(
+        '/hook/mark-rushakoff/right_hook/issue',
+        {payload: payload},
+        generate_secret_header('secret', URI.encode_www_form(payload: payload)).merge(
+          'HTTP_X_GITHUB_EVENT' => 'ping'
+        ),
+      )
+    end
   end
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: right_hook
 version: !ruby/object:Gem::Version
-  version: 0.5.1
+  version: 0.5.2
   prerelease: 
 platform: ruby
 authors:
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2014-02-28 00:00:00.000000000 Z
+date: 2014-06-06 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: bundler
@@ -206,7 +206,7 @@ required_ruby_version: !ruby/object:Gem::Requirement
       version: '0'
       segments:
       - 0
-      hash: -495302186993546898
+      hash: 288642712136343718
 required_rubygems_version: !ruby/object:Gem::Requirement
   none: false
   requirements:
@@ -215,7 +215,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
       segments:
       - 0
-      hash: -495302186993546898
+      hash: 288642712136343718
 requirements: []
 rubyforge_project: 
 rubygems_version: 1.8.23