Not resetting server log level if level is defined (elastic#83651)

chrisronline · Nov 18, 2020 · 4b603da · 4b603da
1 parent 938b762
commit 4b603da
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 2 deletions.
diff --git a/.../public/application/components/builtin_action_types/server_log/server_log_params.test.tsx b/.../public/application/components/builtin_action_types/server_log/server_log_params.test.tsx
@@ -12,6 +12,7 @@ import { coreMock } from 'src/core/public/mocks';
 
 describe('ServerLogParamsFields renders', () => {
   const mocks = coreMock.createSetup();
+  const editAction = jest.fn();
 
   test('all params fields is rendered', () => {
     const actionParams = {
@@ -22,14 +23,15 @@ describe('ServerLogParamsFields renders', () => {
       <ServerLogParamsFields
         actionParams={actionParams}
         errors={{ message: [] }}
-        editAction={() => {}}
+        editAction={editAction}
         index={0}
         defaultMessage={'test default message'}
         docLinks={{ ELASTIC_WEBSITE_URL: '', DOC_LINK_VERSION: '' } as DocLinksStart}
         toastNotifications={mocks.notifications.toasts}
         http={mocks.http}
       />
     );
+    expect(editAction).not.toHaveBeenCalled();
     expect(wrapper.find('[data-test-subj="loggingLevelSelect"]').length > 0).toBeTruthy();
     expect(
       wrapper.find('[data-test-subj="loggingLevelSelect"]').first().prop('value')

diff --git a/...ns_ui/public/application/components/builtin_action_types/server_log/server_log_params.tsx b/...ns_ui/public/application/components/builtin_action_types/server_log/server_log_params.tsx
@@ -25,7 +25,9 @@ export const ServerLogParamsFields: React.FunctionComponent<ActionParamsProps<
   ];
 
   useEffect(() => {
-    editAction('level', 'info', index);
+    if (!actionParams?.level) {
+      editAction('level', 'info', index);
+    }
     // eslint-disable-next-line react-hooks/exhaustive-deps
   }, []);