tests: coap_client: wait enough for requests to be unallocated

As confirmable requests will stay allocated for (3 * ACK_TIMEOUT), we
need to adjust the timings so all requests are unallocated by the end
of the test.

Signed-off-by: Francois Gervais <francoisgervais@gmail.com>
This commit is contained in:
Francois Gervais 2024-09-24 18:51:56 -04:00 committed by Mahesh Mahadevan
commit 13cad59542
2 changed files with 5 additions and 15 deletions

View file

@ -26,7 +26,7 @@ add_compile_definitions(CONFIG_COAP_CLIENT_MESSAGE_HEADER_SIZE=48)
add_compile_definitions(CONFIG_COAP_CLIENT_STACK_SIZE=1024)
add_compile_definitions(CONFIG_COAP_CLIENT_THREAD_PRIORITY=10)
add_compile_definitions(CONFIG_COAP_LOG_LEVEL=4)
add_compile_definitions(CONFIG_COAP_INIT_ACK_TIMEOUT_MS=200)
add_compile_definitions(CONFIG_COAP_INIT_ACK_TIMEOUT_MS=10)
add_compile_definitions(CONFIG_COAP_CLIENT_MAX_REQUESTS=2)
add_compile_definitions(CONFIG_COAP_CLIENT_MAX_INSTANCES=2)
add_compile_definitions(CONFIG_COAP_MAX_RETRANSMIT=4)

View file

@ -377,7 +377,6 @@ ZTEST(coap_client, test_get_request)
zassert_true(ret >= 0, "Sending request failed, %d", ret);
set_socket_events(ZSOCK_POLLIN);
k_sleep(K_MSEC(5));
k_sleep(K_MSEC(100));
zassert_equal(last_response_code, COAP_RESPONSE_CODE_OK, "Unexpected response");
}
@ -406,7 +405,7 @@ ZTEST(coap_client, test_resend_request)
LOG_INF("Send request");
ret = coap_client_req(&client, 0, &address, &client_request, NULL);
zassert_true(ret >= 0, "Sending request failed, %d", ret);
k_sleep(K_MSEC(300));
k_sleep(K_MSEC(15));
set_socket_events(ZSOCK_POLLIN);
k_sleep(K_MSEC(100));
@ -440,7 +439,6 @@ ZTEST(coap_client, test_echo_option)
zassert_true(ret >= 0, "Sending request failed, %d", ret);
set_socket_events(ZSOCK_POLLIN);
k_sleep(K_MSEC(5));
k_sleep(K_MSEC(100));
zassert_equal(last_response_code, COAP_RESPONSE_CODE_OK, "Unexpected response");
}
@ -471,7 +469,6 @@ ZTEST(coap_client, test_echo_option_next_req)
zassert_true(ret >= 0, "Sending request failed, %d", ret);
set_socket_events(ZSOCK_POLLIN);
k_sleep(K_MSEC(5));
k_sleep(K_MSEC(100));
zassert_equal(last_response_code, COAP_RESPONSE_CODE_OK, "Unexpected response");
@ -486,7 +483,6 @@ ZTEST(coap_client, test_echo_option_next_req)
zassert_true(ret >= 0, "Sending request failed, %d", ret);
set_socket_events(ZSOCK_POLLIN);
k_sleep(K_MSEC(5));
k_sleep(K_MSEC(100));
zassert_equal(last_response_code, COAP_RESPONSE_CODE_OK, "Unexpected response");
}
@ -540,7 +536,6 @@ ZTEST(coap_client, test_send_large_data)
zassert_true(ret >= 0, "Sending request failed, %d", ret);
set_socket_events(ZSOCK_POLLIN);
k_sleep(K_MSEC(5));
k_sleep(K_MSEC(100));
zassert_equal(last_response_code, COAP_RESPONSE_CODE_OK, "Unexpected response");
}
@ -574,8 +569,8 @@ ZTEST(coap_client, test_no_response)
ret = coap_client_req(&client, 0, &address, &client_request, &params);
zassert_true(ret >= 0, "Sending request failed, %d", ret);
k_sleep(K_MSEC(300));
k_sleep(K_MSEC(700));
zassert_equal(last_response_code, -ETIMEDOUT, "Unexpected response");
}
@ -605,9 +600,7 @@ ZTEST(coap_client, test_separate_response)
zassert_true(ret >= 0, "Sending request failed, %d", ret);
set_socket_events(ZSOCK_POLLIN);
k_sleep(K_MSEC(5));
k_sleep(K_MSEC(100));
zassert_equal(last_response_code, COAP_RESPONSE_CODE_OK, "Unexpected response");
}
@ -638,11 +631,9 @@ ZTEST(coap_client, test_multiple_requests)
ret = coap_client_req(&client, 0, &address, &client_request, NULL);
zassert_true(ret >= 0, "Sending request failed, %d", ret);
k_sleep(K_MSEC(5));
k_sleep(K_MSEC(100));
zassert_equal(last_response_code, COAP_RESPONSE_CODE_OK, "Unexpected response");
k_sleep(K_MSEC(5));
k_sleep(K_MSEC(100));
zassert_equal(last_response_code, COAP_RESPONSE_CODE_OK, "Unexpected response");
}
@ -676,9 +667,8 @@ ZTEST(coap_client, test_unmatching_tokens)
zassert_true(ret >= 0, "Sending request failed, %d", ret);
set_socket_events(ZSOCK_POLLIN);
k_sleep(K_MSEC(1));
k_sleep(K_MSEC(1));
k_sleep(K_MSEC(2));
clear_socket_events();
k_sleep(K_MSEC(500));
k_sleep(K_MSEC(700));
zassert_equal(last_response_code, -ETIMEDOUT, "Unexpected response");
}