improved the sml linter (#445)

* improved the sml linter

* made matching operator agnostic to user settings

* added tests for sml

1 files changed, 91 insertions, 0 deletions

diff --git a/test/handler/test_sml_handler.vader b/test/handler/test_sml_handler.vader
new file mode 100644
index 00000000..26c85710
--- /dev/null
+++ b/test/handler/test_sml_handler.vader
@@ -0,0 +1,91 @@
+Before:
+  runtime ale_linters/sml/smlnj.vim
+
+Execute (Testing on EOF error):
+  AssertEqual [
+  \ {
+  \   'bufnr': 42,
+  \   'lnum': 2,
+  \   'col': 15,
+  \   'type': 'E',
+  \   'text': 'Error: syntax error found at EOF',
+  \ },
+  \],
+  \ ale_linters#sml#smlnj#Handle(42, [
+  \ "Standard ML of New Jersey v110.78 [built: Thu Jul 23 11:21:58 2015]",
+  \ "[opening a.sml]",
+  \ "a.sml:2.16 Error: syntax error found at EOF",
+  \ '/usr/lib/smlnj/bin/sml: Fatal error -- Uncaught exception Compile with "syntax error" raised at ../compiler/Parse/main/smlfile.sml:15.24-15.46',
+  \])
+
+Execute (Testing if the handler can handle multiple errors on the same line):
+  AssertEqual [
+  \ {
+  \   'bufnr': 42,
+  \   'lnum': 1,
+  \   'col': 5,
+  \   'type': 'E',
+  \   'text': "Error: can't find function arguments in clause",
+  \ },
+  \ {
+  \   'bufnr': 42,
+  \   'lnum': 1,
+  \   'col': 12,
+  \   'type': 'E',
+  \   'text': 'Error: unbound variable or constructor: wow',
+  \ },
+  \],
+  \ ale_linters#sml#smlnj#Handle(42, [
+  \ "Standard ML of New Jersey v110.78 [built: Thu Jul 23 11:21:58 2015]",
+  \ "[opening test.sml]",
+  \ "a.sml:1.6-1.10 Error: can't find function arguments in clause",
+  \ "a.sml:1.13-1.16 Error: unbound variable or constructor: wow",
+  \ "/usr/lib/smlnj/bin/sml: Fatal error -- Uncaught exception Error with 0",
+  \ "raised at ../compiler/TopLevel/interact/evalloop.sml:66.19-66.27",
+  \])
+
+Execute (Testing rarer errors):
+  AssertEqual [
+  \ {
+  \   'bufnr': 42,
+  \   'lnum': 5,
+  \   'col': 18,
+  \   'type': 'E',
+  \   'text': "Error: syntax error found at ID",
+  \ },
+  \ {
+  \   'bufnr': 42,
+  \   'lnum': 7,
+  \   'col': 0,
+  \   'type': 'E',
+  \   'text': "Error: value type in structure doesn't match signature spec",
+  \ },
+  \],
+  \ ale_linters#sml#smlnj#Handle(42, [
+  \ "Standard ML of New Jersey v110.78 [built: Thu Jul 23 11:21:58 2015]",
+  \ "[opening test.sml]",
+  \ "a.sml:5.19 Error: syntax error found at ID",
+  \ "a.sml:7.1-9.27 Error: value type in structure doesn't match signature spec",
+  \ "/usr/lib/smlnj/bin/sml: Fatal error -- Uncaught exception Error with 0",
+  \ "raised at ../compiler/TopLevel/interact/evalloop.sml:66.19-66.27",
+  \])
+
+Execute (Testing a warning):
+  AssertEqual [
+  \ {
+  \   'bufnr': 42,
+  \   'lnum': 4,
+  \   'col': 4,
+  \   'type': 'W',
+  \   'text': "Warning: match nonexhaustive",
+  \ },
+  \],
+  \ ale_linters#sml#smlnj#Handle(42, [
+  \ "Standard ML of New Jersey v110.78 [built: Thu Jul 23 11:21:58 2015]",
+  \ "[opening a.sml]",
+  \ "a.sml:4.5-4.12 Warning: match nonexhaustive",
+            \ "0 => ...",
+  \ "val f = fn : int -> int",
+  \ "-",
+  \])
+