From 82fde0ea51292d9356f153ef36a51df832fa58d2 Mon Sep 17 00:00:00 2001
From: w0rp <devw0rp@gmail.com>
Date: Mon, 27 Mar 2017 12:56:54 +0100
Subject: [PATCH] Use the same formatting as other files for the dogma linter
 file, and cover the Handler function with tests

---
 ale_linters/elixir/dogma.vim          | 65 ++++++++++++++-------------
 test/handler/test_dogma_handler.vader | 28 ++++++++++++
 2 files changed, 61 insertions(+), 32 deletions(-)
 create mode 100644 test/handler/test_dogma_handler.vader

diff --git a/ale_linters/elixir/dogma.vim b/ale_linters/elixir/dogma.vim
index a7d5b27..ba21e37 100644
--- a/ale_linters/elixir/dogma.vim
+++ b/ale_linters/elixir/dogma.vim
@@ -1,44 +1,45 @@
 " Author: archseer - https://github.com/archSeer
 
 function! ale_linters#elixir#dogma#Handle(buffer, lines) abort
-  " Matches patterns line the following:
-  "
-  " lib/filename.ex:19:7: F: Pipe chain should start with a raw value.
-  let l:pattern = '\v:(\d+):?(\d+)?: (.): (.+)$'
-  let l:output = []
+    " Matches patterns line the following:
+    "
+    " lib/filename.ex:19:7: F: Pipe chain should start with a raw value.
+    let l:pattern = '\v:(\d+):?(\d+)?: (.): (.+)$'
+    let l:output = []
 
-  for l:line in a:lines
-    let l:match = matchlist(l:line, l:pattern)
+    for l:line in a:lines
+        let l:match = matchlist(l:line, l:pattern)
 
-    if len(l:match) == 0
-        continue
-    endif
+        if len(l:match) == 0
+            continue
+        endif
 
-    let l:type = l:match[3]
-    let l:text = l:match[4]
+        let l:type = l:match[3]
+        let l:text = l:match[4]
 
-    if l:type ==# 'C'
-      let l:type = 'E'
-    elseif l:type ==# 'R'
-      let l:type = 'W'
-    endif
+        if l:type ==# 'C'
+            let l:type = 'E'
+        elseif l:type ==# 'R'
+            let l:type = 'W'
+        endif
 
-    " vcol is Needed to indicate that the column is a character.
-    call add(l:output, {
-    \   'bufnr': a:buffer,
-    \   'lnum': l:match[1] + 0,
-    \   'col': l:match[2] + 0,
-    \   'type': l:type,
-    \   'text': l:text,
-    \})
-  endfor
+        " vcol is Needed to indicate that the column is a character.
+        call add(l:output, {
+        \   'bufnr': a:buffer,
+        \   'lnum': l:match[1] + 0,
+        \   'col': l:match[2] + 0,
+        \   'type': l:type,
+        \   'text': l:text,
+        \})
+    endfor
 
-  return l:output
+    return l:output
 endfunction
 
 call ale#linter#Define('elixir', {
-      \ 'name': 'dogma',
-      \ 'executable': 'mix',
-      \ 'command': 'mix dogma %s --format=flycheck',
-      \ 'lint_file': 1,
-      \ 'callback': 'ale_linters#elixir#dogma#Handle' })
+\   'name': 'dogma',
+\   'executable': 'mix',
+\   'command': 'mix dogma %s --format=flycheck',
+\   'lint_file': 1,
+\   'callback': 'ale_linters#elixir#dogma#Handle',
+\})
diff --git a/test/handler/test_dogma_handler.vader b/test/handler/test_dogma_handler.vader
new file mode 100644
index 0000000..ee9795e
--- /dev/null
+++ b/test/handler/test_dogma_handler.vader
@@ -0,0 +1,28 @@
+Execute(The dogma handler should parse lines correctly):
+  runtime ale_linters/elixir/dogma.vim
+
+  AssertEqual
+  \ [
+  \   {
+  \     'bufnr': 347,
+  \     'lnum': 18,
+  \     'col': 5,
+  \     'text': 'Some error',
+  \     'type': 'E',
+  \   },
+  \   {
+  \     'bufnr': 347,
+  \     'lnum': 19,
+  \     'col': 7,
+  \     'text': 'Some warning',
+  \     'type': 'W',
+  \   },
+  \ ],
+  \ ale_linters#elixir#dogma#Handle(347, [
+  \   'This line should be ignored completely',
+  \   'lib/filename.ex:18:5: C: Some error',
+  \   'lib/filename.ex:19:7: R: Some warning',
+  \ ])
+
+After:
+  call ale#linter#Reset()