Merge pull request #3156 from m4tthumphrey/api-system-hooks

Added methods to manage system hooks from API

1 files changed, 69 insertions, 0 deletions

diff --git a/spec/requests/api/system_hooks_spec.rb b/spec/requests/api/system_hooks_spec.rb
new file mode 100644
index 00000000000..9842ae91ec3
--- /dev/null
+++ b/spec/requests/api/system_hooks_spec.rb
@@ -0,0 +1,69 @@
+require 'spec_helper'

+

+describe Gitlab::API do

+  include ApiHelpers

+

+  let(:user) { create(:user) }

+  let(:admin) { create(:admin) }

+  let!(:hook) { create(:system_hook, url: "http://example.com") }

+

+  before { stub_request(:post, hook.url) }

+

+  describe "GET /hooks" do

+    context "when not an admin" do

+      it "should return forbidden error" do

+        get api("/hooks", user)

+        response.status.should == 403

+      end

+    end

+

+    context "when authenticated as admin" do

+      it "should return an array of hooks" do

+        get api("/hooks", admin)

+        response.status.should == 200

+        json_response.should be_an Array

+        json_response.first['url'].should == hook.url

+      end

+    end

+  end

+

+  describe "POST /hooks" do

+    it "should create new hook" do

+      expect {

+        post api("/hooks", admin), url: 'http://example.com'

+      }.to change { SystemHook.count }.by(1)

+    end

+

+    it "should respond with 404 on failure" do

+      post api("/hooks", admin)

+      response.status.should == 404

+    end

+

+    it "should not create new hook without url" do

+      expect {

+        post api("/hooks", admin)

+      }.to_not change { SystemHook.count }

+    end

+  end

+

+  describe "GET /hooks/:id" do

+    it "should return hook by id" do

+      get api("/hooks/#{hook.id}", admin)

+      response.status.should == 200

+      json_response['event_name'].should == 'project_create'

+    end

+

+    it "should return 404 on failure" do

+      get api("/hooks/404", admin)

+      response.status.should == 404

+    end

+  end

+

+  describe "DELETE /hooks/:id" do

+    it "should delete a hook" do

+      expect {

+        delete api("/hooks/#{hook.id}", admin)

+      }.to change { SystemHook.count }.by(-1)

+    end

+  end

+end
\ No newline at end of file