Blame SOURCES/Package-Stash-0.37-old-Test::More.patch

385ffa
--- t/addsub.t
385ffa
+++ t/addsub.t
385ffa
@@ -2,7 +2,7 @@
385ffa
 use strict;
385ffa
 use warnings;
385ffa
 use lib 't/lib';
385ffa
-use Test::More;
385ffa
+use Test::More tests => 7;
385ffa
 use Test::Fatal;
385ffa
 
385ffa
 BEGIN { $^P |= 0x210 } # PERLDBf_SUBLINE
385ffa
@@ -42,5 +42,3 @@ $foo_stash->add_symbol(
385ffa
 
385ffa
 is $DB::sub{'Foo::dunk'}, sprintf "%s:%d-%d", "FileName", 100, 199,
385ffa
     '... got the right %DB::sub value for dunk with specified args';
385ffa
-
385ffa
-done_testing;
385ffa
--- t/anon-basic.t
385ffa
+++ t/anon-basic.t
385ffa
@@ -16,6 +16,8 @@ BEGIN {
385ffa
 use Test::Requires 'Package::Anon';
385ffa
 use Symbol;
385ffa
 
385ffa
+plan tests => 122;
385ffa
+
385ffa
 my $Foo = Package::Anon->new('Foo');
385ffa
 $Foo->{SOME_CONSTANT} = \1;
385ffa
 
385ffa
@@ -402,5 +404,3 @@ $Quuux->{quuuux} = -1;
385ffa
 
385ffa
 is_deeply([Package::Stash->new('Quuux')->list_all_symbols], [],
385ffa
           "Quuux:: isn't touched");
385ffa
-
385ffa
-done_testing;
385ffa
--- t/anon.t
385ffa
+++ t/anon.t
385ffa
@@ -16,6 +16,8 @@ BEGIN {
385ffa
 use Test::Requires 'Package::Anon';
385ffa
 use Symbol;
385ffa
 
385ffa
+plan tests => 7;
385ffa
+
385ffa
 my $anon = Package::Anon->new;
385ffa
 my $stash = Package::Stash->new($anon);
385ffa
 my $obj = $anon->bless({});
385ffa
@@ -47,5 +49,3 @@ my $obj = $anon->bless({});
385ffa
         qr/Undefined subroutine \&__ANON__::baz called/
385ffa
     );
385ffa
 }
385ffa
-
385ffa
-done_testing;
385ffa
--- t/bare-anon-basic.t
385ffa
+++ t/bare-anon-basic.t
385ffa
@@ -17,6 +17,8 @@ BEGIN {
385ffa
 
385ffa
 use Symbol;
385ffa
 
385ffa
+plan tests => 10;
385ffa
+
385ffa
 my $Foo = {};
385ffa
 $Foo->{SOME_CONSTANT} = \1;
385ffa
 
385ffa
@@ -396,5 +398,3 @@ $Quuux->{quuuux} = -1;
385ffa
 
385ffa
 is_deeply([Package::Stash->new('Quuux')->list_all_symbols], [],
385ffa
           "Quuux:: isn't touched");
385ffa
-
385ffa
-done_testing;
385ffa
--- t/bare-anon.t
385ffa
+++ t/bare-anon.t
385ffa
@@ -17,6 +17,8 @@ BEGIN {
385ffa
 
385ffa
 use Symbol;
385ffa
 
385ffa
+plan tests => 10;
385ffa
+
385ffa
 my $anon = {};
385ffa
 my $stash = Package::Stash->new($anon);
385ffa
 # no way to bless something into a hashref yet
385ffa
@@ -61,5 +63,3 @@ my $stash = Package::Stash->new($anon);
385ffa
         undef
385ffa
     );
385ffa
 }
385ffa
-
385ffa
-done_testing;
385ffa
--- t/basic.t
385ffa
+++ t/basic.t
385ffa
@@ -2,7 +2,7 @@
385ffa
 use strict;
385ffa
 use warnings;
385ffa
 use lib 't/lib';
385ffa
-use Test::More;
385ffa
+use Test::More tests => 131;
385ffa
 use Test::Fatal;
385ffa
 
385ffa
 use Package::Stash;
385ffa
@@ -444,5 +444,3 @@ like(
385ffa
     qr/^Package::Stash->new must be passed the name of the package to access/,
385ffa
     "module name must be a string"
385ffa
 );
385ffa
-
385ffa
-done_testing;
385ffa
--- t/compile-time.t
385ffa
+++ t/compile-time.t
385ffa
@@ -2,8 +2,6 @@
385ffa
 use strict;
385ffa
 use warnings;
385ffa
 use lib 't/lib';
385ffa
-use Test::More;
385ffa
+use Test::More tests => 1;
385ffa
 
385ffa
 use_ok('CompileTime');
385ffa
-
385ffa
-done_testing;
385ffa
--- t/edge-cases.t
385ffa
+++ t/edge-cases.t
385ffa
@@ -2,7 +2,7 @@
385ffa
 use strict;
385ffa
 use warnings;
385ffa
 use lib 't/lib';
385ffa
-use Test::More;
385ffa
+use Test::More tests => 19;
385ffa
 use Test::Fatal;
385ffa
 
385ffa
 use Package::Stash;
385ffa
@@ -109,5 +109,3 @@ SKIP: {
385ffa
     my $stash = Package::Stash->new('HasISA');
385ffa
     is_deeply([$stash->list_all_symbols('SCALAR')], []);
385ffa
 }
385ffa
-
385ffa
-done_testing;
385ffa
--- t/extension.t
385ffa
+++ t/extension.t
385ffa
@@ -2,7 +2,7 @@
385ffa
 use strict;
385ffa
 use warnings;
385ffa
 use lib 't/lib';
385ffa
-use Test::More;
385ffa
+use Test::More tests => 15;
385ffa
 use Test::Fatal;
385ffa
 
385ffa
 {
385ffa
@@ -72,5 +72,3 @@ is(exception {
385ffa
 }, undef, '... created %Foo::baz successfully');
385ffa
 
385ffa
 ok(!defined($Foo::{baz}), '... the %baz slot has still not been created');
385ffa
-
385ffa
-done_testing;
385ffa
--- t/get.t
385ffa
+++ t/get.t
385ffa
@@ -2,7 +2,7 @@
385ffa
 use strict;
385ffa
 use warnings;
385ffa
 use lib 't/lib';
385ffa
-use Test::More;
385ffa
+use Test::More tests => 36;
385ffa
 
385ffa
 use Package::Stash;
385ffa
 use Scalar::Util;
385ffa
@@ -182,5 +182,3 @@ use Scalar::Util;
385ffa
              "stash has the right variable");
385ffa
     }
385ffa
 }
385ffa
-
385ffa
-done_testing;
385ffa
--- t/impl-selection/basic-pp.t
385ffa
+++ t/impl-selection/basic-pp.t
385ffa
@@ -1,7 +1,7 @@
385ffa
 #!/usr/bin/env perl
385ffa
 use strict;
385ffa
 use warnings;
385ffa
-use Test::More;
385ffa
+use Test::More tests => 133;
385ffa
 use Test::Fatal;
385ffa
 
385ffa
 BEGIN { $Package::Stash::IMPLEMENTATION = 'PP' }
385ffa
@@ -448,5 +448,3 @@ like(
385ffa
     qr/^Package::Stash->new must be passed the name of the package to access/,
385ffa
     "module name must be a string"
385ffa
 );
385ffa
-
385ffa
-done_testing;
385ffa
--- t/impl-selection/basic-xs.t
385ffa
+++ t/impl-selection/basic-xs.t
385ffa
@@ -1,7 +1,7 @@
385ffa
 #!/usr/bin/env perl
385ffa
 use strict;
385ffa
 use warnings;
385ffa
-use Test::More;
385ffa
+use Test::More tests => 133;
385ffa
 use Test::Fatal;
385ffa
 use Test::Requires 'Package::Stash::XS';
385ffa
 
385ffa
@@ -449,5 +449,3 @@ like(
385ffa
     qr/^Package::Stash->new must be passed the name of the package to access/,
385ffa
     "module name must be a string"
385ffa
 );
385ffa
-
385ffa
-done_testing;
385ffa
--- t/impl-selection/bug-rt-78272.t
385ffa
+++ t/impl-selection/bug-rt-78272.t
385ffa
@@ -1,7 +1,7 @@
385ffa
 #!/usr/bin/env perl
385ffa
 use strict;
385ffa
 use warnings;
385ffa
-use Test::More;
385ffa
+use Test::More tests => 2;
385ffa
 use Test::Fatal;
385ffa
 
385ffa
 # https://rt.cpan.org/Public/Bug/Display.html?id=78272
385ffa
@@ -22,18 +22,3 @@ like(
385ffa
     'Sanity check: forcing package reload throws the exception again'
385ffa
 );
385ffa
 
385ffa
-is(
385ffa
-    exception {
385ffa
-        $ENV{PACKAGE_STASH_IMPLEMENTATION} = "PP";
385ffa
-        delete $INC{'Package/Stash.pm'};
385ffa
-        require Package::Stash;
385ffa
-        new_ok(
385ffa
-            'Package::Stash' => ['Foo'],
385ffa
-            'Loaded and able to create instances'
385ffa
-        );
385ffa
-    },
385ffa
-    undef,
385ffa
-    'Valid $ENV value loads correctly'
385ffa
-);
385ffa
-
385ffa
-done_testing;
385ffa
--- t/impl-selection/choice.t
385ffa
+++ t/impl-selection/choice.t
385ffa
@@ -1,7 +1,7 @@
385ffa
 #!/usr/bin/env perl
385ffa
 use strict;
385ffa
 use warnings;
385ffa
-use Test::More;
385ffa
+use Test::More tests => 2;
385ffa
 
385ffa
 my $has_xs = eval "require Package::Stash::XS; 1";
385ffa
 
385ffa
@@ -13,5 +13,3 @@ my $expected = $has_xs ? 'XS' : 'PP';
385ffa
 is($Package::Stash::IMPLEMENTATION, $expected,
385ffa
    "autodetected properly: $expected");
385ffa
 can_ok('Package::Stash', 'new');
385ffa
-
385ffa
-done_testing;
385ffa
--- t/impl-selection/env.t
385ffa
+++ t/impl-selection/env.t
385ffa
@@ -1,7 +1,7 @@
385ffa
 #!/usr/bin/env perl
385ffa
 use strict;
385ffa
 use warnings;
385ffa
-use Test::More;
385ffa
+use Test::More tests => 6;
385ffa
 
385ffa
 # XXX: work around dumb core segfault bug when you delete stashes
385ffa
 sub get_impl { eval '$Package::Stash::IMPLEMENTATION' }
385ffa
@@ -35,5 +35,3 @@ SKIP: {
385ffa
     is(get_impl, 'PP', '$ENV takes precedence over $Package::Stash::IMPLEMENTATION');
385ffa
     can_ok('Package::Stash', 'new');
385ffa
 }
385ffa
-
385ffa
-done_testing;
385ffa
--- t/impl-selection/var.t
385ffa
+++ t/impl-selection/var.t
385ffa
@@ -1,7 +1,7 @@
385ffa
 #!/usr/bin/env perl
385ffa
 use strict;
385ffa
 use warnings;
385ffa
-use Test::More;
385ffa
+use Test::More tests => 4;
385ffa
 
385ffa
 # XXX: work around dumb core segfault bug when you delete stashes
385ffa
 sub get_impl { eval '$Package::Stash::IMPLEMENTATION' }
385ffa
@@ -25,5 +25,3 @@ SKIP: {
385ffa
     is(get_impl, 'XS', "autodetected properly: XS");
385ffa
     can_ok('Package::Stash', 'new');
385ffa
 }
385ffa
-
385ffa
-done_testing;
385ffa
--- t/io.t
385ffa
+++ t/io.t
385ffa
@@ -2,7 +2,7 @@
385ffa
 use strict;
385ffa
 use warnings;
385ffa
 use lib 't/lib';
385ffa
-use Test::More;
385ffa
+use Test::More tests => 11;
385ffa
 use Test::Fatal;
385ffa
 
385ffa
 {
385ffa
@@ -47,5 +47,3 @@ use Package::Stash;
385ffa
     ok($stash->has_symbol('baz'), "has baz");
385ffa
     is($stash->get_symbol('baz'), *Foo::foo{IO}, "got the right baz");
385ffa
 }
385ffa
-
385ffa
-done_testing;
385ffa
--- t/isa.t
385ffa
+++ t/isa.t
385ffa
@@ -2,7 +2,7 @@
385ffa
 use strict;
385ffa
 use warnings;
385ffa
 use lib 't/lib';
385ffa
-use Test::More;
385ffa
+use Test::More tests => 8;
385ffa
 
385ffa
 use Package::Stash;
385ffa
 
385ffa
@@ -19,7 +19,10 @@ use Package::Stash;
385ffa
     my $stash = Package::Stash->new('Foo');
385ffa
     my @ISA = ('Bar');
385ffa
     @{$stash->get_or_add_symbol('@ISA')} = @ISA;
385ffa
+    SKIP: {
385ffa
+    skip "Test::More >= 0.88 needed for isa_ok() tests on classes", 1 if $Test::More::VERSION < 0.88;
385ffa
     isa_ok('Foo', 'Bar');
385ffa
+    }
385ffa
     isa_ok(bless({}, 'Foo'), 'Bar');
385ffa
 }
385ffa
 
385ffa
@@ -33,17 +36,21 @@ use Package::Stash;
385ffa
     {
385ffa
         my $isa = $stash->get_or_add_symbol('@ISA');
385ffa
         @$isa = ('Baz');
385ffa
+        SKIP: {
385ffa
+        skip "Test::More >= 0.88 needed for isa_ok() tests on classes", 1 if $Test::More::VERSION < 0.88;
385ffa
         isa_ok('Quux', 'Baz');
385ffa
+        }
385ffa
         isa_ok(bless({}, 'Quux'), 'Baz');
385ffa
         ok(Quux->can('foo'));
385ffa
     }
385ffa
     {
385ffa
         my $isa = $stash->get_or_add_symbol('@ISA');
385ffa
         @$isa = ('Bar');
385ffa
+        SKIP: {
385ffa
+        skip "Test::More >= 0.88 needed for isa_ok() tests on classes", 1 if $Test::More::VERSION < 0.88;
385ffa
         isa_ok('Quux', 'Bar');
385ffa
+        }
385ffa
         isa_ok(bless({}, 'Quux'), 'Bar');
385ffa
         ok(Quux->can('bar'));
385ffa
     }
385ffa
 }
385ffa
-
385ffa
-done_testing;
385ffa
--- t/magic.t
385ffa
+++ t/magic.t
385ffa
@@ -2,7 +2,7 @@
385ffa
 use strict;
385ffa
 use warnings;
385ffa
 use lib 't/lib';
385ffa
-use Test::More;
385ffa
+use Test::More tests => 14;
385ffa
 
385ffa
 use Package::Stash;
385ffa
 
385ffa
@@ -77,4 +77,3 @@ SKIP: {
385ffa
     is($store, 0, "get_or_add_symbol stores (populated slot)");
385ffa
 }
385ffa
 
385ffa
-done_testing;
385ffa
--- t/paamayim_nekdotayim.t
385ffa
+++ t/paamayim_nekdotayim.t
385ffa
@@ -2,7 +2,7 @@
385ffa
 use strict;
385ffa
 use warnings;
385ffa
 use lib 't/lib';
385ffa
-use Test::More;
385ffa
+use Test::More tests => 3;
385ffa
 use Test::Fatal;
385ffa
 
385ffa
 use Package::Stash;
385ffa
@@ -24,5 +24,3 @@ like(
385ffa
     qr/^Variable names may not contain ::/,
385ffa
     "can't add symbol with ::"
385ffa
 );
385ffa
-
385ffa
-done_testing;
385ffa
--- t/scalar-values.t
385ffa
+++ t/scalar-values.t
385ffa
@@ -2,7 +2,7 @@
385ffa
 use strict;
385ffa
 use warnings;
385ffa
 use lib 't/lib';
385ffa
-use Test::More;
385ffa
+use Test::More tests => 14;
385ffa
 use Test::Fatal;
385ffa
 
385ffa
 use B;
385ffa
@@ -49,5 +49,3 @@ my $vstring = v1.2.3;
385ffa
 is(reftype(\$vstring), ($] < 5.010 ? 'SCALAR' : 'VSTRING'));
385ffa
 is(exception { $Bar->add_symbol('$vstring', \$vstring) }, undef,
385ffa
    "can add vstring values");
385ffa
-
385ffa
-done_testing;
385ffa
--- t/stash-deletion.t
385ffa
+++ t/stash-deletion.t
385ffa
@@ -2,7 +2,7 @@
385ffa
 use strict;
385ffa
 use warnings;
385ffa
 use lib 't/lib';
385ffa
-use Test::More;
385ffa
+use Test::More tests => 2;
385ffa
 
385ffa
 use Package::Stash;
385ffa
 
385ffa
@@ -20,5 +20,3 @@ use Package::Stash;
385ffa
     }
385ffa
     ok(!$delete->has_symbol('&bar'), "method goes away when stash is deleted");
385ffa
 }
385ffa
-
385ffa
-done_testing;
385ffa
--- t/synopsis.t
385ffa
+++ t/synopsis.t
385ffa
@@ -2,7 +2,7 @@
385ffa
 use strict;
385ffa
 use warnings;
385ffa
 use lib 't/lib';
385ffa
-use Test::More;
385ffa
+use Test::More tests => 3;
385ffa
 
385ffa
 use Package::Stash;
385ffa
 
385ffa
@@ -15,5 +15,3 @@ $stash->add_symbol('%foo', {bar => 1});
385ffa
 ok(!$stash->has_symbol('$foo'), "doesn't have anything in scalar slot");
385ffa
 my $namespace = $stash->namespace;
385ffa
 is_deeply(*{ $namespace->{foo} }{HASH}, {bar => 1}, "namespace works properly");
385ffa
-
385ffa
-done_testing;
385ffa
--- t/warnings-taint.t
385ffa
+++ t/warnings-taint.t
385ffa
@@ -2,7 +2,7 @@
385ffa
 use strict;
385ffa
 use warnings;
385ffa
 use lib 't/lib';
385ffa
-use Test::More;
385ffa
+use Test::More tests => 1;
385ffa
 
385ffa
 use Package::Stash;
385ffa
 
385ffa
@@ -19,4 +19,3 @@ BEGIN {
385ffa
 
385ffa
 is($warnings, '');
385ffa
 
385ffa
-done_testing;
385ffa
--- t/warnings.t
385ffa
+++ t/warnings.t
385ffa
@@ -2,7 +2,7 @@
385ffa
 use strict;
385ffa
 use warnings;
385ffa
 use lib 't/lib';
385ffa
-use Test::More;
385ffa
+use Test::More tests => 1;
385ffa
 
385ffa
 use Package::Stash;
385ffa
 
385ffa
@@ -18,5 +18,3 @@ BEGIN {
385ffa
 }
385ffa
 
385ffa
 is($warnings, '');
385ffa
-
385ffa
-done_testing;
385ffa
--- xt/author/leaks-debug.t
385ffa
+++ xt/author/leaks-debug.t
385ffa
@@ -6,6 +6,8 @@ use Test::More;
385ffa
 use Test::Fatal;
385ffa
 use Test::LeakTrace;
385ffa
 
385ffa
+plan tests => 45;
385ffa
+
385ffa
 BEGIN { $^P |= 0x210 } # PERLDBf_SUBLINE
385ffa
 
385ffa
 use Package::Stash;
385ffa
@@ -176,7 +178,10 @@ use Symbol;
385ffa
     ok($foo->has_symbol('@ISA'));
385ffa
     is(ref($foo->get_symbol('@ISA')), 'ARRAY');
385ffa
     is_deeply($foo->get_symbol('@ISA'), ['Exporter']);
385ffa
+    SKIP: {
385ffa
+    skip "Test::More >= 0.88 needed for isa_ok() tests on classes", 1 if $Test::More::VERSION < 0.88;
385ffa
     isa_ok('Foo', 'Exporter');
385ffa
+    }
385ffa
 }
385ffa
 
385ffa
 {
385ffa
@@ -226,5 +231,3 @@ use Symbol;
385ffa
         eval { $foo->add_symbol('&blorg') };
385ffa
     } "doesn't leak on errors";
385ffa
 }
385ffa
-
385ffa
-done_testing;
385ffa
--- xt/author/leaks.t
385ffa
+++ xt/author/leaks.t
385ffa
@@ -6,6 +6,8 @@ use Test::More;
385ffa
 use Test::Fatal;
385ffa
 use Test::LeakTrace;
385ffa
 
385ffa
+plan tests => 45;
385ffa
+
385ffa
 use Package::Stash;
385ffa
 use Symbol;
385ffa
 
385ffa
@@ -174,7 +176,10 @@ use Symbol;
385ffa
     ok($foo->has_symbol('@ISA'));
385ffa
     is(ref($foo->get_symbol('@ISA')), 'ARRAY');
385ffa
     is_deeply($foo->get_symbol('@ISA'), ['Exporter']);
385ffa
+    SKIP: {
385ffa
+    skip "Test::More >= 0.88 needed for isa_ok() tests on classes", 1 if $Test::More::VERSION < 0.88;
385ffa
     isa_ok('Foo', 'Exporter');
385ffa
+    }
385ffa
 }
385ffa
 
385ffa
 {
385ffa
@@ -224,5 +229,3 @@ use Symbol;
385ffa
         eval { $foo->add_symbol('&blorg') };
385ffa
     } "doesn't leak on errors";
385ffa
 }
385ffa
-
385ffa
-done_testing;
385ffa
--- xt/release/no-tabs.t
385ffa
+++ xt/release/no-tabs.t
385ffa
@@ -3,7 +3,7 @@ use warnings;
385ffa
 
385ffa
 # this test was generated with Dist::Zilla::Plugin::Test::NoTabs 0.08
385ffa
 
385ffa
-use Test::More 0.88;
385ffa
+use Test::More;
385ffa
 use Test::NoTabs;
385ffa
 
385ffa
 my @files = (
385ffa
@@ -41,5 +41,6 @@ my @files = (
385ffa
     't/warnings.t'
385ffa
 );
385ffa
 
385ffa
+plan tests => scalar @files;
385ffa
+
385ffa
 notabs_ok($_) foreach @files;
385ffa
-done_testing;