Project hooks: set events to trigger on

lib/gitlab/client/projects.rb

@@ -159,9 +159,14 @@ def project_hook(project, id)
     #
     # @param  [Integer, String] project The ID or name of a project.
     # @param  [String] url The hook URL.
+    # @param  [Hash] options Events list (`{push_events: true, merge_requests_events: false}`).
     # @return [Gitlab::ObjectifiedHash] Information about added hook.
-    def add_project_hook(project, url)
-      post("/projects/#{project}/hooks", :body => {:url => url})
+    def add_project_hook(project, url, options = {})
+      available_events = [:push_events, :merge_requests_events, :issues_events]
+      passed_events = available_events.select { |event| options[event] }
+      events = Hash[passed_events.map { |event| [event, options[event]] }]
+
+      post("/projects/#{project}/hooks", :body => {:url => url}.merge(events))
     end
 
     # Updates a project hook URL.

spec/gitlab/client/projects_spec.rb

@@ -174,18 +174,36 @@
   end
 
   describe ".add_project_hook" do
-    before do
-      stub_post("/projects/1/hooks", "project_hook")
-      @hook = Gitlab.add_project_hook(1, "https://api.example.net/v1/webhooks/ci")
-    end
-
-    it "should get the correct resource" do
-      body = {:url => "https://api.example.net/v1/webhooks/ci"}
-      a_post("/projects/1/hooks").with(:body => body).should have_been_made
-    end
-
-    it "should return information about an added hook" do
-      @hook.url.should == "https://api.example.net/v1/webhooks/ci"
+    context "without specified events" do
+      before do
+        stub_post("/projects/1/hooks", "project_hook")
+        @hook = Gitlab.add_project_hook(1, "https://api.example.net/v1/webhooks/ci")
+      end
+
+      it "should get the correct resource" do
+        body = {:url => "https://api.example.net/v1/webhooks/ci"}
+        a_post("/projects/1/hooks").with(:body => body).should have_been_made
+      end
+
+      it "should return information about an added hook" do
+        @hook.url.should == "https://api.example.net/v1/webhooks/ci"
+      end
+    end
+
+    context "with specified events" do
+      before do
+        stub_post("/projects/1/hooks", "project_hook")
+        @hook = Gitlab.add_project_hook(1, "https://api.example.net/v1/webhooks/ci", push_events: true, merge_requests_events: true)
+      end
+
+      it "should get the correct resource" do
+        body = {:url => "https://api.example.net/v1/webhooks/ci", push_events: "true", merge_requests_events: "true"}
+        a_post("/projects/1/hooks").with(:body => body).should have_been_made
+      end
+
+      it "should return information about an added hook" do
+        @hook.url.should == "https://api.example.net/v1/webhooks/ci"
+      end
     end
   end