He-Pin opened a new pull request, #979:
URL: https://github.com/apache/pekko-http/pull/979

   ## Motivation
   
   A `CONNECT` method directive test used `Await.result(f, 100.millis)` — a 100 
ms hard timeout that can fail immediately on overloaded CI machines or slow 
developer workstations where the future hasn't resolved within that window. The 
upstream akka-http fix (commit `b85460691`) replaced the Await call with 
ScalaTest's `futureValue`.
   
   ## Modification
   
   Replace `Await.result(…, 100.millis)` with `futureValue` in 
`MethodDirectivesSpec`, which honors the configured `PatienceConfig` timeout 
(default 5 s) and retries automatically. Remove now-unused `Await` and `Future` 
imports.
   
   Ported from akka-http commit: `b85460691`
   
   ## Result
   
   Test no longer flakes on loaded CI or slow machines due to an artificially 
tight 100 ms timeout.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to