浏览代码

Refactor tests

shalvah 5 年之前
父节点
当前提交
a81b4a86b9

+ 1 - 1
.gitignore

@@ -1,6 +1,6 @@
 .DS_Store
 composer.lock
-*.composer.lock
+composer.dingo.lock
 .php_cs.cache
 /vendor/
 public/

+ 8 - 8
phpunit.xml

@@ -10,24 +10,24 @@
          processIsolation="false"
          stopOnFailure="false">
     <testsuites>
+        <testsuite name="Full Test">
+            <file>tests/GenerateDocumentationTest.php</file>
+        </testsuite>
         <testsuite name="Generator Tests">
             <file>tests/Unit/DingoGeneratorTest.php</file>
             <file>tests/Unit/LaravelGeneratorTest.php</file>
             <file>tests/Unit/GeneratorPluginSystemTest.php</file>
         </testsuite>
+        <testsuite name="Strategies">
+            <directory>tests/Strategies</directory>
+        </testsuite>
         <testsuite name="RouteMatcher Tests">
             <file>tests/Unit/RouteMatcherDingoTest.php</file>
             <file>tests/Unit/RouteMatcherTest.php</file>
         </testsuite>
-        <testsuite name="Full Test">
-            <file>tests/GenerateDocumentationTest.php</file>
-        </testsuite>
-        <testsuite name="Postman Collection Test">
+        <testsuite name="Other Unit Tests">
             <file>tests/Unit/PostmanCollectionWriterTest.php</file>
-            <file>tests/Tools/AnnotationParserTest.php</file>
-        </testsuite>
-        <testsuite name="Strategies">
-            <directory>tests/Extracting/Strategies</directory>
+            <file>tests/Unit/AnnotationParserTest.php</file>
         </testsuite>
     </testsuites>
     <filter>

+ 1 - 1
tests/Extracting/Strategies/BodyParameters/GetFromBodyParamTagTest.php → tests/Strategies/BodyParameters/GetFromBodyParamTagTest.php

@@ -1,6 +1,6 @@
 <?php
 
-namespace Knuckles\Scribe\Tests\Extracting\Strategies\BodyParameters;
+namespace Knuckles\Scribe\Tests\Strategies\BodyParameters;
 
 use Illuminate\Routing\Route;
 use Knuckles\Scribe\Extracting\Strategies\BodyParameters\GetFromBodyParamTag;

+ 1 - 1
tests/Extracting/Strategies/BodyParameters/GetFromFormRequestTest.php → tests/Strategies/BodyParameters/GetFromFormRequestTest.php

@@ -1,6 +1,6 @@
 <?php
 
-namespace Knuckles\Scribe\Tests\Extracting\Strategies\BodyParameters;
+namespace Knuckles\Scribe\Tests\Strategies\BodyParameters;
 
 use Illuminate\Support\Facades\Validator;
 use Illuminate\Validation\ValidationException;

+ 1 - 1
tests/Extracting/Strategies/Metadata/GetFromDocBlocksTest.php → tests/Strategies/Metadata/GetFromDocBlocksTest.php

@@ -1,6 +1,6 @@
 <?php
 
-namespace Knuckles\Scribe\Tests\Extracting\Strategies\Metadata;
+namespace Knuckles\Scribe\Tests\Strategies\Metadata;
 
 use Knuckles\Scribe\Extracting\Strategies\Metadata\GetFromDocBlocks;
 use Knuckles\Scribe\Tools\DocumentationConfig;

+ 1 - 1
tests/Extracting/Strategies/QueryParameters/GetFromQueryParamTagTest.php → tests/Strategies/QueryParameters/GetFromQueryParamTagTest.php

@@ -1,6 +1,6 @@
 <?php
 
-namespace Knuckles\Scribe\Tests\Extracting\Strategies\QueryParameters;
+namespace Knuckles\Scribe\Tests\Strategies\QueryParameters;
 
 use Illuminate\Routing\Route;
 use Knuckles\Scribe\Extracting\Strategies\QueryParameters\GetFromQueryParamTag;

+ 1 - 1
tests/Extracting/Strategies/ResponseFields/GetFromResponseFieldTagTest.php → tests/Strategies/ResponseFields/GetFromResponseFieldTagTest.php

@@ -1,6 +1,6 @@
 <?php
 
-namespace Knuckles\Scribe\Tests\Extracting\Strategies\ResponseFields;
+namespace Knuckles\Scribe\Tests\Strategies\ResponseFields;
 
 use Knuckles\Scribe\Extracting\Strategies\ResponseFields\GetFromResponseFieldTag;
 use Knuckles\Scribe\Tools\DocumentationConfig;

+ 1 - 3
tests/Extracting/Strategies/Responses/ResponseCallsTest.php → tests/Strategies/Responses/ResponseCallsTest.php

@@ -1,6 +1,6 @@
 <?php
 
-namespace Knuckles\Scribe\Tests\Extracting\Strategies\Responses;
+namespace Knuckles\Scribe\Tests\Strategies\Responses;
 
 use Dingo\Api\Routing\Router;
 use Illuminate\Routing\Route;
@@ -9,10 +9,8 @@ use Knuckles\Scribe\ScribeServiceProvider;
 use Knuckles\Scribe\Tests\Fixtures\TestController;
 use Knuckles\Scribe\Tools\DocumentationConfig;
 use DMS\PHPUnitExtensions\ArraySubset\ArraySubsetAsserts;
-use Knuckles\Scribe\Tools\Flags;
 use Orchestra\Testbench\TestCase;
 use Illuminate\Support\Facades\Route as LaravelRouteFacade;
-use Dingo\Api\Routing\Router as DingoRouter;
 
 class ResponseCallsTest extends TestCase
 {

+ 1 - 3
tests/Extracting/Strategies/Responses/UseApiResourceTagsTest.php → tests/Strategies/Responses/UseApiResourceTagsTest.php

@@ -1,10 +1,8 @@
 <?php
 
-namespace Knuckles\Scribe\Tests\Extracting\Strategies\Responses;
+namespace Knuckles\Scribe\Tests\Strategies\Responses;
 
-use Knuckles\Scribe\Extracting\Generator;
 use Knuckles\Scribe\Extracting\Strategies\Responses\UseApiResourceTags;
-use Knuckles\Scribe\Extracting\Strategies\Responses\UseTransformerTags;
 use Knuckles\Scribe\ScribeServiceProvider;
 use Knuckles\Scribe\Tests\Fixtures\TestUser;
 use Knuckles\Scribe\Tools\DocumentationConfig;

+ 4 - 4
tests/Extracting/Strategies/Responses/UseResponseFileTagTest.php → tests/Strategies/Responses/UseResponseFileTagTest.php

@@ -1,6 +1,6 @@
 <?php
 
-namespace Knuckles\Scribe\Tests\Extracting\Strategies\Responses;
+namespace Knuckles\Scribe\Tests\Strategies\Responses;
 
 use Knuckles\Scribe\Extracting\Strategies\Responses\UseResponseFileTag;
 use Knuckles\Scribe\ScribeServiceProvider;
@@ -30,8 +30,8 @@ class UseResponseFileTagTest extends TestCase
      */
     public function allows_multiple_responsefile_tags_for_multiple_statuses_and_scenarios(array $tags, array $expected)
     {
-        $filePath =  __DIR__ . '/../../../Fixtures/response_test.json';
-        $filePath2 =  __DIR__ . '/../../../Fixtures/response_error_test.json';
+        $filePath =  __DIR__ . '/../../Fixtures/response_test.json';
+        $filePath2 =  __DIR__ . '/../../Fixtures/response_error_test.json';
 
         copy($filePath, storage_path('response_test.json'));
         copy($filePath2, storage_path('response_error_test.json'));
@@ -60,7 +60,7 @@ class UseResponseFileTagTest extends TestCase
     public function can_add_or_replace_key_value_pair_in_response_file()
     {
 
-        $filePath = __DIR__ . '/../../../Fixtures/response_test.json';
+        $filePath = __DIR__ . '/../../Fixtures/response_test.json';
         copy($filePath, storage_path('response_test.json'));
 
         $strategy = new UseResponseFileTag(new DocumentationConfig([]));

+ 1 - 1
tests/Extracting/Strategies/Responses/UseResponseTagTest.php → tests/Strategies/Responses/UseResponseTagTest.php

@@ -1,6 +1,6 @@
 <?php
 
-namespace Knuckles\Scribe\Tests\Extracting\Strategies\Responses;
+namespace Knuckles\Scribe\Tests\Strategies\Responses;
 
 use Knuckles\Scribe\Extracting\Strategies\Responses\UseResponseTag;
 use Knuckles\Scribe\Tools\DocumentationConfig;

+ 1 - 2
tests/Extracting/Strategies/Responses/UseTransformerTagsTest.php → tests/Strategies/Responses/UseTransformerTagsTest.php

@@ -1,10 +1,9 @@
 <?php
 
-namespace Knuckles\Scribe\Tests\Extracting\Strategies\Responses;
+namespace Knuckles\Scribe\Tests\Strategies\Responses;
 
 use Knuckles\Scribe\Extracting\Strategies\Responses\UseTransformerTags;
 use Knuckles\Scribe\ScribeServiceProvider;
-use Knuckles\Scribe\Tests\Fixtures\TestModel;
 use Knuckles\Scribe\Tests\Fixtures\TestUser;
 use Knuckles\Scribe\Tools\DocumentationConfig;
 use Mpociot\Reflection\DocBlock\Tag;

+ 1 - 1
tests/Extracting/Strategies/UrlParameters/GetFromUrlParamTagTest.php → tests/Strategies/UrlParameters/GetFromUrlParamTagTest.php

@@ -1,6 +1,6 @@
 <?php
 
-namespace Knuckles\Scribe\Tests\Extracting\Strategies\UrlParameters;
+namespace Knuckles\Scribe\Tests\Strategies\UrlParameters;
 
 use Knuckles\Scribe\Extracting\Strategies\UrlParameters\GetFromUrlParamTag;
 use Knuckles\Scribe\Tools\DocumentationConfig;

+ 2 - 1
tests/Tools/AnnotationParserTest.php → tests/Unit/AnnotationParserTest.php

@@ -1,7 +1,8 @@
 <?php
 
-namespace Knuckles\Scribe\Tools;
+namespace Knuckles\Scribe\Tests\Unit;
 
+use Knuckles\Scribe\Tools\AnnotationParser;
 use PHPUnit\Framework\TestCase;
 
 class AnnotationParserTest extends TestCase

+ 2 - 3
tests/Unit/GeneratorPluginSystemTestCase.php

@@ -4,13 +4,11 @@ namespace Knuckles\Scribe\Tests\Unit;
 
 use DMS\PHPUnitExtensions\ArraySubset\ArraySubsetAsserts;
 use Illuminate\Routing\Route;
-use Knuckles\Scribe\ScribeServiceProvider;
 use Knuckles\Scribe\Extracting\Generator;
 use Knuckles\Scribe\Extracting\Strategies\Strategy;
+use Knuckles\Scribe\ScribeServiceProvider;
 use Knuckles\Scribe\Tests\Fixtures\TestController;
 use Knuckles\Scribe\Tools\DocumentationConfig;
-use ReflectionClass;
-use ReflectionMethod;
 
 class GeneratorPluginSystemTestCase extends LaravelGeneratorTest
 {
@@ -167,6 +165,7 @@ class GeneratorPluginSystemTestCase extends LaravelGeneratorTest
     }
 }
 
+
 class EmptyStrategy1 extends Strategy
 {
     public static $called = [];