When creating HTML reports in tmp/assets/ include a random string in the filename (#13607)

* When creating HTML reports in tmp/assets/ include a random string in the filename

* allow saveOnDisk option only for sendReport

* fix test and update changelog

1 files changed, 6 insertions, 11 deletions

diff --git a/plugins/MobileMessaging/tests/Integration/MobileMessagingTest.php b/plugins/MobileMessaging/tests/Integration/MobileMessagingTest.php
index 64a5f80435..6979857c9d 100644
--- a/plugins/MobileMessaging/tests/Integration/MobileMessagingTest.php
+++ b/plugins/MobileMessaging/tests/Integration/MobileMessagingTest.php
@@ -61,17 +61,12 @@ class MobileMessagingTest extends IntegrationTestCase
             array("phoneNumbers" => array('33698896656'))
         );
 
-        list($outputFilename, $prettyDate, $websiteName, $additionalFiles) =
-            $APIScheduledReports->generateReport(
-                $reportId,
-                '01-01-2010',
-                'en',
-                2
-            );
-
-        $handle = fopen($outputFilename, "r");
-        $contents = fread($handle, filesize($outputFilename));
-        fclose($handle);
+        $contents = $APIScheduledReports->generateReport(
+            $reportId,
+            '01-01-2010',
+            'en',
+            APIScheduledReports::OUTPUT_RETURN
+        );
 
         $this->assertEquals(
             \Piwik\Piwik::translate('MobileMessaging_MultiSites_Must_Be_Activated'),