Add delete webhook via id functionality.

hub4j · Sep 22, 2021 · 541afca · 541afca
1 parent bcb71a3
commit 541afca
Show file tree

Hide file tree

Showing 4 changed files with 36 additions and 2 deletions.
diff --git a/src/main/java/org/kohsuke/github/GHHooks.java b/src/main/java/org/kohsuke/github/GHHooks.java
@@ -87,6 +87,18 @@ public GHHook createHook(String name, Map<String, String> config, Collection<GHE
             return wrap(hook);
         }
 
+        /**
+         * Deletes hook.
+         *
+         * @param id
+         *            the id
+         * @throws IOException
+         *             the io exception
+         */
+        public void deleteHook(int id) throws IOException {
+            root().createRequest().method("DELETE").withUrlPath(collection() + "/" + id).send();
+        }
+
         abstract String collection();
 
         abstract Class<? extends GHHook[]> collectionClass();

diff --git a/src/main/java/org/kohsuke/github/GHOrganization.java b/src/main/java/org/kohsuke/github/GHOrganization.java
@@ -640,6 +640,16 @@ public GHHook getHook(int id) throws IOException {
         return GHHooks.orgContext(this).getHook(id);
     }
 
+    /**
+     * Deletes hook.
+     *
+     * @param id
+     * @throws IOException
+     */
+    public void deleteHook(int id) throws IOException {
+        GHHooks.orgContext(this).deleteHook(id);
+    }
+
     /**
      * See https://api.github.com/hooks for possible names and their configuration scheme. TODO: produce type-safe
      * binding

diff --git a/src/main/java/org/kohsuke/github/GHRepository.java b/src/main/java/org/kohsuke/github/GHRepository.java
@@ -1648,6 +1648,18 @@ public GHHook getHook(int id) throws IOException {
         return GHHooks.repoContext(this, owner).getHook(id);
     }
 
+    /**
+     * Deletes hook.
+     *
+     * @param id
+     *            the id
+     * @throws IOException
+     *             the io exception
+     */
+    public void deleteHook(int id) throws IOException {
+        GHHooks.repoContext(this, owner).deleteHook(id);
+    }
+
     /**
      * Sets {@link #getCompare(String, String)} to return a {@link GHCompare} that uses a paginated commit list instead
      * of limiting to 250 results.

diff --git a/src/test/java/org/kohsuke/github/AppTest.java b/src/test/java/org/kohsuke/github/AppTest.java
@@ -637,7 +637,7 @@ public void tryHook() throws Exception {
             assertThat(hook2.getConfig().size(), equalTo(3));
             assertThat(hook2.isActive(), equalTo(true));
             hook2.ping();
-            hook2.delete();
+            r.deleteHook((int) hook.getId());
 
             hook = o.createWebHook(new URL("http://www.google.com/"));
             assertThat(hook.getName(), equalTo("web"));
@@ -653,7 +653,7 @@ public void tryHook() throws Exception {
             assertThat(hook2.getConfig().size(), equalTo(3));
             assertThat(hook2.isActive(), equalTo(true));
             hook2.ping();
-            hook2.delete();
+            o.deleteHook((int) hook.getId());
 
             // System.out.println(hook);
         } finally {