From 28fb962cdb111d77e6e19b7e2f6efb520cd696a2 Mon Sep 17 00:00:00 2001
From: oleibman <10341515+oleibman@users.noreply.github.com>
Date: Mon, 20 Feb 2023 06:07:12 -0800
Subject: [PATCH] Complete Functions Coverage for Xls (except GETPIVOTDATA)
 (#3383)

Finish the work started in PR #3371 and PR #3340.
---
 .../Reader/Xls/FormulasTest.php                |  17 +++++++++++++++++
 tests/data/Reader/XLS/formulas.other.xls       | Bin 0 -> 15360 bytes
 2 files changed, 17 insertions(+)
 create mode 100644 tests/data/Reader/XLS/formulas.other.xls

diff --git a/tests/PhpSpreadsheetTests/Reader/Xls/FormulasTest.php b/tests/PhpSpreadsheetTests/Reader/Xls/FormulasTest.php
index c27972054..68434c946 100644
--- a/tests/PhpSpreadsheetTests/Reader/Xls/FormulasTest.php
+++ b/tests/PhpSpreadsheetTests/Reader/Xls/FormulasTest.php
@@ -40,4 +40,21 @@ class FormulasTest extends AbstractFunctional
         self::assertSame($originalArray, $newArray);
         $newSpreadsheet->disconnectWorksheets();
     }
+
+    public function testOtherFormulas(): void
+    {
+        // This file was created with Excel 2003.
+        $filename = 'tests/data/Reader/XLS/formulas.other.xls';
+        $reader = new Xls();
+        $spreadsheet = $reader->load($filename);
+        $sheet = $spreadsheet->getActiveSheet();
+        $originalArray = $sheet->toArray(null, false, false, false);
+
+        $newSpreadsheet = $this->writeAndReload($spreadsheet, 'Xls');
+        $spreadsheet->disconnectWorksheets();
+        $newWorksheet = $newSpreadsheet->getActiveSheet();
+        $newArray = $newWorksheet->toArray(null, false, false, false);
+        self::assertSame($originalArray, $newArray);
+        $newSpreadsheet->disconnectWorksheets();
+    }
 }
diff --git a/tests/data/Reader/XLS/formulas.other.xls b/tests/data/Reader/XLS/formulas.other.xls
new file mode 100644
index 0000000000000000000000000000000000000000..38a71549b2770f066f1027e047254cfb94a0e3c4
GIT binary patch
literal 15360
zcmeHOU2GKB6+W}x#p@py+xQ=255@%iM==;eN?;dk608zH2^C0nWfrqw3*$xWMKv#F
z(>7HrwGx%9Qng5F-qNH^NNE~XN)veNL!&ems7=KKsVY_Cr9W!bMor>=-<g?R?_G~g
zD<L(}+||7^_nUL?IrrRif6krRU%y>@_qkVE-xDM3kuv#czCu<kz%AUjdHEW_^Z9vZ
zz3Up>0}8^&NCOp7bmWSN9PD_j{N@eZmWjM4A-NX5iB#kdh@)~|sznZeAvJ9uOii9V
zmz=gAbl9gdBrVEt1BSO7ca~fQ8j}{4tWML_n!c=buWPzn>SR_vBfmP4*|rm<1?7&1
z5=!??O*=J>A+_W+E%SRt2BcMf3Htd<muc!Emm=0)$;hN6WlA38({{N7c2QBjA-Pm;
z4Jpq_IjQ^y#G$BImZE+c3QGp%bt=`DizOAyP=usu87?BVOH%Y%mh4!POL$p|JLtWo
z>!42eEGY*m2A8FHbQuau;!CupM<!rRJMwj`H2T(-gRrnOG9g*m82j0RB`n=XB9;02
zOj~IWw@Cy&B%NKI>-*NP-!j&{Yjl&B8SQQlufwd4dEA5W{BuvHPJO<Zs8h;iZGmX>
zmg0JlXH^>^YNttM)aH^tEIqPSWQ?Np*q&r=c65^hqusk)ptHXda*b_DjDgE$>BbL5
z7nbX>7Z&QVi%P_$Pd;7D%xOU}KNW2O%XqxNbcgh!KMG8}@_D{qVP#z?c~|OTRU-eG
zxAp2IH*DgZRuuoc_h@OH19dTaKb*BLM*m61EwR}=H#qu?Ag6lxsSAgnRsEOZFZ<wM
z_rbsCgMZ%#f58X;kq_=C&v$(2U-ZHK>i466#iyPJEzi%DFL#;6{q=?q{;CiDnh$>6
z2Y1?A%GZ&nyhNTKYduj9|8{Ndm8vUM5_bBfB9EUw{bt9T9TIlru`q{dyZ!|N4c>U~
z_QCsn@Sa8SO4Pa7c(3yCy!|_Nh+UjRe@l+wwgY=BZP&_TywSsB1>6?NHJ9MVJhsZC
z&+EsrAM7zbz9(?o>eW;0;dwa?`^0l67~}eCZEa&!BlPc#$2t(H9>CYpztb<=%Pgk<
z)gHYef4zs_!uCt{<n`0w;RZeDlc&pUq;3e^u;)T`QX|ml)-4<CetYlucxpO3K(y9G
zw-T*0(QQOmo9H0XdK2AFw82Do5N#wnXv)xJqAEkPiK+}OCaN;DnyAXK#za+yHljOB
z8P=Mp%Fu42Dno~fstmS?stoH)6f#s}dxFuN?}chvU&w=RwR9Eo;8-mi3VHCVmWK*?
zaH*D!g*@ty$|iWqvM+gRE>wd}3P#>ga%%j-R5Cl6o}LS>!mbCSbZ8<qe%2mJO-;>(
z!qS^7o9DGJw?eP|xv*Y0=gM(ykWayYWdH2>spNDro6cOckEUj`bD;{^B4Lro(`hh^
zVRM3M=RhiXYI6FF4Fii|d$j;T<6^RHA%f<`u-#oig9cX0c5D}AWcvICu&BXC6-_ue
zIej*D%9XcT9%hpdr7mPMNtIu60y*nsCl!7uojI4B8hdIYm7<wf$RiZ;P&%8M3sqqg
zhB@}|g{;;dmR-3BN><9F<au;5I|ZUj*{!*BmbKSn%Z_R4STZxscCqCr)JJ;;p5JCp
zFsRXC_6;zy@VjY65k<D&i^&I29@uLHG>dq#Qmmd>f2=&#8rvFciZ#YYVjE*EF-Xw0
zw`*W<9vu{wJDBS@J&s2k5Xr^~9Qq7D;aux@%QZ#ba#e`4h7+)>mV1!#8laWxUsfYM
zC9Cwl7O9|Xqw>KI|N5IJP9BMmDc+-auclv8BnKKVY=EJ_CzYr1<Yt(>DAJde`YW0~
z$1ON0s0&M5j;w~e@xt}9phc}P0R~D+ClFpII^DVNgR&5;Rf;hal46YXCI;&-!6>mw
z$GOtPxc-_LOsxc?HcUFMxhBRwGBK`&CdQ67F_>}*MqQb7G(i)i22G4>n~8CqGBK`2
zCdOViG4{HNVWBR;;Q5tcSh!2DmPIfu%q4U@)mzB-7g(_vXj*@XtAQmfAxev3KAK=!
z<D#?~=Ia%0c2Qal(KbbGpk=U3+D+WGeUuNa07CEji<l&6KXK$nZV->;aoP}T!Z8u>
z6^g%sAx%4qYl#nafV))U<X9`^rQEb6)==CU+LG3rVCuDo*2KKi#<qmha@vza#q=Q;
zrA0CCw0yutX;Va-6HFg*QCb!As*l`W7o}YhwMm<M+(l_wL_3MfVHahuFyE!<QJ~ZU
zt;^8@^-H^oOXu*teec}<XPo+?Pa0R<)4t+zV3=V*b=i=AMmvkk?LXaG`$GRrg5?_X
z&uDKr|41MI_g7zCr{$XT8M)K$3VlF}i^G;M#h`{3rDAW;>L7!Qw<^var{%FlJC)-a
z#ea*X<0Y6$Tc{B-@Ye<|%~%`S+`NZ7SK`hurgVF-Z2gQTBMByMSc@6D45u9k7EV2Q
z_h&9#SouWGM4J$C;|3fX@_@H&n5Qrw_wdoYNF>BQDGnSPcb|FoZ|@(TsCn^QQR&@y
z{oO6Vt|4+{@p%G~BlZ9y^>YG|Ryd2u5pfxjW8f+x?eKX-a{40*wc|?bUU;aC?6UY1
zljE86O!{=z-ak8@no=2Zk&d@}_P=W-Siw72q!;(JrXU0w2s99AAkaXdfj|R+1_BKP
z8VEEHXduu)pn*UGjs}+U|3kOmx_x!?nwq&Q@c;YXe~JG8?+|%jM*sf^BG0XkBk~l8
zem~F0a)|WpzlKP^|1u&^kiLt^^MD^9@_gVr;==QQX5=4BPG;<pnPhU@9!h7<r!)E<
zMf7y&`IaiJA7sAQb}*)FJfm>l5=b>5vf+^f#XFYHoSos@iW%q32~ZFM4FnnpG!SSY
z&_JMpKm&mW0u2Nj2s99AAke@kTLbi!=^XQYIp5{;D;s|JK>wNV^y%yKD+m6^k3Kv7
zcYeV_zn?xnzeV9Ye?IrVO{1^R^8otu{N{t-ckua#h`a{j|3?E)N*cec*s@=yam~m@
z^w#}{Mv3UxFl-U+$c{3_$5@)mW6s4%Yld$h;<p*J0S`Y0?io3Y|G80)vmSikSNsbw
z&z~~fZ^3=;cbyS=N-oGb{F2)C5oqDG%0wiKe`ckD_=ip4&oRXDbHDs~2&IDumxv65
X!??<j(jNHl<k|T@w4Y~k9{>LU^GG4E

literal 0
HcmV?d00001