X-Git-Url: http://git.vpit.fr/?a=blobdiff_plain;f=t%2F40-threads.t;h=6e77c2fd24b27d5285c59265ec1b0f1b4daaa20d;hb=be3665d8224dfbb9cff5f53796500821d9c8185a;hp=bd64663e44166c2a581455384ffb8b438bc27fcf;hpb=a294e41c3d8c7d53b3a0cb56126b227142cb5ccd;p=perl%2Fmodules%2FVariable-Magic.git diff --git a/t/40-threads.t b/t/40-threads.t index bd64663..6e77c2f 100644 --- a/t/40-threads.t +++ b/t/40-threads.t @@ -1,109 +1,121 @@ -#!perl -T +#!perl use strict; use warnings; -sub skipall { - my ($msg) = @_; - require Test::More; - Test::More::plan(skip_all => $msg); -} - -use Config qw/%Config/; - -BEGIN { - my $t_v = '1.67'; - my $ts_v = '1.14'; - skipall 'This perl wasn\'t built to support threads' - unless $Config{useithreads}; - skipall "threads $t_v required to test thread safety" - unless eval "use threads $t_v; 1"; - skipall "threads::shared $ts_v required to test thread safety" - unless eval "use threads::shared $ts_v; 1"; -} - -use Test::More; # after threads +use Variable::Magic qw< + wizard cast dispell getdata + VMG_OP_INFO_NAME VMG_OP_INFO_OBJECT +>; -use Variable::Magic qw/wizard cast dispell getdata VMG_THREADSAFE VMG_OP_INFO_NAME VMG_OP_INFO_OBJECT/; +use lib 't/lib'; +use VPIT::TestHelpers ( + threads => [ 'Variable::Magic' => 'Variable::Magic::VMG_THREADSAFE()' ], +); -BEGIN { - skipall 'This Variable::Magic isn\'t thread safe' unless VMG_THREADSAFE; - plan tests => 2 * (4 * 18 + 1) + 2 * (4 * 13 + 1); - my $v = $threads::VERSION; - diag "Using threads $v" if defined $v; - $v = $threads::shared::VERSION; - diag "Using threads::shared $v" if defined $v; -} +use Test::Leaner 'no_plan'; my $destroyed : shared = 0; sub try { - my ($dispell, $sig, $op_info) = @_; - my $tid = threads->tid(); - my $c = 0; - my $wiz = eval { - wizard data => sub { $_[1] + $tid }, - sig => $sig, - get => sub { ++$c; 0 }, - set => sub { - my $op = $_[-1]; - if ($op_info == VMG_OP_INFO_OBJECT) { - is_deeply { class => ref($op), name => $op->name }, - { class => 'B::BINOP', name => 'sassign' }, - "op object in thread $tid is correct"; - } else { - is $op, 'sassign', "op name in thread $tid is correct"; - } - 0 - }, - free => sub { lock $destroyed; ++$destroyed; 0 }, - op_info => $op_info - }; - is($@, '', "wizard in thread $tid doesn't croak"); - isnt($wiz, undef, "wizard in thread $tid is defined"); - is($c, 0, "wizard in thread $tid doesn't trigger magic"); + my ($dispell, $op_info) = @_; + my $tid = threads->tid; + + my $c = 0; + my $wiz; + + { + local $@; + $wiz = eval { + wizard( + data => sub { $_[1] + $tid }, + get => sub { ++$c; 0 }, + set => sub { + my $op = $_[-1]; + + if ($op_info == VMG_OP_INFO_OBJECT) { + is_deeply { class => ref($op), name => $op->name }, + { class => 'B::BINOP', name => 'sassign' }, + "op object in thread $tid is correct"; + } else { + is $op, 'sassign', "op name in thread $tid is correct"; + } + + return 0; + }, + free => sub { lock $destroyed; ++$destroyed; 0 }, + op_info => $op_info, + ); + }; + is $@, '', "wizard in thread $tid doesn't croak"; + isnt $wiz, undef, "wizard in thread $tid is defined"; + is $c, 0, "wizard in thread $tid doesn't trigger magic"; + } + my $a = 3; - my $res = eval { cast $a, $wiz, sub { 5 }->() }; - is($@, '', "cast in thread $tid doesn't croak"); - is($c, 0, "cast in thread $tid doesn't trigger magic"); - my $b; - eval { $b = $a }; - is($@, '', "get in thread $tid doesn't croak"); - is($b, 3, "get in thread $tid returns the right thing"); - is($c, 1, "get in thread $tid triggers magic"); - my $d = eval { 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"); - eval { $a = 9 }; - is($@, '', "set in thread $tid (check opname) doesn't croak"); - 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; + + { + local $@; + my $res = eval { cast $a, $wiz, sub { 5 }->() }; + is $@, '', "cast in thread $tid doesn't croak"; + is $c, 0, "cast in thread $tid doesn't trigger magic"; + } + + { + local $@; + my $b; eval { $b = $a }; - is($@, '', "get in thread $tid after dispell doesn't croak"); - is($b, 9, "get in thread $tid after dispell returns the right thing"); - is($c, 1, "get in thread $tid after dispell doesn't trigger magic"); + is $@, '', "get in thread $tid doesn't croak"; + is $b, 3, "get in thread $tid returns the right thing"; + is $c, 1, "get in thread $tid triggers magic"; } - return; # Ugly if not here -} -for my $dispell (1, 0) { - for my $sig (undef, Variable::Magic::gensig()) { + { + local $@; + my $d = eval { 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"; + } + + { + local $@; + eval { $a = 9 }; + is $@, '', "set in thread $tid (check opname) doesn't croak"; + } + + if ($dispell) { { - lock $destroyed; - $destroyed = 0; + local $@; + my $res = eval { dispell $a, $wiz }; + is $@, '', "dispell in thread $tid doesn't croak"; + is $c, 1, "dispell in thread $tid doesn't trigger magic"; } - my @t = map { threads->create(\&try, $dispell, $sig, $_) } - (VMG_OP_INFO_NAME) x 2, (VMG_OP_INFO_OBJECT) x 2; - $_->join for @t; - { - lock $destroyed; - is $destroyed, (1 - $dispell) * 4, 'destructors'; + local $@; + my $b; + eval { $b = $a }; + is $@, '', "get in thread $tid after dispell doesn't croak"; + is $b, 9, "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 +} + +for my $dispell (1, 0) { + { + lock $destroyed; + $destroyed = 0; + } + + my @threads = map spawn(\&try, $dispell, $_), + (VMG_OP_INFO_NAME) x 2, (VMG_OP_INFO_OBJECT) x 2; + $_->join for @threads; + + { + lock $destroyed; + is $destroyed, (1 - $dispell) * 4, 'destructors'; + } }