From f3ef7f042fa058937627f352abd1241b4b4fc72c Mon Sep 17 00:00:00 2001 From: Moritz Lenz Date: Thu, 23 Feb 2012 20:42:20 +0100 Subject: [PATCH] [Test.pm] replace complicated, non-working multis with a short, simple, do-it-all only sub --- Pugs/ext/Test/lib/Test.pm | 13 ++----------- 1 file changed, 2 insertions(+), 11 deletions(-) diff --git a/Pugs/ext/Test/lib/Test.pm b/Pugs/ext/Test/lib/Test.pm index a411375..7332aab 100644 --- a/Pugs/ext/Test/lib/Test.pm +++ b/Pugs/ext/Test/lib/Test.pm @@ -249,18 +249,9 @@ sub todo (Str $reason = "fix", *%deadline) returns Bool is export { return False; } -multi sub skip (Str $reason?, :$depends) returns Bool is export { - Test::proclaim(1, "", "skip $reason", :$depends); -} - -multi sub skip (Str $reason, Int $count, :$depends) returns Bool is export { +sub skip (Str $reason = '', Int $count = 1, :$depends) returns Bool is export { for (1 .. $count) { - # Hack -- PIL2JS doesn't support multisubs correctly yet - if $*OS eq "browser" { - Test::proclaim(1, "", "skip $reason", :$depends); - } else { - Test::skip $reason, :$depends; - } + Test::proclaim(1, "", "skip $reason", :$depends); } }