Skip to content

Enable wifi tests hil #69

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 5 commits into from
Aug 21, 2025
Merged

Enable wifi tests hil #69

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
5 commits
Select commit Hold shift + click to select a range
edab762
src/corelibs/wifi: Remove while loops to run the tests on CI/CD.-s
ramya-subramanyam Aug 13, 2025
9c25a58
src/corelibs/wire: Disbale tracing.-s
ramya-subramanyam Aug 13, 2025
3cb69a3
Makefile: Add UDP tests.
ramya-subramanyam Aug 18, 2025
92371c9
src/corelibs/wire: Disable tracing.
ramya-subramanyam Aug 19, 2025
ac43c50
src/corelibs/wifi: Disable scan network test.
ramya-subramanyam Aug 19, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 2 additions & 0 deletions Makefile
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -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

Expand Down
4 changes: 0 additions & 4 deletions src/corelibs/wifi/test_wifi_ap.cpp
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -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);
Copy link
Member

@jaenrig-ifx jaenrig-ifx Aug 20, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is not required to be disabled then?

Copy link
Collaborator Author

@ramya-subramanyam ramya-subramanyam Aug 20, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No it was not needed. It works now without this while. If we see any issue then we can perhaps add a small delay before disconnecting.

RUN_TEST_CASE(wifi_ap, wifi_end);
}
Expand Down
8 changes: 0 additions & 8 deletions src/corelibs/wifi/test_wifi_client.cpp
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -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) {};
}
8 changes: 0 additions & 8 deletions src/corelibs/wifi/test_wifi_extras.cpp
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -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) {};
}
8 changes: 0 additions & 8 deletions src/corelibs/wifi/test_wifi_server.cpp
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -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) {};
}
4 changes: 2 additions & 2 deletions src/corelibs/wifi/test_wifi_sta.cpp
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -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);
Expand All @@ -203,7 +204,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) {};
}


1 change: 0 additions & 1 deletion src/corelibs/wifi/test_wifi_udp_client.cpp
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -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
}
1 change: 0 additions & 1 deletion src/corelibs/wifi/test_wifi_udp_server.cpp
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -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
}
1 change: 0 additions & 1 deletion src/corelibs/wire/test_wire_connected1_pingpong.cpp
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,6 @@
#include <Wire.h>

// defines
#define TRACE_OUTPUT

// variables
const static uint8_t slaveAddress = 8U;
Expand Down
1 change: 0 additions & 1 deletion src/corelibs/wire/test_wire_connected2_masterpingpong.cpp
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,6 @@
#include <Wire.h>

// defines
#define TRACE_OUTPUT

// variables
const static uint8_t slaveAddress = 8U;
Expand Down
1 change: 0 additions & 1 deletion src/corelibs/wire/test_wire_connected2_slavepingpong.cpp
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,6 @@
#include <Wire.h>

// defines
#define TRACE_OUTPUT

// variables
const static uint8_t slaveAddress = 8U;
Expand Down