feat: Add delete webhook functionality

Author: jepser

src/webhooks.js

@@ -9,7 +9,25 @@ export const getWebhook = (http, { uid, tag }) => {
   })
 }
 
-export const createWebhook = (http, { uid, tag, url, enable = false }) => {
+export const createWebhook = (http, args) => {
+  return createOrUpdateWebhook(http, args)
+}
+
+export const updateWebhook = (http, args) => {
+  return createOrUpdateWebhook(http, args)
+}
+
+export const deleteWebhook = (http, { uid, tag }) => {
+  return http.request({
+    method: 'delete',
+    url: `/forms/${uid}/webhooks/${tag}`
+  })
+}
+
+export const createOrUpdateWebhook = (
+  http,
+  { uid, tag, url, enable = false }
+) => {
   if (url === undefined) {
     throw `Please provide an url for ${tag}`
   }

tests/unit/webhooks.test.js

@@ -1,7 +1,11 @@
 import axios from 'axios'
 
 import { stub, spy } from 'sinon'
-import { getWebhook, createWebhook } from '../../src/webhooks'
+import {
+  getWebhook,
+  createOrUpdateWebhook,
+  deleteWebhook
+} from '../../src/webhooks'
 
 beforeEach(() => {
   stub(axios, 'request').returns({})
@@ -18,7 +22,7 @@ test('List webhooks has the correct path and method', () => {
 })
 
 test('Create a new webhooks has the correct path and method', () => {
-  createWebhook(axios, {
+  createOrUpdateWebhook(axios, {
     uid: 2,
     tag: 'test',
     url: 'http://test.com',
@@ -29,10 +33,20 @@ test('Create a new webhooks has the correct path and method', () => {
 })
 
 test('Create a new webhooks requires a url', () => {
-  expect(() => createWebhook(axios, { uid: 2, tag: 'test' })).toThrow()
+  expect(() => createOrUpdateWebhook(axios, { uid: 2, tag: 'test' })).toThrow()
 })
 
 test('Create a new webhooks sends the correct payload', () => {
-  createWebhook(axios, { uid: 2, tag: 'test', url: 'http://example.com' })
+  createOrUpdateWebhook(axios, {
+    uid: 2,
+    tag: 'test',
+    url: 'http://example.com'
+  })
   expect(axios.request.args[0][0].data.url).toBe('http://example.com')
 })
+
+test('Delete a webhook has the correct path and method', () => {
+  deleteWebhook(axios, { uid: 2, tag: 'test' })
+  expect(axios.request.args[0][0].method).toBe('delete')
+  expect(axios.request.args[0][0].url).toBe('/forms/2/webhooks/test')
+})