hoist all the copies of cmpimg into a common module
authorTony Cook <tony@develop-help.com>
Sun, 6 May 2012 02:22:48 +0000 (12:22 +1000)
committerTony Cook <tony@develop-help.com>
Sun, 6 May 2012 02:22:48 +0000 (12:22 +1000)
12 files changed:
MANIFEST
t/lib/Imager/Graph/Test.pm [new file with mode: 0644]
t/t10pie.t
t/t14bar.t
t/t30many_points.t
t/t31tic_color.t
t/t32series_labels.t
t/t33_long_labels.t
t/t34horizontal_many_points.t
t/t40area.t
t/x50vstyle.t
t/x51hstyle.t

index 0840018..4b70aca 100644 (file)
--- a/MANIFEST
+++ b/MANIFEST
@@ -15,6 +15,7 @@ MANIFEST
 MANIFEST.SKIP
 README
 t/lib/Imager/Font/Test.pm
+t/lib/Imager/Graph/Test.pm
 t/t00load.t
 t/t10pie.t
 t/t11line.t
diff --git a/t/lib/Imager/Graph/Test.pm b/t/lib/Imager/Graph/Test.pm
new file mode 100644 (file)
index 0000000..93ad59b
--- /dev/null
@@ -0,0 +1,33 @@
+package Imager::Graph::Test;
+use strict;
+use Test::More;
+use Imager::Test qw(is_image_similar);
+
+require Exporter;
+our @ISA = qw(Exporter);
+
+our @EXPORT_OK = qw(cmpimg);
+
+sub cmpimg ($$;$$) {
+  my ($img, $filename, $error, $note) = @_;
+
+  local $Test::Builder::Level = $Test::Builder::Level + 1;
+
+  $note ||= $filename;
+  $error ||= 10_000;
+ SKIP: {
+    $Imager::formats{png}
+      or skip("png not available", 1);
+
+    my $cmpim = Imager->new;
+    if ($cmpim->read(file => $filename)) {
+      is_image_similar($img, $cmpim, $error, $note);
+    }
+    else {
+      fail("$note: load");
+      diag("loading $filename: " . $cmpim->errstr);
+    }
+  }
+}
+
+1;
index 15f7108..d5277b7 100644 (file)
@@ -3,6 +3,7 @@ use strict;
 use Imager::Graph::Pie;
 use lib 't/lib';
 use Imager::Font::Test;
+use Imager::Graph::Test qw(cmpimg);
 use Test::More;
 
 -d 'testout' 
@@ -269,21 +270,3 @@ cmpimg($img6, "testimg/t10_hlegend.png", 550_000);
   print "# ", $pie->error, "\n";
   cmp_ok($pie->error, '=~', qr/^Error creating image/, "check error message");
 }
-
-sub cmpimg {
-  my ($img, $file, $limit) = @_;
-
-  $limit ||= 10000;
-
- SKIP:
-  {
-    $Imager::formats{png}
-      or skip("No PNG support", 1);
-
-    my $cmpimg = Imager->new;
-    $cmpimg->read(file=>$file)
-      or return ok(0, "Cannot read $file: ".$cmpimg->errstr);
-    my $diff = Imager::i_img_diff($img->{IMG}, $cmpimg->{IMG});
-    cmp_ok($diff, '<', $limit, "Comparison to $file ($diff)");
-  }
-}
index f426a0f..cde1635 100644 (file)
@@ -4,6 +4,7 @@ use Imager::Graph::Bar;
 use lib 't/lib';
 use Imager::Font::Test;
 use Test::More;
+use Imager::Graph::Test 'cmpimg';
 
 -d 'testout' 
   or mkdir "testout", 0700 
@@ -110,15 +111,3 @@ unless (is(@warned, 0, "should be no warnings")) {
   diag($_) for @warned;
 }
 
-sub cmpimg {
-  my ($img, $file, $limit) = @_;
-
-  $limit ||= 10000;
-
-  my $cmpimg = Imager->new;
-  $cmpimg->read(file=>$file)
-    or return ok(0, "Cannot read $file: ".$cmpimg->errstr);
-  my $diff = Imager::i_img_diff($img->{IMG}, $cmpimg->{IMG});
-  cmp_ok($diff, '<', $limit, "Comparison to $file ($diff)");
-}
-
index 0ee7476..1ede862 100644 (file)
@@ -4,6 +4,7 @@ use Imager::Graph::Line;
 use lib 't/lib';
 use Imager::Font::Test;
 use Test::More;
+use Imager::Graph::Test 'cmpimg';
 
 -d 'testout' 
   or mkdir "testout", 0700 
@@ -46,16 +47,3 @@ cmpimg($img1, 'testimg/t30_points.ppm', 40_000);
 unless (is(@warned, 0, "should be no warnings")) {
   diag($_) for @warned;
 }
-
-
-sub cmpimg {
-  my ($img, $file, $limit) = @_;
-
-  $limit ||= 10000;
-
-  my $cmpimg = Imager->new;
-  $cmpimg->read(file=>$file)
-    or return ok(0, "Cannot read $file: ".$cmpimg->errstr);
-  my $diff = Imager::i_img_diff($img->{IMG}, $cmpimg->{IMG});
-  cmp_ok($diff, '<', $limit, "Comparison to $file ($diff)");
-}
index 09abf6c..c563ceb 100644 (file)
@@ -4,6 +4,7 @@ use Imager::Graph::Line;
 use lib 't/lib';
 use Imager::Font::Test;
 use Test::More;
+use Imager::Graph::Test 'cmpimg';
 
 -d 'testout' 
   or mkdir "testout", 0700 
@@ -58,16 +59,3 @@ unless (is(@warned, 0, "should be no warnings")) {
   diag($_) for @warned;
 }
 
-
-sub cmpimg {
-  my ($img, $file, $limit) = @_;
-
-  $limit ||= 10000;
-
-  my $cmpimg = Imager->new;
-  $cmpimg->read(file=>$file)
-    or return ok(0, "Cannot read $file: ".$cmpimg->errstr);
-  my $diff = Imager::i_img_diff($img->{IMG}, $cmpimg->{IMG});
-  cmp_ok($diff, '<', $limit, "Comparison to $file ($diff)");
-}
-
index e22e2a2..66a39e3 100644 (file)
@@ -4,6 +4,7 @@ use Imager::Graph::Line;
 use lib 't/lib';
 use Imager::Font::Test;
 use Test::More;
+use Imager::Graph::Test 'cmpimg';
 
 -d 'testout' 
   or mkdir "testout", 0700 
@@ -48,18 +49,3 @@ cmpimg($img1, 'testimg/t32_series.ppm', 80_000);
 unless (is(@warned, 0, "should be no warnings")) {
   diag($_) for @warned;
 }
-
-
-sub cmpimg {
-  my ($img, $file, $limit) = @_;
-
-  $limit ||= 10000;
-
-  my $cmpimg = Imager->new;
-  $cmpimg->read(file=>$file)
-    or return ok(0, "Cannot read $file: ".$cmpimg->errstr);
-  my $diff = Imager::i_img_diff($img->{IMG}, $cmpimg->{IMG});
-  cmp_ok($diff, '<', $limit, "Comparison to $file ($diff)");
-
-}
-
index 712bec5..77156c3 100644 (file)
@@ -1,9 +1,11 @@
+#!perl
 use strict;
 
 use Imager::Graph::Line;
 use lib 't/lib';
 use Imager::Font::Test;
 use Test::More;
+use Imager::Graph::Test 'cmpimg';
 
 -d 'testout' 
   or mkdir "testout", 0700 
@@ -42,18 +44,3 @@ $img->write(file=>'testout/t33_long_labels.ppm') or die "Can't save img1: ".$img
 unless (is(@warned, 0, "should be no warnings")) {
   diag($_) for @warned;
 }
-
-
-sub cmpimg {
-  my ($img, $file, $limit) = @_;
-
-  $limit ||= 10000;
-
-  my $cmpimg = Imager->new;
-  $cmpimg->read(file=>$file)
-    or return ok(0, "Cannot read $file: ".$cmpimg->errstr);
-  my $diff = Imager::i_img_diff($img->{IMG}, $cmpimg->{IMG});
-  cmp_ok($diff, '<', $limit, "Comparison to $file ($diff)");
-
-}
-
index 149aeec..def8ded 100644 (file)
@@ -4,6 +4,7 @@ use Imager::Graph::Bar;
 use lib 't/lib';
 use Imager::Font::Test;
 use Test::More;
+use Imager::Graph::Test 'cmpimg';
 
 -d 'testout' 
   or mkdir "testout", 0700 
@@ -44,17 +45,3 @@ cmpimg($img1, 'testimg/t34_points.ppm', 80_000);
 unless (is(@warned, 0, "should be no warnings")) {
   diag($_) for @warned;
 }
-
-
-sub cmpimg {
-  my ($img, $file, $limit) = @_;
-
-  $limit ||= 10000;
-
-  my $cmpimg = Imager->new;
-  $cmpimg->read(file=>$file)
-    or return ok(0, "Cannot read $file: ".$cmpimg->errstr);
-  my $diff = Imager::i_img_diff($img->{IMG}, $cmpimg->{IMG});
-  cmp_ok($diff, '<', $limit, "Comparison to $file ($diff)");
-}
-
index 0dcc0bf..62b7e2b 100644 (file)
@@ -5,6 +5,7 @@ use lib 't/lib';
 use Imager::Font::Test;
 use Test::More;
 use Imager::Test qw(is_image_similar);
+use Imager::Graph::Test qw(cmpimg);
 
 -d 'testout' 
   or mkdir "testout", 0700 
@@ -102,20 +103,3 @@ END {
   }
 }
 
-sub cmpimg {
-  my ($img, $file, $limit) = @_;
-
-  $limit ||= 10000;
-
- SKIP:
-  {
-    $Imager::formats{png}
-      or skip("No PNG support", 1);
-
-    my $cmpimg = Imager->new;
-    $cmpimg->read(file=>$file)
-      or return ok(0, "Cannot read $file: ".$cmpimg->errstr);
-    my $diff = Imager::i_img_diff($img->{IMG}, $cmpimg->{IMG});
-    is_image_similar($img, $cmpimg, $limit, "Comparison to $file ($diff)");
-  }
-}
index fb46a21..1de4861 100644 (file)
@@ -4,7 +4,8 @@ use Imager::Graph::Line;
 use lib 't/lib';
 use Imager::Font::Test;
 use Test::More;
-use Imager::Test qw(is_image_similar is_image);
+use Imager::Test qw(is_image);
+use Imager::Graph::Test 'cmpimg';
 
 -d 'testout' 
   or mkdir "testout", 0700 
@@ -220,20 +221,3 @@ END {
   }
 }
 
-sub cmpimg {
-  my ($img, $file, $limit) = @_;
-
-  $limit ||= 10000;
-
- SKIP:
-  {
-    $Imager::formats{png}
-      or skip("No PNG support", 1);
-
-    my $cmpimg = Imager->new;
-    $cmpimg->read(file=>$file)
-      or return ok(0, "Cannot read $file: ".$cmpimg->errstr);
-    my $diff = Imager::i_img_diff($img->{IMG}, $cmpimg->{IMG});
-    is_image_similar($img, $cmpimg, $limit, "Comparison to $file ($diff)");
-  }
-}
index cee81ce..5eacb9a 100644 (file)
@@ -4,7 +4,8 @@ use Imager::Graph::Bar;
 use lib 't/lib';
 use Imager::Font::Test;
 use Test::More;
-use Imager::Test qw(is_image_similar is_image);
+use Imager::Test qw(is_image);
+use Imager::Graph::Test 'cmpimg';
 
 -d 'testout' 
   or mkdir "testout", 0700 
@@ -182,20 +183,3 @@ END {
   }
 }
 
-sub cmpimg {
-  my ($img, $file, $limit) = @_;
-
-  $limit ||= 10000;
-
- SKIP:
-  {
-    $Imager::formats{png}
-      or skip("No PNG support", 1);
-
-    my $cmpimg = Imager->new;
-    $cmpimg->read(file=>$file)
-      or return ok(0, "Cannot read $file: ".$cmpimg->errstr);
-    my $diff = Imager::i_img_diff($img->{IMG}, $cmpimg->{IMG});
-    is_image_similar($img, $cmpimg, $limit, "Comparison to $file ($diff)");
-  }
-}