Merge pull request #2691 from elebow/mdl-use-json-output

mdl: Use JSON output instead of parsing text

test/command_callback/test_markdown_mdl_command_callback.vader

@@ -5,15 +5,15 @@ After:
   call ale#assert#TearDownLinterTest()
 
 Execute(The default command should be correct):
-  AssertLinter 'mdl', ale#Escape('mdl')
+  AssertLinter 'mdl', ale#Escape('mdl') . ' -j'
 
 Execute(The executable and options should be configurable):
   let g:ale_markdown_mdl_executable = 'foo bar'
   let g:ale_markdown_mdl_options = '--wat'
 
-  AssertLinter 'foo bar', ale#Escape('foo bar') . ' --wat'
+  AssertLinter 'foo bar', ale#Escape('foo bar') . ' -j --wat'
 
 Execute(Setting bundle appends 'exec mdl'):
   let g:ale_markdown_mdl_executable = 'path to/bundle'
 
-  AssertLinter 'path to/bundle', ale#Escape('path to/bundle') . ' exec mdl'
+  AssertLinter 'path to/bundle', ale#Escape('path to/bundle') . ' exec mdl -j'

test/handler/test_mdl_handler.vader

@@ -0,0 +1,25 @@
+Before:
+  runtime ale_linters/markdown/mdl.vim
+
+After:
+  call ale#linter#Reset()
+
+Execute(The mdl handler should parse output correctly):
+  AssertEqual
+  \ [
+  \   {
+  \     'lnum': 1,
+  \     'code': 'MD002/first-header-h1',
+  \     'text': 'First header should be a top level header',
+  \     'type': 'W'
+  \   },
+  \   {
+  \     'lnum': 18,
+  \     'code': 'MD033/no-inline-html',
+  \     'text': 'Inline HTML',
+  \     'type': 'W'
+  \   }
+  \ ],
+  \ ale_linters#markdown#mdl#Handle(0, [
+  \     '[{"filename":"README.md","line":1,"rule":"MD002","aliases":["first-header-h1"],"description":"First header should be a top level header"},{"filename":"README.md","line":18,"rule":"MD033","aliases":["no-inline-html"],"description":"Inline HTML"}]'
+  \ ])