From edab762cddb7b028993ac7c5565d0d338a931cb3 Mon Sep 17 00:00:00 2001 From: Ramya Subramanyam Date: Wed, 13 Aug 2025 11:31:28 +0530 Subject: [PATCH 1/5] src/corelibs/wifi: Remove while loops to run the tests on CI/CD.-s --- src/corelibs/wifi/test_wifi_ap.cpp | 4 ---- src/corelibs/wifi/test_wifi_client.cpp | 8 -------- src/corelibs/wifi/test_wifi_extras.cpp | 8 -------- src/corelibs/wifi/test_wifi_server.cpp | 8 -------- src/corelibs/wifi/test_wifi_sta.cpp | 1 - src/corelibs/wifi/test_wifi_udp_client.cpp | 1 - src/corelibs/wifi/test_wifi_udp_server.cpp | 1 - 7 files changed, 31 deletions(-) diff --git a/src/corelibs/wifi/test_wifi_ap.cpp b/src/corelibs/wifi/test_wifi_ap.cpp index 648e645..82ac13b 100644 --- a/src/corelibs/wifi/test_wifi_ap.cpp +++ b/src/corelibs/wifi/test_wifi_ap.cpp @@ -134,10 +134,6 @@ TEST_GROUP_RUNNER(wifi_ap) { RUN_TEST_CASE(wifi_ap, check_ssid); RUN_TEST_CASE(wifi_ap, check_bssid); RUN_TEST_CASE(wifi_ap, check_encryption_type); - /* Wait forever for now. */ - /* This allows to check the sta - test manually. */ - while(true) {}; RUN_TEST_CASE(wifi_ap, disconnect); RUN_TEST_CASE(wifi_ap, wifi_end); } diff --git a/src/corelibs/wifi/test_wifi_client.cpp b/src/corelibs/wifi/test_wifi_client.cpp index ee9276c..f554a18 100644 --- a/src/corelibs/wifi/test_wifi_client.cpp +++ b/src/corelibs/wifi/test_wifi_client.cpp @@ -201,12 +201,4 @@ TEST_GROUP_RUNNER(wifi_client) { RUN_TEST_CASE(wifi_client, client_server_disconnect); RUN_TEST_CASE(wifi_client, wifi_disconnect); RUN_TEST_CASE(wifi_client, wifi_end); - /** - * TODO: Remove when this is added to cicd - * and more tests are added to the suite together with the - * WiFi ones. - * For now it is more controllable like this to run tests - * manually. - */ - while(true) {}; } \ No newline at end of file diff --git a/src/corelibs/wifi/test_wifi_extras.cpp b/src/corelibs/wifi/test_wifi_extras.cpp index 167d30f..03b4461 100644 --- a/src/corelibs/wifi/test_wifi_extras.cpp +++ b/src/corelibs/wifi/test_wifi_extras.cpp @@ -115,12 +115,4 @@ TEST_GROUP_RUNNER(wifi_extras) { RUN_TEST_CASE(wifi_extras, check_set_dns); RUN_TEST_CASE(wifi_extras, wifi_disconnect); RUN_TEST_CASE(wifi_extras, wifi_end); - /** - * TODO: Remove when this is added to cicd - * and more tests are added to the suite together with the - * WiFi ones. - * For now it is more controllable like this to run tests - * manually. - */ - while(true) {}; } \ No newline at end of file diff --git a/src/corelibs/wifi/test_wifi_server.cpp b/src/corelibs/wifi/test_wifi_server.cpp index c872b48..771461f 100644 --- a/src/corelibs/wifi/test_wifi_server.cpp +++ b/src/corelibs/wifi/test_wifi_server.cpp @@ -180,12 +180,4 @@ TEST_GROUP_RUNNER(wifi_server) { RUN_TEST_CASE(wifi_server, server_end); RUN_TEST_CASE(wifi_server, wifi_disconnect); RUN_TEST_CASE(wifi_server, wifi_end); - /** - * TODO: Remove when this is added to cicd - * and more tests are added to the suite together with the - * WiFi ones. - * For now it is more controllable like this to run tests - * manually. - */ - while(true) {}; } \ No newline at end of file diff --git a/src/corelibs/wifi/test_wifi_sta.cpp b/src/corelibs/wifi/test_wifi_sta.cpp index c9abf9a..41bed71 100644 --- a/src/corelibs/wifi/test_wifi_sta.cpp +++ b/src/corelibs/wifi/test_wifi_sta.cpp @@ -203,7 +203,6 @@ TEST_GROUP_RUNNER(wifi_sta) { RUN_TEST_CASE(wifi_sta, check_encryption_type); RUN_TEST_CASE(wifi_sta, disconnect); RUN_TEST_CASE(wifi_sta, wifi_end); - while(true) {}; } diff --git a/src/corelibs/wifi/test_wifi_udp_client.cpp b/src/corelibs/wifi/test_wifi_udp_client.cpp index 39798ed..1c129a2 100644 --- a/src/corelibs/wifi/test_wifi_udp_client.cpp +++ b/src/corelibs/wifi/test_wifi_udp_client.cpp @@ -174,5 +174,4 @@ TEST_GROUP_RUNNER(wifi_udp_client) { RUN_TEST_CASE(wifi_udp_client, udp_client_end); RUN_TEST_CASE(wifi_udp_client, wifi_disconnect); RUN_TEST_CASE(wifi_udp_client, wifi_end); - while (true) {}; // Keep the client running } \ No newline at end of file diff --git a/src/corelibs/wifi/test_wifi_udp_server.cpp b/src/corelibs/wifi/test_wifi_udp_server.cpp index 1f35773..8741569 100644 --- a/src/corelibs/wifi/test_wifi_udp_server.cpp +++ b/src/corelibs/wifi/test_wifi_udp_server.cpp @@ -168,5 +168,4 @@ TEST_GROUP_RUNNER(wifi_udp_server) { RUN_TEST_CASE(wifi_udp_server, udp_server_end); RUN_TEST_CASE(wifi_udp_server, wifi_disconnect); RUN_TEST_CASE(wifi_udp_server, wifi_end); - while (true) {}; // Keep the server running } \ No newline at end of file From 9c25a58bf403263ee1f5487bc9308028ee8b7593 Mon Sep 17 00:00:00 2001 From: Ramya Subramanyam Date: Wed, 13 Aug 2025 11:32:23 +0530 Subject: [PATCH 2/5] src/corelibs/wire: Disbale tracing.-s --- src/corelibs/wire/test_wire_connected1_pingpong.cpp | 2 +- src/corelibs/wire/test_wire_connected2_masterpingpong.cpp | 2 +- src/corelibs/wire/test_wire_connected2_slavepingpong.cpp | 2 +- 3 files changed, 3 insertions(+), 3 deletions(-) diff --git a/src/corelibs/wire/test_wire_connected1_pingpong.cpp b/src/corelibs/wire/test_wire_connected1_pingpong.cpp index 7d86d82..07e4797 100644 --- a/src/corelibs/wire/test_wire_connected1_pingpong.cpp +++ b/src/corelibs/wire/test_wire_connected1_pingpong.cpp @@ -7,7 +7,7 @@ #include // defines -#define TRACE_OUTPUT +#define TRACE_OUTPUT 0 // variables const static uint8_t slaveAddress = 8U; diff --git a/src/corelibs/wire/test_wire_connected2_masterpingpong.cpp b/src/corelibs/wire/test_wire_connected2_masterpingpong.cpp index c117dbe..27a6bd2 100644 --- a/src/corelibs/wire/test_wire_connected2_masterpingpong.cpp +++ b/src/corelibs/wire/test_wire_connected2_masterpingpong.cpp @@ -7,7 +7,7 @@ #include // defines -#define TRACE_OUTPUT +#define TRACE_OUTPUT 0 // variables const static uint8_t slaveAddress = 8U; diff --git a/src/corelibs/wire/test_wire_connected2_slavepingpong.cpp b/src/corelibs/wire/test_wire_connected2_slavepingpong.cpp index b90a182..f4e35ab 100644 --- a/src/corelibs/wire/test_wire_connected2_slavepingpong.cpp +++ b/src/corelibs/wire/test_wire_connected2_slavepingpong.cpp @@ -7,7 +7,7 @@ #include // defines -#define TRACE_OUTPUT +#define TRACE_OUTPUT 0 // variables const static uint8_t slaveAddress = 8U; From 3cb69a3ddb633d7cd15e3480af1f80722d84419c Mon Sep 17 00:00:00 2001 From: Ramya Subramanyam Date: Mon, 18 Aug 2025 13:52:32 +0530 Subject: [PATCH 3/5] Makefile: Add UDP tests. Signed-off-by: Ramya Subramanyam --- Makefile | 2 ++ 1 file changed, 2 insertions(+) diff --git a/Makefile b/Makefile index 70fb47e..aed2762 100644 --- a/Makefile +++ b/Makefile @@ -97,6 +97,8 @@ test_wifi_sta: TESTS=-DTEST_WIFI_STA test_wifi_ap: TESTS=-DTEST_WIFI_AP test_wifi_client: TESTS=-DTEST_WIFI_CLIENT test_wifi_server: TESTS=-DTEST_WIFI_SERVER +test_wifi_udp_client: TESTS=-DTEST_WIFI_UDP_CLIENT +test_wifi_udp_server: TESTS=-DTEST_WIFI_UDP_SERVER test_wifi_extras: TESTS=-DTEST_WIFI_EXTRAS test_wifi_exceptions: TESTS=-DTEST_WIFI_EXCEPTIONS From 92371c96c3d4cd85418d1236bde327015387428d Mon Sep 17 00:00:00 2001 From: "Subramanyam Ramya (DES DOS SW ESW EPE)" Date: Tue, 19 Aug 2025 21:11:37 +0530 Subject: [PATCH 4/5] src/corelibs/wire: Disable tracing. Signed-off-by: Subramanyam Ramya (DES DOS SW ESW EPE) --- src/corelibs/wire/test_wire_connected1_pingpong.cpp | 1 - src/corelibs/wire/test_wire_connected2_masterpingpong.cpp | 1 - src/corelibs/wire/test_wire_connected2_slavepingpong.cpp | 1 - 3 files changed, 3 deletions(-) diff --git a/src/corelibs/wire/test_wire_connected1_pingpong.cpp b/src/corelibs/wire/test_wire_connected1_pingpong.cpp index 07e4797..7e88c8d 100644 --- a/src/corelibs/wire/test_wire_connected1_pingpong.cpp +++ b/src/corelibs/wire/test_wire_connected1_pingpong.cpp @@ -7,7 +7,6 @@ #include // defines -#define TRACE_OUTPUT 0 // variables const static uint8_t slaveAddress = 8U; diff --git a/src/corelibs/wire/test_wire_connected2_masterpingpong.cpp b/src/corelibs/wire/test_wire_connected2_masterpingpong.cpp index 27a6bd2..928d2da 100644 --- a/src/corelibs/wire/test_wire_connected2_masterpingpong.cpp +++ b/src/corelibs/wire/test_wire_connected2_masterpingpong.cpp @@ -7,7 +7,6 @@ #include // defines -#define TRACE_OUTPUT 0 // variables const static uint8_t slaveAddress = 8U; diff --git a/src/corelibs/wire/test_wire_connected2_slavepingpong.cpp b/src/corelibs/wire/test_wire_connected2_slavepingpong.cpp index f4e35ab..8020a42 100644 --- a/src/corelibs/wire/test_wire_connected2_slavepingpong.cpp +++ b/src/corelibs/wire/test_wire_connected2_slavepingpong.cpp @@ -7,7 +7,6 @@ #include // defines -#define TRACE_OUTPUT 0 // variables const static uint8_t slaveAddress = 8U; From ac43c508104be5072d170adac4796f1aa749c52d Mon Sep 17 00:00:00 2001 From: "Subramanyam Ramya (DES DOS SW ESW EPE)" Date: Tue, 19 Aug 2025 21:12:33 +0530 Subject: [PATCH 5/5] src/corelibs/wifi: Disable scan network test. Signed-off-by: Subramanyam Ramya (DES DOS SW ESW EPE) --- src/corelibs/wifi/test_wifi_sta.cpp | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/src/corelibs/wifi/test_wifi_sta.cpp b/src/corelibs/wifi/test_wifi_sta.cpp index 41bed71..7c3649b 100644 --- a/src/corelibs/wifi/test_wifi_sta.cpp +++ b/src/corelibs/wifi/test_wifi_sta.cpp @@ -188,7 +188,8 @@ TEST_IFX(wifi_sta, wifi_end) { } TEST_GROUP_RUNNER(wifi_sta) { - RUN_TEST_CASE(wifi_sta, scan_networks); + // TODO: Fix the test "scan_networks" which hangs during end -> cy_wcm_deinit when running on HIL setup + // RUN_TEST_CASE(wifi_sta, scan_networks); RUN_TEST_CASE(wifi_sta, config_ip_static); RUN_TEST_CASE(wifi_sta, connect_to_ap); RUN_TEST_CASE(wifi_sta, is_status_connected);