Separate "Valid" and "Invalid" tests

Author: pan93412

tests/Unit/ConfigurationTest.php

@@ -26,26 +26,30 @@ public function testValidConfig(array $inputConfig, array $expectedConfig): void
     /**
      * @param mixed $value
      *
-     * @dataProvider dataTestSettingsDynamicChecker
+     * @dataProvider dataTestSettingsDynamicCheckerInvalid
      */
-    public function testSettingsDynamicChecker($value, bool $passed): void
+    public function testSettingsDynamicCheckerInvalid($value): void
     {
-        if (!$passed) {
-            $this->assertConfigurationIsInvalid([
-                'meilisearch' => [
-                    'indices' => [
-                        [
-                            'name' => 'items',
-                            'class' => 'App\Entity\Post',
-                            'settings' => $value,
-                        ],
+        $this->assertConfigurationIsInvalid([
+            'meilisearch' => [
+                'indices' => [
+                    [
+                        'name' => 'items',
+                        'class' => 'App\Entity\Post',
+                        'settings' => $value,
                     ],
                 ],
-            ], 'Settings must be an array.');
-
-            return;
-        }
+            ],
+        ], 'Settings must be an array.');
+    }
 
+    /**
+     * @param mixed $value
+     *
+     * @dataProvider dataTestSettingsDynamicCheckerValid
+     */
+    public function testSettingsDynamicCheckerValid($value): void
+    {
         $this->assertConfigurationIsValid([
             'meilisearch' => [
                 'indices' => [
@@ -330,32 +334,21 @@ public static function dataTestConfigurationTree(): iterable
         ];
     }
 
-    public static function dataTestSettingsDynamicChecker(): iterable
+    public static function dataTestSettingsDynamicCheckerInvalid(): iterable
     {
-        yield 'string is not acceptable' => [
-            'settings' => 'hello',
-            'passed' => false,
-        ];
-
-        yield 'int is not acceptable' => [
-            'settings' => 1,
-            'passed' => false,
-        ];
-
-        yield 'bool is not acceptable' => [
-            'settings' => true,
-            'passed' => false,
-        ];
-
-        yield 'array is acceptable' => [
-            'settings' => [],
-            'passed' => true,
-        ];
+        yield 'string is not acceptable' => ['hello'];
+        yield 'int is not acceptable' => [1];
+        yield 'bool is not acceptable' => [true];
+    }
 
-        yield 'null is acceptable' => [
-            'settings' => null,
-            'passed' => true,
-        ];
+    public static function dataTestSettingsDynamicCheckerValid(): iterable
+    {
+        yield 'array is acceptable' => [[]];
+        yield 'array with arbitrary key is acceptable' => [[
+            'key' => 'value',
+            'key2' => 'value2',
+        ]];
+        yield 'null is acceptable' => [null];
     }
 
     protected function getConfiguration(): Configuration