hato 0.0.6 → 0.0.7

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: ce9895dcdaf6ee77a09e2816cee3a20f5034194b
-  data.tar.gz: 2bb1f36ebcd0e3f717bb9fdd0be0e4ec8f307edb
+  metadata.gz: 202679221627cc28218e8f0fb405f4961eaaea3d
+  data.tar.gz: e54790c53ba76c24dbfbb90bd6721090fcfcf96d
 SHA512:
-  metadata.gz: 47208c46f29d95adcf2a3ba59d1423d483945e47927f84e341ff10327832f2387763719c2f0d23a55b26c51b7491f54d34f7ce81cb2ccc56f9af05d9cc2ef6e6
-  data.tar.gz: 46e5c4c1a57c36a4b6ebb0b25807fba47975e22d54deb712f628a052e2a8e26d1d3106f912c5e07eb92830cea39f8d6e364e3d856be988730e72f3f15bd5945a
+  metadata.gz: c7915729dc929ede55d761e1308a78fbd08a713294b6e5297571f4a2d27d4c32f95e3a0b9c9b25ee07e09bf1cfcf11ed2dfeaa0e10f061083333df716a0a18f0
+  data.tar.gz: f1793d36de2b22e9e20d3332ff902922dab820a262d6463ad8326d1abc0ec110b6d4e6fac0da62de0dfb59cd5078bf08e77fd6d8dbe02d0ab9d121251c1f2bad

data/lib/hato/httpd.rb

@@ -49,7 +49,8 @@ module Hato
         )
       end
 
-      post '/webhook' do
+      post '/webhook/:owner/:repository' do
+        tag     = ['webhook', params[:owner], params[:repository]].join('.')
         payload = params[:payload]
 
         if !payload
@@ -59,18 +60,6 @@ module Hato
           )
         end
 
-        owner      = payload['repository'] && payload['repository']['owner'] && payload['repository']['owner']['name']
-        repository = payload['repository'] && payload['repository']['name']
-
-        if owner && repository
-          tag = ['webhook', owner, repository].join('.')
-        else
-          halt 400, JSON.dump(
-            status:  :error,
-            message: 'Invalid JSON message: both `repository.owner.name` and `repository.name` are required',
-          )
-        end
-
         settings.observer.update(
           tag:     tag,
           payload: payload,

data/lib/hato/version.rb

@@ -1,3 +1,3 @@
 module Hato
-  VERSION = "0.0.6"
+  VERSION = "0.0.7"
 end

data/spec/lib/hato/httpd_spec.rb

@@ -62,18 +62,8 @@ describe Hato::Httpd do
 
   describe 'webhook' do
     context 'success' do
-      let(:payload) {
-        {
-          repository: {
-            name:  'hato',
-            owner: {
-              name: 'kentaro',
-            },
-          }
-        }
-      }
       it 'should be success with correct payload' do
-        post '/webhook', payload: payload, api_key: 'test'
+        post '/webhook/kentaro/hato', payload: {foo: 'bar'}, api_key: 'test'
         expect(last_response).to be_ok
         expect(last_response.body).to eq('{"status":"success","message":"Successfully sent the message you notified to me."}')
       end
@@ -81,22 +71,10 @@ describe Hato::Httpd do
 
     context 'error' do
       it 'should be error when payload is not passed' do
-        post '/webhook', api_key: 'test'
+        post '/webhook/kentaro/hato', api_key: 'test'
         expect(last_response).to be_bad_request
         expect(last_response.body).to eq('{"status":"error","message":"Missing mandatory parameter: `payload`"}')
       end
-
-      it 'should be error when repository.name is not passed' do
-        post '/webhook', api_key: 'test', payload: {repository: {owner: {name: 'kentaro'}}}
-        expect(last_response).to be_bad_request
-        expect(last_response.body).to eq('{"status":"error","message":"Invalid JSON message: both `repository.owner.name` and `repository.name` are required"}')
-      end
-
-      it 'should be error when repository.owner.name is not passed' do
-        post '/webhook', api_key: 'test', payload: {repository: {name: 'hato'}}
-        expect(last_response).to be_bad_request
-        expect(last_response.body).to eq('{"status":"error","message":"Invalid JSON message: both `repository.owner.name` and `repository.name` are required"}')
-      end
     end
   end
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: hato
 version: !ruby/object:Gem::Version
-  version: 0.0.6
+  version: 0.0.7
 platform: ruby
 authors:
 - Kentaro Kuribayashi
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2013-11-25 00:00:00.000000000 Z
+date: 2013-11-26 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: sinatra