Skip to content

Commit f8a6059

Browse files
lukash82weierophinney
authored andcommitted
Manual CS fixes
Signed-off-by: Alex Pindyurin <[email protected]>
1 parent 4afd9bb commit f8a6059

File tree

4 files changed

+10
-7
lines changed

4 files changed

+10
-7
lines changed

src/ApiProblem.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -298,7 +298,7 @@ protected function getTitle()
298298
return $this->problemStatusTitles[$this->status];
299299
}
300300

301-
if ($this->detail instanceof Throwable || $this->detail instanceof Exception) {
301+
if ($this->detail instanceof Throwable) {
302302
return get_class($this->detail);
303303
}
304304

test/ApiProblemTest.php

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -28,6 +28,7 @@ public function statusCodes(): array
2828

2929
/**
3030
* @dataProvider statusCodes
31+
* @param int $status
3132
*/
3233
public function testStatusIsUsedVerbatim(int $status)
3334
{
@@ -132,6 +133,7 @@ public function knownStatusCodes(): array
132133

133134
/**
134135
* @dataProvider knownStatusCodes
136+
* @param int $status
135137
*/
136138
public function testKnownStatusResultsInKnownTitle(int $status)
137139
{
@@ -247,6 +249,7 @@ public function invalidStatusCodes(): array
247249
/**
248250
* @dataProvider invalidStatusCodes
249251
* @group api-tools-118
252+
* @param int $code
250253
*/
251254
public function testInvalidHttpStatusCodesAreCastTo500(int $code)
252255
{

test/ModuleTest.php

Lines changed: 3 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -10,14 +10,13 @@
1010
use Laminas\Mvc\MvcEvent;
1111
use Laminas\Mvc\SendResponseListener;
1212
use Laminas\ServiceManager\ServiceLocatorInterface;
13+
use PHPUnit\Framework\MockObject\MockObject;
1314
use PHPUnit\Framework\TestCase;
1415
use ReflectionClass;
1516

1617
class ModuleTest extends TestCase
1718
{
18-
/**
19-
* @return EventManager
20-
*/
19+
/** @return EventManager */
2120
public function marshalEventManager()
2221
{
2322
$r = new ReflectionClass(EventManager::class);
@@ -49,9 +48,7 @@ public function testOnBootstrap()
4948
$module->onBootstrap($event);
5049
}
5150

52-
/**
53-
* @return null|ApiProblemListener|SendResponseListener|SendApiProblemResponseListener
54-
*/
51+
/** @return null|ApiProblemListener|SendResponseListener|SendApiProblemResponseListener */
5552
public function serviceLocator(string $service)
5653
{
5754
switch ($service) {

test/View/ApiProblemStrategyTest.php

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -8,6 +8,7 @@
88
use Laminas\ApiTools\ApiProblem\View\ApiProblemStrategy;
99
use Laminas\Http\Response;
1010
use Laminas\View\Model\JsonModel;
11+
use Laminas\View\Model\ModelInterface as Model;
1112
use Laminas\View\Model\ViewModel;
1213
use Laminas\View\ViewEvent;
1314
use PHPUnit\Framework\TestCase;
@@ -36,6 +37,7 @@ public function invalidViewModels()
3637

3738
/**
3839
* @dataProvider invalidViewModels
40+
* @param null|Model $model
3941
*/
4042
public function testSelectRendererReturnsNullIfModelIsNotAnApiProblemModel(?ViewModel $model)
4143
{
@@ -89,6 +91,7 @@ public function invalidStatusCodes(): array
8991

9092
/**
9193
* @dataProvider invalidStatusCodes
94+
* @param int $status
9295
*/
9396
public function testUsesStatusCode500ForAnyStatusCodesAbove599OrBelow100(int $status)
9497
{

0 commit comments

Comments
 (0)