about summary refs log tree commit
diff options
context:
space:
mode:
authorSteve Hay <steve.m.hay@googlemail.com>2014-10-09 20:59:42 +0100
committerSteve Hay <steve.m.hay@googlemail.com>2014-10-09 20:59:42 +0100
commit70d9fa0ed3c830bfe1a92fd122005d557c3d6e8b (patch)
tree4d96e87134c33d3357586c0b8d0ca48ecfd32969
parent513339073055a1d0742a4e370fe967e16896e45d (diff)
downloadperl-libnet-70d9fa0ed3c830bfe1a92fd122005d557c3d6e8b.tar.gz
Skip Perl Critic, Pod and Pod Coverage tests unless AUTHOR_TESTING
Even after telling Perl::Critic to use its default policy configuration
we may still see unexpected failures from CPAN Testers resulting from
different versions of Perl::Critic having different policies.

Pod tests may likewise have unexpected failures caused by users having
different versions of the testing modules to the author.

None of these tests are relevant to users anyway, so disable them by
checking for $ENV{AUTHOR_TESTING}, as per the Lancaster Consensus.

Resolves CPAN RT#99399.
-rw-r--r--Changes2
-rw-r--r--t/critic.t2
-rw-r--r--t/pod.t2
-rw-r--r--t/pod_coverage.t2
4 files changed, 8 insertions, 0 deletions
diff --git a/Changes b/Changes
index d6100d5..721e660 100644
--- a/Changes
+++ b/Changes
@@ -1,5 +1,7 @@
 libnet 3.00  -- TODO
 
+  * Skip Perl Critic, Pod and Pod Coverage tests unless AUTHOR_TESTING.
+    [Resolves CPAN RT#99399]
   * Synchronize all $VERSIONs to the distribution's version number, bumping that
     to 3.00 so that no $VERSIONs end up going backwards.
 
diff --git a/t/critic.t b/t/critic.t
index 123dbab..c3fd215 100644
--- a/t/critic.t
+++ b/t/critic.t
@@ -27,6 +27,8 @@ use Test::More;
 #===============================================================================
 
 MAIN: {
+    plan skip_all => 'Author testing only' unless $ENV{AUTHOR_TESTING};
+
     my $ok = eval {
         require Test::Perl::Critic;
         Test::Perl::Critic->import(-profile => '');
diff --git a/t/pod.t b/t/pod.t
index 5af23ae..b407b52 100644
--- a/t/pod.t
+++ b/t/pod.t
@@ -27,6 +27,8 @@ use Test::More;
 #===============================================================================
 
 MAIN: {
+    plan skip_all => 'Author testing only' unless $ENV{AUTHOR_TESTING};
+
     my $ok = eval {
         require Test::Pod;
         Test::Pod->import();
diff --git a/t/pod_coverage.t b/t/pod_coverage.t
index d72778c..56f6401 100644
--- a/t/pod_coverage.t
+++ b/t/pod_coverage.t
@@ -27,6 +27,8 @@ use Test::More;
 #===============================================================================
 
 MAIN: {
+    plan skip_all => 'Author testing only' unless $ENV{AUTHOR_TESTING};
+
     my $ok = eval {
         require Test::Pod::Coverage;
         Test::Pod::Coverage->import();