2019-09-12 20:53:23 +00:00
|
|
|
Before:
|
|
|
|
call ale#test#SetDirectory('/testplugin/test')
|
|
|
|
call ale#test#SetFilename('dummy.txt')
|
|
|
|
|
|
|
|
let g:old_filename = expand('%:p')
|
|
|
|
let g:Callback = ''
|
|
|
|
let g:expr_list = []
|
|
|
|
let g:message_list = []
|
|
|
|
let g:handle_code_action_called = 0
|
|
|
|
let g:code_actions = []
|
|
|
|
let g:options = {}
|
|
|
|
let g:capability_checked = ''
|
|
|
|
let g:conn_id = v:null
|
|
|
|
let g:InitCallback = v:null
|
|
|
|
|
|
|
|
runtime autoload/ale/lsp_linter.vim
|
|
|
|
runtime autoload/ale/lsp.vim
|
|
|
|
runtime autoload/ale/util.vim
|
|
|
|
runtime autoload/ale/rename.vim
|
|
|
|
runtime autoload/ale/code_action.vim
|
|
|
|
|
|
|
|
function! ale#lsp_linter#StartLSP(buffer, linter, Callback) abort
|
|
|
|
let g:conn_id = ale#lsp#Register('executable', '/foo/bar', {})
|
|
|
|
call ale#lsp#MarkDocumentAsOpen(g:conn_id, a:buffer)
|
|
|
|
|
|
|
|
if a:linter.lsp is# 'tsserver'
|
|
|
|
call ale#lsp#MarkConnectionAsTsserver(g:conn_id)
|
|
|
|
endif
|
|
|
|
|
|
|
|
let l:details = {
|
|
|
|
\ 'command': 'foobar',
|
|
|
|
\ 'buffer': a:buffer,
|
|
|
|
\ 'connection_id': g:conn_id,
|
|
|
|
\ 'project_root': '/foo/bar',
|
|
|
|
\}
|
|
|
|
|
|
|
|
let g:InitCallback = {-> ale#lsp_linter#OnInit(a:linter, l:details, a:Callback)}
|
|
|
|
endfunction
|
|
|
|
|
|
|
|
function! ale#lsp#HasCapability(conn_id, capability) abort
|
|
|
|
let g:capability_checked = a:capability
|
|
|
|
|
|
|
|
return 1
|
|
|
|
endfunction
|
|
|
|
|
|
|
|
function! ale#lsp#RegisterCallback(conn_id, callback) abort
|
|
|
|
let g:Callback = a:callback
|
|
|
|
endfunction
|
|
|
|
|
|
|
|
function! ale#lsp#Send(conn_id, message) abort
|
|
|
|
call add(g:message_list, a:message)
|
|
|
|
|
|
|
|
return 42
|
|
|
|
endfunction
|
|
|
|
|
|
|
|
function! ale#util#Execute(expr) abort
|
|
|
|
call add(g:expr_list, a:expr)
|
|
|
|
endfunction
|
|
|
|
|
2020-10-15 20:56:21 +00:00
|
|
|
function! ale#code_action#HandleCodeAction(code_action, options) abort
|
2019-09-12 20:53:23 +00:00
|
|
|
let g:handle_code_action_called = 1
|
Dispatch textDocument/didChange after rename (2) (#4049)
* Dispatch textDocument/didChange after rename
Previously whenever we renamed a symbol that was referenced from other
files we'd just edit those files in the background, and the LSP wouldn't
know about these changes. If we tried to rename the same symbol again,
the renaming would fail. In some scenarios, the operation would just be
wrong. Here is an attempt to fix this issue.
I also noticed another bug when using Go with `gopls` LSP and the `gofmt`
fixer. Whenever the file was saved, the `gofmt` would run and reformat
the file. But it seems there was some kind of a race condition so I
disabled saving for now, and all of the modified files will be unsaved,
so the user should call `:wa` to save them. I personally like this even
better because I can inspect exactly what changes happened, and I
instantly see them in the other opened buffers, which was previously not
the case.
Fixes #3343, #3642, #3781.
* Address PR comments
* Remove mode tests in corner case tests
* Address PR comments
* Save after ALERename and ALEOrganizeImports
Also provide options to disable automatic saving, as well as instructions to
enable `set hidden` before doing that.
* Fix broken test
* Save only when !&hidden
* Update doc
* Update doc
* Add silent
2022-02-08 11:07:39 +00:00
|
|
|
AssertEqual !&hidden, get(a:options, 'should_save', 0)
|
2019-09-12 20:53:23 +00:00
|
|
|
call add(g:code_actions, a:code_action)
|
|
|
|
endfunction
|
|
|
|
|
|
|
|
function! ale#util#Input(message, value) abort
|
|
|
|
return 'a-new-name'
|
|
|
|
endfunction
|
|
|
|
|
|
|
|
call ale#rename#SetMap({
|
|
|
|
\ 3: {
|
|
|
|
\ 'old_name': 'oldName',
|
|
|
|
\ 'new_name': 'aNewName',
|
|
|
|
\ },
|
|
|
|
\})
|
|
|
|
|
|
|
|
After:
|
|
|
|
if g:conn_id isnot v:null
|
|
|
|
call ale#lsp#RemoveConnectionWithID(g:conn_id)
|
|
|
|
endif
|
|
|
|
|
|
|
|
call ale#rename#SetMap({})
|
|
|
|
call ale#test#RestoreDirectory()
|
|
|
|
call ale#linter#Reset()
|
|
|
|
|
|
|
|
unlet! g:capability_checked
|
|
|
|
unlet! g:InitCallback
|
|
|
|
unlet! g:old_filename
|
|
|
|
unlet! g:conn_id
|
|
|
|
unlet! g:Callback
|
|
|
|
unlet! g:message_list
|
|
|
|
unlet! g:expr_list
|
|
|
|
unlet! b:ale_linters
|
|
|
|
unlet! g:options
|
|
|
|
unlet! g:code_actions
|
|
|
|
unlet! g:handle_code_action_called
|
|
|
|
|
|
|
|
runtime autoload/ale/lsp_linter.vim
|
|
|
|
runtime autoload/ale/lsp.vim
|
|
|
|
runtime autoload/ale/util.vim
|
|
|
|
runtime autoload/ale/rename.vim
|
|
|
|
runtime autoload/ale/code_action.vim
|
|
|
|
|
|
|
|
Execute(Other messages for the tsserver handler should be ignored):
|
|
|
|
call ale#rename#HandleTSServerResponse(1, {'command': 'foo'})
|
|
|
|
AssertEqual g:handle_code_action_called, 0
|
|
|
|
|
|
|
|
Execute(Failed rename responses should be handled correctly):
|
|
|
|
call ale#rename#SetMap({3: {'old_name': 'oldName', 'new_name': 'a-test'}})
|
|
|
|
call ale#rename#HandleTSServerResponse(
|
|
|
|
\ 1,
|
|
|
|
\ {'command': 'rename', 'request_seq': 3}
|
|
|
|
\)
|
|
|
|
AssertEqual g:handle_code_action_called, 0
|
|
|
|
|
|
|
|
Given typescript(Some typescript file):
|
|
|
|
foo
|
|
|
|
somelongerline
|
|
|
|
bazxyzxyzxyz
|
|
|
|
|
|
|
|
Execute(Code actions from tsserver should be handled):
|
|
|
|
call ale#rename#HandleTSServerResponse(1, {
|
|
|
|
\ 'command': 'rename',
|
|
|
|
\ 'request_seq': 3,
|
|
|
|
\ 'success': v:true,
|
|
|
|
\ 'body': {
|
|
|
|
\ 'locs': [
|
|
|
|
\ {
|
|
|
|
\ 'file': '/foo/bar/file1.ts',
|
|
|
|
\ 'locs': [
|
|
|
|
\ {
|
|
|
|
\ 'start': {
|
|
|
|
\ 'line': 1,
|
|
|
|
\ 'offset': 2,
|
|
|
|
\ },
|
|
|
|
\ 'end': {
|
|
|
|
\ 'line': 3,
|
|
|
|
\ 'offset': 4,
|
|
|
|
\ },
|
|
|
|
\ },
|
|
|
|
\ ],
|
|
|
|
\ },
|
|
|
|
\ {
|
|
|
|
\ 'file': '/foo/bar/file2.ts',
|
|
|
|
\ 'locs': [
|
|
|
|
\ {
|
|
|
|
\ 'start': {
|
|
|
|
\ 'line': 10,
|
|
|
|
\ 'offset': 20,
|
|
|
|
\ },
|
|
|
|
\ 'end': {
|
|
|
|
\ 'line': 30,
|
|
|
|
\ 'offset': 40,
|
|
|
|
\ },
|
|
|
|
\ },
|
|
|
|
\ ],
|
|
|
|
\ },
|
|
|
|
\ ]
|
|
|
|
\ },
|
|
|
|
\})
|
|
|
|
|
|
|
|
AssertEqual
|
|
|
|
\ [
|
|
|
|
\ {
|
|
|
|
\ 'description': 'rename',
|
|
|
|
\ 'changes': [
|
|
|
|
\ {
|
|
|
|
\ 'fileName': '/foo/bar/file1.ts',
|
|
|
|
\ 'textChanges': [
|
|
|
|
\ {
|
|
|
|
\ 'start': {
|
|
|
|
\ 'line': 1,
|
|
|
|
\ 'offset': 2,
|
|
|
|
\ },
|
|
|
|
\ 'end': {
|
|
|
|
\ 'line': 3,
|
|
|
|
\ 'offset': 4,
|
|
|
|
\ },
|
|
|
|
\ 'newText': 'aNewName',
|
|
|
|
\ },
|
|
|
|
\ ],
|
|
|
|
\ },
|
|
|
|
\ {
|
|
|
|
\ 'fileName': '/foo/bar/file2.ts',
|
|
|
|
\ 'textChanges': [
|
|
|
|
\ {
|
|
|
|
\ 'start': {
|
|
|
|
\ 'line': 10,
|
|
|
|
\ 'offset': 20,
|
|
|
|
\ },
|
|
|
|
\ 'end': {
|
|
|
|
\ 'line': 30,
|
|
|
|
\ 'offset': 40,
|
|
|
|
\ },
|
|
|
|
\ 'newText': 'aNewName',
|
|
|
|
\ },
|
|
|
|
\ ],
|
|
|
|
\ },
|
|
|
|
\ ],
|
|
|
|
\ }
|
|
|
|
\ ],
|
|
|
|
\ g:code_actions
|
|
|
|
|
|
|
|
Execute(HandleTSServerResponse does nothing when no data in rename_map):
|
|
|
|
call ale#rename#HandleTSServerResponse(1, {
|
|
|
|
\ 'command': 'rename',
|
|
|
|
\ 'request_seq': -9,
|
|
|
|
\ 'success': v:true,
|
|
|
|
\ 'body': {}
|
|
|
|
\})
|
|
|
|
|
|
|
|
AssertEqual g:handle_code_action_called, 0
|
|
|
|
|
|
|
|
Execute(Prints a tsserver error message when unsuccessful):
|
|
|
|
call ale#rename#HandleTSServerResponse(1, {
|
|
|
|
\ 'command': 'rename',
|
|
|
|
\ 'request_seq': 3,
|
|
|
|
\ 'success': v:false,
|
|
|
|
\ 'message': 'This symbol cannot be renamed',
|
|
|
|
\})
|
|
|
|
|
|
|
|
AssertEqual g:handle_code_action_called, 0
|
|
|
|
AssertEqual ['echom ''Error renaming "oldName" to: "aNewName". ' .
|
|
|
|
\ 'Reason: This symbol cannot be renamed'''], g:expr_list
|
|
|
|
|
|
|
|
Execute(Does nothing when no changes):
|
|
|
|
call ale#rename#HandleTSServerResponse(1, {
|
|
|
|
\ 'command': 'rename',
|
|
|
|
\ 'request_seq': 3,
|
|
|
|
\ 'success': v:true,
|
|
|
|
\ 'body': {
|
|
|
|
\ 'locs': []
|
|
|
|
\ }
|
|
|
|
\})
|
|
|
|
|
|
|
|
AssertEqual g:handle_code_action_called, 0
|
|
|
|
AssertEqual ['echom ''Error renaming "oldName" to: "aNewName"'''], g:expr_list
|
|
|
|
|
|
|
|
Execute(tsserver rename requests should be sent):
|
|
|
|
call ale#rename#SetMap({})
|
|
|
|
call ale#linter#Reset()
|
|
|
|
|
|
|
|
runtime ale_linters/typescript/tsserver.vim
|
|
|
|
call setpos('.', [bufnr(''), 2, 5, 0])
|
|
|
|
|
|
|
|
ALERename
|
|
|
|
|
|
|
|
" We shouldn't register the callback yet.
|
|
|
|
AssertEqual '''''', string(g:Callback)
|
|
|
|
|
|
|
|
AssertEqual type(function('type')), type(g:InitCallback)
|
|
|
|
call g:InitCallback()
|
|
|
|
|
|
|
|
AssertEqual 'rename', g:capability_checked
|
|
|
|
AssertEqual
|
|
|
|
\ 'function(''ale#rename#HandleTSServerResponse'')',
|
|
|
|
\ string(g:Callback)
|
|
|
|
AssertEqual
|
|
|
|
\ [
|
|
|
|
\ ale#lsp#tsserver_message#Change(bufnr('')),
|
|
|
|
\ [0, 'ts@rename', {
|
|
|
|
\ 'file': expand('%:p'),
|
|
|
|
\ 'line': 2,
|
|
|
|
\ 'offset': 5,
|
|
|
|
\ 'arguments': {
|
|
|
|
\ 'findInComments': g:ale_rename_tsserver_find_in_comments,
|
|
|
|
\ 'findInStrings': g:ale_rename_tsserver_find_in_strings,
|
|
|
|
\ },
|
|
|
|
\ }]
|
|
|
|
\ ],
|
|
|
|
\ g:message_list
|
2020-11-21 19:00:53 +00:00
|
|
|
AssertEqual {'42': {'old_name': 'somelongerline', 'new_name': 'a-new-name'}},
|
2019-09-12 20:53:23 +00:00
|
|
|
\ ale#rename#GetMap()
|
|
|
|
|
|
|
|
Given python(Some Python file):
|
|
|
|
foo
|
|
|
|
somelongerline
|
|
|
|
bazxyzxyzxyz
|
|
|
|
|
|
|
|
Execute(Code actions from LSP should be handled):
|
|
|
|
call ale#rename#HandleLSPResponse(1, {
|
|
|
|
\ 'id': 3,
|
|
|
|
\ 'result': {
|
|
|
|
\ 'changes': {
|
|
|
|
\ 'file:///foo/bar/file1.ts': [
|
|
|
|
\ {
|
|
|
|
\ 'range': {
|
|
|
|
\ 'start': {
|
|
|
|
\ 'line': 1,
|
|
|
|
\ 'character': 2,
|
|
|
|
\ },
|
|
|
|
\ 'end': {
|
|
|
|
\ 'line': 3,
|
|
|
|
\ 'character': 4,
|
|
|
|
\ },
|
|
|
|
\ },
|
|
|
|
\ 'newText': 'bla123'
|
|
|
|
\ },
|
|
|
|
\ ],
|
|
|
|
\ },
|
|
|
|
\ },
|
|
|
|
\})
|
|
|
|
|
|
|
|
AssertEqual
|
|
|
|
\ [
|
|
|
|
\ {
|
|
|
|
\ 'description': 'rename',
|
|
|
|
\ 'changes': [
|
|
|
|
\ {
|
|
|
|
\ 'fileName': '/foo/bar/file1.ts',
|
|
|
|
\ 'textChanges': [
|
|
|
|
\ {
|
|
|
|
\ 'start': {
|
|
|
|
\ 'line': 2,
|
|
|
|
\ 'offset': 3,
|
|
|
|
\ },
|
|
|
|
\ 'end': {
|
|
|
|
\ 'line': 4,
|
|
|
|
\ 'offset': 5,
|
|
|
|
\ },
|
|
|
|
\ 'newText': 'bla123',
|
|
|
|
\ },
|
|
|
|
\ ],
|
|
|
|
\ },
|
|
|
|
\ ],
|
|
|
|
\ }
|
|
|
|
\ ],
|
|
|
|
\ g:code_actions
|
|
|
|
|
2020-02-22 06:52:50 +00:00
|
|
|
Execute(DocumentChanges from LSP should be handled):
|
|
|
|
call ale#rename#HandleLSPResponse(1, {
|
|
|
|
\ 'id': 3,
|
|
|
|
\ 'result': {
|
|
|
|
\ 'documentChanges': [
|
|
|
|
\ {
|
|
|
|
\ 'textDocument': {
|
|
|
|
\ 'version': 1.0,
|
|
|
|
\ 'uri': 'file:///foo/bar/file1.ts',
|
|
|
|
\ },
|
|
|
|
\ 'edits': [
|
|
|
|
\ {
|
|
|
|
\ 'range': {
|
|
|
|
\ 'start': {
|
|
|
|
\ 'line': 1,
|
|
|
|
\ 'character': 2,
|
|
|
|
\ },
|
|
|
|
\ 'end': {
|
|
|
|
\ 'line': 3,
|
|
|
|
\ 'character': 4,
|
|
|
|
\ },
|
|
|
|
\ },
|
|
|
|
\ 'newText': 'bla123',
|
|
|
|
\ },
|
|
|
|
\ ],
|
|
|
|
\ },
|
|
|
|
\ ],
|
|
|
|
\ },
|
|
|
|
\})
|
|
|
|
|
|
|
|
AssertEqual
|
|
|
|
\ [
|
|
|
|
\ {
|
|
|
|
\ 'description': 'rename',
|
|
|
|
\ 'changes': [
|
|
|
|
\ {
|
|
|
|
\ 'fileName': '/foo/bar/file1.ts',
|
|
|
|
\ 'textChanges': [
|
|
|
|
\ {
|
|
|
|
\ 'start': {
|
|
|
|
\ 'line': 2,
|
|
|
|
\ 'offset': 3,
|
|
|
|
\ },
|
|
|
|
\ 'end': {
|
|
|
|
\ 'line': 4,
|
|
|
|
\ 'offset': 5,
|
|
|
|
\ },
|
|
|
|
\ 'newText': 'bla123',
|
|
|
|
\ },
|
|
|
|
\ ],
|
|
|
|
\ },
|
|
|
|
\ ],
|
|
|
|
\ }
|
|
|
|
\ ],
|
|
|
|
\ g:code_actions
|
|
|
|
|
|
|
|
Execute(Single DocumentChange from LSP should be handled):
|
|
|
|
call ale#rename#HandleLSPResponse(1, {
|
|
|
|
\ 'id': 3,
|
|
|
|
\ 'result': {
|
|
|
|
\ 'documentChanges': {
|
|
|
|
\ 'textDocument': {
|
|
|
|
\ 'version': 1.0,
|
|
|
|
\ 'uri': 'file:///foo/bar/file1.ts',
|
|
|
|
\ },
|
|
|
|
\ 'edits': [
|
|
|
|
\ {
|
|
|
|
\ 'range': {
|
|
|
|
\ 'start': {
|
|
|
|
\ 'line': 1,
|
|
|
|
\ 'character': 2,
|
|
|
|
\ },
|
|
|
|
\ 'end': {
|
|
|
|
\ 'line': 3,
|
|
|
|
\ 'character': 4,
|
|
|
|
\ },
|
|
|
|
\ },
|
|
|
|
\ 'newText': 'bla123',
|
|
|
|
\ },
|
|
|
|
\ ],
|
|
|
|
\ },
|
|
|
|
\ },
|
|
|
|
\})
|
|
|
|
|
|
|
|
AssertEqual
|
|
|
|
\ [
|
|
|
|
\ {
|
|
|
|
\ 'description': 'rename',
|
|
|
|
\ 'changes': [
|
|
|
|
\ {
|
|
|
|
\ 'fileName': '/foo/bar/file1.ts',
|
|
|
|
\ 'textChanges': [
|
|
|
|
\ {
|
|
|
|
\ 'start': {
|
|
|
|
\ 'line': 2,
|
|
|
|
\ 'offset': 3,
|
|
|
|
\ },
|
|
|
|
\ 'end': {
|
|
|
|
\ 'line': 4,
|
|
|
|
\ 'offset': 5,
|
|
|
|
\ },
|
|
|
|
\ 'newText': 'bla123',
|
|
|
|
\ },
|
|
|
|
\ ],
|
|
|
|
\ },
|
|
|
|
\ ],
|
|
|
|
\ }
|
|
|
|
\ ],
|
|
|
|
\ g:code_actions
|
2019-09-12 20:53:23 +00:00
|
|
|
Execute(LSP should perform no action when no result):
|
|
|
|
call ale#rename#HandleLSPResponse(1, {
|
|
|
|
\ 'id': 3,
|
|
|
|
\})
|
|
|
|
|
|
|
|
AssertEqual g:handle_code_action_called, 0
|
|
|
|
AssertEqual ['echom ''No rename result received from server'''], g:expr_list
|
|
|
|
|
|
|
|
Execute(LSP should perform no action when no changes):
|
|
|
|
call ale#rename#HandleLSPResponse(1, {
|
|
|
|
\ 'id': 3,
|
|
|
|
\ 'result': {},
|
|
|
|
\})
|
|
|
|
|
|
|
|
AssertEqual g:handle_code_action_called, 0
|
|
|
|
AssertEqual ['echom ''No changes received from server'''], g:expr_list
|
|
|
|
|
|
|
|
Execute(LSP should perform no action when changes is empty):
|
|
|
|
call ale#rename#HandleLSPResponse(1, {
|
|
|
|
\ 'id': 3,
|
|
|
|
\ 'result': {
|
|
|
|
\ 'changes': [],
|
|
|
|
\ },
|
|
|
|
\})
|
|
|
|
|
|
|
|
AssertEqual g:handle_code_action_called, 0
|
|
|
|
AssertEqual ['echom ''No changes received from server'''], g:expr_list
|
|
|
|
|
|
|
|
Execute(LSP rename requests should be sent):
|
|
|
|
call ale#rename#SetMap({})
|
2021-07-09 13:40:31 +00:00
|
|
|
runtime ale_linters/python/pylsp.vim
|
|
|
|
let b:ale_linters = ['pylsp']
|
2019-09-12 20:53:23 +00:00
|
|
|
call setpos('.', [bufnr(''), 1, 5, 0])
|
|
|
|
|
2020-11-21 19:00:53 +00:00
|
|
|
ALERename
|
2019-09-12 20:53:23 +00:00
|
|
|
|
|
|
|
" We shouldn't register the callback yet.
|
|
|
|
AssertEqual '''''', string(g:Callback)
|
|
|
|
|
|
|
|
AssertEqual type(function('type')), type(g:InitCallback)
|
|
|
|
call g:InitCallback()
|
|
|
|
|
|
|
|
AssertEqual 'rename', g:capability_checked
|
|
|
|
AssertEqual
|
|
|
|
\ 'function(''ale#rename#HandleLSPResponse'')',
|
|
|
|
\ string(g:Callback)
|
|
|
|
|
|
|
|
AssertEqual
|
|
|
|
\ [
|
|
|
|
\ [1, 'textDocument/didChange', {
|
|
|
|
\ 'textDocument': {
|
2022-03-04 19:03:27 +00:00
|
|
|
\ 'uri': ale#path#ToFileURI(expand('%:p')),
|
2019-09-12 20:53:23 +00:00
|
|
|
\ 'version': g:ale_lsp_next_version_id - 1,
|
|
|
|
\ },
|
|
|
|
\ 'contentChanges': [{'text': join(getline(1, '$'), "\n") . "\n"}]
|
|
|
|
\ }],
|
|
|
|
\ [0, 'textDocument/rename', {
|
2022-03-04 19:03:27 +00:00
|
|
|
\ 'textDocument': {'uri': ale#path#ToFileURI(expand('%:p'))},
|
2019-09-12 20:53:23 +00:00
|
|
|
\ 'position': {'line': 0, 'character': 2},
|
|
|
|
\ 'newName': 'a-new-name',
|
|
|
|
\ }],
|
|
|
|
\ ],
|
|
|
|
\ g:message_list
|
|
|
|
|
2020-11-21 19:00:53 +00:00
|
|
|
AssertEqual {'42': {'old_name': 'foo', 'new_name': 'a-new-name'}},
|
2019-09-12 20:53:23 +00:00
|
|
|
\ ale#rename#GetMap()
|