From ee79db8c889e2c71bc649db5af5ec250a995575a Mon Sep 17 00:00:00 2001
From: Ryan Oldenburg <guzba8@gmail.com>
Date: Sun, 9 May 2021 17:27:34 -0500
Subject: [PATCH] trailing data test for png

---
 tests/images/png/trailing_data.png | Bin 0 -> 1221 bytes
 tests/test_png.nim                 |  36 ++++++++++++++++-------------
 2 files changed, 20 insertions(+), 16 deletions(-)
 create mode 100644 tests/images/png/trailing_data.png

diff --git a/tests/images/png/trailing_data.png b/tests/images/png/trailing_data.png
new file mode 100644
index 0000000000000000000000000000000000000000..f319ebbe2c3bc9ab3165830a3751159aeaa4b8de
GIT binary patch
literal 1221
zcmV;$1UmbPP)<h;3K|Lk000e1NJLTq003YB003YJ1^@s6;+S_h000DwNkl<ZcwX(D
zOL7!342GGrz$KCm%N&G*q;e1rk|mqu5^$EGOpS}Kj>mfavLz5ps#_y}pCvysfy<}O
zf3DZ-b!Q5f%jL55^LKuD`|J|1*+2;>0o8&yb3odwxs@f2%gI&p%vSr{H9!lABS9Nm
z0(tgqcmY;Glmu&htB4Ztzi&f-zP(M)0bd>;O_Rhse)<aV2IL*T1P}yDfDur8UE78z
z_?Fn#ncv?x^;bj^97q~aUU3m)KE_BRKq7$)UlJ1yK3xU$gsFBUfnNe{R8i1SalB^x
zDDL(a)RVHpv%bF)dZ;zvn2Z#n78=#Yv~e}>g{c5-B}fMy34Y5xoPd^;S0zM)A69^e
zgoyA%3qVSU3O~F6goMcOQwXp~hz>uE0F8tU@KXtpNXP;|odAx6Oz=|*kRH6EKOgcS
z79AQgzY>6?sfbkq6wr{blmH}6MXbyB=jZA7a=t!2-3{x#<DkCP#vQNsx8G)$ho2uF
zrfHf4w8&l(Q1JWcy(CaKx9$QwB%tAYNuc1j&s`z_DZvXqS^@|DQUM4FZ9Qsh4mE4I
zz5m{G)^@~&zg&Pt!f`VYS%L)q76BRwqn%hp2~zmm1V|*zw!vo!zuw<j-AB^Gxa~*-
z-z4DM%gcvDnluT{7L{`jX+XK~yCJRwjsV2C?`<xI=YSFXJ}^Ww$7xInXwS|LgR_$0
zNT38D$2*enLBI^YhXhK%8U76Fs7fdaGXY5Wo!wc2N<a&SN{(6s&NG|>DwS|g0BU^a
z{9yu663_x#US|n2X(It%@LO(B0a6L0MOi9f)bmGxj}klua5fspoZ$i_61)VAwtFiC
zNa4Fwz%BvDS>0ie{*Ky<TUt;FxYc$EAb#1p_dsL|stUMKmFyUh0;t7!#Tp>Fa)F-%
zB)#9G0-^|zN`UQxBr|`OBw`ynyN=K-!l=nL$_pX@1-~UIy^5e<OHa<>t|Hn1>N4)H
zM&x8;UB>l~j&dE{^O4?&{&;=;bBjvd9Z|QTsCFc}jy{&Lbl|fEcho+p7Np6g!9R+V
zUPX-ZlvYBIK~4Y)fJT7ygh=He?fd8F?^Ob4_8((uBrF#oh2ICbT7ndQ&r>R4sQ@kf
z)e^MuH4>Hyu)<#@f%6kTPRrv~!V&=}_}&sw@FkVtE&vVRO9C1`SAwemFZgH)Uhp{*
zNCBLFSI_tnsNazBW6O$M0$7;*9x4HTY*~>@01K1fL+J!i2O#gA8Cpdu0TlcWCzFIU
z0=V$ANk}0;3O|#C@B*~(vq%UnzzRQugs=io@S{rzDF6*WvV?E~yx>Qb5K4eI{D>05
z2#~@*AGWOTqffZpcp(I!;GyDh;US*eE&vS;9S;F{79*7a?j4Ur0yn0Yh%^GEptusG
zanT}D2+%^}NYKVci3l&i3W$<mjc*lE0*wD#YY89-R0R-#sBcAEzf|QNKYayw1M-gl
z&kJY+)K&on-xAvjzQ4bH5#~VBfbxoq0-sI*df+{twz^RHq^SVZfX{JC0P(N4dx5VC
jxDnv?YA+GHE8sU^v2zZx^_YVI0000<MNUMnLSTXbF+vvY

literal 0
HcmV?d00001

diff --git a/tests/test_png.nim b/tests/test_png.nim
index 056bdfc..57a618e 100644
--- a/tests/test_png.nim
+++ b/tests/test_png.nim
@@ -1,4 +1,4 @@
-import pixie/common, pixie/fileformats/png, pngsuite, strformat
+import pixie, pixie/fileformats/png, pngsuite, strformat
 
 # for file in pngSuiteFiles:
 #   let
@@ -13,19 +13,23 @@ import pixie/common, pixie/fileformats/png, pngsuite, strformat
 #   doAssert decoded.width == decoded2.width
 #   doAssert decoded.data == decoded2.data
 
-for channels in 1 .. 4:
-  var data: seq[uint8]
-  for x in 0 ..< 16:
-    for y in 0 ..< 16:
-      var components = newSeq[uint8](channels)
-      for i in 0 ..< channels:
-        components[i] = (x * 16).uint8
-      data.add(components)
-  let encoded = encodePng(16, 16, channels, data[0].addr, data.len)
+block:
+  for channels in 1 .. 4:
+    var data: seq[uint8]
+    for x in 0 ..< 16:
+      for y in 0 ..< 16:
+        var components = newSeq[uint8](channels)
+        for i in 0 ..< channels:
+          components[i] = (x * 16).uint8
+        data.add(components)
+    let encoded = encodePng(16, 16, channels, data[0].addr, data.len)
 
-for file in pngSuiteCorruptedFiles:
-  try:
-    discard decodePng(readFile(&"tests/images/png/pngsuite/{file}.png"))
-    doAssert false
-  except PixieError:
-    discard
+  for file in pngSuiteCorruptedFiles:
+    try:
+      discard decodePng(readFile(&"tests/images/png/pngsuite/{file}.png"))
+      doAssert false
+    except PixieError:
+      discard
+
+block:
+  discard readImage("tests/images/png/trailing_data.png")