Fix the didSave tests so they work in NeoVim 0.1.7, and do not send the message for tsserver

autoload/ale/engine.vim

@@ -704,8 +704,8 @@ function! s:CheckWithLSP(buffer, linter) abort
     let l:request_id = ale#lsp#Send(l:id, l:change_message, l:root)
 
     " If this was a file save event, also notify the server of that.
-    let l:is_save = getbufvar(a:buffer, 'ale_save_event_fired', 0)
+    if a:linter.lsp isnot# 'tsserver'
-    if l:is_save != 0
+    \&& getbufvar(a:buffer, 'ale_save_event_fired', 0)
       let l:save_message = ale#lsp#message#DidSave(a:buffer)
       let l:request_id = ale#lsp#Send(l:id, l:save_message, l:root)
     endif

test/lsp/test_did_save_event.vader

@@ -16,13 +16,21 @@ Before:
   let g:message_list = []
   let g:Callback = ''
 
+  function! LanguageCallback() abort
+    return 'foobar'
+  endfunction
+
+  function! ProjectRootCallback() abort
+    return expand('.')
+  endfunction
+
   call ale#linter#Define('foobar', {
   \ 'name': 'dummy_linter',
   \ 'lsp': 'stdio',
   \ 'command': 'cat - > /dev/null',
   \ 'executable': has('win32') ? 'cmd' : 'echo',
-  \ 'language_callback': {buffer -> 'foobar'},
+  \ 'language_callback': 'LanguageCallback',
-  \ 'project_root_callback': {buffer -> expand('.')},
+  \ 'project_root_callback': 'ProjectRootCallback',
   \ })
   let g:ale_linters = {'foobar': ['dummy_linter']}
 
@@ -48,6 +56,9 @@ After:
   unlet! g:Callback
   unlet! g:message_list
 
+  delfunction LanguageCallback
+  delfunction ProjectRootCallback
+
   call ale#test#RestoreDirectory()
   call ale#linter#Reset()