From 68d8056d2e66b9fbdc52f0a18c3e3f34c06dc388 Mon Sep 17 00:00:00 2001
From: Lukas <lukas.pupkalipinski@lpl-mind.de>
Date: Sat, 14 Mar 2026 17:58:35 +0100
Subject: [PATCH] fix: resolve ESLint no-confusing-void-expression in LogTab
 tests

---
 .../config/__tests__/LogTab.test.tsx          | 32 +++++++++----------
 1 file changed, 16 insertions(+), 16 deletions(-)

diff --git a/frontend/src/components/config/__tests__/LogTab.test.tsx b/frontend/src/components/config/__tests__/LogTab.test.tsx
index 8667011..9b62bc8 100644
--- a/frontend/src/components/config/__tests__/LogTab.test.tsx
+++ b/frontend/src/components/config/__tests__/LogTab.test.tsx
@@ -102,7 +102,7 @@ describe("LogTab", () => {
 
     renderTab();
 
-    await waitFor(() => expect(screen.queryByText(/loading log viewer/i)).toBeNull());
+    await waitFor(() => { expect(screen.queryByText(/loading log viewer/i)).toBeNull(); });
 
     expect(screen.getByText("Running")).toBeInTheDocument();
     expect(screen.getByText("1.0.2")).toBeInTheDocument();
@@ -116,7 +116,7 @@ describe("LogTab", () => {
 
     renderTab();
 
-    await waitFor(() => expect(screen.queryByText(/loading log viewer/i)).toBeNull());
+    await waitFor(() => { expect(screen.queryByText(/loading log viewer/i)).toBeNull(); });
 
     expect(screen.getByText("Offline")).toBeInTheDocument();
     expect(screen.getByText(/not running or unreachable/i)).toBeInTheDocument();
@@ -128,9 +128,9 @@ describe("LogTab", () => {
 
     renderTab();
 
-    await waitFor(() =>
-      expect(screen.getByText(/2025-01-01 12:00:00 INFO/)).toBeInTheDocument(),
-    );
+    await waitFor(() => {
+      expect(screen.getByText(/2025-01-01 12:00:00 INFO/)).toBeInTheDocument();
+    });
 
     expect(screen.getByText(/2025-01-01 12:00:01 WARNING/)).toBeInTheDocument();
     expect(screen.getByText(/2025-01-01 12:00:02 ERROR/)).toBeInTheDocument();
@@ -142,9 +142,9 @@ describe("LogTab", () => {
 
     renderTab();
 
-    await waitFor(() =>
-      expect(screen.getByText(/fail2ban is logging to/i)).toBeInTheDocument(),
-    );
+    await waitFor(() => {
+      expect(screen.getByText(/fail2ban is logging to/i)).toBeInTheDocument();
+    });
 
     expect(screen.getByText(/STDOUT/)).toBeInTheDocument();
     expect(screen.queryByText(/Refresh/)).toBeNull();
@@ -156,9 +156,9 @@ describe("LogTab", () => {
 
     renderTab();
 
-    await waitFor(() =>
-      expect(screen.getByText(/no log entries found/i)).toBeInTheDocument(),
-    );
+    await waitFor(() => {
+      expect(screen.getByText(/no log entries found/i)).toBeInTheDocument();
+    });
   });
 
   it("shows truncation notice when total_lines > lines.length", async () => {
@@ -167,9 +167,9 @@ describe("LogTab", () => {
 
     renderTab();
 
-    await waitFor(() =>
-      expect(screen.getByText(/showing last/i)).toBeInTheDocument(),
-    );
+    await waitFor(() => {
+      expect(screen.getByText(/showing last/i)).toBeInTheDocument();
+    });
   });
 
   it("calls fetchFail2BanLog again on Refresh button click", async () => {
@@ -179,11 +179,11 @@ describe("LogTab", () => {
     const user = userEvent.setup();
     renderTab();
 
-    await waitFor(() => expect(screen.getByText(/Refresh/)).toBeInTheDocument());
+    await waitFor(() => { expect(screen.getByText(/Refresh/)).toBeInTheDocument(); });
 
     const refreshBtn = screen.getByRole("button", { name: /refresh/i });
     await user.click(refreshBtn);
 
-    await waitFor(() => expect(mockFetchLog).toHaveBeenCalledTimes(2));
+    await waitFor(() => { expect(mockFetchLog).toHaveBeenCalledTimes(2); });
   });
 });