From 5556fcbd1c271001d22b88a9d4533b3d64e0e394 Mon Sep 17 00:00:00 2001 From: w0rp Date: Sun, 19 Nov 2017 23:09:20 +0000 Subject: [PATCH] Stop the completion tests from failing randomly --- autoload/ale/completion.vim | 13 ++++++++++--- test/test_completion.vader | 4 ++++ 2 files changed, 14 insertions(+), 3 deletions(-) diff --git a/autoload/ale/completion.vim b/autoload/ale/completion.vim index 90c9ae29ea..e471a3bdc4 100644 --- a/autoload/ale/completion.vim +++ b/autoload/ale/completion.vim @@ -282,6 +282,15 @@ function! s:TimerHandler(...) abort endif endfunction +" Stop any completion timer that is queued. This is useful for tests. +function! ale#completion#StopTimer() abort + if s:timer_id != -1 + call timer_stop(s:timer_id) + endif + + let s:timer_id = -1 +endfunction + function! ale#completion#Queue() abort let l:time = get(b:, 'ale_complete_done_time', 0) @@ -298,9 +307,7 @@ function! ale#completion#Queue() abort let b:ale_completion_info.request_id = 0 endif - if s:timer_id != -1 - call timer_stop(s:timer_id) - endif + call ale#completion#StopTimer() let s:timer_id = timer_start(g:ale_completion_delay, function('s:TimerHandler')) endfunction diff --git a/test/test_completion.vader b/test/test_completion.vader index 18e50f5b74..3dfc205663 100644 --- a/test/test_completion.vader +++ b/test/test_completion.vader @@ -41,6 +41,10 @@ After: delfunction CheckCompletionCalled + " Stop any timers we left behind. + " This stops the tests from failing randomly. + call ale#completion#StopTimer() + runtime autoload/ale/completion.vim runtime autoload/ale/lsp.vim