From df0800f071608f71fd070b61bed7a7d6433cf4a6 Mon Sep 17 00:00:00 2001
From: mike cullerton <michaelc@cullerton.com>
Date: Tue, 16 Nov 2021 11:57:24 -0500
Subject: [PATCH] New test and xls file for failing test

---
 .../enum_options_with_search/sponsors.xls     | Bin 0 -> 9728 bytes
 tests/test_lookup_service.py                  |  18 ++++++++++++++++++
 2 files changed, 18 insertions(+)
 create mode 100644 tests/data/enum_options_with_search/sponsors.xls

diff --git a/tests/data/enum_options_with_search/sponsors.xls b/tests/data/enum_options_with_search/sponsors.xls
new file mode 100644
index 0000000000000000000000000000000000000000..7c6b1730a53b9dba4ba88bd6b26ff5ea056f899d
GIT binary patch
literal 9728
zcmeI1Yiv}<701tA@2=OzHoG>@HE~xaCA@+{{GlijVBXLGrZFzGrRA~M#tqnX4HTqA
zSxC|*0#vt&D6Ud9>6f-tiinz~QKcy6TckKG4bVJ9DkrL{O8sK0qBd2+_WwUKcXsdf
z8W4Oc%Gh`Aoij7PIdkScd;X89kr!W1e<UHdL@MN3p-SQ<au45ic=(p2{_A%Ej-nJE
zet+)%e=JayKtsl3av?cf$u?wre<Tq(8$AbK<YPR`HN4YfcrkoWo`<APS~bN<^_$hN
zgYU`jH01{>IU!L<JlUbLb6!Huq5IH2i=`gCdu8jAzkO*DzSys}Mw+565|gRoHOt-7
zEHh<3o|$r=Y?nFmplp-bkOpJ6A*Lz17^=@0FWWcdVd<3J^3Cf2mP)1EH5RY{WnZs!
z6K`}_CkbdbM)gv}^))Gl^~hYgU%c(E#&abz#vZ{~TBm3+>hvJ&=39*Un_=)~YfHwo
zqA}R4x9mL35^Qs}XeMD$Lx#3GjrWG0Y_6}ruj@c&)!}UC-prQ1+#~&aIy(;_U!kyQ
z^}6MpQ8aErJLL;ZUoeOp``0k@#L85GnnW(Wed+lHjmc+!oRE3*KY3{-WU-L!QOH<g
z1s>P}CNlHdY`iDnxet$TAcpDu_auBX4;@C)ExGaZCwJtjpZ)Wbwmr#T{}B4$^PBgn
z%QQTU<HIt2-SBy9j^nB+x&{6m)7i%BW|WWD!OC_m?Pg8UBN)ODjNgw-2S>241`8_=
zjq-hZsVpt+*S7F(nI{X&)hk@=*xH2)EY`7HQ41w4Z=wYfGQg&Ri+Rf=Cq1%L_CkgE
zG74ESw$59N8ELN^f`0(1vPhSajR+mYLs}G^V)`I$hP(h;l`AXmZASh*(kJ`y9uf9T
zj*E=QU-c8-P=<0UceMcdziv*>{#7phcydz9rN5Kq9_{R$Z|8Z&4*&5ZeEFPl<nzao
z&mBizRaVZ@RG%5aaw|9tsOO`p)bZHySTb6z{lTSQluAvA*Tj>ypOngLO5_!w1Ua?J
z8OyKDJR6BhGG9^PJC%>?yFMS)cSF8X-;x)7Ps+y%Kf-Rq%>FGcEpqkhRk?imvYa_{
zhO3M44<E<?x0`#-0U7wt0|-Ns&&$xz5MFu7p1CF;-h00kuAY#MA8(QFR6(x&Vzqp)
zeT)43^?@74%Wwhze1=inpbR2h7#x=2LA-|trKPJNEyKf*LmfVYc)CQox(4OcsZ)Xk
z(lQ7^3;yv8>H}E|@<OmJS*NQFE2G<Hc?#q55SKxTd+iv-ojNBhlz&LIH&J9X!a*zX
zQJwpvX3nRm^ts?o7<~z{qf9Et)PF+#C)IyS{qL*)wEEAee_Z`Hq#CPQUduV^QLkh8
z_T&vOg<buS*LL)E-M4x2mbWM?^0ub@gZk&y|C9Rfczj(@|DyVr)c>=``Qlym3*I!W
zVoM>2V6)PEr3Nu4v@R#*;&URc(sYF@TP>zP#3iQU`8O83PvulB1390G;;oPlX_w8?
zCL3fGws+exd3NA?xrS=XCs?M5r<99aA?sxsi0vSexD32?c-C-N*&$zrAcM_QMunTT
zkd2V7x6(Dplu?yd;oXLe>*Z@QSuioPK{JTv29y(hkM!cZAF>rXVeiM3-77~h*?a5o
zu^THuKh}gCL@0Wh?82W(o0I*>kwf|e%H*cDAJ0z6vM6T((yd3_ZhWDVTur_U(JD~8
zvDUDd-3Xg(Ow96F)&gl1uN?*Fupm)KYHYF@Z95UeuWrbLT4PCBj^w?d?8W;C612l7
zwG*omtLdxMB;FcL>MwbWY=>GblWc}(;aLGxA0(YvHjR>5sED04i{7i{bR*KJNK3J1
zzS4&g+NVI8sYgK>bc;U7bEto{c&iaOh%CDy@{_>SD-hg;r~@ExK+Y`E_n{otD+?(@
zy(;eyD3+!PSkNH=dXSt|HYn3pY_lFfg4w1U%dz2TD`ulHSZ>(JG{btN$YMY0tw%{&
zdBjc>Y|stJmeZQ_qKj@sh?)?%3cgXE%}v|-t=y@a?$tgMY_JCL+7!qA1TDv^QA27<
zP1tMLATv#eWmGpZm|jVJOrNJ@KjIFmW?60BHAqL>?+<{I;;lp}Ijbt$Hr)92?4PCv
z4`~TTonFw`SGtj%_Qk}&-n|q0`3;+ql^WH9M7@H_QGM#nzP=NAve1WvBNx!gCZrx5
zTzy(luxh4AOf_EyfgRoqt<u6V0gTnhn5P%XOEt@21}-6$Mq!MV27N6^2-9JJg?eJ;
zsg<qGEg{F-9$1KDtge`#Tr_l8ST<jc$yY8<0VmajyqXZ<T%M>F`IaYg7RRJGj#bMs
zk)Ot@<vT=+xphbeZXIf|9(W)`Ff1qPcM3jFISKjnS6ANIuxr!O?JA$E@_FhXQ%N3a
zeD_7{HgF!rN*IxI(GK`kx*whlW6TzB7G{@Om`kp={hqI8?RNd5895f>Fg{Bja<-*?
zXUQgcLe+Ru{TIQ--UP+-kIEapUfnUWlmWgyc}&eD^7V$vnucMf&)-uFM;MbXhAVK5
zB2~q3jGG}FmWb{haCt!&!;^?`0p)iJ4Iz5Ur;LHHj9P|eJPe0rY{9UMWrbzDKnlw$
z%gESZ;dE7HWb7{CbUc8EW$b!k8G7Cr8Eq9#hfUTP87&x2$4zHg#!IZQtf7pIN3w9b
z$z^0y%E+(<8k3is!7y)n85u8O!s!~z$ar2TP4^iVN;>KzVnQBw6)qG(h`q;)u+1xY
zX{TRNZ0{=2Cz+BO!L|ONl?3+kw7F_QN{$e$f%@g=_R!BwZjx>8wI(;G2ONtbCyOH_
z4UJ9t5m$c15!j0@VUr>7@RG(op&?WR2o;XNv1W8&S(dQX5ID7@aZzXp(EuUp2vJ9%
zewMJq5MoMT?`JJ|byciI%n@RaKy58yk0H=<X-sN{5DyUIju3YQYHJDm4WUvAY+ve7
z86Z?TLZu^6TT3`>2-vGa2aZD$5=!6*WNRiIA>jzr))MlDP^AQpP!g&FI#fAAl_OAF
zOL*E4*q_q+g9Hie5k*T@J3_T1P+LnFFa#`i$ivZ3!h`@}f+I|D1lG_JP8$Me?zCRS
zk-){Ps6&k-)Hnj`YY8tH0=5sx!<mHyE`miuts~Sr0_QJF_@yCmq^0#TodjON7YRIS
znsHg@2&}IqoHYb&8jy#x771K@i-d`eFwqe>H(J7LhQPHYt;;M4$p9hg2uVj^eJx?g
z5K>Cuj7S2P{-O>kM@TsW=Uq#9(-5#lL>|spB-958^^Q>Q2&}IqoHqpQ`9R=oOF~0{
z(BKFSj=(wH65cZeUa+Mxdl<r`0AZ3NOmYO)*Ahky0Xs$zD5DNs@r!nt><E(`f%Ub7
zzZ(MQ=rsNefpc_`FvSt3I0EZy38RL99)mnwC8)#H0AZ>lOmzg-*AlEHrzwGJ3JKE!
z1kA0rPfl|L=C%ZDNvsUefqNPfrUwYq9bvj7u)daHEs0|p2u+rd4iLBsnQ_4t#mx4s
zuO(PZ;!q627c8MMKxlM?Mn_<MEx}q6r%DifJIn|WW;ntOB`k^5%PXAjrIns_cVRqo
zC4<ww9NxMx`&!>-`#-EF8y;PF+&8xlEUSl}b7q6dAeUvj9UWhpbMkR+53m)rj{1dv
z1JcC#C71s4@=v||Tf?y{AH1tKLO#gDT?vn<BEOwWUf3loejkvJ#yhlJKOLrh-dEu_
zD7{a^u6V5T%k|$(!J$j%x$BHJoc+ITgk^j`=Jd*+JXd)9ZkK}749jN|@$Di?KKye1
zH|@05#&3cBb2!ZN-k}d~-`OUYS1x{-E0nRz$bTQJp}qX_xmCH@@>#}fwx7oILSUc%
zZonHS9*(<IC;pP)pP}*V!L|45OWP;&{?J5*XWRO7qjR(6vyZQ(5^yn`fq9*AP>-qX
zfxMu&u|6g@f4ejGXW4YVy!7qc-G9@z1EmRW^V^hc{IV$9|HZz?)3a>en-Jbk`TP}@
zV~20{A+{sGxoPD4b9ZN3z;Anw3BUb0KH2`<_H*OT^@f{gZhyIfWxwDil5akr-PtC(
Hqigp+L5-ZR

literal 0
HcmV?d00001

diff --git a/tests/test_lookup_service.py b/tests/test_lookup_service.py
index dda8772c..15157e46 100644
--- a/tests/test_lookup_service.py
+++ b/tests/test_lookup_service.py
@@ -176,3 +176,21 @@ class TestLookupService(BaseTest):
         first_result = result[0]
         self.assertEquals(1000, first_result['CUSTOMER_NUMBER'])
         self.assertEquals('UVA - INTERNAL - GM USE ONLY', first_result['CUSTOMER_NAME'])
+
+    def test_lookup_fails_for_xls(self):
+        BaseTest.load_test_spec('enum_options_with_search')
+
+        # Using an old xls file should raise an error
+        file_model_xls = session.query(FileModel).filter(FileModel.name == 'sponsors.xls').first()
+        file_data_model_xls = session.query(FileDataModel).filter(FileDataModel.file_model_id == file_model_xls.id).first()
+        with self.assertRaises(ApiError) as ae:
+            LookupService.build_lookup_table(file_data_model_xls, 'CUSTOMER_NUMBER', 'CUSTOMER_NAME')
+        self.assertIn('Error opening excel file', ae.exception.args[0])
+
+        # Using an xlsx file should work
+        file_model_xlsx = session.query(FileModel).filter(FileModel.name == 'sponsors.xlsx').first()
+        file_data_model_xlsx = session.query(FileDataModel).filter(FileDataModel.file_model_id == file_model_xlsx.id).first()
+        lookup_model = LookupService.build_lookup_table(file_data_model_xlsx, 'CUSTOMER_NUMBER', 'CUSTOMER_NAME')
+        self.assertEqual(28, len(lookup_model.dependencies))
+        self.assertIn('CUSTOMER_NAME', lookup_model.dependencies[0].data.keys())
+        self.assertIn('CUSTOMER_NUMBER', lookup_model.dependencies[0].data.keys())