Merge pull request #6709 from mgazdzik/patchApiTesting

modified test case to allow overriding requests Collection and it's logic

tests/PHPUnit/Framework/TestCase/SystemTestCase.php

@@ -432,7 +432,7 @@ abstract class SystemTestCase extends PHPUnit_Framework_TestCase
             $this->changeLanguage($testConfig->language);
         }
 
-        $testRequests = new Collection($api, $testConfig, $api);
+        $testRequests = $this->getTestRequestsCollection($api, $testConfig, $api);
 
         foreach ($testRequests->getRequestUrls() as $apiId => $requestUrl) {
             $this->_testApiUrl($testName . $testConfig->testSuffix, $apiId, $requestUrl, $testConfig->compareAgainst, $testConfig->xmlFieldsToRemove, $params);
@@ -460,6 +460,11 @@ abstract class SystemTestCase extends PHPUnit_Framework_TestCase
         return count($this->comparisonFailures) == 0;
     }
 
+    protected function getTestRequestsCollection($api, $testConfig, $api)
+    {
+       return new Collection($api, $testConfig, $api);
+    }
+
     private function printComparisonFailures()
     {
         $messages = '';

tests/PHPUnit/Framework/TestRequest/Collection.php

@@ -288,7 +288,7 @@ class Collection
         return $result;
     }
 
-    private function shouldSkipApiMethod($moduleName, $methodName) {
+    protected function shouldSkipApiMethod($moduleName, $methodName) {
         $apiId = $moduleName . '.' . $methodName;
 
         // If Api to test were set, we only test these