]> git.imager.perl.org - imager.git/blobdiff - t/t022double.t
improved thread safety for Imager
[imager.git] / t / t022double.t
index 87fdbbb6994e894d288d65d3d7e375282f7b5ea9..05807715fa32fb0e5e6445feea42b9bf67b4a149 100644 (file)
@@ -1,46 +1,45 @@
 #!perl -w
 use strict;
-BEGIN { $| = 1; print "1..42\n"; }
-my $loaded;
-END {print "not ok 1\n" unless $loaded;}
-use Imager qw(:all :handy);
-#use Data::Dumper;
-$loaded = 1;
-print "ok 1\n";
+use Test::More tests => 136;
+BEGIN { use_ok(Imager => qw(:all :handy)) }
 
-require "t/testtools.pl";
+use Imager::Test qw(test_image is_image is_color3);
 
-init_log("testout/t022double.log", 1);
+-d "testout" or mkdir "testout";
+
+Imager->open_log(log => "testout/t022double.log");
+
+use Imager::Test qw(image_bounds_checks test_colorf_gpix test_colorf_glin mask_tests);
 
 use Imager::Color::Float;
 
 my $im_g = Imager::i_img_double_new(100, 101, 1);
 
-ok(2, Imager::i_img_getchannels($im_g) == 1, 
+ok(Imager::i_img_getchannels($im_g) == 1, 
    "1 channel image channel count mismatch");
-ok(3, Imager::i_img_getmask($im_g) & 1, "1 channel image bad mask");
-ok(4, Imager::i_img_virtual($im_g) == 0, 
+ok(Imager::i_img_getmask($im_g) & 1, "1 channel image bad mask");
+ok(Imager::i_img_virtual($im_g) == 0, 
   "1 channel image thinks it is virtual");
 my $double_bits = length(pack("d", 1)) * 8;
 print "# $double_bits double bits\n";
-ok(5, Imager::i_img_bits($im_g) == $double_bits, 
+ok(Imager::i_img_bits($im_g) == $double_bits, 
    "1 channel image has bits != $double_bits");
-ok(6, Imager::i_img_type($im_g) == 0, "1 channel image isn't direct");
+ok(Imager::i_img_type($im_g) == 0, "1 channel image isn't direct");
 
 my @ginfo = i_img_info($im_g);
-ok(7, $ginfo[0] == 100, "1 channel image width incorrect");
-ok(8, $ginfo[1] == 101, "1 channel image height incorrect");
+ok($ginfo[0] == 100, "1 channel image width incorrect");
+ok($ginfo[1] == 101, "1 channel image height incorrect");
 
 undef $im_g;
 
 my $im_rgb = Imager::i_img_double_new(100, 101, 3);
 
-ok(9, Imager::i_img_getchannels($im_rgb) == 3,
+ok(Imager::i_img_getchannels($im_rgb) == 3,
    "3 channel image channel count mismatch");
-ok(10, (Imager::i_img_getmask($im_rgb) & 7) == 7, "3 channel image bad mask");
-ok(11, Imager::i_img_bits($im_rgb) == $double_bits,
+ok((Imager::i_img_getmask($im_rgb) & 7) == 7, "3 channel image bad mask");
+ok(Imager::i_img_bits($im_rgb) == $double_bits,
   "3 channel image has bits != $double_bits");
-ok(12, Imager::i_img_type($im_rgb) == 0, "3 channel image isn't direct");
+ok(Imager::i_img_type($im_rgb) == 0, "3 channel image isn't direct");
 
 my $redf = NCF(1, 0, 0);
 my $greenf = NCF(0, 1, 0);
@@ -50,95 +49,250 @@ my $bluef = NCF(0, 0, 1);
 for my $y (0..101) {
   Imager::i_plinf($im_rgb, 0, $y, ($redf) x 100);
 }
-print "ok 13\n";
+
 # basic sanity
-test_colorf_gpix(14, $im_rgb, 0,  0,   $redf);
-test_colorf_gpix(16, $im_rgb, 99, 0,   $redf);
-test_colorf_gpix(18, $im_rgb, 0,  100, $redf);
-test_colorf_gpix(20, $im_rgb, 99, 100, $redf);
-test_colorf_glin(22, $im_rgb, 0,  0,   ($redf) x 100);
-test_colorf_glin(24, $im_rgb, 0,  100, ($redf) x 100);
+test_colorf_gpix($im_rgb, 0,  0,   $redf);
+test_colorf_gpix($im_rgb, 99, 0,   $redf);
+test_colorf_gpix($im_rgb, 0,  100, $redf);
+test_colorf_gpix($im_rgb, 99, 100, $redf);
+test_colorf_glin($im_rgb, 0,  0,   [ ($redf) x 100 ], 'sanity glin @0');
+test_colorf_glin($im_rgb, 0,  100, [ ($redf) x 100 ], 'sanity glin @100');
 
 Imager::i_plinf($im_rgb, 20, 1, ($greenf) x 60);
-test_colorf_glin(26, $im_rgb, 0, 1, 
-                 ($redf) x 20, ($greenf) x 60, ($redf) x 20);
+test_colorf_glin($im_rgb, 0, 1, 
+                 [ ($redf) x 20, ($greenf) x 60, ($redf) x 20 ],
+                'check after write');
 
 # basic OO tests
 my $ooimg = Imager->new(xsize=>200, ysize=>201, bits=>'double');
-ok(28, $ooimg, "couldn't make double image");
-ok(29, $ooimg->bits eq 'double', "oo didn't give double image");
+ok($ooimg, "couldn't make double image");
+is($ooimg->bits, 'double', "oo didn't give double image");
+ok(!$ooimg->is_bilevel, 'not monochrome');
 
 # check that the image is copied correctly
 my $oocopy = $ooimg->copy;
-ok(30, $oocopy->bits eq 'double', "oo copy didn't give double image");
+is($oocopy->bits, 'double', "oo copy didn't give double image");
 
-my $num = 31;
-okn($num++, !Imager->new(xsize=>0, ysize=>1, bits=>'double'),
+ok(!Imager->new(xsize=>0, ysize=>1, bits=>'double'),
     "fail making 0 width image");
-matchn($num++, Imager->errstr, qr/Image sizes must be positive/,
+cmp_ok(Imager->errstr, '=~', qr/Image sizes must be positive/,
        "and correct message");
-okn($num++, !Imager->new(xsize=>1, ysize=>0, bits=>'double'),
+ok(!Imager->new(xsize=>1, ysize=>0, bits=>'double'),
     "fail making 0 height image");
-matchn($num++, Imager->errstr, qr/Image sizes must be positive/,
+cmp_ok(Imager->errstr, '=~', qr/Image sizes must be positive/,
        "and correct message");
-okn($num++, !Imager->new(xsize=>-1, ysize=>1, bits=>'double'),
+ok(!Imager->new(xsize=>-1, ysize=>1, bits=>'double'),
     "fail making -ve width image");
-matchn($num++, Imager->errstr, qr/Image sizes must be positive/,
+cmp_ok(Imager->errstr, '=~', qr/Image sizes must be positive/,
        "and correct message");
-okn($num++, !Imager->new(xsize=>1, ysize=>-1, bits=>'double'),
+ok(!Imager->new(xsize=>1, ysize=>-1, bits=>'double'),
     "fail making -ve height image");
-matchn($num++, Imager->errstr, qr/Image sizes must be positive/,
+cmp_ok(Imager->errstr, '=~', qr/Image sizes must be positive/,
        "and correct message");
-okn($num++, !Imager->new(xsize=>1, ysize=>1, bits=>'double', channels=>0),
+ok(!Imager->new(xsize=>1, ysize=>1, bits=>'double', channels=>0),
     "fail making 0 channel image");
-matchn($num++, Imager->errstr, qr/channels must be between 1 and 4/,
+cmp_ok(Imager->errstr, '=~', qr/channels must be between 1 and 4/,
        "and correct message");
-okn($num++, !Imager->new(xsize=>1, ysize=>1, bits=>'double', channels=>5),
+ok(!Imager->new(xsize=>1, ysize=>1, bits=>'double', channels=>5),
     "fail making 5 channel image");
-matchn($num++, Imager->errstr, qr/channels must be between 1 and 4/,
+cmp_ok(Imager->errstr, '=~', qr/channels must be between 1 and 4/,
        "and correct message");
 
-sub NCF {
-  return Imager::Color::Float->new(@_);
-}
+{
+  # https://rt.cpan.org/Ticket/Display.html?id=8213
+  # check for handling of memory allocation of very large images
+  # only test this on 32-bit machines - on a 64-bit machine it may
+  # result in trying to allocate 4Gb of memory, which is unfriendly at
+  # least and may result in running out of memory, causing a different
+  # type of exit
+  use Config;
+  SKIP: 
+  {
+    $Config{ptrsize} == 4
+      or skip "don't want to allocate 4Gb", 8;
+    my $uint_range = 256 ** $Config{intsize};
+    my $dbl_size = $Config{doublesize} || 8;
+    my $dim1 = int(sqrt($uint_range/$dbl_size))+1;
+    
+    my $im_b = Imager->new(xsize=>$dim1, ysize=>$dim1, channels=>1, bits=>'double');
+    is($im_b, undef, "integer overflow check - 1 channel");
+    
+    $im_b = Imager->new(xisze=>$dim1, ysize=>1, channels=>1, bits=>'double');
+    ok($im_b, "but same width ok");
+    $im_b = Imager->new(xisze=>1, ysize=>$dim1, channels=>1, bits=>'double');
+    ok($im_b, "but same height ok");
+    cmp_ok(Imager->errstr, '=~', qr/integer overflow/,
+           "check the error message");
+
+    # do a similar test with a 3 channel image, so we're sure we catch
+    # the same case where the third dimension causes the overflow
+    my $dim3 = int(sqrt($uint_range / 3 / $dbl_size))+1;
+    
+    $im_b = Imager->new(xsize=>$dim3, ysize=>$dim3, channels=>3, bits=>'double');
+    is($im_b, undef, "integer overflow check - 3 channel");
+    
+    $im_b = Imager->new(xsize=>$dim3, ysize=>1, channels=>3, bits=>'double');
+    ok($im_b, "but same width ok");
+    $im_b = Imager->new(xsize=>1, ysize=>$dim3, channels=>3, bits=>'double');
+    ok($im_b, "but same height ok");
 
-sub test_colorf_gpix {
-  my ($test_base, $im, $x, $y, $expected) = @_;
-  my $c = Imager::i_gpixf($im, $x, $y);
-  $c or print "not ";
-  print "ok ",$test_base++,"\n";
-  colorf_cmp($c, $expected) == 0 or print "not ";
-  print "ok ",$test_base++,"\n";
+    cmp_ok(Imager->errstr, '=~', qr/integer overflow/,
+           "check the error message");
+  }
 }
 
-sub test_colorf_glin {
-  my ($test_base, $im, $x, $y, @pels) = @_;
+{ # check the channel mask function
+  
+  my $im = Imager->new(xsize => 10, ysize=>10, bits=>'double');
+
+  mask_tests($im);
+}
 
-  my @got = Imager::i_glinf($im, $x, $x+@pels, $y);
-  @got == @pels or print "not ";
-  print "ok ",$test_base++,"\n";
-  grep(colorf_cmp($pels[$_], $got[$_]), 0..$#got) and print "not ";
-  print "ok ",$test_base++,"\n";
+{ # bounds checking
+  my $im = Imager->new(xsize => 10, ysize=>10, bits=>'double');
+  image_bounds_checks($im);
 }
 
-sub colorf_cmp {
-  my ($c1, $c2) = @_;
-  my @s1 = map { int($_*65535.99) } $c1->rgba;
-  my @s2 = map { int($_*65535.99) } $c2->rgba;
 
-  # print "# (",join(",", @s1[0..2]),") <=> (",join(",", @s2[0..2]),")\n";
-  return $s1[0] <=> $s2[0] 
-    || $s1[1] <=> $s2[1]
-      || $s1[2] <=> $s2[2];
+{ # convert to rgb double
+  my $im = test_image();
+  my $imdb = $im->to_rgb_double;
+  print "# check conversion to double\n";
+  is($imdb->bits, "double", "check bits");
+  is_image($im, $imdb, "check image data matches");
+}
+
+{ # empty image handling
+  my $im = Imager->new;
+  ok($im, "make empty image");
+  ok(!$im->to_rgb_double, "convert empty image to double");
+  is($im->errstr, "empty input image", "check message");
 }
 
-sub ok {
-  my ($test_num, $ok, $comment) = @_;
+my $psamp_outside_error = "Image position outside of image";
+{ # psamp
+  print "# psamp\n";
+  my $imraw = Imager::i_img_double_new(10, 10, 3);
+  {
+    is(Imager::i_psamp($imraw, 0, 2, undef, [ 255, 128, 64 ]), 3,
+       "i_psamp def channels, 3 samples");
+    is_color3(Imager::i_get_pixel($imraw, 0, 2), 255, 128, 64,
+             "check color written");
+    Imager::i_img_setmask($imraw, 5);
+    is(Imager::i_psamp($imraw, 1, 3, undef, [ 64, 128, 192 ]), 3,
+       "i_psamp def channels, 3 samples, masked");
+    is_color3(Imager::i_get_pixel($imraw, 1, 3), 64, 0, 192,
+             "check color written");
+    is(Imager::i_psamp($imraw, 1, 7, [ 0, 1, 2 ], [ 64, 128, 192 ]), 3,
+       "i_psamp channels listed, 3 samples, masked");
+    is_color3(Imager::i_get_pixel($imraw, 1, 7), 64, 0, 192,
+             "check color written");
+    Imager::i_img_setmask($imraw, ~0);
+    is(Imager::i_psamp($imraw, 2, 4, [ 0, 1 ], [ 255, 128, 64, 32 ]), 4,
+       "i_psamp channels [0, 1], 4 samples");
+    is_color3(Imager::i_get_pixel($imraw, 2, 4), 255, 128, 0,
+             "check first color written");
+    is_color3(Imager::i_get_pixel($imraw, 3, 4), 64, 32, 0,
+             "check second color written");
+    is(Imager::i_psamp($imraw, 0, 5, [ 0, 1, 2 ], [ (128, 63, 32) x 10 ]), 30,
+       "write a full row");
+    is_deeply([ Imager::i_gsamp($imraw, 0, 10, 5, [ 0, 1, 2 ]) ],
+             [ (128, 63, 32) x 10 ],
+             "check full row");
+    is(Imager::i_psamp($imraw, 8, 8, [ 0, 1, 2 ],
+                      [ 255, 128, 32, 64, 32, 16, 32, 16, 8 ]),
+       6, "i_psamp channels [0, 1, 2], 9 samples, but room for 6");
+  }
+  { # errors we catch
+    is(Imager::i_psamp($imraw, 6, 8, [ 0, 1, 3 ], [ 255, 128, 32 ]),
+       undef, "i_psamp channels [0, 1, 3], 3 samples (invalid channel number)");
+    is(_get_error(), "No channel 3 in this image",
+       "check error message");
+    is(Imager::i_psamp($imraw, 6, 8, [ 0, 1, -1 ], [ 255, 128, 32 ]),
+       undef, "i_psamp channels [0, 1, -1], 3 samples (invalid channel number)");
+    is(_get_error(), "No channel -1 in this image",
+       "check error message");
+    is(Imager::i_psamp($imraw, 0, -1, undef, [ 0, 0, 0 ]), undef,
+       "negative y");
+    is(_get_error(), $psamp_outside_error, "check error message");
+    is(Imager::i_psamp($imraw, 0, 10, undef, [ 0, 0, 0 ]), undef,
+       "y overflow");
+    is(_get_error(), $psamp_outside_error, "check error message");
+    is(Imager::i_psamp($imraw, -1, 0, undef, [ 0, 0, 0 ]), undef,
+       "negative x");
+    is(_get_error(), $psamp_outside_error, "check error message");
+    is(Imager::i_psamp($imraw, 10, 0, undef, [ 0, 0, 0 ]), undef,
+       "x overflow");
+    is(_get_error(), $psamp_outside_error, "check error message");
+  }
+  print "# end psamp tests\n";
+}
 
-  if ($ok) {
-    print "ok $test_num\n";
+{ # psampf
+  print "# psampf\n";
+  my $imraw = Imager::i_img_double_new(10, 10, 3);
+  {
+    is(Imager::i_psampf($imraw, 0, 2, undef, [ 1, 0.5, 0.25 ]), 3,
+       "i_psampf def channels, 3 samples");
+    is_color3(Imager::i_get_pixel($imraw, 0, 2), 255, 128, 64,
+             "check color written");
+    Imager::i_img_setmask($imraw, 5);
+    is(Imager::i_psampf($imraw, 1, 3, undef, [ 0.25, 0.5, 0.75 ]), 3,
+       "i_psampf def channels, 3 samples, masked");
+    is_color3(Imager::i_get_pixel($imraw, 1, 3), 64, 0, 191,
+             "check color written");
+    is(Imager::i_psampf($imraw, 1, 7, [ 0, 1, 2 ], [ 0.25, 0.5, 0.75 ]), 3,
+       "i_psampf channels listed, 3 samples, masked");
+    is_color3(Imager::i_get_pixel($imraw, 1, 7), 64, 0, 191,
+             "check color written");
+    Imager::i_img_setmask($imraw, ~0);
+    is(Imager::i_psampf($imraw, 2, 4, [ 0, 1 ], [ 1, 0.5, 0.25, 0.125 ]), 4,
+       "i_psampf channels [0, 1], 4 samples");
+    is_color3(Imager::i_get_pixel($imraw, 2, 4), 255, 128, 0,
+             "check first color written");
+    is_color3(Imager::i_get_pixel($imraw, 3, 4), 64, 32, 0,
+             "check second color written");
+    is(Imager::i_psampf($imraw, 0, 5, [ 0, 1, 2 ], [ (0.5, 0.25, 0.125) x 10 ]), 30,
+       "write a full row");
+    is_deeply([ Imager::i_gsamp($imraw, 0, 10, 5, [ 0, 1, 2 ]) ],
+             [ (128, 64, 32) x 10 ],
+             "check full row");
+    is(Imager::i_psampf($imraw, 8, 8, [ 0, 1, 2 ],
+                       [ 1.0, 0.5, 0.125, 0.25, 0.125, 0.0625, 0.125, 0, 1 ]),
+       6, "i_psampf channels [0, 1, 2], 9 samples, but room for 6");
   }
-  else {
-    print "not ok $test_num # $comment\n";
+  { # errors we catch
+    is(Imager::i_psampf($imraw, 6, 8, [ 0, 1, 3 ], [ 1, 0.5, 0.125 ]),
+       undef, "i_psampf channels [0, 1, 3], 3 samples (invalid channel number)");
+    is(_get_error(), "No channel 3 in this image",
+       "check error message");
+    is(Imager::i_psampf($imraw, 6, 8, [ 0, 1, -1 ], [ 1, 0.5, 0.125 ]),
+       undef, "i_psampf channels [0, 1, -1], 3 samples (invalid channel number)");
+    is(_get_error(), "No channel -1 in this image",
+       "check error message");
+    is(Imager::i_psampf($imraw, 0, -1, undef, [ 0, 0, 0 ]), undef,
+       "negative y");
+    is(_get_error(), $psamp_outside_error, "check error message");
+    is(Imager::i_psampf($imraw, 0, 10, undef, [ 0, 0, 0 ]), undef,
+       "y overflow");
+    is(_get_error(), $psamp_outside_error, "check error message");
+    is(Imager::i_psampf($imraw, -1, 0, undef, [ 0, 0, 0 ]), undef,
+       "negative x");
+    is(_get_error(), $psamp_outside_error, "check error message");
+    is(Imager::i_psampf($imraw, 10, 0, undef, [ 0, 0, 0 ]), undef,
+       "x overflow");
+    is(_get_error(), $psamp_outside_error, "check error message");
   }
+  print "# end psampf tests\n";
+}
+
+Imager->close_log;
+
+unless ($ENV{IMAGER_KEEP_FILES}) {
+  unlink "testout/t022double.log";
+}
+
+sub _get_error {
+  my @errors = Imager::i_errors();
+  return join(": ", map $_->[0], @errors);
 }