Merge pull request #1111 from elebow/perlcritic-only-warnings

Perlcritic: All issues are warnings

ale_linters/perl/perlcritic.vim

@@ -61,6 +61,7 @@ function! ale_linters#perl#perlcritic#Handle(buffer, lines) abort
         \   'lnum': l:match[1],
         \   'col': l:match[2],
         \   'text': l:match[3],
+        \   'type': 'W'
         \})
     endfor
 

test/handler/test_perlcritic_handler.vader

@@ -0,0 +1,20 @@
+Before:
+  runtime ale_linters/perl/perlcritic.vim
+
+After:
+  call ale#linter#Reset()
+
+Execute(The Perl::Critic handler should create all issues as warnings):
+  AssertEqual
+  \  [
+  \    {
+  \      'lnum': '21',
+  \      'col': '17',
+  \      'text': 'Regular expression without "/m" flag',
+  \      'type': 'W',
+  \    }
+  \  ],
+  \  ale_linters#perl#perlcritic#Handle(99, [
+  \      '21:17 Regular expression without "/m" flag'
+  \  ])
+