X-Git-Url: http://git.vpit.fr/?a=blobdiff_plain;f=t%2F40-threads.t;h=151116bab11f44b9aebe5b25a1e6240380d8576a;hb=b1c264655ccc1eca4423e9dd9479855938bbac24;hp=7f55f22fc43cea8ee85dcf3301ae06a3b6a14290;hpb=51804c2b718abc276a95b4626f3fb30f829e51fe;p=perl%2Fmodules%2FVariable-Magic.git

diff --git a/t/40-threads.t b/t/40-threads.t
index 7f55f22..151116b 100644
--- a/t/40-threads.t
+++ b/t/40-threads.t
@@ -1,16 +1,11 @@
-#!perl -T
+#!perl
 
 use strict;
 use warnings;
 
-use Variable::Magic qw<
- wizard cast dispell getdata
- VMG_OP_INFO_NAME VMG_OP_INFO_OBJECT
->;
-
 use lib 't/lib';
 use VPIT::TestHelpers (
- threads => [ 'Variable::Magic' => Variable::Magic::VMG_THREADSAFE ],
+ threads => [ 'Variable::Magic' => 'Variable::Magic::VMG_THREADSAFE()' ],
 );
 
 use Test::Leaner 'no_plan';
@@ -24,16 +19,21 @@ sub try {
  my $c = 0;
  my $wiz;
 
+ {
+  local $@;
+  eval { require Variable::Magic; 1 } or return;
+ }
+
  {
   local $@;
   $wiz = eval {
-   wizard(
+   Variable::Magic::wizard(
     data    => sub { $_[1] + $tid },
     get     => sub { ++$c; 0 },
     set     => sub {
      my $op = $_[-1];
 
-     if ($op_info == VMG_OP_INFO_OBJECT) {
+     if ($op_info eq 'object') {
       is_deeply { class => ref($op),   name => $op->name },
                 { class => 'B::BINOP', name => 'sassign' },
                 "op object in thread $tid is correct";
@@ -44,7 +44,8 @@ sub try {
      return 0;
     },
     free    => sub { lock $destroyed; ++$destroyed; 0 },
-    op_info => $op_info,
+    op_info => $op_info eq 'object' ? Variable::Magic::VMG_OP_INFO_OBJECT()
+                                    : Variable::Magic::VMG_OP_INFO_NAME()
    );
   };
   is $@,     '',    "wizard in thread $tid doesn't croak";
@@ -56,7 +57,7 @@ sub try {
 
  {
   local $@;
-  my $res = eval { cast $a, $wiz, sub { 5 }->() };
+  my $res = eval { &Variable::Magic::cast(\$a, $wiz, sub { 5 }->()) };
   is $@, '', "cast in thread $tid doesn't croak";
   is $c, 0,  "cast in thread $tid doesn't trigger magic";
  }
@@ -72,7 +73,7 @@ sub try {
 
  {
   local $@;
-  my $d = eval { getdata $a, $wiz };
+  my $d = eval { &Variable::Magic::getdata(\$a, $wiz) };
   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";
@@ -87,7 +88,7 @@ sub try {
  if ($dispell) {
   {
    local $@;
-   my $res = eval { dispell $a, $wiz };
+   my $res = eval { &Variable::Magic::dispell(\$a, $wiz) };
    is $@, '', "dispell in thread $tid doesn't croak";
    is $c, 1,  "dispell in thread $tid doesn't trigger magic";
   }
@@ -101,7 +102,8 @@ sub try {
    is $c, 1,  "get in thread $tid after dispell doesn't trigger magic";
   }
  }
- return; # Ugly if not here
+
+ return 1;
 }
 
 for my $dispell (1, 0) {
@@ -110,12 +112,16 @@ for my $dispell (1, 0) {
   $destroyed = 0;
  }
 
- my @threads = map spawn(\&try, $dispell, $_),
-                              (VMG_OP_INFO_NAME) x 2, (VMG_OP_INFO_OBJECT) x 2;
- $_->join for @threads;
+ my $completed = 0;
+
+ my @threads = map spawn(\&try, $dispell, $_), ('name') x 2, ('object') x 2;
+ for my $thr (@threads) {
+  my $res = $thr->join;
+  $completed += $res if defined $res;
+ }
 
  {
   lock $destroyed;
-  is $destroyed, (1 - $dispell) * 4, 'destructors';
+  is $destroyed, (1 - $dispell) * $completed, 'destructors';
  }
 }