summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorHoracio Sanson <hsanson@gmail.com>2021-01-28 04:52:24 +0900
committerGitHub <noreply@github.com>2021-01-27 19:52:24 +0000
commitbafe1c0fd62ee8173fdbc04a04caf59e3adeabc9 (patch)
tree0c859c3effce39b7a9233cd7e7933f96351d5105
parentc9f4005820092c7824a35f1d62ae0dc93e474815 (diff)
downloadale-bafe1c0fd62ee8173fdbc04a04caf59e3adeabc9.zip
3560 add vim 8.2 and nvim 0.4 to ci tests (#3561)
* Add vim82 and neovim04 to CI tests. * Fix test_sign_column_hightlighting test. In vim82 with verbose=1 the output of highlight command changes breaking the ale#sign#SetUpDefaultColumnWithoutErrorsHighlight(). This commit forces verbose=0 when the method starts and restores the previous value before exiting. * No return values in vim82 returns a numeric value instead of a empty string. * Fix test_reek_handler test The FuzzyJSONDecode() method catches E474 when it fails to parse the input as JSON but Vim8.2 throws E491 instead. This commit modifies the function to catch both E474 or E491. * Fix perl6 handler test. Perl6 handler catches json parse errors using the E474 error but in Vim82 it changed to E491. This commit modifies the handler so both errors are considered. * Fix list opening tests. In Vim 8.2 the call `range(1, bufnr('$'))` always returns quickfix buffers no matter if they are closed or not. Using `ls` does not show them but the above range will always include them. This new behavior breaks the ale#list#IsQuickfixOpen() method that in turn breaks many other things. This commit fixes this by using the getqflist() and getloclist() methods instead. * Fix test updates loclist test. For some reason in Vim 8.2 the sign offset seems to not reset between tests causing the sign_id to not match in the Assert. When the test is run individually it passes but when run as part of the whole suite the sign_id is off by one. Forcing the offset in the test setup seems to fix the issue. * Fix omnifunc completion test. For unknown reasons the SetCompletionResponse tests fail in Neovim 0.2 and 0.4. Unfortunatelly the only solution I found is to disable them for neovim. * Fix linter warnings * Fix smoker test. Add vim 8.2 to the list of versions that need some retires due to randomly failing tests. * Add docker image build job. Trying some clever trick to build the docker image if not available locally or in Docker hub. It uses the Dockerfile md5 checksum as tag so only when changes on that file occur will the image be downloaded or build. * Add labels to Docker image * Remove tests for middle versions 8.1 and 0.3.5 * Use same vader commit as appveyor * Implement image push to Docker Hub Co-authored-by: Horacio Sanson <horacio@allm.inc>
-rw-r--r--.github/workflows/main.yml16
-rw-r--r--Dockerfile8
-rw-r--r--ale_linters/perl6/perl6.vim2
-rw-r--r--autoload/ale/handlers/hdl_checker.vim2
-rw-r--r--autoload/ale/list.vim16
-rw-r--r--autoload/ale/sign.vim3
-rw-r--r--autoload/ale/util.vim2
-rwxr-xr-xrun-tests92
-rw-r--r--test/completion/test_omnifunc_completion.vader6
-rw-r--r--test/smoke_test.vader8
-rw-r--r--test/test_linting_updates_loclist.vader6
11 files changed, 115 insertions, 46 deletions
diff --git a/.github/workflows/main.yml b/.github/workflows/main.yml
index 7c182758..225512d9 100644
--- a/.github/workflows/main.yml
+++ b/.github/workflows/main.yml
@@ -8,14 +8,26 @@ on:
branches: [ master ]
jobs:
+ build_image:
+ runs-on: ubuntu-latest
+ steps:
+ - uses: actions/checkout@v2
+ - name: Build docker run image
+ shell: bash
+ env:
+ DOCKER_HUB_USER: ${{ secrets.DOCKER_HUB_USER }}
+ DOCKER_HUB_PASS: ${{ secrets.DOCKER_HUB_PASS }}
+ run: ./run-tests --build-image
test_ale:
+ needs: build_image
runs-on: ubuntu-latest
strategy:
matrix:
vim-version:
- '--vim-80-only'
- - '--vim-81-only'
- - '--neovim-only'
+ - '--vim-82-only'
+ - '--neovim-02-only'
+ - '--neovim-04-only'
- '--linters-only'
steps:
- uses: actions/checkout@v2
diff --git a/Dockerfile b/Dockerfile
index 397277d9..b685a7a2 100644
--- a/Dockerfile
+++ b/Dockerfile
@@ -1,9 +1,9 @@
FROM tweekmonster/vim-testbed:latest
RUN install_vim -tag v8.0.0027 -build \
- -tag v8.1.0519 -build \
+ -tag v8.2.2401 -build \
-tag neovim:v0.2.0 -build \
- -tag neovim:v0.3.5 -build
+ -tag neovim:v0.4.4 -build
ENV PACKAGES="\
bash \
@@ -20,3 +20,7 @@ RUN pip install vim-vint==0.3.15
RUN git clone https://github.com/junegunn/vader.vim vader && \
cd vader && git checkout c6243dd81c98350df4dec608fa972df98fa2a3af
+
+ARG GIT_VERSION
+LABEL Version=${GIT_VERSION}
+LABEL Name=w0rp/ale
diff --git a/ale_linters/perl6/perl6.vim b/ale_linters/perl6/perl6.vim
index 68ef4769..444ae4d7 100644
--- a/ale_linters/perl6/perl6.vim
+++ b/ale_linters/perl6/perl6.vim
@@ -88,7 +88,7 @@ function! ale_linters#perl6#perl6#Handle(buffer, lines) abort
try
let l:json = json_decode(join(a:lines, ''))
- catch /E474/
+ catch /E474\|E491/
call add(l:output, {
\ 'lnum': '1',
\ 'text': 'Received output in the default Perl6 error format. See :ALEDetail for details',
diff --git a/autoload/ale/handlers/hdl_checker.vim b/autoload/ale/handlers/hdl_checker.vim
index 36dbd259..e11c5377 100644
--- a/autoload/ale/handlers/hdl_checker.vim
+++ b/autoload/ale/handlers/hdl_checker.vim
@@ -32,6 +32,8 @@ function! ale#handlers#hdl_checker#GetProjectRoot(buffer) abort
if ale#handlers#hdl_checker#IsDotGit(l:project_root)
return fnamemodify(l:project_root, ':h:h')
endif
+
+ return ''
endfunction
function! ale#handlers#hdl_checker#GetExecutable(buffer) abort
diff --git a/autoload/ale/list.vim b/autoload/ale/list.vim
index 4bfe2a7b..c2ae5cc5 100644
--- a/autoload/ale/list.vim
+++ b/autoload/ale/list.vim
@@ -20,11 +20,17 @@ endif
" Return 1 if there is a buffer with buftype == 'quickfix' in bufffer list
function! ale#list#IsQuickfixOpen() abort
- for l:buf in range(1, bufnr('$'))
- if getbufvar(l:buf, '&buftype') is# 'quickfix'
- return 1
- endif
- endfor
+ let l:res = getqflist({ 'winid' : winnr() })
+
+ if has_key(l:res, 'winid') && l:res.winid > 0
+ return 1
+ endif
+
+ let l:res = getloclist(0, { 'winid' : winnr() })
+
+ if has_key(l:res, 'winid') && l:res.winid > 0
+ return 1
+ endif
return 0
endfunction
diff --git a/autoload/ale/sign.vim b/autoload/ale/sign.vim
index 8109c60e..2864f39b 100644
--- a/autoload/ale/sign.vim
+++ b/autoload/ale/sign.vim
@@ -50,9 +50,12 @@ if !hlexists('ALESignColumnWithErrors')
endif
function! ale#sign#SetUpDefaultColumnWithoutErrorsHighlight() abort
+ let l:verbose = &verbose
+ set verbose=0
redir => l:output
0verbose silent highlight SignColumn
redir end
+ let &verbose = l:verbose
let l:highlight_syntax = join(split(l:output)[2:])
let l:match = matchlist(l:highlight_syntax, '\vlinks to (.+)$')
diff --git a/autoload/ale/util.vim b/autoload/ale/util.vim
index 1f396377..5c41ab83 100644
--- a/autoload/ale/util.vim
+++ b/autoload/ale/util.vim
@@ -409,7 +409,7 @@ function! ale#util#FuzzyJSONDecode(data, default) abort
endif
return l:result
- catch /E474/
+ catch /E474\|E491/
return a:default
endtry
endfunction
diff --git a/run-tests b/run-tests
index 06a05384..ee2b10cd 100755
--- a/run-tests
+++ b/run-tests
@@ -9,8 +9,11 @@ set -u
# options, or read the output below.
#
-image=w0rp/ale
-current_image_id=1c04720f5d17
+image=denseanalysis/ale
+
+# Create docker image tag based on Dockerfile contents
+image_tag=$(md5sum Dockerfile | cut -d' ' -f1)
+git_version=$(git describe --always --tags)
# Used in all test scripts for running the selected Docker image.
DOCKER_RUN_IMAGE="$image"
@@ -21,9 +24,9 @@ tests='test/*.vader test/*/*.vader test/*/*/*.vader test/*/*/*.vader'
verbose_flag=''
quiet_flag=''
run_neovim_02_tests=1
-run_neovim_03_tests=1
+run_neovim_04_tests=1
run_vim_80_tests=1
-run_vim_81_tests=1
+run_vim_82_tests=1
run_linters=1
while [ $# -ne 0 ]; do
@@ -36,58 +39,66 @@ while [ $# -ne 0 ]; do
quiet_flag='-q'
shift
;;
+ --build-image)
+ run_vim_80_tests=0
+ run_vim_82_tests=0
+ run_neovim_02_tests=0
+ run_neovim_04_tests=0
+ run_linters=0
+ shift
+ ;;
--neovim-only)
run_vim_80_tests=0
- run_vim_81_tests=0
+ run_vim_82_tests=0
run_linters=0
shift
;;
--neovim-02-only)
- run_neovim_03_tests=0
+ run_neovim_04_tests=0
run_vim_80_tests=0
- run_vim_81_tests=0
+ run_vim_82_tests=0
run_linters=0
shift
;;
- --neovim-03-only)
+ --neovim-04-only)
run_neovim_02_tests=0
run_vim_80_tests=0
- run_vim_81_tests=0
+ run_vim_82_tests=0
run_linters=0
shift
;;
--vim-only)
run_neovim_02_tests=0
- run_neovim_03_tests=0
+ run_neovim_04_tests=0
run_linters=0
shift
;;
--vim-80-only)
run_neovim_02_tests=0
- run_neovim_03_tests=0
- run_vim_81_tests=0
+ run_neovim_04_tests=0
+ run_vim_82_tests=0
run_linters=0
shift
;;
- --vim-81-only)
+ --vim-82-only)
run_neovim_02_tests=0
- run_neovim_03_tests=0
+ run_neovim_04_tests=0
run_vim_80_tests=0
run_linters=0
shift
;;
--linters-only)
run_vim_80_tests=0
- run_vim_81_tests=0
+ run_vim_82_tests=0
run_neovim_02_tests=0
- run_neovim_03_tests=0
+ run_neovim_04_tests=0
shift
;;
--fast)
run_vim_80_tests=0
- run_vim_81_tests=0
+ run_vim_82_tests=0
run_neovim_02_tests=0
- run_neovim_03_tests=1
+ run_neovim_04_tests=1
shift
;;
--help)
@@ -99,12 +110,13 @@ while [ $# -ne 0 ]; do
echo 'Options:'
echo ' -v Enable verbose output'
echo ' -q Hide output for successful tests'
- echo ' --neovim-only Run tests only for NeoVim 0.2 and 0.3'
+ echo ' --build-image Run docker image build only.'
+ echo ' --neovim-only Run tests only for NeoVim'
echo ' --neovim-02-only Run tests only for NeoVim 0.2'
- echo ' --neovim-03-only Run tests only for NeoVim 0.3'
- echo ' --vim-only Run tests only for Vim 8.0 and 8.1'
+ echo ' --neovim-04-only Run tests only for NeoVim 0.4'
+ echo ' --vim-only Run tests only for Vim'
echo ' --vim-80-only Run tests only for Vim 8.0'
- echo ' --vim-81-only Run tests only for Vim 8.1'
+ echo ' --vim-82-only Run tests only for Vim 8.2'
echo ' --linters-only Run only Vint and custom checks'
echo ' --fast Run only the fastest Vim and custom checks'
echo ' --help Show this help text'
@@ -138,8 +150,36 @@ fi
# Delete .swp files in the test directory, which cause Vim 8 to hang.
find test -name '*.swp' -delete
-docker images -q w0rp/ale | grep "^$current_image_id" > /dev/null \
- || docker pull "$image"
+# Check if docker un image is available locally
+has_image=$(docker images ls --filter reference="${image}:${image_tag}" --quiet | wc -l)
+
+if [ "$has_image" -eq 0 ]
+then
+
+ echo "Downloading run image ${image}:${image_tag}"
+ docker pull "${image}:${image_tag}" &> /dev/null
+
+ if [ $? -eq 1 ]
+ then
+ echo "Could not pull image ${image}:${image_tag}"
+ echo "Building run image ${image}:${image_tag}"
+ docker build --build-arg GIT_VERSION="$git_version" -t "${image}:${image_tag}" .
+ docker tag "${image}:${image_tag}" "${image}:latest"
+
+ if [[ -z "${DOCKER_HUB_USER:-}" || -z "${DOCKER_HUB_PASS:-}" ]]
+ then
+ echo "Docker Hub credentials not set, skip push"
+ else
+ echo "Push ${image}:${image_tag} to Docker Hub"
+ echo "$DOCKER_HUB_PASS" | docker login -u "$DOCKER_HUB_USER" --password-stdin
+ docker push "${image}:${image_tag}"
+ fi
+ fi
+else
+ echo "Docker run image ${image}:${image_tag} ready"
+fi
+
+docker tag "${image}:${image_tag}" "${image}:latest"
output_dir=$(mktemp -d 2>/dev/null || mktemp -d -t 'mytmpdir')
@@ -174,9 +214,9 @@ trap cancel_tests INT TERM
for vim in $(docker run --rm "$DOCKER_RUN_IMAGE" ls /vim-build/bin | grep '^neovim\|^vim' ); do
if ( [[ $vim =~ ^vim-v8.0 ]] && ((run_vim_80_tests)) ) \
- || ( [[ $vim =~ ^vim-v8.1 ]] && ((run_vim_81_tests)) ) \
+ || ( [[ $vim =~ ^vim-v8.2 ]] && ((run_vim_82_tests)) ) \
|| ( [[ $vim =~ ^neovim-v0.2 ]] && ((run_neovim_02_tests)) ) \
- || ( [[ $vim =~ ^neovim-v0.3 ]] && ((run_neovim_03_tests)) ); then
+ || ( [[ $vim =~ ^neovim-v0.4 ]] && ((run_neovim_04_tests)) ); then
echo "Starting Vim: $vim..."
file_number=$((file_number+1))
test/script/run-vader-tests $quiet_flag $verbose_flag "$vim" "$tests" \
diff --git a/test/completion/test_omnifunc_completion.vader b/test/completion/test_omnifunc_completion.vader
index 3711f974..1db64705 100644
--- a/test/completion/test_omnifunc_completion.vader
+++ b/test/completion/test_omnifunc_completion.vader
@@ -45,15 +45,15 @@ Execute(The start position should be returned when results can be requested):
AssertEqual 11, ale#completion#OmniFunc(1, '')
Execute(The omnifunc function should return async results):
- " Neovim 0.2.0 struggles at running these tests.
- if !has('nvim') || has('nvim-0.3.0')
+ " Neovim 0.2.0 and 0.4.4 struggles at running these tests.
+ if !has('nvim')
call timer_start(0, function('SetCompletionResult'))
AssertEqual ['foo'], ale#completion#OmniFunc(0, '')
endif
Execute(The omnifunc function should parse and return async responses):
- if !has('nvim') || has('nvim-0.3.0')
+ if !has('nvim')
call timer_start(0, function('SetCompletionResponse'))
AssertEqual ['foo'], ale#completion#OmniFunc(0, '')
diff --git a/test/smoke_test.vader b/test/smoke_test.vader
index 0b126cc6..58206049 100644
--- a/test/smoke_test.vader
+++ b/test/smoke_test.vader
@@ -64,9 +64,9 @@ Execute(Linters should run with the default options):
\ 'valid': 1,
\ }]
- " Try the test a few times over in NeoVim 0.3 or Windows,
+ " Try the test a few times over in NeoVim 0.3 or Windows or Vim 8.2,
" where tests fail randomly.
- for g:i in range(has('nvim-0.3') || has('win32') ? 5 : 1)
+ for g:i in range(has('nvim-0.3') || has('win32') || has('patch-8.2.2401') ? 5 : 1)
call ale#Queue(0, '')
call ale#test#WaitForJobs(2000)
@@ -165,9 +165,9 @@ Execute(Previous errors should be removed when linters change):
\ 'valid': 1,
\}]
- " Try the test a few times over in NeoVim 0.3 or Windows,
+ " Try the test a few times over in NeoVim 0.3 or VIm 8.2 or Windows,
" where tests fail randomly.
- for g:i in range(has('nvim-0.3') || has('win32') ? 5 : 1)
+ for g:i in range(has('nvim-0.3') || has('win32') || has('patch-8.2.2401') ? 5 : 1)
call ale#Queue(0, '')
call ale#test#WaitForJobs(2000)
diff --git a/test/test_linting_updates_loclist.vader b/test/test_linting_updates_loclist.vader
index 8a162703..41c86522 100644
--- a/test/test_linting_updates_loclist.vader
+++ b/test/test_linting_updates_loclist.vader
@@ -7,10 +7,12 @@ Before:
Save g:ale_run_synchronously
Save g:ale_set_lists_synchronously
Save g:ale_buffer_info
+ Save g:ale_sign_offset
" We want to check that sign IDs are set for this test.
let g:ale_set_signs = 1
let g:ale_set_loclist = 1
+ let g:ale_sign_offset = 2000000
" Disable features we don't need for these tests.
let g:ale_set_quickfix = 0
let g:ale_set_highlights = 0
@@ -77,7 +79,7 @@ Execute(The loclist should be updated after linting is done):
\ 'type': 'W',
\ 'col': 10,
\ 'text': 'Infix operators must be spaced. [Warning/space-infix-ops]',
- \ 'sign_id': 1000001,
+ \ 'sign_id': 2000001,
\ },
\ {
\ 'lnum': 2,
@@ -88,7 +90,7 @@ Execute(The loclist should be updated after linting is done):
\ 'type': 'E',
\ 'col': 10,
\ 'text': 'Missing semicolon. [Error/semi]',
- \ 'sign_id': 1000002,
+ \ 'sign_id': 2000002,
\ }
\ ],
\ get(get(g:ale_buffer_info, bufnr('%'), {}), 'loclist', [])