Merge pull request #8 from hirokinoue/dataprovider-as-generator

dataProviderをGeneratorで書き直す
hirokinoue · Feb 27, 2024 · 10a9fcd · 10a9fcd
2 parents 10cda81 + 42d09ec
commit 10a9fcd
Show file tree

Hide file tree

Showing 5 changed files with 151 additions and 180 deletions.
diff --git a/tests/ClassManipulator/ClassLikeWrapperTest.php b/tests/ClassManipulator/ClassLikeWrapperTest.php
@@ -34,29 +34,26 @@ public function testクラスの名前が取得できること(ClassLike $classL
 
     /**
      * @noinspection NonAsciiCharacters
-     * @return array<string, array<int, Class_|string>>
      */
-    public function dataクラスの名前が取得できること(): array
+    public function dataクラスの名前が取得できること(): \Generator
     {
         $class1 = new Class_(new Identifier('Bar'));
         $class1->namespacedName = new FullyQualified('Foo\Bar');
         $class2 = new Class_(null);
         $class2->namespacedName = null;
         $class3 =  new Class_(new Identifier('Bar'));
         $class3->namespacedName = null;
-        return [
-            '完全修飾名が取得できること' => [
-                $class1,
-                '\Foo\Bar',
-            ],
-            '名前が空であること' => [
-                $class2,
-                '',
-            ],
-            '完全修飾名ではない名前が取得できること' => [
-               $class3,
-                'Bar',
-            ],
+        yield '完全修飾名が取得できること' => [
+            $class1,
+            '\Foo\Bar',
+        ];
+        yield '名前が空であること' => [
+            $class2,
+            '',
+        ];
+        yield '完全修飾名ではない名前が取得できること' => [
+            $class3,
+            'Bar',
         ];
     }
 
@@ -78,31 +75,28 @@ public function test宣言する要素が取得できること(ClassLike $classL
 
     /**
      * @noinspection NonAsciiCharacters
-     * @return array<string, array<int, ClassLike|string>>
      */
-    public function data宣言する要素が取得できること(): array
+    public function data宣言する要素が取得できること(): \Generator
     {
-        return [
-            '具象クラス' => [
-                new Class_(''),
-                'class',
-            ],
-            '抽象クラス' => [
-                new Class_('', ['flags' => Modifiers::ABSTRACT]),
-                'abstract',
-            ],
-            'インタフェース' => [
-                new Interface_(''),
-                'interface',
-            ],
-            'トレイト' => [
-                new Trait_(''),
-                'trait',
-            ],
-            'enum' => [
-                new Enum_(''),
-                'enum',
-            ],
+        yield '具象クラス' => [
+            new Class_(''),
+            'class',
+        ];
+        yield '抽象クラス' => [
+            new Class_('', ['flags' => Modifiers::ABSTRACT]),
+            'abstract',
+        ];
+        yield 'インタフェース' => [
+            new Interface_(''),
+            'interface',
+        ];
+        yield 'トレイト' => [
+            new Trait_(''),
+            'trait',
+        ];
+        yield 'enum' => [
+            new Enum_(''),
+            'enum',
         ];
     }
 

diff --git a/tests/ClassManipulator/ClassLoaderTest.php b/tests/ClassManipulator/ClassLoaderTest.php
@@ -33,37 +33,34 @@ public function test対象に応じてロードできること(
 
     /**
      * @noinspection NonAsciiCharacters
-     * @return array<string, array<int, string|FullyQualified>>
      */
-    public function data対象に応じてロードできること(): array
+    public function data対象に応じてロードできること(): \Generator
     {
         $userDefinedClassCode = <<<CODE
 <?php
 namespace Hirokinoue\DependencyVisualizer\Tests\data;
 class UserDefinedClass{}
 
 CODE;
-        return [
-            'ユーザー定義クラスはクラス名とコードが取得できる' => [
-                new FullyQualified('Hirokinoue\DependencyVisualizer\Tests\data\UserDefinedClass'),
-                '\Hirokinoue\DependencyVisualizer\Tests\data\UserDefinedClass',
-                $userDefinedClassCode,
-            ],
-            '内部クラスはクラス名のみ取得できる' => [
-                new FullyQualified('Exception'),
-                '\Exception',
-                '',
-            ],
-            '定数はクラス名もコードも取得できない' => [
-                new FullyQualified('FOO'),
-                '',
-                '',
-            ],
-            'キーワードはクラス名もコードも取得できない' => [
-                new FullyQualified('true'),
-                '',
-                '',
-            ],
+        yield 'ユーザー定義クラスはクラス名とコードが取得できる' => [
+            new FullyQualified('Hirokinoue\DependencyVisualizer\Tests\data\UserDefinedClass'),
+            '\Hirokinoue\DependencyVisualizer\Tests\data\UserDefinedClass',
+            $userDefinedClassCode,
+        ];
+        yield '内部クラスはクラス名のみ取得できる' => [
+            new FullyQualified('Exception'),
+            '\Exception',
+            '',
+        ];
+        yield '定数はクラス名もコードも取得できない' => [
+            new FullyQualified('FOO'),
+            '',
+            '',
+        ];
+        yield 'キーワードはクラス名もコードも取得できない' => [
+            new FullyQualified('true'),
+            '',
+            '',
         ];
     }
 
@@ -90,9 +87,8 @@ public function testロードしたことのあるクラスを再びロードし
 
     /**
      * @noinspection NonAsciiCharacters
-     * @return array<string, array<int, FullyQualified|string|bool>>
      */
-    public function dataロードしたことのあるクラスを再びロードしないこと(): array
+    public function dataロードしたことのあるクラスを再びロードしないこと(): \Generator
     {
         $userDefinedClass = 'Hirokinoue\DependencyVisualizer\Tests\data\UserDefinedClass';
         $userDefinedClassCode = <<<CODE
@@ -101,17 +97,15 @@ public function dataロードしたことのあるクラスを再びロードし
 class UserDefinedClass{}
 
 CODE;
-        return [
-            '初回はロードする' => [
-                new FullyQualified($userDefinedClass),
-                '\\' . $userDefinedClass,
-                $userDefinedClassCode,
-            ],
-            '2回目以降はロードしない' => [
-                new FullyQualified($userDefinedClass),
-                '\\' . $userDefinedClass,
-                '',
-            ],
+        yield '初回はロードする' => [
+            new FullyQualified($userDefinedClass),
+            '\\' . $userDefinedClass,
+            $userDefinedClassCode,
+        ];
+        yield '2回目以降はロードしない' => [
+            new FullyQualified($userDefinedClass),
+            '\\' . $userDefinedClass,
+            '',
         ];
     }
 

diff --git a/tests/DependencyVisualizerTest.php b/tests/DependencyVisualizerTest.php
@@ -31,9 +31,8 @@ public function test分析結果をテキスト形式で出力できること(st
 
     /**
      * @noinspection NonAsciiCharacters
-     * @return array<string, array<int, string>>
      */
-    public function data分析結果をテキスト形式で出力できること(): array
+    public function data分析結果をテキスト形式で出力できること(): \Generator
     {
         $rootIsClass = <<<RESULT
 \Hirokinoue\DependencyVisualizer\Tests\data\Foo
@@ -64,23 +63,21 @@ public function data分析結果をテキスト形式で出力できること():
   \Hirokinoue\DependencyVisualizer\Tests\data\RedundantDependency\C
 
 RESULT;
-        return [
-            '始点がクラスの時ルートがクラス名' => [
-                __DIR__ . '/data/Foo.php',
-                $rootIsClass,
-            ],
-            '始点がクラスではない時ルートがroot' => [
-                __DIR__ . '/data/foo.php',
-                $rootIsNotClass,
-            ],
-            '循環依存があっても無限ループせず解析が完了する' => [
-                __DIR__ . '/data/InfiniteLoop/A.php',
-                $infiniteLoop,
-            ],
-            '同じクラスを複数回使用する場合1つだけ図示する' => [
-                __DIR__ . '/data/RedundantDependency/A.php',
-                $performanceEnhancement,
-            ],
+        yield '始点がクラスの時ルートがクラス名' => [
+            __DIR__ . '/data/Foo.php',
+            $rootIsClass,
+        ];
+        yield '始点がクラスではない時ルートがroot' => [
+            __DIR__ . '/data/foo.php',
+            $rootIsNotClass,
+        ];
+        yield '循環依存があっても無限ループせず解析が完了する' => [
+            __DIR__ . '/data/InfiniteLoop/A.php',
+            $infiniteLoop,
+        ];
+        yield '同じクラスを複数回使用する場合1つだけ図示する' => [
+            __DIR__ . '/data/RedundantDependency/A.php',
+            $performanceEnhancement,
         ];
     }
 }