More tests

Author: sebastianbergmann

.php-cs-fixer.dist.php

@@ -24,6 +24,8 @@
     ->notName('TestAttributeOnHookMethodsTest.php')
     // InvokableConstraintAndPipeOperatorTest.php uses PHP 8.5 syntax
     ->notName('InvokableConstraintAndPipeOperatorTest.php')
+    // NoReturnTypeTest.php must have a method without a return type declaration
+    ->notPath('repeat/_files/NoReturnTypeTest.php')
     // FirstPartyClass.php and PhpDeprecationTest.php must not use declare(strict_types=1);
     ->notPath('error-handler/_files/php-deprecation/src/FirstPartyClass.php')
     ->notPath('error-handler/_files/php-deprecation/tests/PhpDeprecationTest.php')

phpunit.xml

@@ -35,6 +35,7 @@
             <directory suffix=".phpt">tests/end-to-end/migration</directory>
             <directory suffix=".phpt">tests/end-to-end/phpt</directory>
             <directory suffix=".phpt">tests/end-to-end/regression</directory>
+            <directory suffix=".phpt">tests/end-to-end/repeat</directory>
             <directory suffix=".phpt">tests/end-to-end/test-doubles/generator</directory>
             <directory suffix=".phpt">tests/end-to-end/test-doubles/runtime</directory>
             <directory suffix=".phpt">tests/end-to-end/testdox</directory>
@@ -44,6 +45,7 @@
             <exclude>tests/end-to-end/groups-from-configuration/_files</exclude>
             <exclude>tests/end-to-end/logging/_files</exclude>
             <exclude>tests/end-to-end/migration/_files</exclude>
+            <exclude>tests/end-to-end/repeat/_files</exclude>
             <exclude>tests/end-to-end/self-direct-indirect/_files</exclude>
             <exclude>tests/end-to-end/testdox/_files</exclude>
         </testsuite>

tests/_files/Metadata/Attribute/tests/RepeatTest.php

@@ -0,0 +1,21 @@
+<?php declare(strict_types=1);
+/*
+ * This file is part of PHPUnit.
+ *
+ * (c) Sebastian Bergmann <sebastian@phpunit.de>
+ *
+ * For the full copyright and license information, please view the LICENSE
+ * file that was distributed with this source code.
+ */
+namespace PHPUnit\TestFixture\Metadata\Attribute;
+
+use PHPUnit\Framework\Attributes\Repeat;
+use PHPUnit\Framework\TestCase;
+
+final class RepeatTest extends TestCase
+{
+    #[Repeat(5, failureThreshold: 2)]
+    public function testOne(): void
+    {
+    }
+}

tests/end-to-end/repeat/_files/NoReturnTypeTest.php

@@ -14,7 +14,7 @@
 final class NoReturnTypeTest extends TestCase
 {
     /** @phpstan-ignore missingType.return */
-    public function testWithoutReturnType(): void
+    public function testWithoutReturnType()
     {
         $this->assertTrue(true);
     }

tests/end-to-end/repeat/repeat-skips-no-return-type.phpt

@@ -1,5 +1,5 @@
 --TEST--
---repeat does not repeat test without return type declaration
+--repeat skips test without return type declaration
 --FILE--
 <?php declare(strict_types=1);
 $_SERVER['argv'][] = '--no-configuration';

tests/unit/Metadata/MetadataCollectionTest.php

@@ -41,6 +41,7 @@
 #[UsesClass(PostCondition::class)]
 #[UsesClass(PreCondition::class)]
 #[UsesClass(PreserveGlobalState::class)]
+#[UsesClass(Repeat::class)]
 #[UsesClass(RequiresFunction::class)]
 #[UsesClass(RequiresMethod::class)]
 #[UsesClass(RequiresOperatingSystem::class)]
@@ -571,6 +572,14 @@ public function test_Can_be_filtered_for_UsesMethod(): void
         $this->assertTrue($collection->asArray()[0]->isUsesMethod());
     }
 
+    public function test_Can_be_filtered_for_Repeat(): void
+    {
+        $collection = $this->collectionWithOneOfEach()->isRepeat();
+
+        $this->assertCount(1, $collection);
+        $this->assertTrue($collection->asArray()[0]->isRepeat());
+    }
+
     public function test_Can_be_filtered_for_WithoutErrorHandler(): void
     {
         $collection = $this->collectionWithOneOfEach()->isWithoutErrorHandler();
@@ -650,6 +659,7 @@ private function collectionWithOneOfEach(): MetadataCollection
                 Metadata::usesTrait(''),
                 Metadata::usesFunction(''),
                 Metadata::usesMethod('', ''),
+                Metadata::repeat(3, 1),
                 Metadata::withEnvironmentVariableOnClass('foo', 'bar'),
                 Metadata::withoutErrorHandler(),
             ],

tests/unit/Metadata/MetadataTest.php

@@ -15,6 +15,7 @@
 use PHPUnit\Metadata\DependsOnClass;
 use PHPUnit\Metadata\DependsOnMethod;
 use PHPUnit\Metadata\InvalidAttributeException;
+use PHPUnit\Metadata\Repeat;
 use PHPUnit\Metadata\RequiresEnvironmentVariable;
 use PHPUnit\Metadata\RequiresPhp;
 use PHPUnit\Metadata\RequiresPhpExtension;
@@ -52,6 +53,7 @@
 use PHPUnit\TestFixture\Metadata\Attribute\PhpunitAttributeThatDoesNotExistTest;
 use PHPUnit\TestFixture\Metadata\Attribute\PreserveGlobalStateTest;
 use PHPUnit\TestFixture\Metadata\Attribute\ProcessIsolationTest;
+use PHPUnit\TestFixture\Metadata\Attribute\RepeatTest;
 use PHPUnit\TestFixture\Metadata\Attribute\RequiresEnvironmentVariableTest;
 use PHPUnit\TestFixture\Metadata\Attribute\RequiresFunctionTest;
 use PHPUnit\TestFixture\Metadata\Attribute\RequiresMethodTest;
@@ -1201,6 +1203,22 @@ public function test_parses_Ticket_attribute_on_method(): void
         $this->assertSame('another-ticket', $metadata->asArray()[1]->groupName());
     }
 
+    #[TestDox('Parses #[Repeat] attribute on method')]
+    public function test_parses_Repeat_attribute_on_method(): void
+    {
+        $metadata = $this->parser()->forMethod(RepeatTest::class, 'testOne')->isRepeat();
+
+        $this->assertCount(1, $metadata);
+        $this->assertTrue($metadata->asArray()[0]->isRepeat());
+
+        $repeat = $metadata->asArray()[0];
+
+        assert($repeat instanceof Repeat);
+
+        $this->assertSame(5, $repeat->times());
+        $this->assertSame(2, $repeat->failureThreshold());
+    }
+
     #[TestDox('Parses #[WithoutErrorHandler] attribute on method')]
     public function test_parses_WithoutErrorHandler_attribute_on_method(): void
     {