summaryrefslogtreecommitdiff
path: root/test/smoke_test.vader
AgeCommit message (Collapse)Author
2021-01-273560 add vim 8.2 and nvim 0.4 to ci tests (#3561)Horacio Sanson
* 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>
2020-08-28Close #3285 - lint_file is now dynamicw0rp
`lint_file` can now be computed dynamically with a callback function, which can return a deferred result, as per `ale#command#Run`. This allows linters to dynamically switch between checking files on disk, or checking code on the fly. Some tests have been fixed on Windows.
2019-02-06#2132 - lint and fix with ale#command#Runw0rp
A new function is added here which will later be modified for public use in linter and fixer callbacks. All linting and fixing now goes through this new function, to prove that it works in all cases.
2018-10-29#2017 Simplify lint cycles for supporting other sources laterw0rp
2018-07-23Get tests to pass in Vim 8.1w0rp
2018-07-16Make NeoVim 0.3 tests fail a bit lessw0rp
2018-07-13Retry the smoke test on Windows toow0rp
2018-07-11Try the smoke test a few times over in NeoVim 0.3, which fails randomlyw0rp
2018-07-03Revert "Add a failing test to test re-running tests in AppVeyor"w0rp
This reverts commit 1565859eaed03f8cbec575feacfde6d353eac982.
2018-07-03Add a failing test to test re-running tests in AppVeyorw0rp
2018-06-28Get tests running and passing with NeoVim 0.2 and 0.3w0rp
2018-05-16TUI / GUI tooltip with content from ALEHover (#1556)Gerry Agbobada
* Guard the ballooneval settings * Mark main objectives to do to get nice Hover * Make tweaks to make the tooltip work - See " XXX: comments * Guard balloon_show call * Use return instead of finish for functions * ale#hover#show : Add optional arguments to specify arbtirary position This change is requested to be able to call the function with mouse position to enable hover information in vim's balloon * ale#ballon#Disable : Remove feature guards * ale#balloon : Show 'ALEHover' output on balloon if no diagnostic found * ale#hover#HandleLSPResponse : remove the check for cursor position This check prevented the 'ALEHover in balloon' feature, since mouse position is almost never cursor position. * ale#balloon#MessageForPos : Change the return of balloonexpr balloonexpr evaluation now works even without balloon_show for basic diagnostics, leaving the balloon_show call to ale#hover#Show, which can then feature guard the call to avoid errors * ale#hover#Response : Feature guard balloon_show calls * ale#hover : always display 'Hover' information in messages Also add a small comment to warn readers the different outputs the ale#hover#Show will write to * {LSP,TS}Response : use only variables from the Response It is clearer that we only rely on l:options to get the relevant data to build the LSP Response string * hover#ShowDetails : fix an issue where not having focus broke balloons The issue was caused by not using a buffer-specific version of getline() to cap the value of the column sent in the message to LSP. Therefore a cursor on column 10 in an inactive window could send a message with column=0, if the active window had a buffer with too few lines * {LSP,TS}Response : Remove redundant checks for balloon_show call With the upcoming change in ale_set_balloons default value (see Pull Request w0rp/ale#1565), this check will be useless * balloonexpr? : Add a flag to separate hover#Show() calls The goal of this flag is to make `:ALEHover` calls not pop a balloon under the cursor, since the user has probably no interest in their cursor while typing the command The flag is a default argument which is overridden only in ballonexpr call of ale#hover#Show, and stays set in the hover_map until the callback for the LSP handles it. There are no automated tests for this feature right now, and the nature of the addition (one optional argument in the API) should make it transparent to existing tests. Since the differentiation is now possible, the check for moved cursor has been put back in ale#hover#HandleLSPResponse * ale#hover#hover_map : Protect accesses to hover_map Using get() is safer than trying to access directly with ., as the tests show. * Raise timeout to try to get Appveyor happy * Review : Fix comments * Review : pass the optional argument 'called_from_balloonexpr' in a Dict This optional dictionary has documentation just before the function using it, ale#hover#Show, and allows easier extension in the future.
2018-01-17Fix #1298 - Escape commands for PowerShellw0rp
2017-09-08Fix the smoke_test.vader file on Windowsw0rp
2017-06-08Experimental code for showing results as soon as each linter completesw0rp
2017-05-12Refactor jobs into a Vim version agnostic API which can be used for other ↵w0rp
purposes
2017-03-21Remove a test for experimental functionality which was removedw0rp
2017-02-10Fix #283 Add an option for using ch_sendraw(), which can be better for some ↵w0rp
users