Merge pull request #564 from adriaanzon/php-end-columns

Add end columns on php linter

1 files changed, 30 insertions, 11 deletions

diff --git a/test/handler/test_php_handler.vader b/test/handler/test_php_handler.vader
index 5dac0948..0d4d4278 100644
--- a/test/handler/test_php_handler.vader
+++ b/test/handler/test_php_handler.vader
@@ -1,3 +1,6 @@
+Before:
+  runtime ale_linters/php/php.vim
+
 Given (Some invalid lines of PHP):
   [foo;]
   class Foo { / }
@@ -5,31 +8,52 @@ Given (Some invalid lines of PHP):
   ['foo' 'bar']
   function count() {}
 
-Execute(The php handler should parse lines correctly):
-  runtime ale_linters/php/php.vim
-
+Execute(The php handler should calculate column numbers):
   AssertEqual
   \ [
   \   {
   \     'lnum': 1,
   \     'col': 5,
+  \     'end_col': 5,
   \     'text': "syntax error, unexpected ';', expecting ']'",
   \   },
   \   {
   \     'lnum': 2,
   \     'col': 13,
+  \     'end_col': 13,
   \     'text': "syntax error, unexpected '/', expecting function (T_FUNCTION) or const (T_CONST)",
   \   },
   \   {
   \     'lnum': 3,
   \     'col': 5,
+  \     'end_col': 5,
   \     'text': "syntax error, unexpected ')'",
   \   },
   \   {
   \     'lnum': 4,
   \     'col': 8,
+  \     'end_col': 12,
   \     'text': "syntax error, unexpected ''bar'' (T_CONSTANT_ENCAPSED_STRING), expecting ']'",
   \   },
+  \ ],
+  \ ale_linters#php#php#Handle(347, [
+  \   "This line should be ignored completely",
+  \   "Parse error:  syntax error, unexpected ';', expecting ']' in - on line 1",
+  \   "Parse error:  syntax error, unexpected '/', expecting function (T_FUNCTION) or const (T_CONST) in - on line 2",
+  \   "Parse error:  syntax error, unexpected ')' in - on line 3",
+  \   "Parse error:  syntax error, unexpected ''bar'' (T_CONSTANT_ENCAPSED_STRING), expecting ']' in - on line 4",
+  \ ])
+
+Execute (The php handler should ignore lines starting with 'PHP Parse error'):
+  AssertEqual
+  \ [],
+  \ ale_linters#php#php#Handle(347, [
+  \   "PHP Parse error:  syntax error, This line should be ignored completely in - on line 1",
+  \ ])
+
+Execute (The php handler should parse lines without column indication):
+  AssertEqual
+  \ [
   \   {
   \     'lnum': 5,
   \     'col': 0,
@@ -47,15 +71,10 @@ Execute(The php handler should parse lines correctly):
   \   },
   \ ],
   \ ale_linters#php#php#Handle(347, [
-  \   'This line should be ignored completely',
-  \   "PHP Parse error:  syntax error, This line should be ignored completely in - on line 1",
-  \   "Parse error:  syntax error, unexpected ';', expecting ']' in - on line 1",
-  \   "Parse error:  syntax error, unexpected '/', expecting function (T_FUNCTION) or const (T_CONST) in - on line 2",
-  \   "Parse error:  syntax error, unexpected ')' in - on line 3",
-  \   "Parse error:  syntax error, unexpected ''bar'' (T_CONSTANT_ENCAPSED_STRING), expecting ']' in - on line 4",
+  \   "This line should be ignored completely",
   \   "Fatal error:  Cannot redeclare count() in - on line 5",
-  \   'Parse error:  syntax error, unexpected end of file in - on line 21',
-  \   'Parse error: Invalid numeric literal in - on line 47',
+  \   "Parse error:  syntax error, unexpected end of file in - on line 21",
+  \   "Parse error: Invalid numeric literal in - on line 47",
   \ ])
 
 After: