From f0f5edd38cac5c092ce0baba3c20c1687483cc6a Mon Sep 17 00:00:00 2001
From: Benaka Moorthi <benaka.moorthi@gmail.com>
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 a5a9dfd475..cfc3a39bfb 100755
--- a/tests/PHPUnit/Integration/CsvExportTest.php
+++ b/tests/PHPUnit/Integration/CsvExportTest.php
@@ -45,13 +45,15 @@ class Test_Piwik_Integration_CsvExport extends IntegrationTestCase
                                     '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;<!i@$+B+mc6Oe2c6MfV{r6ujYgo-%HnOHAyc>2#J9Lo5-cP>|t<QMu_iSJ%
z_5f6L^Wle9x1Mz~<KH{Td#DCZX5*33c|59)Z^t^0OP#g`uM3_#G*4@@95eDB!s7ys
z>UP7IcwF#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%a4<Yp#*z<4n4jrd52C<=WuMPomAWy6P!*c#VY9
zbvz@ikDQ*fdzfb)683k3&t!2=>C0QC+ZB)Z^mO!GG!l8{F&2;fh=d0lqpj9tR%&+q
z7(DM8X$z*cT%>&sIU!<JC-%~cKNUa5;GWARgUNbm1+?3>^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;v27<dHuJep{w#Wc#q>CwP%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<bD2k@4Qn{Y+a4g
z8B6*EnNg=_6W>#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

-- 
GitLab