Skip to content

Commit 50bc2e9

Browse files
author
Veijo Pesonen
committed
Bugfix - drops the assumption that there is more than SSID in vicinity
1 parent 7995e8b commit 50bc2e9

File tree

3 files changed

+7
-7
lines changed

3 files changed

+7
-7
lines changed

TESTS/network/wifi/main.cpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -85,7 +85,7 @@ Case cases[] = {
8585
Case("WIFI-CONNECT-DISCONNECT-REPEAT", wifi_connect_disconnect_repeat),
8686
#endif
8787
Case("WIFI-SCAN-NULL", wifi_scan_null),
88-
#if defined(MBED_CONF_APP_WIFI_SECURE_SSID) && defined(MBED_CONF_APP_WIFI_UNSECURE_SSID)
88+
#if defined(MBED_CONF_APP_WIFI_SECURE_SSID) || defined(MBED_CONF_APP_WIFI_UNSECURE_SSID)
8989
Case("WIFI-SCAN", wifi_scan),
9090
#endif
9191
};

TESTS/network/wifi/wifi_scan.cpp

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -24,16 +24,14 @@
2424

2525
using namespace utest::v1;
2626

27-
#if defined(MBED_CONF_APP_WIFI_SECURE_SSID) && defined(MBED_CONF_APP_WIFI_UNSECURE_SSID)
28-
2927
void wifi_scan(void)
3028
{
3129
WiFiInterface *wifi = get_interface();
3230

3331
WiFiAccessPoint ap[MBED_CONF_APP_MAX_SCAN_SIZE];
3432

3533
int size = wifi->scan(ap, MBED_CONF_APP_MAX_SCAN_SIZE);
36-
TEST_ASSERT(size >= 2);
34+
TEST_ASSERT(size >= 1);
3735

3836
bool secure_found = false;
3937
bool unsecure_found = false;
@@ -49,17 +47,19 @@ void wifi_scan(void)
4947
nsapi_security_t security = ap[i].get_security();
5048
int8_t rssi = ap[i].get_rssi();
5149
TEST_ASSERT_INT8_WITHIN(-10, -100, rssi);
50+
#if defined(MBED_CONF_APP_WIFI_SECURE_SSID)
5251
if (strcmp(MBED_CONF_APP_WIFI_SECURE_SSID, ssid) == 0) {
5352
secure_found = true;
5453
TEST_ASSERT_EQUAL_INT(get_security(), security);
5554
}
55+
#endif
56+
#if defined(MBED_CONF_APP_WIFI_UNSECURE_SSID)
5657
if (strcmp(MBED_CONF_APP_WIFI_UNSECURE_SSID, ssid) == 0) {
5758
unsecure_found = true;
5859
TEST_ASSERT_EQUAL_INT(NSAPI_SECURITY_NONE, security);
5960
}
61+
#endif
6062
}
6163
// Finding one SSID is enough
6264
TEST_ASSERT_TRUE(secure_found || unsecure_found);
6365
}
64-
65-
#endif // defined(MBED_CONF_APP_WIFI_SECURE_SSID) && defined(MBED_CONF_APP_WIFI_UNSECURE_SSID)

TESTS/network/wifi/wifi_scan_null.cpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -26,6 +26,6 @@ using namespace utest::v1;
2626
void wifi_scan_null(void)
2727
{
2828
WiFiInterface *wifi = get_interface();
29-
TEST_ASSERT(wifi->scan(NULL, 0) >= 2);
29+
TEST_ASSERT(wifi->scan(NULL, 0) >= 1);
3030
}
3131

0 commit comments

Comments
 (0)