Merge pull request #1827 from lluiscampos/QA-1223-longer-timeouts-http

Lluis Campos · web-flow · commit f8a26fab5191 · 2025-09-24T09:56:24.000+02:00
QA-1223: Add longer timeouts for HTTP tests reaching external services
diff --git a/tests/src/common/http_test.cpp b/tests/src/common/http_test.cpp
@@ -1646,7 +1646,7 @@ TEST(HttpTest, SerialRequestsWithSameObjectAfterCancel) {
 }
 
 TEST(HttpTest, DestroyClientBeforeRequestComplete) {
-	TestEventLoop loop;
+	TestEventLoop loop(chrono::seconds(30));
 
 	bool client_hit_header = false;
 	bool client_hit_body = false;
@@ -2200,7 +2200,7 @@ TEST(HttpsTest, NoCertificateError) {
 }
 
 TEST(HttpsTest, CorrectDefaultCertificateStoreVerification) {
-	TestEventLoop loop;
+	TestEventLoop loop(chrono::seconds(30));
 
 	bool client_hit_header {false};
 	bool client_hit_body {false};
@@ -2527,7 +2527,7 @@ TEST(HttpsTest, MtlsSuccess) {
 }
 
 TEST(HttpsTest, CertsAndKeysLoadFailures) {
-	TestEventLoop loop;
+	TestEventLoop loop(chrono::seconds(30));
 
 	auto handler = [](http::ExpectedIncomingResponsePtr exp_resp) {
 		ASSERT_TRUE(false) << "Should never get here";

Original file line number	Diff line number	Diff line change
`@@ -1646,7 +1646,7 @@ TEST(HttpTest, SerialRequestsWithSameObjectAfterCancel) {`
`1646`	`1646`	`}`
`1647`	`1647`
`1648`	`1648`	`TEST(HttpTest, DestroyClientBeforeRequestComplete) {`
`1649`		`- TestEventLoop loop;`
	`1649`	`+ TestEventLoop loop(chrono::seconds(30));`
`1650`	`1650`
`1651`	`1651`	`bool client_hit_header = false;`
`1652`	`1652`	`bool client_hit_body = false;`
`@@ -2200,7 +2200,7 @@ TEST(HttpsTest, NoCertificateError) {`
`2200`	`2200`	`}`
`2201`	`2201`
`2202`	`2202`	`TEST(HttpsTest, CorrectDefaultCertificateStoreVerification) {`
`2203`		`- TestEventLoop loop;`
	`2203`	`+ TestEventLoop loop(chrono::seconds(30));`
`2204`	`2204`
`2205`	`2205`	`bool client_hit_header {false};`
`2206`	`2206`	`bool client_hit_body {false};`
`@@ -2527,7 +2527,7 @@ TEST(HttpsTest, MtlsSuccess) {`
`2527`	`2527`	`}`
`2528`	`2528`
`2529`	`2529`	`TEST(HttpsTest, CertsAndKeysLoadFailures) {`
`2530`		`- TestEventLoop loop;`
	`2530`	`+ TestEventLoop loop(chrono::seconds(30));`
`2531`	`2531`
`2532`	`2532`	`auto handler = [](http::ExpectedIncomingResponsePtr exp_resp) {`
`2533`	`2533`	`ASSERT_TRUE(false) << "Should never get here";`