From: Vincent Pit <vince@profvince.com>
Date: Sat, 20 Sep 2008 22:01:07 +0000 (+0200)
Subject: Test destructors
X-Git-Tag: v0.20~6
X-Git-Url: http://git.vpit.fr/?a=commitdiff_plain;h=c6abed5ccaa4d139bd636acc79f53df4e1e254bc;p=perl%2Fmodules%2FVariable-Magic.git

Test destructors
---

diff --git a/t/40-threads.t b/t/40-threads.t
index c319860..67afabb 100644
--- a/t/40-threads.t
+++ b/t/40-threads.t
@@ -21,16 +21,16 @@ use Test::More;
 use Variable::Magic qw/wizard cast dispell getdata VMG_THREADSAFE/;
 
 if (VMG_THREADSAFE) {
- plan tests => 2 * (2 * 16 + 1);
+ plan tests => 2 * (2 * 16 + 1) + 2 * (2 * 11 + 1);
 } else {
  plan skip_all => 'This Variable::Magic isn\'t thread safe';
 }
 
 my $destroyed : shared = 0;
-
 my $sig = undef;
 
 sub try {
+ my ($dispell) = @_;
  my $tid = threads->tid();
  my $c   = 0;
  my $wiz = eval {
@@ -55,29 +55,35 @@ sub try {
  is($@, '',       "getdata in thread $tid doesn't croak");
  is($d, 5 + $tid, "getdata in thread $tid returns the right thing");
  is($c, 1,        "getdata in thread $tid doesn't trigger magic");
- $res = eval { dispell $a, $wiz };
- is($@, '', "dispell in thread $tid doesn't croak");
- is($c, 1,  "dispell in thread $tid doesn't trigger magic");
- undef $b;
- eval { $b = $a };
- is($@, '', "get in thread $tid after dispell doesn't croak");
- is($b, 3,  "get in thread $tid after dispell returns the right thing");
- is($c, 1,  "get in thread $tid after dispell doesn't trigger magic");
- return;
+ if ($dispell) {
+  $res = eval { dispell $a, $wiz };
+  is($@, '', "dispell in thread $tid doesn't croak");
+  is($c, 1,  "dispell in thread $tid doesn't trigger magic");
+  undef $b;
+  eval { $b = $a };
+  is($@, '', "get in thread $tid after dispell doesn't croak");
+  is($b, 3,  "get in thread $tid after dispell returns the right thing");
+  is($c, 1,  "get in thread $tid after dispell doesn't trigger magic");
+ }
+ return; # Ugly if not here
 }
 
-my @t = map { threads->create(\&try) } 1 .. 2;
-$t[0]->join;
-$t[1]->join;
+for my $dispell (1, 0) {
+ $destroyed = 0;
+ $sig = undef;
 
-is($destroyed, 0, 'destructors didn\'t fired');
+ my @t = map { threads->create(\&try, $dispell) } 1 .. 2;
+ $t[0]->join;
+ $t[1]->join;
 
-$destroyed = 0;
-$sig = Variable::Magic::gensig();
+ is($destroyed, (1 - $dispell) * 2, 'destructors');
 
-@t = map { threads->create(\&try) } 1 .. 2;
-$t[0]->join;
-$t[1]->join;
+ $destroyed = 0;
+ $sig = Variable::Magic::gensig();
 
-is($destroyed, 0, 'destructors didn\'t fired');
+ @t = map { threads->create(\&try, $dispell) } 1 .. 2;
+ $t[0]->join;
+ $t[1]->join;
 
+ is($destroyed, (1 - $dispell) * 2, 'destructors');
+}