#333 Keep any loclist items which have match_id values set on them

1 files changed, 22 insertions, 0 deletions

diff --git a/test/test_highlight_placement.vader b/test/test_highlight_placement.vader
index 2bf37d19..52e861d9 100644
--- a/test/test_highlight_placement.vader
+++ b/test/test_highlight_placement.vader
@@ -33,11 +33,13 @@ After:
   delfunction GenerateResults
   call ale#linter#Reset()
   let g:ale_buffer_info = {}
+  call clearmatches()
 
 Given testft(A Javscript file with warnings/errors):
   foo
   bar
   baz wat
+  line four
 
 Execute(Highlights should be set when a linter runs):
   call ale#Lint()
@@ -52,3 +54,23 @@ Execute(Highlights should be set when a linter runs):
   \ getmatches()
 
   AssertEqual [4, 5, 6], map(copy(g:ale_buffer_info[bufnr('')].loclist), 'v:val.match_id')
+
+Execute(Existing highlights should be kept):
+  call matchaddpos('ALEError', [[1, 2, 1]], 10, 347)
+  call matchaddpos('ALEWarning', [[2, 2, 1]], 10, 348)
+
+  call ale#highlight#SetHighlights(bufnr('%'), [
+  \ {'bufnr': bufnr('%'), 'match_id': 347, 'type': 'E', 'lnum': 1, 'col': 2},
+  \ {'bufnr': bufnr('%'), 'match_id': 348, 'type': 'W', 'lnum': 2, 'col': 2},
+  \ {'bufnr': bufnr('%'), 'type': 'E', 'lnum': 3, 'col': 2},
+  \ {'bufnr': bufnr('%'), 'type': 'W', 'lnum': 4, 'col': 1},
+  \])
+
+  AssertEqual
+  \ [
+  \   {'group': 'ALEError', 'id': 347, 'priority': 10, 'pos1': [1, 2, 1]},
+  \   {'group': 'ALEWarning', 'id': 348, 'priority': 10, 'pos1': [2, 2, 1]},
+  \   {'group': 'ALEError', 'id': 7, 'priority': 10, 'pos1': [3, 2, 1]},
+  \   {'group': 'ALEWarning', 'id': 8, 'priority': 10, 'pos1': [4, 1, 1]},
+  \ ],
+  \ getmatches()