From 7e8517d08e9860ceede6c0833b4e37edabb11e65 Mon Sep 17 00:00:00 2001
From: Andreas Lind <andreas.lind@peakon.com>
Date: Wed, 3 Apr 2019 16:39:35 +0200
Subject: [PATCH] Add column limit support, similar to #541

---
 lib/xlsx/xform/sheet/row-xform.js             |   6 +++-
 lib/xlsx/xform/sheet/worksheet-xform.js       |   3 +-
 spec/integration/data/many-columns.xlsx       | Bin 0 -> 8885 bytes
 spec/integration/workbook-xlsx-reader.spec.js |  29 ++++++++++++++++++
 4 files changed, 36 insertions(+), 2 deletions(-)
 create mode 100644 spec/integration/data/many-columns.xlsx

diff --git a/lib/xlsx/xform/sheet/row-xform.js b/lib/xlsx/xform/sheet/row-xform.js
index 27ec1bd48..9e08b8308 100644
--- a/lib/xlsx/xform/sheet/row-xform.js
+++ b/lib/xlsx/xform/sheet/row-xform.js
@@ -5,7 +5,8 @@ var BaseXform = require('../base-xform');
 
 var CellXform = require('./cell-xform');
 
-var RowXform = module.exports = function() {
+var RowXform = module.exports = function(options) {
+  this.maxItems = options && options.maxItems;
   this.map = {
     c: new CellXform()
   };
@@ -119,6 +120,9 @@ utils.inherits(RowXform, BaseXform, {
     if (this.parser) {
       if (!this.parser.parseClose(name)) {
         this.model.cells.push(this.parser.model);
+        if (this.maxItems && this.model.cells.length > this.maxItems) {
+          throw new Error('Max column count exceeded');
+        }
         this.parser = undefined;
       }
       return true;
diff --git a/lib/xlsx/xform/sheet/worksheet-xform.js b/lib/xlsx/xform/sheet/worksheet-xform.js
index 50b3c0446..853b66e4d 100644
--- a/lib/xlsx/xform/sheet/worksheet-xform.js
+++ b/lib/xlsx/xform/sheet/worksheet-xform.js
@@ -30,13 +30,14 @@ var RowBreaksXform = require('./row-breaks-xform');
 
 var WorkSheetXform = module.exports = function(options) {
   var maxRows = options && options.maxRows;
+  var maxCols = options && options.maxCols;
   this.map = {
     sheetPr: new SheetPropertiesXform(),
     dimension: new DimensionXform(),
     sheetViews: new ListXform({tag: 'sheetViews', count: false, childXform: new SheetViewXform()}),
     sheetFormatPr: new SheetFormatPropertiesXform(),
     cols: new ListXform({tag: 'cols', count: false, childXform: new ColXform()}),
-    sheetData: new ListXform({tag: 'sheetData', count: false, empty: true, childXform: new RowXform(), maxItems: maxRows}),
+    sheetData: new ListXform({tag: 'sheetData', count: false, empty: true, childXform: new RowXform({maxItems: maxCols}), maxItems: maxRows}),
     autoFilter: new AutoFilterXform(),
     mergeCells: new ListXform({tag: 'mergeCells', count: true, childXform: new MergeCellXform()}),
     rowBreaks: new RowBreaksXform(),
diff --git a/spec/integration/data/many-columns.xlsx b/spec/integration/data/many-columns.xlsx
new file mode 100644
index 0000000000000000000000000000000000000000..c6c02482173e4e58a7b62e224f885ed04942b872
GIT binary patch
literal 8885
zcmeHN1y@|j)@|Gg4#7P*1a~I{_u%d{5@;NP25kt?KyU&95*&iNJAt6V9fG_2*O__m
z%}i##U+`Y_TDMlOb<eqV>+W6q)H$W900)l;Km;HI001h0*+GVl5extj3l9L`0+3<#
zrJNjHEgfBrHN9S2f(+R_9qcJT!oxCT0brr`|F8WQ&p>g)pmHY%mdxeLJE?V6`S}WQ
zWS;$?UK}Ph(YCJmo<ei|Ok3N>%!peo$!vT(zArf6=6yJihiobw9cqHYdm7cz!GV2k
zT1F&%?A^V4bghI>Auf8lhgo<e;?JKN8%JBE0g|0-JJq?RS0v<%iEZ!+L<80`W_w>@
zFZI-;7wY8;^v$&|E5BOA<{u~+$66c3ov^cI^f5YJW0ivkQ~S&lxEtlEQt4<gnyJ^j
z+$1TqfEenyCQZy?7db*>n=u!Lmk=9U_{f#Gi7N^s&kFT=Vu8%9z6QSDtqkwLXR0?1
zy!bZ8tooVafmA_b(*RW<xQd{J)A8GzE;0NZ#lfQv%7t6H6<pG;B@&B<%$QgUWg=<^
zmgT(OOlcJXKBgeQ&wc%M{B0`x=A^!kpGJ|^;bpEK@h-5p8&2CKvHIJnXy1hGFZXQ=
z`Rt8s=m#Ghzu)%fd_V*M9v|TVs(+znwH61>85C=ZP}ZSAX=&_YX%Av&`+5Bz9si3t
z_@`GdhA63Ya-fGEzPt<TyPRBz#gbI;ka*chrRf(SH~*wMI)|2Iq4gOlmL_p9oUDJd
z-`&96f^hUsAJxSwcQF_TPl%?*qa-xx#@P*tiQeV?3+LjME^PP7^T~@8S$Qu8_vRR;
zqPqNSh5jX)7vsm$U$DNhX%nO1=8=XHh@=GS^(g5snq8H`OiF6(mxPwr@@4JBe@pY9
zdjDw~Lo8fCac?XIzYk<)JyYu2XHRv0O{}G1BVb!@mgyo$?PY9X-*zgU){1-Y%c`8-
zr%ug{bH({hwwEsJ++VMX`*<+JtBVh%v#9T=KL|{bzX(;Rzmp_TG$H8}IwK8Gk`M!s
zVLa{Gf9HvZlMB$y$qD#Vz5dM^7^s9nz5I8#Vs#~jE)J|_w1+Sbk2H6D>={>f>OHO9
zC&>L(jC1r<Tz+RuBuov4I^Px8;haLf4u^W(FL^Lm;c(B}ne!uY;JtC}4}}n%hE9jz
zkj?f>$b)1NaB=o_c8m7WFjGOe^`ekImP|bUA6E=?Ps4IPQA?lIkA-sE4Rg40+JQ#M
z2B4T3tXdDad8@Hk5>!=NOPR`OweeUlHFtsdA;^nj^z<o;oG^wLV03GOiutNt<^uVo
z>CQ^YYZ%y`bA!G?TBJZ;$Rpw#>Pi{bCrjba2mx(;Olj0QTyd4W9bjyoJ7F0-JG5Hw
z@10zNe(AqMrXMJgYX%1Z93ue$PoQ@|A@gUj6l*y+<?v%Y_>J8ob$Jk`)dHAwL$F*E
zFO;=v=kDuV+To31KsC1NTDMD1ijuRA+MgXS3nzNWPp?u^tV2(q+?G}!q-YwP6(|jt
zJA7%u!#}JoxW8@?k8@yDx{a*)XkBpMmT+%+ee1L-M0MyGQdbGc(k@uA5utsTGZ5;g
znGq3qZc-4kmmiupFK8DV^{UWR(W7L|j8u?pqxp^a%XNJYX~R><#z#`2@4Il}7Oa!v
z#yq~NTzc9q2&FkEj-;xL4KS3BUW4bI@OJu!DF*gFRND?{j)!t4`*e4bSI>wLJz7u{
z%Sv}V8AEWYMd55V#cvWH;f3BdA-6xcp)ZE63qck@@twHLPg~-(JUdW7v>`uOa>I_L
zRzbv%79$AK>*_B~ntrf03cQ^9*m*>_ulPxC&aPlAEfV-D?*jI{S+9|OpL5@f#h^r!
zC%rY5!u{9kdiX`#(>p+{6Z9uYogD6PAe^^f?0DiuChb|hF$9Hg&Wlph%BP|MA^O^T
zAL0}cLlACYjZJecC7mYeCZl-<c&jtMp{9Fe@n)B54&Dwtm)S0V(?8pazZIQ`i4>$S
zFd3$yrj%h_kyi#>fA&^E0T%<Q!#T^>^Ffrm&49j=pS`+j`I`C>8RbU$kjjuUaz|%~
zFg9ju*L?jN9I|_DgS!rciYg|pW>DAH*+sK5U<ePq!)S^%RYw)v`N04aoHE;T2{-=1
zRZJgQ`Gt|l!ee-hf8W<7jLQAA;|cCYl4189ITkD1{Z{eyZno-Repn~+A9*Ap<?~8C
z>`h}zc8zFLV*B^)S2Zv_v}Z%~cp;|mXJb-!8;oiq)g)G7OQb$zB`OHxA-whUlDLyW
zgyHz&|H9U;NZcW{#E&UN<<lh2){+(PXoz)6ve3GW4E@{dYDf*;%O%C}_HYxALbRGG
zibSK#PV~Yx@t7;m>b5U2M?Gk<6y0o*zTaekVjg_$X7NrHZR@?h1dzuZ0p(SdABWMV
zGpYq;bgYNV%x>K~QB101?NrOx>GlNDPcH&K8ppz3)Ale7N>U(rtstO5%J?LXrN7r1
zw^s7V44?7lc)zXS!51pQ<s#@^7`J*e&HrOtW|U65zW?j-{-dDIydvY>ipU5&C8kw+
z1s@M$&ijqlf289Pvd&o^boEw;20<9V(-GwAWp4@kxpL=fIRNK4upUIG{jqKi2He+J
zP-+IpCRDe2vdH#|%myR!jiP9+hkG7wmEDa<1#3qAB9u4BLEjcY>SJljs8!}YE+w3#
zh9N}HwRR$lltNoY@!Qd9cJmAKM<Ojfzp=FRFSRO^%%NS>S$?GoQ048^-%LfIs+s4M
z+fQ^%4q~t7g$PckVtk1<t*Us4#iPaSWQ<E-7N%sexE7;?q;A%r$g`5{T&1ENT!;|7
zs4$p!O5k;64X`6rvWaN0u``=UF^4SPQpc?4jlP;XKz$ui3N)%na^190w()uayFNuF
zgMzXKo)tQk&qaA|-`niVuL3Mdm&tQXgnJ&C@6^7kyM0)A^q~n9KA>?6x1SGCiIU5F
zuV)iX`VuJv+$5{k9~!RDbT#|>u()8|Oswt})I1x;-=BRc8tF+M*4;tH))k?(hY~=e
zA|cuv#bgqUc`@D+c!o;v5U{=WfZhDum7=?U3$Ts*V_Po`mHu`<CWl!Cp@<BN$G7sl
zXdL6Nv>KG<m-#B)vFc+-5@$d<I>ToRN#UFA$5+NC9z5zLEV+$koamQ*d@DARR7jvU
z5?*ruC|;A+rvz^2bfT2+Y>Z<?^JI+<(fu!`wA$85DMuPMI-JjuIs$avHVNJu!EBZp
zEv0J^Y)N<9&7xkq-XyP7-bZLPB_1>Gpe@->Nm*Rki+SdKEcn(gcxhtVWFk|t^7O&0
z_Kw4N0pFPeEveGPWe8%D$)YX6S#MbSs#`d!bIv#iX&Gax)xYcnkL`O0sweH&>}jcp
z7;EED*|b1Eu^e%quW!gzI%DMU5WC%o5aJm#5GuFbaPZ{t%09(!7^>6K9P~4Ph=`RQ
zJzDlW>F;pvhu4-Zd50R^Oo?Ovs17f{Br)udc<uC$cy1@eYEgztOAqDG-Sltr;%Z~*
zV9Ea5_4!X!@97SKN%-(u32(&EK^}KJYcX_-%irwd7icXq5=g6S_S9Z+aYC9%aAD~L
z&ea)m^O7W;L=wit;V_!dVo+$SwkbxR4NcROY|y6J%j>ygfMp+=KYzZ=^z%M-JsxXL
z_LfaX>wvsBZc#lENspRNrX7Fp59DXy_EmYA5QS}t;&_e5a;Gr&!OsQTp&z+-8tz)!
zKKng|oWL9tK{s+}TwR+Lf8Y{*RPW0UnhylMq-4R1$pp-K>Lo-pkSn7gGgY5AoK!S>
z7>V-!J_SuGE6?L3F`^ijv<tt`Wq(=gtdI;3*-}3sqnbT2&3rCIzUnh}$$*37XLC(=
zze!ljPB8k4UU~yjSIRN#k+WXW#8+n&bh6!UH%5sZtml(w43}@vd%k<G*IG078ODY?
zDDhLxwBT-{W2`BL>dj<!qxPZJAi?5@v0-IBSz%1bRF<ZHc_;;X!?FxonFOP#L1$Qo
z)o1J7sVqEW{mv<ceIwP{GoY%poJ(COTS&E?NBu0mMaR}V+(q<YiDn4nH?>%dqCIsD
zMICCs4ZWxXPEkMrtU9gSYJWGn$_R2r9hgC2xoeJY92v(c6o=lVPLmNsUhAq%H1OdQ
zO=JxRXKtdQNrVoG`h}-iSHEJ-9dp(sA0>OHh|j~(_GwtFi2vEvb*FJfeKnJUBNCm4
z#=&Ha@#FnH;i7Tt!_m$$-}c0_<BqP@yX(QE*2iPP^YYbseD;<%CnwwSgiCMEH&tTD
zX6H!Bl8>k&z8saVm><%EO#?A+k72|zx;P}eAmZ3t?)e9UeXvIQlsg@6<@+6Tq&hta
zO~S3fnFfmEdfIm>WiP%U%$rBK6L|<O7p4v#9lg#}T!=rdk7$e-i31+6$rl4IH@UL3
zSJh{t%PoIkMB+qz;%^_*X#FA;MOM@(6}~Yg!nvwg9&vVnRTdmRPABDJLZ{AFC1n;q
z*kMWecJ^dw(V?s}A2=!EYe8cQ9S4VBc)>=`pbeYap1Oeaa~`r5u*Ev3i+{T;eJMMw
zw09)q>5VzdTZJEjVdumWt*Gd0n78DHUAzzlm#wjSFcDXri2HQ3sxW9)5z^P?M~_J*
z6Xy{uN9Y?fBP1;%MXFlye7+W?rsLj<x(;F8E;Zp<_>JW-*_WoMlx7M-*PwE+<*abc
zboEdKmyqCTLE>y9k%ntj50Ou;C`f0-lxuLaepH*<B#~@SzL>dhSnE991jZ6l2>j0I
zl7!4`d3l`RP;Twy9OOyB5P`jvdrd68AaBUOk*?{T@eWgInFn@*VjVorIEApq0cmWN
z+l@xnwNCOTIPf)m&Ftq?*=u6FD6-!I|J2v+Ddk1&#SoSnFi6-m6d4tra3Lrh|G_{y
zqe##GAR%o!jrR#0-iuewxbf@J-QOZAp51v2kbA&@5D{YZk8cwM(-<r_r2{KbE-6@m
z-Zq)@ZLcB4pzmH<Afw5v$J;`yaU;<(%VA*3sb0_d#kquCN|SHk0R7>~R;EhRmklne
zgo8nZ?nX(EH}FlPQAiy5b7~FG>g`~LsjpIbl3C^xXjs>r>BXLL&c=+O>q%KIw53NX
zo@;t?(Px?GhPg50e;6sbRX+8Mi5^$Bt}kv#EkHwnTzfJaL`x;ff7;R$Ib1B6*!A1e
z^w3R0WSAKzGF7x9QF9|wGscv}GqQebG?St;>Z&uUx|5Wx_riGtmBRra-*e+)S*Cd-
zOk!3d)QU7=(;WYERZ3G-De7>_3-5TYf)p54Gj=Y@7*~y%24^ojH`~HynOs(yDA&qp
zjxmlP!W9;wq4&4CJq;;g6PzmbX7wa4TszTEuz@nv&*}o{=`}{WgdDG{?T~AkXt;sb
zxLZV>Q=W+RG!y!SA2sUSpUP!AUU`_le@a*pCY>N?5U)Id_&hGzC!vegnAJZKgL{<p
zqXV;c0G6oUTB<lRMVd2~t5BaOXFH8;=mL>irZPp~ibm<{hr;e9vSQj9(vD_|=$K|W
z!A(BwTpb;DW~6r!0pMVlouTJ}Y32BF8`mDICIdE8O_5^)3GZkG!K&^blPxkkcp^%_
zj;b(I5UI?hwrB6M2`3lAT{NpK@KH<-fDmneSbyqf+L>^T-wqR6$0$0Hny6sPbG%n(
zbtwh1)nnpJddWAfOuPZZwR5n3_{yoI+oaqXSy`zzSH;DGjIJ`ld*$Hvf?wpg^qE*+
z##cQzah6^AjH{c0`KC!LB#%I0@7MnPpVhZiC7qIJ7;eT$`Vx3e!1M9C(a1|p0?$|<
z{TS_$7UZW{6NpV)u{49#v$1Q6Lfna74c?a<?6@&KSvHANGr%0*y@4?>9g%UjE$f|P
zJ6PLb9SXq)gB6Xx5>~I=thN};_&OhAQ)b*&4`k^&xZ8zTRf{mdH!J6i?0pvuF)GnN
z#PD7yA0-t&YNo-!oSk%3-Q`2AcB$1zwxhH3%hAn8N|YDMuO+$9x~F6|H`!o(W6=V*
zv}>$+{KtgABxk%`9W<QWfl4sZ@ACU|cL%buv~&fr|JMDc!Rd)Wr&$iX&{maM|MELB
z7Z^zr38qrkW_!aqhY!(f0h6@D_HZSI_m>%Ov2ki6Rq0~)gfD!2eHYWw*^D4W4$v)L
zuGC(YLzIrZOyBq<)}wD~?J$RWYA#iT%s}W>BBL+1?>*-<%60MgC|h{c0^-%`?<r2Y
zk)^tln)((j2skJySQr}h6`J%<QNT)a_VqRUx+p7*i_BLico^6+=@kW8X>aqq*{O9E
zxzmkHygp&tp@_(^QJr|jSLeUvjMX_w-kp-f%XxbP)?%X7?TNhvkJI5bzr3nHIwbAD
z`?e$_&z~q6@v>H8X7<DNo14w=^)faEXQGes7U`vQGw(MMgyri$2tme~tLJlcHsj5V
z7~dNwo$(2iuGWrDWDI2-7xNs{8x`^GdzTy+3)w04WVB3W%)RsAeFO(f^ksK=y?R+;
z6E6GiaovoW2=C##dq!ES=Vkl`4fyFKx*0#w#)SduaXPy}8x0Ab))z``TFq0|v~qs(
zRoOnxdq%T1ELa`Ow>wu@4n{;bMjz?a?5S6mgeQQ=DD298)D;ou!PAD%Tl9dQI})q)
zpt~ct0X&4k4z<}{;ZM@>CY0Tn66Y(^Of=Vx>sM@BZ|b;+Mtt{?34-4TRn-KFl-Im|
zb@L&*=xafxTw=W&Z%Ku6#!z9gzV#`h`%GIY@$|<)L%#AfM0b75W=<ru($-=L$Iy5n
z06a+S`2wS`*l8kbU^t4HyIeOhgM=ravzMdG5`*|%U7<uL4w#r**x4`yv9Z8`8y%z{
z+$)ba+)^a5rq{}2uzNZCVr}>kI8t{;!DBPOCx!bpP1Y8scN*n>N2-o*zGY~SMEu38
zy?1^1D)^DQn%ohhFI<*n_IRVJ`%b=(9-3C+>4Dcd^gH<yC1lIzMl5Nl&P*Y-6(`JO
zchHQ*-*Xvh%ukdLp(y8qA|CH|w1aF+T`Vm$TwQD(t$*7?S;R@$b#b7}T!t*RdoDSr
zN;1+Nv!IGd_yANsMqt#WN37d!UYy6uwCn9$KeE3OCiU0Q)ShOrAU!|L9!GsW0BW<G
zSTg>g;-rTyly3MHIj03?JLiorQMis4`&hon>5U`05;-JC$_Qq6P{66*nXI;Zq#-b^
zAMSPJr^5#Acmwg0PX@8F_hUdU4g*x*y{yA3GKT3u)?4vlQ=#PBrb-c&5g$LaX554I
zqwfQsNEIZhMDVfu$n`+1ggN2Dgb6-&BnEn|EPaa+-4a-Jur<1jPOB}Hn(!Jux@Uc2
ztb#I^F^c!Dw@||WeS-KRR|H-_7YbYG1Ytw-gBDKasxD5iLG0#EE|&kQZ2d1S2t`g5
z#89P^k|^X*>Fx=}C9CVgL|hG+VrKGvGJnqTySBz?9R`@Hxz;3i{gPp{!vK&c|0c$@
z{pZ3h+~W9ZHq}%tyucsTvECY4o0GyV5^<-BXs;%TQLB)`tTr}ehA}p(LuB<y%8i1+
zuyR5JN`W<tB#~d%^i{vowPy#{m6n)^+V3Nh^eoLF9)-s;7q@Y>3wN?|@QPrQ#RXly
zJT)!Cq~s4p#j`nlotk?!x7*Hixf!DTlt1yz2Un164Ra=lqHD1kRKdrq6wj?oNVBeT
zyz6p2p%tCyXx$BySihX0Z_-k;Q9@w4@iAnAx)&~+%&a@zIMl0*F1+!c{ji?A&M+E|
zR2na6R4O|D1hODj{3D+<&XqXcS+PPZb+kVLvRiv`O>sGw6$a;qeyp;Zj3fCh2j&=_
z0pxojFu)F-!M}sA!$=o7779KRD0i{`3_jD>um6WV=#>4dr9+gQ=6;qj;BF|;7f1vg
z)X)Qq&0cDsm7Bpp+^v;AnP*qfh8M@}O{vt3PRzK^MpLjGiGIvKz|3IwR<sB%>vB<x
zgl$HtKEIV0J|t%o0|ycLVz7A@cVWxTZ-Z6r;w8Jf2nzZLGMPp>dlM5Z^!5S>>i0pP
zQ3~LOZD$8ZxW6yI;cz1Jey?rm#eCeh1uSM?>sM8LSCsiURfTD_oRhTh+{&b+rE9Ag
zX1HuZ<$JDp_R1<s)H!KBYYRELAl)WsqCl<9B4i_nEiwJG9k6XhFlGdUUy@D?bqCv2
zX9>}sy2RV)+=f3lXs@iPjalsuz5}KhM}!tFlWUnW4f5PQj(?kis#zV}muo?w*!?^m
zxcQ}dbWkle7_?%-5xCwXusU2Jb+c$sc`O#51X_(VUCnr11GmHt8s-xfP`T;!A!|T>
zNGAEbm3CfsM(VLbPP8=TWu!Srfb;ztNqJWkf5Lpx!X>!B0im_`#_-g`oIf~e`}_Hc
zyP8qbwXro1omi>f<A3UqFtDspJ^AMY0{{MM|L*_cxPYp{Umg6ll=*MNpM4UP8h<Km
z{%ZKw^3I<Po1kO;|BF4p`uR2I|A(hT=oyG#DgeJ4|C*uv!`Kh~H{(C@mA{()nhg5G
zR2f=&fIj!v)X=XEehp{;a4<yhpMn1^xc$}3uVKO;UK*eN{szAW4ZnK$tB(F*4*<*)
r0|0+h)nCp3x)T1`T$k)m=Koq3RTU7RUl#yCh5iDd!c9T>^WXmgr#yWB

literal 0
HcmV?d00001

diff --git a/spec/integration/workbook-xlsx-reader.spec.js b/spec/integration/workbook-xlsx-reader.spec.js
index c57ca4943..b9c650df1 100644
--- a/spec/integration/workbook-xlsx-reader.spec.js
+++ b/spec/integration/workbook-xlsx-reader.spec.js
@@ -53,6 +53,35 @@ describe('WorkbookReader', function() {
         return workbook.xlsx.readFile('./spec/integration/data/fibonacci.xlsx', {maxRows: 20});
       });
     });
+
+    describe('Column limit', function() {
+      it('should bail out if the file contains more cells than the limit', function() {
+        const workbook = new Excel.Workbook();
+        // The many-columns sheet has 20 columns in row 2
+        return workbook.xlsx.readFile('./spec/integration/data/many-columns.xlsx', {maxCols: 15})
+          .then(function() {
+            throw new Error('Promise unexpectedly fulfilled');
+          }, function(err) {
+            expect(err.message).to.equal('Max column count exceeded');
+          });
+      });
+
+      it('should fail fast on a huge file', function() {
+        this.timeout(20000);
+        const workbook = new Excel.Workbook();
+        return workbook.xlsx.readFile('./spec/integration/data/huge.xlsx', {maxCols: 10})
+          .then(function() {
+            throw new Error('Promise unexpectedly fulfilled');
+          }, function(err) {
+            expect(err.message).to.equal('Max column count exceeded');
+          });
+      });
+
+      it('should parse fine if the limit is not exceeded', function() {
+        const workbook = new Excel.Workbook();
+        return workbook.xlsx.readFile('./spec/integration/data/many-columns.xlsx', {maxCols: 40});
+      });
+    });
   });
 
   describe('#read', function() {