From 5e110d72167461ee5842b56a347d8e159b88c22c Mon Sep 17 00:00:00 2001 From: Andreas Beeker Date: Sat, 16 Jan 2021 09:55:42 +0000 Subject: [PATCH] integration tests: NullPointerExceptions contain now a description in Java 16+ git-svn-id: https://svn.apache.org/repos/asf/poi/trunk@1885549 13f79535-47bb-0310-9956-ffa450edef68 --- .../org/apache/poi/stress/TestAllFiles.java | 10 +++++++++- test-data/spreadsheet/stress.xls | Bin 37888 -> 38400 bytes 2 files changed, 9 insertions(+), 1 deletion(-) diff --git a/src/integrationtest/org/apache/poi/stress/TestAllFiles.java b/src/integrationtest/org/apache/poi/stress/TestAllFiles.java index 9bce94f0db..347559330f 100644 --- a/src/integrationtest/org/apache/poi/stress/TestAllFiles.java +++ b/src/integrationtest/org/apache/poi/stress/TestAllFiles.java @@ -88,7 +88,15 @@ import org.opentest4j.AssertionFailedError; @Execution(ExecutionMode.CONCURRENT) public class TestAllFiles { private static final File ROOT_DIR = new File("test-data"); + private static final int jreVersion; + static { + String version = System.getProperty("java.version"); + String major = version.startsWith("1.") + ? version.substring(2, 3) + : version.substring(0, version.indexOf(".")); + jreVersion = Integer.parseInt(major); + } public static final String[] SCAN_EXCLUDES = { "**/.svn/**", @@ -204,7 +212,7 @@ public class TestAllFiles { } else if (exClass != null) { Exception e = assertThrows((Class)exClass, exec); String actMsg = e.getMessage(); - if (exMessage == null) { + if ((NullPointerException.class.isAssignableFrom(exClass) && jreVersion < 16) || exMessage == null) { assertNull(actMsg); } else { assertNotNull(actMsg); diff --git a/test-data/spreadsheet/stress.xls b/test-data/spreadsheet/stress.xls index 004069cffe2dc1d2186633ab7fdb02d443084794..065ad65564bf825eb53dd468be2f636755fd3888 100644 GIT binary patch delta 10242 zcmZWu30z!98K2z*l91yOAa5ZHVHX0+LLiU?2v;tSkTislKrSF;3kx9uw&e&pnl5Rp zJ)^CoR;@ACv$gf=T5o%JSFNp9TeaS|R*hP<+G^u`-}`3Xe!nKayfE`W=9_tQe9W7> z&pY>^_s+_t?SlTK3(@-Vrl-6)Evn+xD_5?-zwPw%4*37BuIC?`|7wc&V9lG-vb^4y zw9{Trsh=mrro@WHkIc_FDnxT+Vj?yj7@as7I}r^O$iG_tnyr(Qk<)vlgR#k>ijnAa zTWoTCW-QXJ2oqB!;ex=`(ZR^flwwCFqk{#3(W$`1%-C4s87r23V&pZJvB{B&$Tg9{ zW6_FhVxtw&Q`3`?!Rgp!#h%U`-Kx}TDpXrg!I~J4PDh3!(-Ffm?d8fyC_B+xve28- zDE^i%j0axRJoISu?$~scIy5*In~3fojqHt1MJGoiW1~~i6yui zz{$wiOf(Q1#!<K7}vE%N2f+rm~r{p{3% zPf$L(&F9IN&m|#C&?P9XW}l}(hMHT|b6hxpX{uFv5T%ZFpO!IWUks!a*Ttj3h!eb=fIjG52)Wvd!n zY$R5;yuB?Ls?#Nmp`J9^Mw5e@2Kh`|IIg)ljGJS)E0Nnsjw{(aUa2OyC7KKd@oq9D zh~bvA%rSCKlv zyaL!+?G-Sw1r^&CyGRpcu?1OdLDw&?x)DUf4hk-43=5cH0SmBz1z6x3D!&5eS77`K z(Qh{eScracy=fJ)2n(5EAsTj4fQ7~oUcW80;c6D(Y8K#X=C|7Qi>ueW+8Dx~Y>Pb< zVG$d`A{Jl~Gb}>GE();7_`xpj`W3SPi_tHx-*GWBEM|tq#t;qyTkNF(LoC1$>v0JE z^n4DXpPtVl<5v=Ah9xY*5@uL}hWbn>LBoAY4GW~i7{c_oMK=W)W&wuT2!_!wu2)DH z4SUEiYz*P3at%vafTb+JQZ|C6%&*k=!4YSR{S;uCvkB@+SjHkOV}@mDcz^;dGlnoC zuHhOM;2IX-8s@hK{o>kijWL9S))ogT!g4l*P&KY! z1q-l3{--lDuD8(&45^PCDvSdh=eFoqPuR=WvWV7VL`Mt;5$Sij7X1duZ>{mGWPX+W zAgV+|y)9R=F{@;TmBtXV1X~yWIu>kJnX4O?6-rxaI@r5l-Oe5<|Bl_viG#Wq13vF?&dP0|*;EQMEd{J>>IsgmY&2Tg zXtc7?XhlCg8m-0;`fQ7{O?x83-`K5L8X$!~{z{;(VG4%Fyh|8^J+WWh=!3wz7)*Z7^t zgQN!fwbSq*%d*9s9PJHT`kh_nc*=WweV#e_8FJXg9CjH8NRMrCUcS=nzV!|kRfpk1 zMhe`k`kbhCGtb?I2N|s`Zjg8Nxt^WOv(s=PDFyB?`f77-R6j%i_87Da(VJvdzw6Y+ zoVpAbvPWCoB+vJk_rjao%c9$Bcrf#A@jCSc);={KEuqh!efWMe4VL`7(?0Zjy}Z)z ztGk&P-Avuh)ZI8c^a|}Jb>|UZ-7Tc&0r_W9;LR#>*Xb-zLo0sGxdI^ z-p|zgNqt$VZzJ^qR4+6oICu5QasbsjKRAHuh}=BjtGk`}2bJ2rm^p}T-(fHqHN7n# z#C~6pZ&d0x5aSS2A7bi5IBHsb2-Rl>*50MuV744KZx(VyTf9j& z9ragBYO5Fh-fT*6O1_1Zy+#SSp)KAjFC;1Zn6eMs)XF}igzV84cgvq7Df^kSpDFu| z5^_shyiJxz+-Q$5O&&WZAewNTlgH9s)IeGC| z(E7YWVT`5}YKbq%Cy)8^9-%tOc`Wg$DLD+!sLy(g>SmDfxD1VkExkp};FSBK92oW0 zeTjG{(OqvcC-Dtb8w|$0*py(*zbrqRq@G3f5@O6U^(@XQt)9iU$K{nI^(j>AgZmUy zpF*`xyH26HR(2lu)qRCpJIx|L&D5t^+ow_eWqBz{eTGGThN;gm^%;u%@^RnxuTtA* znffeKpGCFKH_xIP5;u64*Qx(4@ip0T!XN*-se@70{OcGWQXfk^LCkZk)pOXYb~(pd zJ;z)iv9ZKA$mM#+MgH!@(lV{So~f^AYDj4;@lBaO7Ph`c9&uzmDG!YG$G=T=bLgwz z<{TT7IpuGeTkSbiL)r*qavuGkG9{SH+T}cRInP|qJ1)yecmvuzEw_&Q2O;MI>l;v@ zgT4U;-ysY#8B08EO3s(?5Ew#4^k)vVw1tn5(F=j7PpM_NDH8imTsV|rkM5-5x zjo6V9b=iphg|KYI#;_Dm6FCT8CNL)nOj942pof1Y3_er2Gl74m z@+0$a#PlFPc&^whFO-5C&g-TwOk5|0E1P-Y6__ z*Vzl+_`j$!zyv`QyhIp$QE|=SR@BsmIw8Bde^biOi16R%7`Zi8rIf4sJ39zrLI$%~H zwvaHm(ssvbEn16Rtf%{s)E5VnqC>k(T@ z*m{P+foh3mgl%v!SmAIWSi&L@u6x}EAp^5SI$<>ogA?2m%L#*9SXZ|Zu?)gCG7K(I zEs=?sTH$bm>I&--%pzeu!x|7)B+aqBI!9(nZE=7(-OJ#Vqvezz#r+}k|opYVFx SyMXR;e+G4FOuWCibs#Po2w%S@1m%6l7skU0{mRfMmd1uajr~LYObN}CW&Y8J0 zcP2LsKk6TT*nelqlzK(~B`8(%#F7{MiC3;(y$b)=832Db=-`-F@7g@}+}JB|{?`|r zicj{}?$K`iXx#iUN>xUtPr5KRvM^y^YQzZz{raW+G~HZ~sRM;~$5p9Aj*2{9IN+qG ztNT=HoKllb)b80z75J38e}Pi_ok)3AvoAQBd6&mt$Ix!(pEpUy~zS zj~Hz(X%kRuMgz48tW891p1xEYD4RptL^W2WD4Qh_ogs22vNH*t`8vNYlvhB`B-9p? zHi@;CM0Z7Ic_&RUnW z{q>=+t-(d@JkpvwOm`_Isx_YwF~v+0SIy^^-;A}HPsCuX%D4r0eW7W=9ZBJMspg8S z7Lh7dCp84~7TXrqP)?py_N1YwLU%QUSequaX{=30ZKXcj5GbpnxaoKomv9RzTWVTR zSvord*A-2H3_H&=(b+)GOw8FBInZ?4 z@2l7GYXW6WgtCM%D=MTES)3w>DOTw&NCb^`$P&IA;szYP?M|L^^kGeoCU&JfX)ue zS-?(s=z{ZNOwK~#EELW{b{3(tQ>V8C$~Ktqu&9dAxshA2sJcuG7F7{D;o2C=IxGzTAVS6hgKWCJe50&T#B91!+1SMAglZQk%MLZ=;uMd;i`r7U76?0w)I zY4c7fi&5K8+QqDeoe{K`OqSi|I}AoSX0ZcUj?U}ISgh zJ1faK)b7<+iQ0Xntz<3ix31cwFPbd2zA8~)m8h?Zov=5%YQIkJ@J>)mPd)N?I3e~KIvxBP!^u?}F*p5jJY6nSM!&)fPXw4@S zUMmKvRt!?D7^GS;NVOafwkTH(QM@{RpgYL*)}iN!VNh=!d*DYWcWRZ0w2C9aVaZj;bW?B9bQqJ>Ox7R?>odJxlrC z1{@SPo<&>Fi zb%!1`aql$Wp@wdZdl$E$qEn^?ed*@7J?PZBX{+b#5zZdWWSu?ighQIEKB~{eID3V& zS2%mw2}d|r-K{UjIQxXNPdNM7xe1;3=!$KD4BO95!nrBxj2zf@+TVKG$f3KN&18wb zUfHb1nAvqNUx#y1RAGO#--0PVMv5)$+!A%_p6x;38GUSfitk>1&-P&2$4R-3mD^DH z3H_p>_Yu8@=`~0{spEG9eV@{WJ90DcH<}CP#m}|a5qyz^HxQ0juKKjzzau5>0gAJo zhXRgUuKJ9=e@BY%LH*Q@VED77*~z(fVy=hyy6x>wjsiz7SAEWW!j##CPWu?yg^TK8 zW-xLPE2@N8k8sOOu6gRLX~JOc;_z@(~cK!AdCSglW73wjXi`8h=2oF zpgoEYa6mYMc>xb&y!jOGu!wgUk#Gd6)yjIn7buCfT6q@t_DeC<+|J0&R~6 zIUbzHym&*R!$Z;i(e`*q1RN4Q9^!y-h;!6;sK6s4;1N;a5sYV#p+_(t9DLzc!tu^g zzGrm&o3*=aB=24&b}bNIdLFlK{U0D}VOE=N7b zEm%7?%5f3pxQKE*8f7XOPhglI=_`8!+u*c)(zP^DROrYGi`U%x$)8Sx(s`Fyh@g@+=3=f~s(iKBj^ z-#Q%dy`MA`TVy$v+MiKPNU0nVsw7 zuwTC*HXfCixCPZ&<#?fladFfyNjV`J6J`kv@&qBBh|*sXn<&yx6v}v1{+g8WLK!cV zlTi5^QcfbJ$q(r#2`N0)9Q7(m;Wt8FpC$|C6jc6}lv9LqicrD~bJXuhIn^q|n0~5| zIw<`;NzGD^N$&_{0xJJN$^@ZI5XwYUzDCMKR)(SXi9(u$($`6v#8OZu2_?)mN4-JH zX+k+oD3ek7M^YwxO8xGUl<=SE`gFW*SB#6EKNE8WOF?#-82mEEQP!7=>`h{+$aYgJ z4cT9~1-{*O4yGggE5UR$+Nqq5-KioLK;>K9ioHVtp`3xrw@EnzW7beB1KHo;#~>Kn z3;|~%{5wf!3YLlN9b%bL7HK|uyFdI7x|k&lK{UKeEGSquvVRiGMrM1RjeW7FZrEKM z^)FJ+63SVq{I?y-9F+ctq&Y&GgVO&J%SGiVw_qsWqt|KnCH%R}})u{_N1o-+Ai zDCZ!&LU0bjC<`IGN-QK;J~DF&qvQ)#fXqj%0GXd!g~-PEOnyjTC}0u7u_P@*W;f8e z$l{32MP>)T7}+>t#Uf@gj^T(=K)w=GjyFnN&Lu)xif{sjDHUuUvWdjz2?o2TqvDCp z7Ys^t)FfgHJQg<9z&XuPlL^8*YV**d7Hrp!n!>GEwg}l&VvB@tF*1kPV!_IhB_IpK zd*Ilh6$lf_SRq&?vLs@ag275RuO^9A3AO}TGO;DGY##nP)lt(4E{%n185&(;%c3lD z%Sp3tQ|RJyVOW8NRATV*)Eml`$kK?d6s#IqIO-7V-^OblKQdDC|*vT|Zu1>1(Kf*8CiGQLRN$bj$MZG&SbnaA>SqZl6h rT%5l7j^Oy0;~%;E%~yOnI5s)*^c_$7eYN`FsgwSelabL=