diff options
author | w0rp <devw0rp@gmail.com> | 2023-09-16 22:22:01 +0100 |
---|---|---|
committer | w0rp <devw0rp@gmail.com> | 2023-09-16 22:23:30 +0100 |
commit | ae1d0515049ed135878672e06b51b083f5ae384d (patch) | |
tree | 1556645edaf786bc934be9cc8cc63408e65054d4 /test/sign/test_sign_placement.vader | |
parent | 4b11cf21dc8406b579b89e50ae3f08ef2c0856e1 (diff) | |
download | ale-ae1d0515049ed135878672e06b51b083f5ae384d.zip |
#4454 Clean up more tests and code
* Remove some tests we no longer need
* Delete blocks of redundant code
* Compress some tests together to simplify them
* Remove a little code for ancient linter versions
* Escape more executables we didn't escape before
* Rename a deno option that didn't match our conventions
Diffstat (limited to 'test/sign/test_sign_placement.vader')
-rw-r--r-- | test/sign/test_sign_placement.vader | 33 |
1 files changed, 8 insertions, 25 deletions
diff --git a/test/sign/test_sign_placement.vader b/test/sign/test_sign_placement.vader index 7b80d83c..82532876 100644 --- a/test/sign/test_sign_placement.vader +++ b/test/sign/test_sign_placement.vader @@ -133,11 +133,11 @@ Execute(ale#sign#GetSignName should return the right sign names): \]) Given testft(A file with warnings/errors): - foo - bar - baz - fourth line - fifth line + Foo + Bar + Baz + Fourth line + Fifth line Execute(The current signs should be set for running a job): ALELint @@ -277,26 +277,6 @@ Execute(Signs should be upgraded correctly): \ sort(ParseSigns()) Execute(It should be possible to clear signs with empty lists): - let g:loclist = [ - \ {'bufnr': bufnr(''), 'lnum': 16, 'col': 2, 'type': 'E', 'text': 'f'}, - \] - - call ale#sign#SetSigns(bufnr(''), g:loclist) - - AssertEqual - \ [ - \ ['16', '1000001', 'ALEErrorSign'], - \ ], - \ sort(ParseSigns()) - - call ale#sign#SetSigns(bufnr(''), []) - - AssertEqual [], ParseSigns() - -Execute(No exceptions should be thrown when setting signs for invalid buffers): - call ale#sign#SetSigns(123456789, [{'lnum': 15, 'col': 2, 'type': 'W', 'text': 'e'}]) - -Execute(Signs should be removed when lines have multiple sign IDs on them): " We can fail to remove signs if there are multiple signs on one line, " say after deleting lines in Vim, etc. if has('nvim-0.4.2') || has('patch-8.1.614') @@ -313,3 +293,6 @@ Execute(Signs should be removed when lines have multiple sign IDs on them): call ale#sign#SetSigns(bufnr(''), []) AssertEqual [], ParseSigns() + +Execute(No exceptions should be thrown when setting signs for invalid buffers): + call ale#sign#SetSigns(123456789, [{'lnum': 15, 'col': 2, 'type': 'W', 'text': 'e'}]) |