Fix TrackerTest.php (includes fix to RequestCommand due to cache use in...

Fix TrackerTest.php (includes fix to RequestCommand due to cache use in UrlHelper::getArrayFromQueryString).

core/CliMulti/RequestCommand.php

@@ -89,7 +89,7 @@ class RequestCommand extends ConsoleCommand
         Url::setHost($hostname);
 
         $query = $input->getArgument('url-query');
-        $query = UrlHelper::getArrayFromQueryString($query);
+        $query = UrlHelper::getArrayFromQueryString($query); // NOTE: this method can create the StaticContainer now
         foreach ($query as $name => $value) {
             $_GET[$name] = $value;
         }

core/Tracker/ScheduledTasksRunner.php

@@ -81,8 +81,7 @@ class ScheduledTasksRunner
             $tokens = CronArchive::getSuperUserTokenAuths();
             $tokenAuth = reset($tokens);
 
-            $invokeScheduledTasksUrl = SettingsPiwik::getPiwikUrl()
-                . "?module=API&format=csv&convertToUnicode=0&method=CoreAdminHome.runScheduledTasks&trigger=archivephp&token_auth=$tokenAuth";
+            $invokeScheduledTasksUrl = "?module=API&format=csv&convertToUnicode=0&method=CoreAdminHome.runScheduledTasks&trigger=archivephp&token_auth=$tokenAuth";
 
             $cliMulti = new CliMulti();
             $responses = $cliMulti->request(array($invokeScheduledTasksUrl));

core/UrlHelper.php

@@ -156,6 +156,9 @@ class UrlHelper
             return array();
         }
 
+        // TODO: this method should not use a cache. callers should instead have their own cache, configured through DI.
+        //       one undesirable side effect of using a cache here, is that this method can now init the StaticContainer, which makes setting
+        //       test environment for RequestCommand more complicated.
         $cache    = Cache::getTransientCache();
         $cacheKey = 'arrayFromQuery' . $urlQuery;