From f0f5edd38cac5c092ce0baba3c20c1687483cc6a Mon Sep 17 00:00:00 2001 From: Benaka Moorthi Date: Thu, 12 Sep 2013 14:53:48 -0400 Subject: [PATCH] Refs #4063, test for this fix. --- tests/PHPUnit/Integration/CsvExportTest.php | 16 +++++++++------- ...csvExport__Live.getLastVisitsDetails_day.csv | Bin 0 -> 7880 bytes 2 files changed, 9 insertions(+), 7 deletions(-) create mode 100644 tests/PHPUnit/Integration/expected/test_csvExport__Live.getLastVisitsDetails_day.csv diff --git a/tests/PHPUnit/Integration/CsvExportTest.php b/tests/PHPUnit/Integration/CsvExportTest.php index a5a9dfd475f..cfc3a39bfbd 100755 --- a/tests/PHPUnit/Integration/CsvExportTest.php +++ b/tests/PHPUnit/Integration/CsvExportTest.php @@ -45,13 +45,15 @@ public function getApiForTesting() 'language' => 'de', 'testSuffix' => '_xp1_inner1_trans-de')), - // TODO: we cannot currently test the csv output of Live.getLastVisitsDetails. The API method includes - // random output. Normally we remove this when it's in XML output, but it's harder to do that w/ - // CSV output. For now, we use compareOutput => false to just check that no errors result. - array('Live.getLastVisitsDetails', array('idSite' => $idSite, - 'date' => $dateTime, - 'format' => 'csv', - 'compareOutput' => false)) + array('Live.getLastVisitsDetails', array('idSite' => $idSite, + 'date' => $dateTime, + 'format' => 'csv', + 'otherRequestParameters' => array( + 'hideColumns' => 'serverDate,actionDetails,serverTimestamp,serverTimePretty,' + . 'serverDatePretty,serverDatePrettyFirstAction,serverTimePrettyFirstAction,' + . 'goalTimePretty,serverTimePretty,visitorId,visitServerHour,date,' + . 'prettyDate,serverDateTimePrettyFirstAction' + ))) ); } diff --git a/tests/PHPUnit/Integration/expected/test_csvExport__Live.getLastVisitsDetails_day.csv b/tests/PHPUnit/Integration/expected/test_csvExport__Live.getLastVisitsDetails_day.csv new file mode 100644 index 0000000000000000000000000000000000000000..acd2ac66fb2c9c2354d7de3ca008a683e83b14de GIT binary patch literal 7880 zcmeI1ZEqVz5Xa{=pTa7<0Oe}eaT*{JLWnptBG9N#Qs7OL*iPccPuNbK@M-v@!0*57 z_1>Ky;2#J9Lo5-cP>|tUP7IcwF#3V)oKs{kr2lqTj(n!fnxBTHE2DTiMQ+;`|J@^ez7jyd=MkKC5xtUQx5{ z6*PN{SJX`0uCbOhO|Q8;E7;U~S(EWQ>^WdQU2OOO<)V=qqaCc?gKK)7kAT#jtP}Xl z&htlKfhSpsU4b`1&d9!lGOTo<7F_EkpKtTEF5W)0+F5~bTC07hiN?J}mW>@d>x{sa zb>eQ5W$&=s2xtRY4`^D0pX}JQqlxBcK)glgVI52@u^caOo842d(XN3!+12lO&8Z8- zYc>m$-lFjMeg*?KH{r(qBD@2)D%a4C0QC+ZB)Z^mO!GG!l8{F&2;fh=d0lqpj9tR%&+q z7(DM8X$z*cT%>&sIU!^P23$1WlD|7LCW%UXE8I z!pk-zi#A+N5jDJou7_=vEe3XgrX4(|?L2Go*gn`pyiOUT?|jYsYL+5ulZ7vm9_O3I zKXbZiO}!K^xe2tJ%GdH@MG-}bn8oE~L+F%S;trX|A!^lk*xG?UPiFf8Z%7V$!IK^h zW7;v27CwP%nVdXLl=E2LudJ9|D|qmS@`~BDVxPE5LR`!C(sr## zTeJ#~GOu>xn0JEJ)pngXDBgOAs*o7%v$dRZ={3J5@#D4ggoO9VRYt0iRAc!%sDKZJ zN~D0i8|P6sRsci0?2dJ9(7S|J0h$32WhS2!HKGct?&9th5=H;cZlUTpjr;aHG?I3Q zq_N$z)BZs7b*MgO8owr|#JhcF_Zd%_Ror7WcW7U*x*p=&UB+JW?AX`5y$5WAlZS3Y zS>}1}-fAmYyiB_XuIDdmdB_W_EbgJ_-&O3ci05nkrt2ldJ;h*OVW^H$)=%%Zp_Rx2 zK1M1^#cW<;eLK*~w}f@YN~`C;2l#FuzRK$Umz5;gDdV@!H^#KH{TqB`1eZOymw31E znidpA^ccWdGB5C|lG8!EM007=|8z2P?rf_%N%dh9?^JEIpXwD%`A*)fiX#1c3sx6O zpCbzPiybrmcKX(6fG)C_@~Y+_TyL4I&6KZ86P`sdTIj7xTbkY?Ik6w6-v#@e)$;SW zV(i|&U_1*c;15OT1M6y(%|FuafM3*2y<{q-fj`LNvar11mbuEq#uN?4P3=%iTo>;t6cwe{XD2~lo@tv|cGoS=J@5+L$TpC@F^<#|GT zKhOC$R!Ehxyh|sD5w^^FV>sc(ox902A9tqFIzKhjC}+%OnzDU~Pbq^e$~4L`+x8W0 z#o<-RQj))Gf4a)K%e#zUZ8DE?(kAoxGeoQ&9?ONB%;W2*eB6#%Cgy|)n~n3KT6@gi zc{w4^L@5ChGmrNBf0KC>nN~UD98zf|)ZSH2!UZa;L*B3Ws@hM~ZzuLE-)Gzs9uYUI zU@G1{HHManf~sz$lWKcktqZzC6kQfW%PxBA91vsZ4qVbPbY~MobJe>}AK`XNYa4sM zdiQq?+C8e5d;G4cNp(KfIajq#fyy*}lQ#cM>rX<$)Lol=J$nKfbN(2+`Y*~TQmN}- zgc4GsUf+@dzL`qj%1Pe>H=d}s+=+V2on3Su^3t=CRhflZ1&Ome{|&VMwofM^)ymoW zF0EnumQR@TB*Z%K|9ckFX*(b3R*jIWXCz4p=aS7?XkB~f=|cXQmyzP_neM7|Z`)>< o`@5+#BKsQ`>BH^6?xu8>pLcJy-VM}qZesLX=X|iCU4B3K59^Yn4*&oF literal 0 HcmV?d00001