test 14_warnings.t fails under cpanm set's $ENV{AUTOMATED_TESTING} true

[AnotherOneAckap]

see Changeset 17365.
Padre::Document::Perl::Beginner doesn't like 'Padre/Wx/Scintilla?.pm'
need to add it to %skip_files

Index: 14_warnings.t
===================================================================
--- 14_warnings.t	(revision 18156)
+++ 14_warnings.t	(working copy)
@@ -48,6 +48,7 @@
 	'Padre/Wx/Constant.pm'              => 'Bad substitution',
 	'Padre/Wx/Theme.pm'                 => 'Bad substitution',
 	'Padre/Wx/Dialog/Patch.pm'          => 'equals in an if',
+	'Padre/Wx/Scintilla.pm'             => 'PHP language keywords'
 );
 
 my @files = File::Find::Rule->relative->file->name('*.pm')->in('lib');

[bowtie]

Perl 5.15.7
Padre Updated to revision 18158.

kevin@vinny:~/src/Padre/Padre$ prove -lv t/14_warnings.t

not ok 73 - Check Padre/Wx/Scintilla?.pm

# Failed test 'Check Padre/Wx/Scintilla?.pm'
# at t/14_warnings.t line 68.
# got: 'Line 201: 'elseif' is wrong syntax, correct if 'elsif'.'
# expected:

test's are confused as this refers to keywords, not perl it's self
php line 201,
vbscript line 321,

[azawawi]

Is this fixed or not in trunk?

[bowtie]

using perl 5.15.8
export AUTOMATED_TESTING=1
~/.cpan/build/Padre-0.94-Zcd3Os$ prove -lv t/14_warnings.t

not ok 71 - Check Padre/Wx/Scintilla.pm

#   Failed test 'Check Padre/Wx/Scintilla.pm'
#   at t/14_warnings.t line 68.
#          got: 'Line 201: 'elseif' is wrong syntax, correct if 'elsif'.'
#     expected: ''

As far as I could ascertain, the test is just wrong, as this is not an error as it's php

should we just remove this test?

[bowtie]

This only happens when AUTOMATED_TESTING=1

sorry, I forgot that.

otherwise it passes, ok

[bowtie]

fixed in r19118

added Padre/Wx/Scintilla?.pm to skip list