From 10eb6193b711e9b007edb060721b61737de550a5 Mon Sep 17 00:00:00 2001
From: "Brendan C. Ward" <bcward@astutespruce.com>
Date: Fri, 8 Nov 2019 05:50:54 -0800
Subject: [PATCH] Add test for invalid tile format, resolves #58

---
 mbtiles/mbtiles_test.go                      |   6 ++++++
 mbtiles/testdata/invalid-tile-format.mbtiles | Bin 0 -> 172032 bytes
 2 files changed, 6 insertions(+)
 create mode 100644 mbtiles/testdata/invalid-tile-format.mbtiles

diff --git a/mbtiles/mbtiles_test.go b/mbtiles/mbtiles_test.go
index 2828560..cbb321d 100644
--- a/mbtiles/mbtiles_test.go
+++ b/mbtiles/mbtiles_test.go
@@ -58,4 +58,10 @@ func Test_NewDB(t *testing.T) {
 	if err == nil {
 		t.Error("Invalid tileset did not raise validation error")
 	}
+
+	// invalid tile image format should raise error
+	_, err = mbtiles.NewDB("./testdata/invalid-tile-format.mbtiles")
+	if err == nil {
+		t.Error("Invalid tileset did not raise validation error")
+	}
 }
diff --git a/mbtiles/testdata/invalid-tile-format.mbtiles b/mbtiles/testdata/invalid-tile-format.mbtiles
new file mode 100644
index 0000000000000000000000000000000000000000..c641c4fd7ed48d1dab3d67ca97cff0972c7b9dcb
GIT binary patch
literal 172032
zcmeI*Piz}m9l-G!kN?}rv~jY=`LmfeDVr>*HTL-L0clGys8CvMo5BiIMdQr4GV8=?
z;&i)kU<({LA#p}X+!n4JS7<NX_Jjl{B#xYr5E3^o%X>4nlc-^}@+ztA`A${m&C7c~
z|K8_0Re$;O^6>4u?VfA2x`%sq&#;s$imEDa8HS=L5%Kp;@po{gL?b-75pUH~?f=(|
zDEY`+Gvd9;)F+DmR9~F^)9n82wV97+p3LOZe-)iv2q1s}0tg_000IagFoM8OW69V;
zHXHxh(`Rk>y<WTH9(fP9A2e=lH;nJzX?)M{I*eNnlZLn+HtsfVZ+lI7yJsKdf7I>n
z?R4B{Zbu9<PVTnOjFM4ycDtS9y?r_C)a^Or4!iFS2Z{E{i1wcS#68M;vmLu<4@b!x
z@A<y*;Qn`HTHejK?%%m*_;ZZ=gO)$L?HFJ6()VJuo!g0Mc40-`ZSOno^L=~I-E(`k
z7qxHk=-!>TA2p0S_rBhE$B+YL#K9nAL-uUmjYqSqIkoEr@t=36?f8wvd2xngPSP_r
z`WbCLibb<;<ka`QXz#l3i&gD(y4`n=5B!_>c@fWylTrN+V?(^%@shNovFvI_eX-)j
zl|cuM*m(hm1HARx2gb&!Ek3i4{*BnL&2LAd*&8?1AN1^|ctt(!h-z`?sAu;Ir`yr~
zvu)q{+TDh6`aK)HW@oGIJl~RYHXnwg*_9Rb$6l}j_~32iOprmZvEgmMw@W?udCz|V
zJ+nK<t|9M}n~$_uHkVUhM7*f}Hu#U>GZFp1e!P>HU9;0|`a%1fH3%Cz9oDD!AGI@K
z2b<!hIoKIvyYbGpA60h!@Tj}*btN}nq}5n9BlhekK7xH9b|rKsqUi9mJlP%bTHS7Z
zv+=+S>E+q~_<N~ee!RWZDLz7m2|YP%JK~kxPe<0~A^o?C{%7&Pg#ZEwAb<b@2q1s}
z0tg_000OT>ARme*;_=6dS+=SbXSdd>TD6v0shh2)U9#%C<$_ZzwMuojZkv{6*?Bb@
zm*bnJT`t$1YNgsNl^xf#s!qvvTW+alIhI|lmYi0rXt_06|BEmBb0L5L0tg_000Iag
zfB*srAb`LW3wU4uPrt6r>f(V50R#|0009ILKmY**0xa<J4{nAczxW{i_*1T_7Z9)W
zC9&Zu8$R`0?JrPlcuLm)3B9i9|I+_1eg@!=`tS5#>%Y{$FS@u8KmY**5I_I{1Q0*~
z0R#|0U^SuT>bvfK&po_Zt>lZvYOP${vaEcmZkpBdR;d`*ay!lL@xF6(vsTDgth!~^
zwu-fUrBEvr%&nVMGhZu+?&_9R%UhMASt_Jr+LFF!KbJrFUr0seO?zM7m~#<rZNcvK
z4%^M+Ub`!v_NX@(7JZI;w0qb-81$tyZDBzkmLLb{A#Ex9%so636AF1VZ(dQgm8?8I
zKpsd?ls&Tk*YuAR{h#_@#2YRI5I_I{1Q0*~0R#|0009IL7%PFS7S60LiL(NSr#cpm
zg8cJZD4bi(oIVFp9QP;Qmr=tj3z@;Wit=*=b)%3JYYOTAQS|?c2QCB<KmY**5I_I{
z1Q0*~0R#}37J+C)oG7S<A`x*=U_#daoc}j1Tgy5SKmY**5I_I{1Q0*~0R#{@U%>zR
z|A&hB?*9+ZA3-ky2q1s}0tg_000IagfB*srT#~?7w8x3h`z!snw8!e_{dIvMS^p>W
zUn%;(^?&Gp(?8Ka*8iaYUjL2g;z9rc1Q0*~0R#|0009ILKmdV@5=d!FYX8W-P)b`#
z3{LBdYYVFUnSD(T=${H0_1gY%fiYcMQ%@ZR7*1+A)jyvwA)Ee5gmF3Q)G>k)v5a&;
ze%%4fL;wK<5I_I{1Q0*~0R#|0Ag}_W{+|i#;#oKX2q1s}0tg_000IagfB*sk5fJtN
zY(SREG7&%k0R#|0009ILKmY**5D2V*sQ>lAE}n%WfB*srAb<b@2q1s}0tg@w5CKvD
zUkS)kStbGqAb<b@2q1s}0tg_000My(5cU7nz%HJJBY*$`2q1s}0tg_000Iag5D)=T
z|IY<vsbjj#<+*gQe*?p^mK@Wqxs<f#uPK-EJmdK!sj#Xf#xu3c$SSp@ssY~q<?E?r
zfS3R2d`Qh)#><T7KmY**5I_I{1Wp7*{m=hEPB>^GfB*srAb<b@2q1s}0tg^5l>(yv
zpP$Nwu_y!(KmY**5I_I{1Q0*~0R#{j3W)lDVR%CW0R#|0009ILKmY**5I_I{1g26z
z)c>zd<-%AL0tg_000IagfB*srAb<b@2n+>8{huA)&_DnI1Q0*~0R#|0009ILKmdWM
z6cF|Qm!@)IED8Yx5I_I{1Q0*~0R#|0009Js0;2xU4R2^5fB*srAb<b@2q1s}0tg_0
zz*Gu|`hRgM7sjFxKmY**5I_I{1Q0*~0R#|0U??E!e`R<>0|5jOKmY**5I_I{1Q0*~
zfe{3nrd=-Aooc1pER`MCw5m?Yc3W<#W;vE!td^Wst7y43S%Y%DGJ)k0ww5UfAb<b@
z2q1s}0tg_000IagFhK(1vkog0lrE_vfB*srAb<b@2q1s}0tg_0zz71O{(pUhX-q)?
z0R#|0009ILKmY**5I_Kd2@(+X|LO##OR5MUfB*srAb<b@2q1s}0tg^5f`F+1jS;3X
z1px#QKmY**5I_I{1Q0*~0R$#UK-B-&CMaD}MF0T=5I_I{1Q0*~0R#|00D%z%ME!q#
zglSAc009ILKmY**5I_I{1Q0*~fe8{2_5a!grAw*^Ab<b@2q1s}0tg_000IagFoJ-n
z|JO&D#uNk)KmY**5I_I{1Q0*~0R#}3AOTVT-<Y6uNfiME5I_I{1Q0*~0R#|0009I>
z5D@kMmq(b!6a)}J009ILKmY**5I_I{1Q3`Y0a5?IF+u5)Dgp=~fB*srAb<b@2q1s}
z0tk#CAnN~(5vDN(0R#|0U;+e|78kFkmnR_63rcbAdS-q;d_ln{A?#WvnMh1Rq8F3n
z`r2Yb)h;IX_(#oT=C9`BsyhCeUPy*>sf>0t9#1G25`El*uBVcjl&XsV<&S%&7n0w4
zYH2>e%U{;k)2W2q{tM9^`=H@aI-N+XN<jbZTMuUvV)s9D_2OTK&9UFdFOcARIGOxx
zfiDo8sR$r|00IagfB*srAb<b@2uz58sQ=%bkW@(%0R#|0009ILKmY**5I_I{1V$7P
z^?yLWIb=jIrXhd;0tg_000IagfB*sryd+?jt!l;Dt+lFFtz}l~W~*tJtom-b;1o-(
TQr)fFre#^StU<Y6jlll^=or!G

literal 0
HcmV?d00001