Ruff use json-lines output format (#4656)

* Ruff use json-lines output format
* Fix Ruff: add -q to prevent non json output

Using the json-lines output format allows for setting of the end_line,
end_col and code field of the handle output.

Additionally, the first letter of the code is used to determine the type
field.

Co-authored-by: w0rp <w0rp@users.noreply.github.com>

1 files changed, 18 insertions, 17 deletions

diff --git a/test/linter/test_ruff.vader b/test/linter/test_ruff.vader
index 049f04fa..358268bd 100644
--- a/test/linter/test_ruff.vader
+++ b/test/linter/test_ruff.vader
@@ -6,7 +6,8 @@ Before:
   call ale#assert#SetUpLinterTest('python', 'ruff')
 
   let b:bin_dir = has('win32') ? 'Scripts' : 'bin'
-  let b:command_tail = ' --format text --stdin-filename %s -'
+  let b:command_head = ale#Escape('ruff') . ' -q'
+  let b:command_tail = ' --format json-lines --stdin-filename %s -'
 
   GivenCommandOutput ['ruff 0.0.83']
 
@@ -19,51 +20,51 @@ After:
 
 Execute(The ruff callbacks should return the correct default values):
   AssertLinterCwd expand('%:p:h')
-  AssertLinter 'ruff', ale#Escape('ruff') . b:command_tail
+  AssertLinter 'ruff', b:command_head . b:command_tail
 
 Execute(ruff should run with the file path of buffer in old versions):
   " version `0.0.69` supports liniting input from stdin
   GivenCommandOutput ['ruff 0.0.68']
 
   AssertLinterCwd expand('%:p:h')
-  AssertLinter 'ruff', ale#Escape('ruff') . b:command_tail[:-23] . ' %s'
+  AssertLinter 'ruff', b:command_head . b:command_tail[:-23] . ' %s'
 
 Execute(ruff should run with the --output-format flag in new versions):
   GivenCommandOutput ['ruff 0.1.0']
 
   AssertLinterCwd expand('%:p:h')
-  AssertLinter 'ruff', ale#Escape('ruff') . ' --output-format text --stdin-filename %s -'
+  AssertLinter 'ruff', b:command_head . ' --output-format json-lines --stdin-filename %s -'
 
 Execute(ruff should run with the stdin in new enough versions):
   GivenCommandOutput ['ruff 0.0.83']
 
   AssertLinterCwd expand('%:p:h')
-  AssertLinter 'ruff', ale#Escape('ruff') . b:command_tail[:-3] . ' -'
-  " AssertLinter 'ruff', ale#Escape('ruff') . b:command_tail[:-3] . '--format text -'
+  AssertLinter 'ruff', b:command_head . b:command_tail[:-3] . ' -'
+  " AssertLinter 'ruff', b:command_head . b:command_tail[:-3] . '--format json-lines -'
 
 Execute(The option for disabling changing directories should work):
   let g:ale_python_ruff_change_directory = 0
 
   AssertLinterCwd ''
-  AssertLinter 'ruff', ale#Escape('ruff') . b:command_tail
+  AssertLinter 'ruff', b:command_head . b:command_tail
 
 Execute(The ruff executable should be configurable, and escaped properly):
   let g:ale_python_ruff_executable = 'executable with spaces'
 
-  AssertLinter 'executable with spaces', ale#Escape('executable with spaces') . b:command_tail
+  AssertLinter 'executable with spaces', ale#Escape('executable with spaces') . ' -q' . b:command_tail
 
 Execute(The ruff command callback should let you set options):
   let g:ale_python_ruff_options = '--some-flag'
-  AssertLinter 'ruff', ale#Escape('ruff') . ' --some-flag' . b:command_tail
+  AssertLinter 'ruff', b:command_head . ' --some-flag' . b:command_tail
 
   let g:ale_python_ruff_options = '--some-option value'
-  AssertLinter 'ruff', ale#Escape('ruff') . ' --some-option value' . b:command_tail
+  AssertLinter 'ruff', b:command_head . ' --some-option value' . b:command_tail
 
 Execute(The ruff callbacks shouldn't detect virtualenv directories where they don't exist):
   call ale#test#SetFilename('../test-files/python/no_virtualenv/subdir/foo/bar.py')
 
   AssertLinterCwd ale#path#Simplify(g:dir . '/../test-files/python/no_virtualenv/subdir')
-  AssertLinter 'ruff', ale#Escape('ruff') . b:command_tail
+  AssertLinter 'ruff', b:command_head . b:command_tail
 
 Execute(The ruff callbacks should detect virtualenv directories):
   call ale#test#SetFilename('../test-files/python/with_virtualenv/subdir/foo/bar.py')
@@ -71,20 +72,20 @@ Execute(The ruff callbacks should detect virtualenv directories):
   \ g:dir . '/../test-files/python/with_virtualenv/env/' . b:bin_dir . '/ruff'
   \)
   AssertLinterCwd ale#path#Simplify(g:dir . '/../test-files/python/with_virtualenv/subdir')
-  AssertLinter b:executable, ale#Escape(b:executable) . b:command_tail
+  AssertLinter b:executable, ale#Escape(b:executable) . ' -q' . b:command_tail
 
 Execute(You should able able to use the global ruff instead):
   call ale#test#SetFilename('../test-files/python/with_virtualenv/subdir/foo/bar.py')
   let g:ale_python_ruff_use_global = 1
 
   AssertLinterCwd ale#path#Simplify(g:dir . '/../test-files/python/with_virtualenv/subdir')
-  AssertLinter 'ruff', ale#Escape('ruff') . b:command_tail
+  AssertLinter 'ruff', b:command_head . b:command_tail
 
 Execute(Setting executable to 'pipenv' appends 'run ruff'):
   let g:ale_python_ruff_executable = 'path/to/pipenv'
   let g:ale_python_ruff_use_global = 1
 
-  AssertLinter 'path/to/pipenv', ale#Escape('path/to/pipenv') . ' run ruff'
+  AssertLinter 'path/to/pipenv', ale#Escape('path/to/pipenv') . ' run ruff -q'
   \   . b:command_tail
 
 Execute(Pipenv is detected when python_ruff_auto_pipenv is set):
@@ -92,14 +93,14 @@ Execute(Pipenv is detected when python_ruff_auto_pipenv is set):
   call ale#test#SetFilename('../test-files/python/pipenv/whatever.py')
 
   AssertLinterCwd expand('%:p:h')
-  AssertLinter 'pipenv', ale#Escape('pipenv') . ' run ruff'
+  AssertLinter 'pipenv', ale#Escape('pipenv') . ' run ruff -q'
   \   . b:command_tail
 
 Execute(Setting executable to 'poetry' appends 'run ruff'):
   let g:ale_python_ruff_executable = 'path/to/poetry'
   let g:ale_python_ruff_use_global = 1
 
-  AssertLinter 'path/to/poetry', ale#Escape('path/to/poetry') . ' run ruff'
+  AssertLinter 'path/to/poetry', ale#Escape('path/to/poetry') . ' run ruff -q'
   \   . b:command_tail
 
 Execute(poetry is detected when python_ruff_auto_poetry is set):
@@ -107,5 +108,5 @@ Execute(poetry is detected when python_ruff_auto_poetry is set):
   call ale#test#SetFilename('../test-files/python/poetry/whatever.py')
 
   AssertLinterCwd expand('%:p:h')
-  AssertLinter 'poetry', ale#Escape('poetry') . ' run ruff'
+  AssertLinter 'poetry', ale#Escape('poetry') . ' run ruff -q'
   \   . b:command_tail