From 46c4c7cbc6d562d5f1b784fea9527c998c190b99 Mon Sep 17 00:00:00 2001 From: Daniel Gustafsson Date: Thu, 3 Apr 2025 20:41:09 +0200 Subject: [PATCH] oauth: Remove timeout from t/002_client when not needed The connect_timeout=1 setting for the --hang-forever test was left in place and used by later tests, causing unexpected timeouts on slower buildfarm animals. Remove it when no longer needed. Per buildfarm member skink, reported by Andres on Discord. Author: Jacob Champion Reported-by: Andres Freund --- src/test/modules/oauth_validator/t/002_client.pl | 3 +++ 1 file changed, 3 insertions(+) diff --git a/src/test/modules/oauth_validator/t/002_client.pl b/src/test/modules/oauth_validator/t/002_client.pl index ab83258d736..54769f12f57 100644 --- a/src/test/modules/oauth_validator/t/002_client.pl +++ b/src/test/modules/oauth_validator/t/002_client.pl @@ -122,6 +122,9 @@ test( flags => ["--hang-forever"], expected_stderr => qr/failed: timeout expired/); +# Remove the timeout for later tests. +$common_connstr = "$base_connstr oauth_issuer=$issuer oauth_client_id=myID"; + # Test various misbehaviors of the client hook. my @cases = ( { -- 2.39.5