Get rid of t/99-kwalitee.t

New versions started to throw warnings if specific environment variables
are not set. This may be appropriate for dist manager users, but not really
for the rest of the world.

Since kwalitee metrics are either obvious or useless, there's no big loss
in removing this test anyway.

diff --git a/MANIFEST b/MANIFEST
index 70c1d21fd7267c1e0ad5e1934e261868c8b50fa7..5e5947e59fdfb5a6697d8e52d5ed5c0c6bb32f1c 100644 (file)
--- a/MANIFEST
+++ b/MANIFEST
@@ -39,5 +39,4 @@ t/81-suppressions-demangle.t
 t/91-pod.t
 t/92-pod-coverage.t
 t/95-portability-files.t
-t/99-kwalitee.t
 t/lib/Test/Valgrind/Test/Action.pm

diff --git a/t/99-kwalitee.t b/t/99-kwalitee.t
deleted file mode 100644 (file)
index 4a66457..0000000
--- a/t/99-kwalitee.t
+++ /dev/null
@@ -1,27 +0,0 @@
-#!perl
-
-use strict;
-use warnings;
-
-use Test::More;
-
-eval { require Parse::RecDescent; 'Parse::RecDescent'->VERSION('1.967006') }
-  or plan skip_all => 'Parse::RecDescent version 1.967006 or greater required';
-
-eval { require Module::ExtractUse; 'Module::ExtractUse'->VERSION('0.24') }
-  or plan skip_all => 'Module::ExtractUse version 0.24 or greater required';
-
-eval { require Test::Kwalitee; 1 }
-  or plan skip_all => 'Test::Kwalitee required';
-
-SKIP: {
- eval { Test::Kwalitee->import(); };
- if (my $err = $@) {
-  1 while chomp $err;
-  require Test::Builder;
-  my $Test = Test::Builder->new;
-  my $plan = $Test->has_plan;
-  $Test->skip_all($err) if not defined $plan or $plan eq 'no_plan';
-  skip $err => $plan - $Test->current_test;
- }
-}