Add methods

Author: jmini

gitlab4j-api/src/main/java/org/gitlab4j/api/SystemHooksApi.java

@@ -141,6 +141,36 @@ public SystemHook addSystemHook(String url, String token, SystemHook systemHook)
         return (response.readEntity(SystemHook.class));
     }
 
+    /**
+     * Add a new system hook. This method requires admin access.
+     *
+     *  <pre><code>GitLab Endpoint: PUT /hooks/:hook_id</code></pre>
+     *
+     * @param systemHook the systemHook to update
+     * @param token secret token to validate received payloads, optional
+     * @return an SystemHook instance with info on the added system hook
+     * @throws GitLabApiException if any exception occurs
+     */
+    public SystemHook updateSystemHook(SystemHook systemHook, String token) throws GitLabApiException {
+
+        if (systemHook.getId() == null) {
+            throw new RuntimeException("systemHook id cannot be null");
+        }
+
+        GitLabApiForm formData = new GitLabApiForm()
+                .withParam("url", systemHook.getUrl())
+                .withParam("token", token)
+                .withParam("name", systemHook.getName())
+                .withParam("description", systemHook.getDescription())
+                .withParam("push_events", systemHook.getPushEvents())
+                .withParam("tag_push_events", systemHook.getTagPushEvents())
+                .withParam("merge_requests_events", systemHook.getMergeRequestsEvents())
+                .withParam("repository_update_events", systemHook.getRepositoryUpdateEvents())
+                .withParam("enable_ssl_verification", systemHook.getEnableSslVerification());
+        Response response = putWithFormData(Response.Status.OK, formData, "hooks", systemHook.getId());
+        return (response.readEntity(SystemHook.class));
+    }
+
     /**
      * Deletes a system hook. This method requires admin access.
      *
@@ -212,7 +242,7 @@ public void testSystemHook(Long hookId) throws GitLabApiException {
     }
 
     /**
-     * Add a new system hook. This method requires admin access.
+     * Add a new URL variable.
      *
      *  <pre><code>GitLab Endpoint: PUT /hooks/:hook_id/url_variables/:key</code></pre>
      *
@@ -225,4 +255,17 @@ public void addSystemHookUrlVariable(Long hookId, String key, String value) thro
         GitLabApiForm formData = new GitLabApiForm().withParam("value", value, true);
         put(Response.Status.CREATED, formData.asMap(), "hooks", hookId, "url_variables", key);
     }
+
+    /**
+     * Delete a URL variable.
+     *
+     *  <pre><code>GitLab Endpoint: DELETE /hooks/:hook_id/url_variables/:key</code></pre>
+     *
+     * @param hookId the ID of the system hook
+     * @param key Key of the URL variable
+     * @throws GitLabApiException if any exception occurs
+     */
+    public void deleteSystemHookUrlVariable(Long hookId, String key) throws GitLabApiException {
+        delete(Response.Status.NO_CONTENT, null, "hooks", hookId, "url_variables", key);
+    }
 }

gitlab4j-models/src/main/java/org/gitlab4j/api/models/SystemHook.java

@@ -2,6 +2,7 @@
 
 import java.io.Serializable;
 import java.util.Date;
+import java.util.List;
 
 import org.gitlab4j.models.utils.JacksonJson;
 
@@ -18,6 +19,7 @@ public class SystemHook implements Serializable {
     private Boolean enableSslVerification;
     private Boolean repositoryUpdateEvents;
     private Boolean mergeRequestsEvents;
+    private List<SystemHook.UrlVariable> urlVariables;
 
     public Long getId() {
         return id;
@@ -99,48 +101,85 @@ public Boolean getMergeRequestsEvents() {
         return mergeRequestsEvents;
     }
 
+    public List<SystemHook.UrlVariable> getUrlVariables() {
+        return urlVariables;
+    }
+
+    public void setUrlVariables(List<SystemHook.UrlVariable> urlVariables) {
+        this.urlVariables = urlVariables;
+    }
+
     public SystemHook withId(Long id) {
         this.id = id;
-        return (this);
+        return this;
+    }
+
+    public SystemHook withName(String name) {
+        this.name = name;
+        return this;
+    }
+
+    public SystemHook withDescription(String description) {
+        this.description = description;
+        return this;
     }
 
     public SystemHook withUrl(String url) {
         this.url = url;
-        return (this);
+        return this;
     }
 
     public SystemHook withCreatedAt(Date createdAt) {
         this.createdAt = createdAt;
-        return (this);
+        return this;
     }
 
     public SystemHook withPushEvents(Boolean pushEvents) {
         this.pushEvents = pushEvents;
-        return (this);
+        return this;
     }
 
     public SystemHook withTagPushEvents(Boolean tagPushEvents) {
         this.tagPushEvents = tagPushEvents;
-        return (this);
+        return this;
     }
 
     public SystemHook withEnableSslVerification(Boolean enableSslVerification) {
         this.enableSslVerification = enableSslVerification;
-        return (this);
+        return this;
     }
 
     public SystemHook withRepositoryUpdateEvents(Boolean repositoryUpdateEvents) {
         this.repositoryUpdateEvents = repositoryUpdateEvents;
-        return (this);
+        return this;
     }
 
     public SystemHook withMergeRequestsEvents(Boolean mergeRequestsEvents) {
         this.mergeRequestsEvents = mergeRequestsEvents;
-        return (this);
+        return this;
     }
 
     @Override
     public String toString() {
         return (JacksonJson.toJsonString(this));
     }
+
+    public static class UrlVariable implements Serializable {
+        private static final long serialVersionUID = 1L;
+
+        private String key;
+
+        public String getKey() {
+            return key;
+        }
+
+        public void setKey(String key) {
+            this.key = key;
+        }
+
+        @Override
+        public String toString() {
+            return (JacksonJson.toJsonString(this));
+        }
+    }
 }

gitlab4j-models/src/test/resources/org/gitlab4j/models/system-hook.json

@@ -1,9 +1,15 @@
 {
   "id": 1,
   "url": "https://gitlab.example.com/hook",
+  "name": "Hook name",
+  "description": "Hook description",
   "created_at": "2016-10-31T12:32:15.192Z",
   "push_events": true,
   "tag_push_events": false,
-  "enable_ssl_verification": true
-}
-
+  "merge_requests_events": true,
+  "repository_update_events": true,
+  "enable_ssl_verification": true,
+  "url_variables": [{
+    "key": "example"
+  }]
+}