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 a711a59cd40fcf5cfa6872d2a29d3e2aa8c6b5a9..b24bd74dcf40ed56ac89f2ba0eab1d41c5e5172a 100644 (file)
--- a/MANIFEST
+++ b/MANIFEST
@@ -187,7 +187,6 @@ t/91-pod.t
 t/92-pod-coverage.t
 t/93-pod-spelling.t
 t/95-portability-files.t
 t/92-pod-coverage.t
 t/93-pod-spelling.t
 t/95-portability-files.t

-t/99-kwalitee.t

 t/lib/VPIT/TestHelpers.pm
 t/re-engine-Hooks-TestDist/Makefile.PL
 t/re-engine-Hooks-TestDist/TestDist.xs
 t/lib/VPIT/TestHelpers.pm
 t/re-engine-Hooks-TestDist/Makefile.PL
 t/re-engine-Hooks-TestDist/TestDist.xs

diff --git a/t/99-kwalitee.t b/t/99-kwalitee.t
deleted file mode 100644 (file)
index 2de8e0b..0000000
--- a/t/99-kwalitee.t
+++ /dev/null
@@ -1,29 +0,0 @@
-#!perl
-
-use strict;
-use warnings;
-
-use Test::More;
-
-use lib 't/lib';
-use VPIT::TestHelpers;
-
-my $guard = VPIT::TestHelpers::Guard->new(
- sub { unlink for glob 'Debian_CPANTS.txt*' }
-);
-
-load_or_skip_all('Parse::RecDescent',  '1.967006');
-load_or_skip_all('Module::ExtractUse', '0.24'    );
-load_or_skip_all('Test::Kwalitee',     '1.01'    );
-
-SKIP: {
- eval { Test::Kwalitee->import(tests => [ '-use_strict' ]); };
- 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;
- }
-}