From d1f15f983918808516eaf53dfa57e62f205ffa18 Mon Sep 17 00:00:00 2001 From: Uli Schlachter Date: Mon, 18 Feb 2019 07:47:19 +0100 Subject: [PATCH] tests/_runner: Don't lower timeouts for later steps The first step of a test is called 20 times before a failure is diagnosed. Later steps only get five calls to finish. I guess the idea is that the first step sets everything up and opens all necessary clients, while later steps then do something with these clients. However, looking at our existing tests, they are not structured like that. Later steps open more clients etc. Since this limit has no negative impact on the run-time of tests, but only means that failures are detected later, let's just remove this limit of five calls. Signed-off-by: Uli Schlachter --- tests/_runner.lua | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/tests/_runner.lua b/tests/_runner.lua index 2e7c1649b..fc65b67d5 100644 --- a/tests/_runner.lua +++ b/tests/_runner.lua @@ -89,7 +89,7 @@ runner.run_steps = function(steps, options) -- Next step. step = step+1 step_count = 0 - wait = 5 + wait = 20 t.timeout = 0 t:again() return