From be336277558461c6578d7c36a6f497c8e1d6aa74 Mon Sep 17 00:00:00 2001 From: Vadiem Janssens Date: Mon, 22 Jun 2020 11:25:50 +0200 Subject: [PATCH] Remove webhook_key and webhook_url from JT form payload Signed-off-by: Vadiem Janssens --- .../src/screens/Template/JobTemplateAdd/JobTemplateAdd.jsx | 2 ++ .../screens/Template/JobTemplateAdd/JobTemplateAdd.test.jsx | 2 -- .../src/screens/Template/JobTemplateEdit/JobTemplateEdit.jsx | 2 ++ .../screens/Template/JobTemplateEdit/JobTemplateEdit.test.jsx | 3 ++- 4 files changed, 6 insertions(+), 3 deletions(-) diff --git a/awx/ui_next/src/screens/Template/JobTemplateAdd/JobTemplateAdd.jsx b/awx/ui_next/src/screens/Template/JobTemplateAdd/JobTemplateAdd.jsx index a0a594feca..e09a4b98f2 100644 --- a/awx/ui_next/src/screens/Template/JobTemplateAdd/JobTemplateAdd.jsx +++ b/awx/ui_next/src/screens/Template/JobTemplateAdd/JobTemplateAdd.jsx @@ -16,6 +16,8 @@ function JobTemplateAdd() { initialInstanceGroups, credentials, webhook_credential, + webhook_key, + webhook_url, ...remainingValues } = values; diff --git a/awx/ui_next/src/screens/Template/JobTemplateAdd/JobTemplateAdd.test.jsx b/awx/ui_next/src/screens/Template/JobTemplateAdd/JobTemplateAdd.test.jsx index 6da1ac5b17..4bcae68e9d 100644 --- a/awx/ui_next/src/screens/Template/JobTemplateAdd/JobTemplateAdd.test.jsx +++ b/awx/ui_next/src/screens/Template/JobTemplateAdd/JobTemplateAdd.test.jsx @@ -172,9 +172,7 @@ describe('', () => { playbook: 'Baz', inventory: 2, webhook_credential: undefined, - webhook_key: '', webhook_service: '', - webhook_url: '', }); }); diff --git a/awx/ui_next/src/screens/Template/JobTemplateEdit/JobTemplateEdit.jsx b/awx/ui_next/src/screens/Template/JobTemplateEdit/JobTemplateEdit.jsx index 8c8cb0f4b8..1e48523d85 100644 --- a/awx/ui_next/src/screens/Template/JobTemplateEdit/JobTemplateEdit.jsx +++ b/awx/ui_next/src/screens/Template/JobTemplateEdit/JobTemplateEdit.jsx @@ -101,6 +101,8 @@ class JobTemplateEdit extends Component { initialInstanceGroups, credentials, webhook_credential, + webhook_key, + webhook_url, ...remainingValues } = values; diff --git a/awx/ui_next/src/screens/Template/JobTemplateEdit/JobTemplateEdit.test.jsx b/awx/ui_next/src/screens/Template/JobTemplateEdit/JobTemplateEdit.test.jsx index c575ba3f9d..61b1467bcf 100644 --- a/awx/ui_next/src/screens/Template/JobTemplateEdit/JobTemplateEdit.test.jsx +++ b/awx/ui_next/src/screens/Template/JobTemplateEdit/JobTemplateEdit.test.jsx @@ -271,7 +271,8 @@ describe('', () => { delete expected.id; delete expected.type; delete expected.related; - expected.webhook_url = `${window.location.origin}${mockJobTemplate.related.webhook_receiver}`; + delete expected.webhook_key; + delete expected.webhook_url; expect(JobTemplatesAPI.update).toHaveBeenCalledWith(1, expected); expect(JobTemplatesAPI.disassociateLabel).toHaveBeenCalledTimes(2); expect(JobTemplatesAPI.associateLabel).toHaveBeenCalledTimes(4);