Merge branch 'fix-webserviceshosttest' into 'master'

tomash87 · tomash87 · commit 00225ac4d618 · 2025-05-07T19:51:13.000Z
fix: increase timeouts in WebServicesHostTest to avoid test failure on slow environments

See merge request processm-team/processm!347

Former-commit-id: ec0e800954ab92da722cf254a9af4101ae5f9fe1
diff --git a/processm.services/src/test/kotlin/processm/services/WebServicesHostTest.kt b/processm.services/src/test/kotlin/processm/services/WebServicesHostTest.kt
@@ -21,9 +21,9 @@ class WebServicesHostTest {
             customizeClient {
                 setSSLContext(SSLContextBuilder.create().loadTrustMaterial(TrustSelfSignedStrategy()).build())
                 setSSLHostnameVerifier(NoopHostnameVerifier())
-                connectTimeout = 1000
-                connectionRequestTimeout = 1000
-                socketTimeout = 1000
+                connectTimeout = 5_000
+                connectionRequestTimeout = 10_000
+                socketTimeout = 5_000
             }
         }
     }
@@ -35,16 +35,16 @@ class WebServicesHostTest {
     @Test
     fun startStopStartStopTest() = runBlocking {
         for (i in 0..2) {
-            host.start()
-            var response = client.get(baseURIs).bodyAsText()
             try {
+                host.start()
+                val response = client.get(baseURIs).bodyAsText()
                 assertTrue(response.startsWith("<!DOCTYPE html>", ignoreCase = true))
             } finally {
                 host.stop()
             }
 
             assertFails {
-                response = client.get(baseURIs).bodyAsText()
+                val response = client.get(baseURIs).bodyAsText()
             }
         }
     }

Original file line number	Diff line number	Diff line change
`@@ -21,9 +21,9 @@ class WebServicesHostTest {`
`21`	`21`	`customizeClient {`
`22`	`22`	`setSSLContext(SSLContextBuilder.create().loadTrustMaterial(TrustSelfSignedStrategy()).build())`
`23`	`23`	`setSSLHostnameVerifier(NoopHostnameVerifier())`
`24`		`- connectTimeout = 1000`
`25`		`- connectionRequestTimeout = 1000`
`26`		`- socketTimeout = 1000`
	`24`	`+ connectTimeout = 5_000`
	`25`	`+ connectionRequestTimeout = 10_000`
	`26`	`+ socketTimeout = 5_000`
`27`	`27`	`}`
`28`	`28`	`}`
`29`	`29`	`}`
`@@ -35,16 +35,16 @@ class WebServicesHostTest {`
`35`	`35`	`@Test`
`36`	`36`	`fun startStopStartStopTest() = runBlocking {`
`37`	`37`	`for (i in 0..2) {`
`38`		`- host.start()`
`39`		`- var response = client.get(baseURIs).bodyAsText()`
`40`	`38`	`try {`
	`39`	`+ host.start()`
	`40`	`+ val response = client.get(baseURIs).bodyAsText()`
`41`	`41`	`assertTrue(response.startsWith("<!DOCTYPE html>", ignoreCase = true))`
`42`	`42`	`} finally {`
`43`	`43`	`host.stop()`
`44`	`44`	`}`
`45`	`45`
`46`	`46`	`assertFails {`
`47`		`- response = client.get(baseURIs).bodyAsText()`
	`47`	`+ val response = client.get(baseURIs).bodyAsText()`
`48`	`48`	`}`
`49`	`49`	`}`
`50`	`50`	`}`