fix(ui): prevent double API call on refresh button click

handleRefreshClick is now a no-op since AutoRefresh.handleRefreshClick
already calls onRefresh() before onRefreshClicked(), which means
loadData() is already triggered via handleRefresh.

References: #RI-7761

Author: pawelangelow

redisinsight/ui/src/components/database-overview/hooks/useDatabaseOverview.spec.ts

@@ -158,8 +158,10 @@ describe('useDatabaseOverview', () => {
     expect(actions).toContainEqual(getDatabaseConfigInfo())
   })
 
-  it('should handle refresh click correctly', () => {
-    // Clear previous actions
+  it('should handle refresh click correctly (no-op)', () => {
+    // handleRefreshClick is intentionally a no-op.
+    // AutoRefresh.handleRefreshClick calls onRefresh() before onRefreshClicked(),
+    // so loadData() is already called via handleRefresh.
     mockedStore.clearActions()
 
     const data = renderHelper(mockedStore)
@@ -168,9 +170,8 @@ describe('useDatabaseOverview', () => {
     })
 
     const actions = mockedStore.getActions()
-    // handleRefreshClick now calls loadData which dispatches getDatabaseConfigInfo
-    // The connectivity error will be cleared on successful API response
-    expect(actions).toContainEqual(getDatabaseConfigInfo())
+    // handleRefreshClick should not dispatch any actions
+    expect(actions).toHaveLength(0)
   })
 
   it('should send telemetry event when auto-refresh is enabled', () => {

redisinsight/ui/src/components/database-overview/hooks/useDatabaseOverview.ts

@@ -85,9 +85,11 @@ export const useDatabaseOverview = () => {
   const handleRefresh = () => {
     loadData()
   }
-  const handleRefreshClick = () => {
-    loadData()
-  }
+  // Note: handleRefreshClick is intentionally a no-op.
+  // AutoRefresh.handleRefreshClick calls onRefresh() before onRefreshClicked(),
+  // so loadData() is already called via handleRefresh. This callback exists only
+  // for cases where additional side effects are needed on manual click.
+  const handleRefreshClick = () => {}
   const usedMemoryPercent = getUsedMemoryPercent(
     planMemoryLimit,
     usedMemory,