[Security Solution][Platform] - Bug fix when loading a saved query in detections (#114347) (#114668)

Bug in checking === for null when possibility of undefined, added tests.

Co-authored-by: Yara Tercero <yctercero@users.noreply.github.com>

Author: kibanamachine

x-pack/plugins/security_solution/public/common/components/url_state/helpers.test.ts

@@ -6,7 +6,8 @@
  */
 
 import { navTabs } from '../../../app/home/home_navigations';
-import { getTitle } from './helpers';
+import { getTitle, isQueryStateEmpty } from './helpers';
+import { CONSTANTS } from './constants';
 
 describe('Helpers Url_State', () => {
   describe('getTitle', () => {
@@ -31,4 +32,58 @@ describe('Helpers Url_State', () => {
       expect(result).toEqual('');
     });
   });
+
+  describe('isQueryStateEmpty', () => {
+    test('returns true if queryState is undefined', () => {
+      const result = isQueryStateEmpty(undefined, CONSTANTS.savedQuery);
+      expect(result).toBeTruthy();
+    });
+
+    test('returns true if queryState is null', () => {
+      const result = isQueryStateEmpty(null, CONSTANTS.savedQuery);
+      expect(result).toBeTruthy();
+    });
+
+    test('returns true if url key is "query" and queryState is empty string', () => {
+      const result = isQueryStateEmpty({}, CONSTANTS.appQuery);
+      expect(result).toBeTruthy();
+    });
+
+    test('returns false if url key is "query" and queryState is not empty', () => {
+      const result = isQueryStateEmpty(
+        { query: { query: '*:*' }, language: 'kuery' },
+        CONSTANTS.appQuery
+      );
+      expect(result).toBeFalsy();
+    });
+
+    test('returns true if url key is "filters" and queryState is empty', () => {
+      const result = isQueryStateEmpty([], CONSTANTS.filters);
+      expect(result).toBeTruthy();
+    });
+
+    test('returns false if url key is "filters" and queryState is not empty', () => {
+      const result = isQueryStateEmpty(
+        [{ query: { query: '*:*' }, meta: { key: '123' } }],
+        CONSTANTS.filters
+      );
+      expect(result).toBeFalsy();
+    });
+
+    // TODO: Is this a bug, or intended?
+    test('returns false if url key is "timeline" and queryState is empty', () => {
+      const result = isQueryStateEmpty({}, CONSTANTS.timeline);
+      expect(result).toBeFalsy();
+    });
+
+    test('returns true if url key is "timeline" and queryState id is empty string', () => {
+      const result = isQueryStateEmpty({ id: '', isOpen: true }, CONSTANTS.timeline);
+      expect(result).toBeTruthy();
+    });
+
+    test('returns false if url key is "timeline" and queryState is not empty', () => {
+      const result = isQueryStateEmpty({ id: '123', isOpen: true }, CONSTANTS.timeline);
+      expect(result).toBeFalsy();
+    });
+  });
 });

x-pack/plugins/security_solution/public/common/components/url_state/helpers.ts

@@ -199,8 +199,11 @@ export const updateTimerangeUrl = (
   return timeRange;
 };
 
-export const isQueryStateEmpty = (queryState: ValueUrlState | null, urlKey: KeyUrlState) =>
-  queryState === null ||
+export const isQueryStateEmpty = (
+  queryState: ValueUrlState | undefined | null,
+  urlKey: KeyUrlState
+): boolean =>
+  queryState == null ||
   (urlKey === CONSTANTS.appQuery && isEmpty((queryState as Query).query)) ||
   (urlKey === CONSTANTS.filters && isEmpty(queryState)) ||
   (urlKey === CONSTANTS.timeline && (queryState as TimelineUrl).id === '');