Login
ProhibitSingleCharAlternation test was failing when
authorElliot Shank <perl@galumph.com>
Tue, 22 Jul 2008 13:44:20 +0000 (13:44 +0000)
committerElliot Shank <perl@galumph.com>
Tue, 22 Jul 2008 13:44:20 +0000 (13:44 +0000)
Regexp::Parser wasn't installed, incorrectly.  In addition
to fixing the test, generate a
"_without_optional_dependencies" wrapper around
t/20_policies.t.

Changes
inc/Perl/Critic/BuildUtilities.pm
t/RegularExpressions/ProhibitSingleCharAlternation.run

diff --git a/Changes b/Changes
index 74ed479..dc50819 100644 (file)
--- a/Changes
+++ b/Changes
@@ -1,3 +1,8 @@
+[1.090] Released on 2008-07-22
+
+    Bug Fixes:
+    * Test was incorrectly failing when Regexp::Parser wasn't installed.
+
 [1.089] Released on 2008-07-21
 
     Minor Enhancements:
 [1.089] Released on 2008-07-21
 
     Minor Enhancements:
@@ -20,7 +25,7 @@
       rcs_keywords option to allow for the common case where those require
       dollar signs.
 
       rcs_keywords option to allow for the common case where those require
       dollar signs.
 
-    Bug fixes:
+    Bug Fixes:
     * BuiltinFunctions::ProhibitSleepViaSelect would complain if there were
       three undefs as arguments to select(), but one of them was the timeout.
       RT #37416.
     * BuiltinFunctions::ProhibitSleepViaSelect would complain if there were
       three undefs as arguments to select(), but one of them was the timeout.
       RT #37416.
index db37bcd..7d14f3b 100644 (file)
@@ -69,6 +69,7 @@ sub test_wrappers_to_generate {
         t/13_bundled_policies.t
         t/14_policy_parameters.t
         t/15_statistics.t
         t/13_bundled_policies.t
         t/14_policy_parameters.t
         t/15_statistics.t
+        t/20_policies.t
         t/20_policy_podspelling.t
         t/20_policy_requiretidycode.t
         xt/author/80_policysummary.t
         t/20_policy_podspelling.t
         t/20_policy_requiretidycode.t
         xt/author/80_policysummary.t
index 94445ba..2da8bcb 100644 (file)
@@ -18,6 +18,7 @@ m/\A (?: [a] ) \z/xms; # bad form, but not a violation of this policy
 
 ## name warnings reported by users (App::Ack)
 ## failures 1
 
 ## name warnings reported by users (App::Ack)
 ## failures 1
+## optional_modules Regexp::Parser
 ## cut
 
 return ('shell',TEXT)  if $header =~ /\b(?:ba|c|k|z)?sh\b/;
 ## cut
 
 return ('shell',TEXT)  if $header =~ /\b(?:ba|c|k|z)?sh\b/;