From: David Landgren <david@landgren.net>
Date: Sat, 29 Apr 2006 17:51:28 +0000 (+0200)
Subject: t/op/hashwarn.t using test.pl
X-Git-Url: http://git.shadowcat.co.uk/gitweb/gitweb.cgi?a=commitdiff_plain;h=6647cf4edb2b5f87e78f9df85deeb68eccb8a090;p=p5sagit%2Fp5-mst-13.2.git

t/op/hashwarn.t using test.pl
Message-ID: <44538B80.2060503@landgren.net>

p4raw-id: //depot/perl@28056
---

diff --git a/t/op/hashwarn.t b/t/op/hashwarn.t
index 50c9939..b14e9c2 100755
--- a/t/op/hashwarn.t
+++ b/t/op/hashwarn.t
@@ -2,9 +2,12 @@
 
 BEGIN {
     chdir 't' if -d 't';
-    @INC = '../lib';
+    @INC = qw(. ../lib);
 }
 
+require 'test.pl';
+plan( tests => 16 );
+
 use strict;
 use warnings;
 
@@ -13,72 +16,58 @@ use vars qw{ @warnings };
 BEGIN {
     $SIG{'__WARN__'} = sub { push @warnings, @_ };
     $| = 1;
-    print "1..9\n";
-}
-
-END { print "not ok\n# Uncaught warnings:\n@warnings\n" if @warnings }
-
-sub test ($$;$) {
-    my($num, $bool, $diag) = @_;
-    if ($bool) {
-	print "ok $num\n";
-	return;
-    }
-    print "not ok $num\n";
-    return unless defined $diag;
-    $diag =~ s/\Z\n?/\n/;			# unchomp
-    print map "# $num : $_", split m/^/m, $diag;
-}
-
-sub test_warning ($$$) {
-    my($num, $got, $expected) = @_;
-    my($pattern, $ok);
-    if (($pattern) = ($expected =~ m#^/(.+)/$#s) or
-	(undef, $pattern) = ($expected =~ m#^m([^\w\s])(.+)\1$#s)) {
-	    # it's a regexp
-	    $ok = ($got =~ /$pattern/);
-	    test $num, $ok, "Expected pattern /$pattern/, got '$got'\n";
-    } else {
-	$ok = ($got eq $expected);
-	test $num, $ok, "Expected string '$expected', got '$got'\n";
-    }
-#   print "# $num: $got\n";
 }
 
-my $odd_msg = '/^Odd number of elements in hash assignment/';
-my $odd_msg2 = '/^Odd number of elements in anonymous hash/';
-my $ref_msg = '/^Reference found where even-sized list expected/';
+my $fail_odd      = 'Odd number of elements in hash assignment at ';
+my $fail_odd_anon = 'Odd number of elements in anonymous hash at ';
+my $fail_ref      = 'Reference found where even-sized list expected at ';
+my $fail_not_hr   = 'Not a HASH reference at ';
 
 {
+    @warnings = ();
     my %hash = (1..3);
-    test_warning 1, shift @warnings, $odd_msg;
+    cmp_ok(scalar(@warnings),'==',1,'odd count');
+    cmp_ok(substr($warnings[0],0,length($fail_odd)),'eq',$fail_odd,'odd msg');
 
+    @warnings = ();
     %hash = 1;
-    test_warning 2, shift @warnings, $odd_msg;
+    cmp_ok(scalar(@warnings),'==',1,'scalar count');
+    cmp_ok(substr($warnings[0],0,length($fail_odd)),'eq',$fail_odd,'scalar msg');
 
+    @warnings = ();
     %hash = { 1..3 };
-    test_warning 3, shift @warnings, $odd_msg2;
-    test_warning 4, shift @warnings, $ref_msg;
+    cmp_ok(scalar(@warnings),'==',2,'odd hashref count');
+    cmp_ok(substr($warnings[0],0,length($fail_odd_anon)),'eq',$fail_odd_anon,'odd hashref msg 1');
+    cmp_ok(substr($warnings[1],0,length($fail_ref)),'eq',$fail_ref,'odd hashref msg 2');
 
+    @warnings = ();
     %hash = [ 1..3 ];
-    test_warning 5, shift @warnings, $ref_msg;
+    cmp_ok(scalar(@warnings),'==',1,'arrayref count');
+    cmp_ok(substr($warnings[0],0,length($fail_ref)),'eq',$fail_ref,'arrayref msg');
 
-    %hash = sub { print "ok" };
-    test_warning 6, shift @warnings, $odd_msg;
+    @warnings = ();
+    %hash = sub { print "fenice" };
+    cmp_ok(scalar(@warnings),'==',1,'coderef count');
+    cmp_ok(substr($warnings[0],0,length($fail_odd)),'eq',$fail_odd,'coderef msg');
+
+    @warnings = ();
+    $_ = { 1..10 };
+    cmp_ok(scalar(@warnings),'==',0,'hashref assign');
 
     # Old pseudo-hash syntax, now removed.
+
+    @warnings = ();
     my $avhv = [{x=>1,y=>2}];
     eval {
         %$avhv = (x=>13,'y');
     };
-    test 7, $@ =~ /^Not a HASH reference/;
+    cmp_ok(scalar(@warnings),'==',0,'pseudo-hash 1 count');
+    cmp_ok(substr($@,0,length($fail_not_hr)),'eq',$fail_not_hr,'pseudo-hash 1 msg');
 
-    # Old pseudo-hash syntax, since removed.
+    @warnings = ();
     eval {
         %$avhv = 'x';
     };
-    test 8, $@ =~ /^Not a HASH reference/;
-
-    $_ = { 1..10 };
-    test 9, ! @warnings, "Unexpected warning";
+    cmp_ok(scalar(@warnings),'==',0,'pseudo-hash 2 count');
+    cmp_ok(substr($@,0,length($fail_not_hr)),'eq',$fail_not_hr,'pseudo-hash 2 msg');
 }