From 3d185ec4854bfe85aa6ef0ff2073e33a5fcaae5f Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Javier=20Jaramago=20Fern=C3=A1ndez?= Date: Fri, 17 May 2024 01:59:14 +0200 Subject: [PATCH] Further increase timeouts of 'restapi_return_codes-t' for ASAN builds Added doc elaborating on the limitation. --- test/tap/tests/reg_test_3223-restapi_return_codes-t.cpp | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/test/tap/tests/reg_test_3223-restapi_return_codes-t.cpp b/test/tap/tests/reg_test_3223-restapi_return_codes-t.cpp index fbc7ae275..bd6fa98ba 100644 --- a/test/tap/tests/reg_test_3223-restapi_return_codes-t.cpp +++ b/test/tap/tests/reg_test_3223-restapi_return_codes-t.cpp @@ -280,10 +280,13 @@ int main(int argc, char** argv) { vector i_epts_info {}; const auto ext_i_epts_info = [] (const faulty_req_t& req) { return req.ept_info; }; + // Failed scripts may require to read the full output from ASAN leaks report. This in combination with the + // forked process shutdown slowdown can take a considerable amount of time. A cleaner solution would be + // to disable 'detect_leaks' at runtime, but doesn't look feasible at the moment. int wasan = get_env_int("WITHASAN", 0); if (wasan) { for (auto& req : invalid_requests) { - req.ept_info.timeout += 2000; + req.ept_info.timeout += 8000; } }