Fix duplicated requests on refreshing the overview (#138) (#141)

* Duplicated requests on refreshing the overview

* updated unit test

---------

(cherry picked from commit c8d7ff5)

Signed-off-by: Kishore Kumaar Natarajan <[email protected]>
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
Co-authored-by: Kishore Kumaar Natarajan <[email protected]>
(cherry picked from commit 2c39703)
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>

Author: github-actions[bot]

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(16);
+      expect(mockCore.http.get).toHaveBeenCalledTimes(7);
       expect(mockCore.http.get).toHaveBeenCalledWith('/api/settings', expect.any(Object));
       expect(mockCore.http.get).toHaveBeenCalledWith(
         '/api/top_queries/latency',

public/pages/TopNQueries/TopNQueries.tsx

@@ -163,6 +163,8 @@ const TopNQueries = ({
   // TODO: refactor retrieveQueries and retrieveConfigInfo into a Util function
   const retrieveQueries = useCallback(
     async (start: string, end: string) => {
+      if (loading) return;
+      setLoading(true);
       const nullResponse = { response: { top_queries: [] } };
       const apiParams = {
         query: {
@@ -190,7 +192,6 @@ const TopNQueries = ({
         }
       };
       try {
-        setLoading(true);
         const respLatency = latencySettings.isEnabled
           ? await fetchMetric('/api/top_queries/latency')
           : nullResponse;