remove BETA comment and fix tests

test/api/v3/integration/webhook/GET-user-webhook.test.js

@@ -1,23 +1,22 @@
 import { generateUser } from '../../../../helpers/api-integration/v3';
 
 describe('GET /user/webhook', () => {
-  let user, webhooks;
+  let user;
 
   beforeEach(async () => {
     user = await generateUser();
 
-    webhooks = [];
-    webhooks.push(await user.post('/user/webhook', {
+    await user.post('/user/webhook', {
       url: 'http://some-url.com',
       label: 'Label',
       enabled: true,
       type: 'taskActivity',
       options: { created: true, scored: true },
-    }));
-    webhooks.push(await user.post('/user/webhook', {
+    });
+    await user.post('/user/webhook', {
       url: 'http://some-other-url.com',
       enabled: false,
-    }));
+    });
 
     await user.sync();
   });
@@ -25,9 +24,10 @@ describe('GET /user/webhook', () => {
   it('returns users webhooks', async () => {
     let response = await user.get('/user/webhook');
 
-    // updatedAt times don't match for some reason, so need to omit those from comparison
-    let omitUpdatedAt = a => a.map(v => _.omit(v, 'updatedAt'));
-
-    expect(omitUpdatedAt(response)).to.eql(omitUpdatedAt(webhooks));
+    expect(response).to.eql(user.webhooks.map(w => {
+      w.createdAt = w.createdAt.toISOString();
+      w.updatedAt = w.updatedAt.toISOString();
+      return w;
+    }));
   });
 });

website/server/controllers/api-v3/webhook.js

@@ -96,7 +96,7 @@ api.addWebhook = {
 };
 
 /**
- * @api {get} /api/v3/user/webhook Get webhooks - BETA
+ * @api {get} /api/v3/user/webhook Get webhooks
  * @apiName UserGetWebhook
  * @apiGroup Webhook
  *
@@ -107,7 +107,7 @@ api.getWebhook = {
   middlewares: [authWithHeaders()],
   url: '/user/webhook',
   async handler (req, res) {
-    let user = res.locals.user;
+    const user = res.locals.user;
 
     res.respond(200, user.webhooks);
   },