From c697ef05bb3a084588fb6033c2bf915932917afc Mon Sep 17 00:00:00 2001
From: Bjorn Neergaard <bjorn@neersighted.com>
Date: Thu, 20 Oct 2016 22:52:25 -0500
Subject: Add PEP8 handler test.

---
 autoload/ale/handlers.vim       |  4 ++--
 test/test_common_handlers.vader | 28 ++++++++++++++++++++++++++++
 2 files changed, 30 insertions(+), 2 deletions(-)

diff --git a/autoload/ale/handlers.vim b/autoload/ale/handlers.vim
index a8d2572f..9ac0c192 100644
--- a/autoload/ale/handlers.vim
+++ b/autoload/ale/handlers.vim
@@ -99,11 +99,11 @@ function! ale#handlers#HandleCppCheckFormat(buffer, lines) abort
         \   'nr': -1,
         \})
     endfor
-    return l:output
 
+    return l:output
 endfunction
 
-function! ale#handlers#HandlePEP8Format(buffer, lines)
+function! ale#handlers#HandlePEP8Format(buffer, lines) abort
     " Matches patterns line the following:
     "
     " Matches patterns line the following:
diff --git a/test/test_common_handlers.vader b/test/test_common_handlers.vader
index 6fee00b8..ed3b1647 100644
--- a/test/test_common_handlers.vader
+++ b/test/test_common_handlers.vader
@@ -26,6 +26,34 @@ Then (The loclist should be correct):
   \ },
   \], g:loclist
 
+Execute (Run HandlePEP8Format):
+  let g:loclist = ale#handlers#HandlePEP8Format(42, [
+  \ "stdin:6:6: E111 indentation is not a multiple of four",
+  \ "test.yml:35: [EANSIBLE0002] Trailing whitespace",
+  \])
+
+Then (The loclist should be correct):
+  AssertEqual [
+  \ {
+  \   'bufnr': 42,
+  \   'vcol': 0,
+  \   'nr': -1,
+  \   'lnum': 6,
+  \   'col': 6,
+  \   'type': 'E',
+  \   'text': 'E111: indentation is not a multiple of four',
+  \ },
+  \ {
+  \   'bufnr': 42,
+  \   'vcol': 0,
+  \   'nr': -1,
+  \   'lnum': 35,
+  \   'col': 0,
+  \   'type': 'E',
+  \   'text': "EANSIBLE0002: Trailing whitespace",
+  \ },
+  \], g:loclist
+
 Execute (Run HandleGCCFormat):
   let g:loclist = ale#handlers#HandleGCCFormat(42, [
   \ '<stdin>:8:5: warning: conversion lacks type at end of format [-Wformat=]',
-- 
cgit v1.2.3