Completed
Pull Request — master (#1)
by Dmitry
02:51
created
src/Sync/TestRailSync.php 2 patches
Doc Comments   +8 added lines, -4 removed lines patch added patch discarded remove patch
@@ -27,6 +27,7 @@  discard block
 block discarded – undo
27 27
 
28 28
     /**
29 29
      * TestRailSync constructor.
30
+     * @param string $apiUrl
30 31
      */
31 32
     public function __construct($apiUrl, HttpMethodsClient $httpMethodsClient)
32 33
     {
@@ -103,7 +104,7 @@  discard block
 block discarded – undo
103 104
     }
104 105
 
105 106
     /**
106
-     * @param $normalizeTitle
107
+     * @param string $normalizeTitle
107 108
      *
108 109
      * @return \Closure
109 110
      */
@@ -114,6 +115,9 @@  discard block
 block discarded – undo
114 115
         };
115 116
     }
116 117
 
118
+    /**
119
+     * @param string $resourcePath
120
+     */
117 121
     protected function getResourceUrl($resourcePath)
118 122
     {
119 123
         return $this->getApiUrl().$resourcePath;
@@ -172,7 +176,7 @@  discard block
 block discarded – undo
172 176
     }
173 177
 
174 178
     /**
175
-     * @param $testRunId
179
+     * @param integer $testRunId
176 180
      *
177 181
      * @return array
178 182
      *
@@ -198,7 +202,7 @@  discard block
 block discarded – undo
198 202
 
199 203
     /**
200 204
      * @param TestRailRunReport $testRailReport
201
-     * @param $testRunId
205
+     * @param integer $testRunId
202 206
      *
203 207
      * @return ResponseInterface
204 208
      *
@@ -227,7 +231,7 @@  discard block
 block discarded – undo
227 231
 
228 232
     /**
229 233
      * @param ResponseInterface $request
230
-     * @param $apiUrlResource
234
+     * @param string $apiUrlResource
231 235
      *
232 236
      * @throws TestRailSyncServerException
233 237
      */
Please login to merge, or discard this patch.
Unused Use Statements   -1 removed lines patch added patch discarded remove patch
@@ -4,7 +4,6 @@
 block discarded – undo
4 4
 
5 5
 use Http\Client\Common\HttpMethodsClient;
6 6
 use Http\Client\Exception as HttpClientException;
7
-use Psr\Http\Message\RequestInterface;
8 7
 use Psr\Http\Message\ResponseInterface;
9 8
 use TonicForHealth\ReportAggregator\Entity\Result;
10 9
 use TonicForHealth\ReportAggregator\Entity\TestCase;
Please login to merge, or discard this patch.
tests/TestRailReportAggregatorTest.php 2 patches
Doc Comments   +5 added lines, -5 removed lines patch added patch discarded remove patch
@@ -114,10 +114,10 @@  discard block
 block discarded – undo
114 114
     }
115 115
 
116 116
     /**
117
-     * @param $responseBody
118
-     * @param $responseCode
117
+     * @param string $responseBody
118
+     * @param integer $responseCode
119 119
      *
120
-     * @return \PHPUnit_Framework_MockObject_MockObject|ResponseInterface
120
+     * @return ResponseInterface
121 121
      */
122 122
     protected function getMockResponse($responseBody, $responseCode)
123 123
     {
@@ -137,7 +137,7 @@  discard block
 block discarded – undo
137 137
     }
138 138
 
139 139
     /**
140
-     * @return HttpMethodsClient|PHPUnit_Framework_MockObject_MockObject
140
+     * @return HttpMethodsClient
141 141
      */
142 142
     protected function getHttpMethodsClient()
143 143
     {
@@ -145,7 +145,7 @@  discard block
 block discarded – undo
145 145
     }
146 146
 
147 147
     /**
148
-     * @param HttpMethodsClient|PHPUnit_Framework_MockObject_MockObject $httpMethodsClient
148
+     * @param HttpMethodsClient $httpMethodsClient
149 149
      */
150 150
     protected function setHttpMethodsClient(HttpMethodsClient $httpMethodsClient)
151 151
     {
Please login to merge, or discard this patch.
Unused Use Statements   -1 removed lines patch added patch discarded remove patch
@@ -12,7 +12,6 @@
 block discarded – undo
12 12
 use PHPUnit_Framework_TestCase;
13 13
 use Psr\Http\Message\ResponseInterface;
14 14
 use Psr\Http\Message\StreamInterface;
15
-use TonicForHealth\ReportAggregator\Report\JUnit\JUnitReport;
16 15
 use TonicForHealth\ReportAggregator\Sync\TestRailSync;
17 16
 use TonicForHealth\ReportAggregator\Transformer\TestRail\JUnitToTestRailRunTransformer;
18 17
 
Please login to merge, or discard this patch.