From 7212632a401b797137600947dfbe710f80250b8c Mon Sep 17 00:00:00 2001
From: Neil Kakkar <neilkakkar@gmail.com>
Date: Tue, 1 Oct 2024 10:26:20 +0100
Subject: [PATCH] address comments

---
 .../error-tracking/ErrorTrackingFilters.tsx      |  9 ++-------
 .../scenes/error-tracking/errorTrackingLogic.ts  |  5 -----
 .../hogql_queries/error_tracking_query_runner.py | 12 +++++++-----
 .../test/test_error_tracking_query_runner.py     | 16 ++++++++++++++++
 4 files changed, 25 insertions(+), 17 deletions(-)

diff --git a/frontend/src/scenes/error-tracking/ErrorTrackingFilters.tsx b/frontend/src/scenes/error-tracking/ErrorTrackingFilters.tsx
index d390755d830a2..c377c71d3425c 100644
--- a/frontend/src/scenes/error-tracking/ErrorTrackingFilters.tsx
+++ b/frontend/src/scenes/error-tracking/ErrorTrackingFilters.tsx
@@ -14,17 +14,12 @@ import { errorTrackingSceneLogic } from './errorTrackingSceneLogic'
 
 export const FilterGroup = (): JSX.Element => {
     const { filterGroup, filterTestAccounts, searchQuery } = useValues(errorTrackingLogic)
-    const { setFilterGroup, setFilterTestAccounts, setSearchQueryDebounced } = useActions(errorTrackingLogic)
+    const { setFilterGroup, setFilterTestAccounts, setSearchQuery } = useActions(errorTrackingLogic)
 
     return (
         <div className="flex flex-1 items-center justify-between space-x-2">
             <div className="flex items-center gap-2">
-                <LemonInput
-                    type="search"
-                    placeholder="Search..."
-                    value={searchQuery}
-                    onChange={setSearchQueryDebounced}
-                />
+                <LemonInput type="search" placeholder="Search..." value={searchQuery} onChange={setSearchQuery} />
                 <UniversalFilters
                     rootKey="error-tracking"
                     group={filterGroup}
diff --git a/frontend/src/scenes/error-tracking/errorTrackingLogic.ts b/frontend/src/scenes/error-tracking/errorTrackingLogic.ts
index f043982938640..d8c09e0289521 100644
--- a/frontend/src/scenes/error-tracking/errorTrackingLogic.ts
+++ b/frontend/src/scenes/error-tracking/errorTrackingLogic.ts
@@ -40,7 +40,6 @@ export const errorTrackingLogic = kea<errorTrackingLogicType>([
         setFilterGroup: (filterGroup: UniversalFiltersGroup) => ({ filterGroup }),
         setFilterTestAccounts: (filterTestAccounts: boolean) => ({ filterTestAccounts }),
         setSearchQuery: (searchQuery: string) => ({ searchQuery }),
-        setSearchQueryDebounced: (searchQuery: string) => ({ searchQuery }),
         setSparklineSelectedPeriod: (period: string | null) => ({ period }),
         _setSparklineOptions: (options: SparklineOption[]) => ({ options }),
     }),
@@ -125,9 +124,5 @@ export const errorTrackingLogic = kea<errorTrackingLogicType>([
                 actions._setSparklineOptions([])
             }
         },
-        setSearchQueryDebounced: async ({ searchQuery }, breakpoint) => {
-            await breakpoint(30)
-            actions.setSearchQuery(searchQuery)
-        },
     })),
 ])
diff --git a/posthog/hogql_queries/error_tracking_query_runner.py b/posthog/hogql_queries/error_tracking_query_runner.py
index e228d41c2004d..338557d0b5a49 100644
--- a/posthog/hogql_queries/error_tracking_query_runner.py
+++ b/posthog/hogql_queries/error_tracking_query_runner.py
@@ -128,6 +128,9 @@ def where(self):
 
         if self.query.searchQuery:
             # TODO: Refine this so it only searches the frames inside $exception_list
+            # TODO: Split out spaces and search for each word separately
+            # TODO: Add support for searching for specific properties
+            # TODO: Add fuzzy search support
             props_to_search = ["$exception_list", "$exception_stack_trace_raw", "$exception_type", "$exception_message"]
             or_exprs: list[ast.Expr] = []
             for prop in props_to_search:
@@ -145,12 +148,11 @@ def where(self):
                     )
                 )
 
-            if or_exprs:
-                exprs.append(
-                    ast.Or(
-                        exprs=or_exprs,
-                    )
+            exprs.append(
+                ast.Or(
+                    exprs=or_exprs,
                 )
+            )
 
         return ast.And(exprs=exprs)
 
diff --git a/posthog/hogql_queries/test/test_error_tracking_query_runner.py b/posthog/hogql_queries/test/test_error_tracking_query_runner.py
index 0860b918b2f61..910413bb3abdf 100644
--- a/posthog/hogql_queries/test/test_error_tracking_query_runner.py
+++ b/posthog/hogql_queries/test/test_error_tracking_query_runner.py
@@ -173,6 +173,22 @@ def test_search_query(self):
         self.assertEqual(results[0]["fingerprint"], ["DatabaseNotFound"])
         self.assertEqual(results[0]["occurrences"], 1)
 
+    def test_empty_search_query(self):
+        runner = ErrorTrackingQueryRunner(
+            team=self.team,
+            query=ErrorTrackingQuery(
+                kind="ErrorTrackingQuery",
+                fingerprint=None,
+                dateRange=DateRange(),
+                filterTestAccounts=False,
+                searchQuery="probs not found",
+            ),
+        )
+
+        results = self._calculate(runner)["results"]
+
+        self.assertEqual(len(results), 0)
+
     @snapshot_clickhouse_queries
     def test_fingerprints(self):
         runner = ErrorTrackingQueryRunner(