Skip to content

Commit 555fb86

Browse files
mpociotStyleCIBot
authored andcommitted
Apply fixes from StyleCI
1 parent 90560e9 commit 555fb86

File tree

1 file changed

+12
-16
lines changed

1 file changed

+12
-16
lines changed

tests/Unit/GeneratorPluginSystemTestCase.php

Lines changed: 12 additions & 16 deletions
Original file line numberDiff line numberDiff line change
@@ -2,17 +2,14 @@
22

33
namespace Mpociot\ApiDoc\Tests\Unit;
44

5+
use ReflectionClass;
6+
use ReflectionMethod;
57
use Illuminate\Routing\Route;
6-
use Illuminate\Support\Arr;
7-
use Mpociot\ApiDoc\Strategies\Strategy;
8-
use Orchestra\Testbench\TestCase;
98
use Mpociot\ApiDoc\Tools\Generator;
9+
use Mpociot\ApiDoc\Strategies\Strategy;
1010
use Mpociot\ApiDoc\Tools\DocumentationConfig;
1111
use Mpociot\ApiDoc\Tests\Fixtures\TestController;
1212
use Mpociot\ApiDoc\ApiDocGeneratorServiceProvider;
13-
use Mpociot\ApiDoc\Tests\Fixtures\TestResourceController;
14-
use ReflectionClass;
15-
use ReflectionMethod;
1613

1714
class GeneratorPluginSystemTestCase extends LaravelGeneratorTest
1815
{
@@ -33,12 +30,12 @@ public function only_specified_strategies_are_loaded()
3330
{
3431
$config = [
3532
'strategies' => [
36-
'metadata' => [ EmptyStrategy1::class, ],
33+
'metadata' => [EmptyStrategy1::class],
3734
'bodyParameters' => [
3835
EmptyStrategy1::class,
3936
EmptyStrategy2::class,
4037
],
41-
'responses' => [ EmptyStrategy1::class ],
38+
'responses' => [EmptyStrategy1::class],
4239
],
4340
];
4441
$route = $this->createRoute('GET', '/api/test', 'dummy', true, TestController::class);
@@ -54,15 +51,14 @@ public function only_specified_strategies_are_loaded()
5451
$this->assertArrayNotHasKey('queryParameters', EmptyStrategy1::$called);
5552

5653
$this->assertTrue(EmptyStrategy1::$called['responses']);
57-
5854
}
5955

6056
/** @test */
6157
public function combines_responses_from_different_strategies()
6258
{
6359
$config = [
6460
'strategies' => [
65-
'responses' => [ DummyResponseStrategy200::class, DummyResponseStrategy400::class ],
61+
'responses' => [DummyResponseStrategy200::class, DummyResponseStrategy400::class],
6662
],
6763
];
6864
$route = $this->createRoute('GET', '/api/test', 'dummy', true, TestController::class);
@@ -82,6 +78,7 @@ public function combines_responses_from_different_strategies()
8278
}
8379

8480
// This is a generalized test, as opposed to the one above for responses only
81+
8582
/** @test */
8683
public function combines_results_from_different_strategies_in_same_stage()
8784
{
@@ -110,7 +107,7 @@ public function missing_metadata_is_filled_in()
110107
{
111108
$config = [
112109
'strategies' => [
113-
'metadata' => [ PartialDummyMetadataStrategy2::class, ],
110+
'metadata' => [PartialDummyMetadataStrategy2::class],
114111
],
115112
];
116113
$route = $this->createRoute('GET', '/api/test', 'dummy', true, TestController::class);
@@ -133,7 +130,7 @@ public function overwrites_results_from_previous_strategies_in_same_stage()
133130
{
134131
$config = [
135132
'strategies' => [
136-
'responses' => [ DummyResponseStrategy200::class, StillDummyResponseStrategyAlso200::class ],
133+
'responses' => [DummyResponseStrategy200::class, StillDummyResponseStrategyAlso200::class],
137134
],
138135
];
139136
$route = $this->createRoute('GET', '/api/test', 'dummy', true, TestController::class);
@@ -182,7 +179,6 @@ public function dataResources()
182179
}
183180
}
184181

185-
186182
class EmptyStrategy1 extends Strategy
187183
{
188184
public static $called = [];
@@ -245,22 +241,22 @@ class DummyResponseStrategy200 extends Strategy
245241
{
246242
public function __invoke(Route $route, ReflectionClass $controller, ReflectionMethod $method, array $routeRules, array $context = [])
247243
{
248-
return [ 200 => 'dummy', ];
244+
return [200 => 'dummy'];
249245
}
250246
}
251247

252248
class StillDummyResponseStrategyAlso200 extends Strategy
253249
{
254250
public function __invoke(Route $route, ReflectionClass $controller, ReflectionMethod $method, array $routeRules, array $context = [])
255251
{
256-
return [ 200 => 'stilldummy', ];
252+
return [200 => 'stilldummy'];
257253
}
258254
}
259255

260256
class DummyResponseStrategy400 extends Strategy
261257
{
262258
public function __invoke(Route $route, ReflectionClass $controller, ReflectionMethod $method, array $routeRules, array $context = [])
263259
{
264-
return [ 400 => 'dummy2', ];
260+
return [400 => 'dummy2'];
265261
}
266262
}

0 commit comments

Comments
 (0)