From c10a207a20eb6dbcd628cae80c32de602d29e4b3 Mon Sep 17 00:00:00 2001 From: "Justin M. Keyes" Date: Fri, 20 Apr 2018 23:56:50 +0200 Subject: test/util: throttle retry() (#8296) Avoid a hot loop in retry(), there's no need to retry more than 50/s. Also use luv.sleep() to implement sleep() instead of spinning the event-loop, so events are not silently discarded. --- test/functional/helpers.lua | 10 ++-------- 1 file changed, 2 insertions(+), 8 deletions(-) (limited to 'test') diff --git a/test/functional/helpers.lua b/test/functional/helpers.lua index e690fe350d..655200e6f7 100644 --- a/test/functional/helpers.lua +++ b/test/functional/helpers.lua @@ -309,6 +309,7 @@ local function retry(max, max_ms, fn) error("\nretry() attempts: "..tostring(tries).."\n"..tostring(result)) end tries = tries + 1 + luv.sleep(20) -- Avoid hot loop... end end @@ -467,14 +468,7 @@ end -- sleeps the test runner (_not_ the nvim instance) local function sleep(ms) - local function notification_cb(method, _) - if method == "redraw" then - error("Screen is attached; use screen:sleep() instead.") - end - return true - end - - run(nil, notification_cb, nil, ms) + luv.sleep(ms) end local function curbuf_contents() -- cgit