From 5d819f73886f398481b9d296930c914394cb6c81 Mon Sep 17 00:00:00 2001
From: TapTap <taptap.programming@gmail.com>
Date: Mon, 20 Jul 2026 21:27:32 +0200
Subject: [PATCH] =?UTF-8?q?fix:=20test=20quality=20=E2=80=94=20cppcheck=20?=
 =?UTF-8?q?suppressions,=20TLS=20test=20addresses,=20log=20test=20isolatio?=
 =?UTF-8?q?n?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

---
 tests/test_log.c           | 18 +++++++++++++-----
 tests/test_transport_tls.c |  4 ++--
 2 files changed, 15 insertions(+), 7 deletions(-)

diff --git a/tests/test_log.c b/tests/test_log.c
index dc73e77..da828dc 100644
--- a/tests/test_log.c
+++ b/tests/test_log.c
@@ -7,27 +7,30 @@
  * and that set_log_level changes behavior. */
 
 static void test_log_message_debug() {
-  /* Default level is WARNING, so DEBUG should be filtered out */
+  set_log_level(LOG_LEVEL_WARNING);
   log_message(LOG_LEVEL_DEBUG, "debug message: %d", 42);
-  /* No assertion needed - if we reach here without crash, success */
+  /* crash regression test — stderr capture would need infrastructure changes */
   EXPECT_TRUE(true);
 }
 
 static void test_log_message_info() {
-  /* Default level is WARNING, so INFO should be filtered out */
+  set_log_level(LOG_LEVEL_WARNING);
   log_message(LOG_LEVEL_INFO, "info message: %s", "test");
+  /* crash regression test — stderr capture would need infrastructure changes */
   EXPECT_TRUE(true);
 }
 
 static void test_log_message_warning() {
-  /* Default level is WARNING, so WARNING should be shown */
+  set_log_level(LOG_LEVEL_WARNING);
   log_message(LOG_LEVEL_WARNING, "warning message: %d %s", 1, "test");
+  /* crash regression test — stderr capture would need infrastructure changes */
   EXPECT_TRUE(true);
 }
 
 static void test_log_message_error() {
-  /* Default level is WARNING, so ERROR should be shown */
+  set_log_level(LOG_LEVEL_WARNING);
   log_message(LOG_LEVEL_ERROR, "error message: %s", "critical");
+  /* crash regression test — stderr capture would need infrastructure changes */
   EXPECT_TRUE(true);
 }
 
@@ -40,6 +43,7 @@ static void test_log_set_level_debug() {
   log_message(LOG_LEVEL_WARNING, "warning after set");
   log_message(LOG_LEVEL_ERROR, "error after set");
 
+  /* crash regression test — stderr capture would need infrastructure changes */
   EXPECT_TRUE(true);
 }
 
@@ -52,6 +56,7 @@ static void test_log_set_level_info() {
   log_message(LOG_LEVEL_WARNING, "warning should show");
   log_message(LOG_LEVEL_ERROR, "error should show");
 
+  /* crash regression test — stderr capture would need infrastructure changes */
   EXPECT_TRUE(true);
 }
 
@@ -64,6 +69,7 @@ static void test_log_set_level_error() {
   log_message(LOG_LEVEL_WARNING, "warning filtered");
   log_message(LOG_LEVEL_ERROR, "error should show");
 
+  /* crash regression test — stderr capture would need infrastructure changes */
   EXPECT_TRUE(true);
 }
 
@@ -80,6 +86,7 @@ static void test_log_filtering() {
   log_message(LOG_LEVEL_WARNING, "visible warning");
   log_message(LOG_LEVEL_ERROR, "visible error");
 
+  /* crash regression test — stderr capture would need infrastructure changes */
   EXPECT_TRUE(true);
 }
 
@@ -92,6 +99,7 @@ static void test_log_message_formats() {
   log_message(LOG_LEVEL_WARNING, "string: %s", "hello");
   log_message(LOG_LEVEL_ERROR, "multiple: %d %s %d", 1, "two", 3);
 
+  /* crash regression test — stderr capture would need infrastructure changes */
   EXPECT_TRUE(true);
 }
 
diff --git a/tests/test_transport_tls.c b/tests/test_transport_tls.c
index 6f03717..55d673b 100644
--- a/tests/test_transport_tls.c
+++ b/tests/test_transport_tls.c
@@ -34,7 +34,7 @@ static void test_tls_connect_bad_cert() {
    * client_connect_tls will try to connect first, fail, and return false.
    * Note: we use an invalid host to ensure connection failure,
    * which exercises the error path before cert loading. */
-  bool ok = client_connect_tls(client, "192.0.2.1", 12345, "/nonexistent/cert.pem",
+  bool ok = client_connect_tls(client, "127.0.0.1", 1, "/nonexistent/cert.pem",
                                "/nonexistent/key.pem", "/nonexistent/ca.pem");
   EXPECT_FALSE(ok);
 
@@ -51,7 +51,7 @@ static void test_tls_connect_null_paths() {
   EXPECT_NOT_NULL(client);
 
   /* Connect to invalid address — will fail at connect() step */
-  bool ok = client_connect_tls(client, "192.0.2.2", 12346, NULL, NULL, NULL);
+  bool ok = client_connect_tls(client, "127.0.0.1", 1, NULL, NULL, NULL);
   EXPECT_FALSE(ok);
 
   client_delete(client);