From: Vincent Pit <vince@profvince.com>
Date: Tue, 6 Aug 2013 03:09:32 +0000 (-0300)
Subject: Get rid of t/99-kwalitee.t
X-Git-Url: http://git.vpit.fr/?p=perl%2Fmodules%2FSub-Op.git;a=commitdiff_plain;h=4f98d51f37f3534320ae1cca9c323ef2988770ab

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 4e360b0..1e09ba2 100644
--- a/MANIFEST
+++ b/MANIFEST
@@ -14,7 +14,6 @@ t/12-prototype.t
 t/91-pod.t
 t/92-pod-coverage.t
 t/95-portability-files.t
-t/99-kwalitee.t
 t/Sub-Op-LexicalSub/Makefile.PL
 t/Sub-Op-LexicalSub/LexicalSub.xs
 t/Sub-Op-LexicalSub/lib/Sub/Op/LexicalSub.pm
diff --git a/t/99-kwalitee.t b/t/99-kwalitee.t
deleted file mode 100644
index 24d716b..0000000
--- a/t/99-kwalitee.t
+++ /dev/null
@@ -1,21 +0,0 @@
-#!perl
-
-use strict;
-use warnings;
-
-use Test::More;
-
-eval "use Test::Kwalitee 1.01 ()";
-plan(skip_all => 'Test::Kwalitee 1.01 not installed') if $@;
-
-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;
- }
-}