summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorStewy Slocum <sslocum3@jhu.edu>2018-03-25 13:33:24 -0400
committerStewy Slocum <sslocum3@jhu.edu>2018-03-25 13:33:24 -0400
commit84952918cc48a2de2082c9988e720412f146d3cc (patch)
treef5460e63b9cfe885dc95eac060ef4eeb7cbe1e0a
parent92e6e4d1ba482a4d2d89d850f660c67ccf8a28eb (diff)
downloadale-84952918cc48a2de2082c9988e720412f146d3cc.zip
add support for mercury language using mmc as linter
-rw-r--r--ale_linters/mercury/mmc.vim45
-rw-r--r--doc/ale-mercury.txt26
-rw-r--r--test/command_callback/test_mercury_mmc_command_callback.vader42
-rw-r--r--test/handler/test_mercury_mmc_handler.vader58
4 files changed, 171 insertions, 0 deletions
diff --git a/ale_linters/mercury/mmc.vim b/ale_linters/mercury/mmc.vim
new file mode 100644
index 00000000..c7bfc59d
--- /dev/null
+++ b/ale_linters/mercury/mmc.vim
@@ -0,0 +1,45 @@
+" Author: stewy33 <slocumstewy@gmail.com>
+" Description: Lints mercury files using mmc
+
+call ale#Set('mercury_mmc_executable', 'mmc')
+call ale#Set('mercury_mmc_options', '--make --output-compile-error-lines 100')
+
+function! ale_linters#mercury#mmc#GetExecutable(buffer) abort
+ return ale#Var(a:buffer, 'mercury_mmc_executable')
+endfunction
+
+function! ale_linters#mercury#mmc#GetCommand(buffer) abort
+ let l:module_name = expand('#' . a:buffer . ':t:r')
+
+ return ale#path#BufferCdString(a:buffer)
+ \ . ale_linters#mercury#mmc#GetExecutable(a:buffer)
+ \ . ' --errorcheck-only '
+ \ . ale#Var(a:buffer, 'mercury_mmc_options')
+ \ . ' ' . l:module_name
+endfunction
+
+function! ale_linters#mercury#mmc#Handle(buffer, lines) abort
+ " output format
+ " <filename>:<line>: <issue type>: <message>
+ let l:pattern = '\v^\w+\.m:(\d+):\s+([W|w]arning|.*[E|e]rror.*): (.*)'
+ let l:output = []
+
+ for l:match in ale#util#GetMatches(a:lines, l:pattern)
+ call add(l:output, {
+ \ 'lnum': substitute(l:match[1], '\v^0*', '', '') + 0,
+ \ 'type': l:match[2][0] =~? 'W' ? 'W' : 'E',
+ \ 'text': l:match[2] . ': ' . l:match[3]
+ \})
+ endfor
+
+ return l:output
+endfunction
+
+call ale#linter#Define('mercury', {
+\ 'name': 'mmc',
+\ 'output_stream': 'stderr',
+\ 'executable_callback': 'ale_linters#mercury#mmc#GetExecutable',
+\ 'command_callback': 'ale_linters#mercury#mmc#GetCommand',
+\ 'callback': 'ale_linters#mercury#mmc#Handle',
+\ 'lint_file': 1,
+\})
diff --git a/doc/ale-mercury.txt b/doc/ale-mercury.txt
new file mode 100644
index 00000000..ca06a0a7
--- /dev/null
+++ b/doc/ale-mercury.txt
@@ -0,0 +1,26 @@
+===============================================================================
+ALE Mercury Integration *ale-mercury-options*
+
+
+===============================================================================
+mmc *ale-mercury-mmc*
+
+
+g:ale_mercury_mmc_executable *g:ale_mercury_mmc_executable*
+ *b:ale_mercury_mmc_executable*
+ Type: |String|
+ Default: `'mmc'`
+
+ This variable can be changed to use a different executable for mmc.
+
+
+g:ale_mercury_mmc_options *g:ale_mercury_mmc_options*
+ *b:ale_mercury_mmc_options*
+ Type: |String|
+ Default: `'--make --output-compile-error-lines 100'`
+
+ This variable can be set to pass additional options to mmc.
+
+
+===============================================================================
+ vim:tw=78:ts=2:sts=2:sw=2:ft=help:norl:
diff --git a/test/command_callback/test_mercury_mmc_command_callback.vader b/test/command_callback/test_mercury_mmc_command_callback.vader
new file mode 100644
index 00000000..7ebf49c3
--- /dev/null
+++ b/test/command_callback/test_mercury_mmc_command_callback.vader
@@ -0,0 +1,42 @@
+Before:
+ Save g:ale_mercury_mmc_executable
+ Save g:ale_mercury_mmc_options
+
+ unlet! g:ale_mercury_mmc_executable
+ unlet! b:ale_mercury_mmc_executable
+ unlet! g:ale_mercury_mmc_options
+ unlet! b:ale_mercury_mmc_options
+
+ runtime ale_linters/mercury/mmc.vim
+ call ale#test#SetDirectory('/testplugin/test/command_callback')
+
+After:
+ unlet! b:ale_mercury_mmc_executable
+ unlet! b:ale_mercury_mmc_options
+
+ Restore
+ call ale#test#RestoreDirectory()
+ call ale#linter#Reset()
+
+Execute(The default command should be correct):
+ AssertEqual
+ \ ale#path#BufferCdString(bufnr(''))
+ \ . 'mmc --errorcheck-only --make --output-compile-error-lines 100 dummy',
+ \
+ \ ale_linters#mercury#mmc#GetCommand(bufnr(''))
+
+Execute(The executable should be configurable):
+ let b:ale_mercury_mmc_executable = 'foo'
+ AssertEqual
+ \ ale#path#BufferCdString(bufnr(''))
+ \ . 'foo --errorcheck-only --make --output-compile-error-lines 100 dummy',
+ \
+ \ ale_linters#mercury#mmc#GetCommand(bufnr(''))
+
+Execute(The options should be configurable):
+ let b:ale_mercury_mmc_options = '--bar'
+ AssertEqual
+ \ ale#path#BufferCdString(bufnr(''))
+ \ . 'mmc --errorcheck-only --bar dummy',
+ \
+ \ ale_linters#mercury#mmc#GetCommand(bufnr(''))
diff --git a/test/handler/test_mercury_mmc_handler.vader b/test/handler/test_mercury_mmc_handler.vader
new file mode 100644
index 00000000..e862f287
--- /dev/null
+++ b/test/handler/test_mercury_mmc_handler.vader
@@ -0,0 +1,58 @@
+Before:
+ runtime ale_linters/mercury/mmc.vim
+
+After:
+ call ale#linter#Reset()
+
+Execute(The mmc handler should handle syntax errors):
+ AssertEqual
+ \ [
+ \ {
+ \ 'lnum': 3,
+ \ 'type': 'E',
+ \ 'text': "Syntax error at token ',': operator precedence error."
+ \ }
+ \ ],
+ \ ale_linters#mercury#mmc#Handle(1, [
+ \ "file_name.m:003: Syntax error at token ',': operator precedence error."
+ \ ])
+
+Execute(The mmc handler should handle warnings):
+ AssertEqual
+ \ [
+ \ {
+ \ 'lnum': 10,
+ \ 'type': 'W',
+ \ 'text': 'Warning: reference to uninitialized state variable !.X.'
+ \ },
+ \ {
+ \ 'lnum': 12,
+ \ 'type': 'W',
+ \ 'text': 'warning: determinism declaration could be tighter.'
+ \ }
+ \ ],
+ \ ale_linters#mercury#mmc#Handle(1, [
+ \ 'file_name.m:010: Warning: reference to uninitialized state variable !.X.',
+ \ "file_name.m:012: In `some_predicate':",
+ \ 'file_name.m:012: warning: determinism declaration could be tighter.'
+ \ ])
+
+Execute(The mmc handler should handle semantic errors):
+ AssertEqual
+ \ [
+ \ {
+ \ 'lnum': 7,
+ \ 'type': 'E',
+ \ 'text': "error: undefined type `bar'/0."
+ \ },
+ \ {
+ \ 'lnum': 15,
+ \ 'type': 'E',
+ \ 'text': "Error: circular equivalence type `file_name.foo'/0."
+ \ }
+ \ ],
+ \ ale_linters#mercury#mmc#Handle(1, [
+ \ "file_name.m:007: In clause for predicate `foldit'/4:",
+ \ "file_name.m:007: error: undefined type `bar'/0.",
+ \ "file_name.m:015: Error: circular equivalence type `file_name.foo'/0."
+ \ ])