[Improvement] remove duplicate requests on overview page loading (#179)

* Bug-fix Duplicate Request on page load

Signed-off-by: Kishore Kumaar Natarajan <[email protected]>

* bug fix

Signed-off-by: Kishore Kumaar Natarajan <[email protected]>

* Added mock RetrieveQueries

Signed-off-by: Kishore Kumaar Natarajan <[email protected]>

* Added mock RetrieveQueries

Signed-off-by: Kishore Kumaar Natarajan <[email protected]>

---------

Signed-off-by: Kishore Kumaar Natarajan <[email protected]>
Co-authored-by: Kishore Kumaar Natarajan <[email protected]>

Author: KishoreKicha14

public/pages/QueryInsights/QueryInsights.test.tsx

@@ -33,6 +33,8 @@ const mockDataSourceContext = {
 
 const sampleQueries = MockQueries();
 
+const mockRetrieveQueries = jest.fn();
+
 const renderQueryInsights = () =>
   render(
     <MemoryRouter>
@@ -44,6 +46,7 @@ const renderQueryInsights = () =>
           recentlyUsedRanges={[]}
           currStart="now-15m"
           currEnd="now"
+          retrieveQueries={mockRetrieveQueries}
           // @ts-ignore
           core={mockCore}
           depsStart={{ navigation: {} }}

public/pages/QueryInsights/QueryInsights.tsx

@@ -379,6 +379,7 @@ const QueryInsights = ({
 
   const onRefresh = async ({ start, end }: { start: string; end: string }) => {
     onTimeChange({ start, end });
+    retrieveQueries(start, end);
   };
 
   const filterDuplicates = (options: any[]) =>

public/pages/TopNQueries/TopNQueries.test.tsx

@@ -209,7 +209,7 @@ describe('TopNQueries Component', () => {
     );
     // Verify that the component re-fetches data for the new time range
     await waitFor(() => {
-      expect(mockCore.http.get).toHaveBeenCalledTimes(7);
+      expect(mockCore.http.get).toHaveBeenCalledTimes(4);
       expect(mockCore.http.get).toHaveBeenCalledWith('/api/settings', expect.any(Object));
       expect(mockCore.http.get).toHaveBeenCalledWith(
         '/api/top_queries/latency',

public/pages/TopNQueries/TopNQueries.tsx

@@ -351,7 +351,6 @@ const TopNQueries = ({
     setEnd(end);
     setRecentlyUsedRanges(usedRange.length > 10 ? usedRange.slice(0, 9) : usedRange);
     retrieveConfigInfo(true);
-    retrieveQueries(start, end);
   };
 
   useEffect(() => {