Minor @requires function tests cleanup

fancyweb · fancyweb · commit cf4daad6b7f8 · 2022-04-20T17:01:42.000+02:00
diff --git a/src/Symfony/Bridge/Doctrine/Tests/IdGenerator/UlidGeneratorTest.php b/src/Symfony/Bridge/Doctrine/Tests/IdGenerator/UlidGeneratorTest.php
@@ -29,9 +29,6 @@ public function testUlidCanBeGenerated()
         $this->assertTrue(Ulid::isValid($ulid));
     }
 
-    /**
-     * @requires function \Symfony\Component\Uid\Factory\UlidFactory::create
-     */
     public function testUlidFactory()
     {
         $ulid = new Ulid('00000000000000000000000000');
diff --git a/src/Symfony/Bridge/Doctrine/Tests/IdGenerator/UuidGeneratorTest.php b/src/Symfony/Bridge/Doctrine/Tests/IdGenerator/UuidGeneratorTest.php
@@ -18,9 +18,6 @@
 use Symfony\Component\Uid\UuidV4;
 use Symfony\Component\Uid\UuidV6;
 
-/**
- * @requires function \Symfony\Component\Uid\Factory\UuidFactory::create
- */
 class UuidGeneratorTest extends TestCase
 {
     public function testUuidCanBeGenerated()
diff --git a/src/Symfony/Bundle/FrameworkBundle/Tests/Routing/RedirectableCompiledUrlMatcherTest.php b/src/Symfony/Bundle/FrameworkBundle/Tests/Routing/RedirectableCompiledUrlMatcherTest.php
@@ -18,9 +18,6 @@
 use Symfony\Component\Routing\Route;
 use Symfony\Component\Routing\RouteCollection;
 
-/**
- * @requires function \Symfony\Component\Routing\Matcher\CompiledUrlMatcher::match
- */
 class RedirectableCompiledUrlMatcherTest extends TestCase
 {
     public function testRedirectWhenNoSlash()
diff --git a/src/Symfony/Component/Cache/Tests/Messenger/EarlyExpirationDispatcherTest.php b/src/Symfony/Component/Cache/Tests/Messenger/EarlyExpirationDispatcherTest.php
@@ -26,9 +26,6 @@
 use Symfony\Component\Messenger\Envelope;
 use Symfony\Component\Messenger\MessageBusInterface;
 
-/**
- * @requires function Symfony\Component\DependencyInjection\ReverseContainer::__construct
- */
 class EarlyExpirationDispatcherTest extends TestCase
 {
     public static function tearDownAfterClass(): void
diff --git a/src/Symfony/Component/Cache/Tests/Messenger/EarlyExpirationHandlerTest.php b/src/Symfony/Component/Cache/Tests/Messenger/EarlyExpirationHandlerTest.php
@@ -21,9 +21,6 @@
 use Symfony\Component\DependencyInjection\ServiceLocator;
 use Symfony\Component\Filesystem\Filesystem;
 
-/**
- * @requires function Symfony\Component\DependencyInjection\ReverseContainer::__construct
- */
 class EarlyExpirationHandlerTest extends TestCase
 {
     public static function tearDownAfterClass(): void
diff --git a/src/Symfony/Component/Cache/Tests/Messenger/EarlyExpirationMessageTest.php b/src/Symfony/Component/Cache/Tests/Messenger/EarlyExpirationMessageTest.php
@@ -19,9 +19,6 @@
 use Symfony\Component\DependencyInjection\ReverseContainer;
 use Symfony\Component\DependencyInjection\ServiceLocator;
 
-/**
- * @requires function Symfony\Component\DependencyInjection\ReverseContainer::__construct
- */
 class EarlyExpirationMessageTest extends TestCase
 {
     public function testCreate()
diff --git a/src/Symfony/Component/Console/Tests/Helper/QuestionHelperTest.php b/src/Symfony/Component/Console/Tests/Helper/QuestionHelperTest.php
@@ -714,9 +714,6 @@ public function testNoInteraction()
         $this->assertEquals('not yet', $dialog->ask($this->createStreamableInputInterfaceMock(null, false), $this->createOutputInterface(), $question));
     }
 
-    /**
-     * @requires function mb_strwidth
-     */
     public function testChoiceOutputFormattingQuestionForUtf8Keys()
     {
         $question = 'Lorem ipsum?';
diff --git a/src/Symfony/Component/Form/Tests/ChoiceList/Factory/DefaultChoiceListFactoryTest.php b/src/Symfony/Component/Form/Tests/ChoiceList/Factory/DefaultChoiceListFactoryTest.php
@@ -759,9 +759,6 @@ function ($object, $key, $value) {
         $this->assertFlatViewWithAttr($view);
     }
 
-    /**
-     * @requires function Symfony\Component\Translation\TranslatableMessage::__construct
-     */
     public function testPassTranslatableMessageAsLabelDoesntCastItToString()
     {
         $view = $this->factory->createView(
diff --git a/src/Symfony/Component/PropertyAccess/Tests/PropertyAccessorTest.php b/src/Symfony/Component/PropertyAccess/Tests/PropertyAccessorTest.php
@@ -907,9 +907,6 @@ public function testGetDynamicPublicPropertyWithMagicGetterAllow()
         $this->assertSame($value, $this->propertyAccessor->getValue($object, $path));
     }
 
-    /**
-     * @requires PHP 7.4
-     */
     public function testSetValueWrongTypeShouldThrowWrappedException()
     {
         $object = new TestClassTypedProperty();
diff --git a/src/Symfony/Component/VarDumper/Tests/Caster/ExceptionCasterTest.php b/src/Symfony/Component/VarDumper/Tests/Caster/ExceptionCasterTest.php
@@ -184,9 +184,6 @@ public function testHtmlDump()
         $this->assertStringMatchesFormat($expectedDump, $dump);
     }
 
-    /**
-     * @requires function Twig\Template::getSourceContext
-     */
     public function testFrameWithTwig()
     {
         require_once \dirname(__DIR__).'/Fixtures/Twig.php';
diff --git a/src/Symfony/Component/VarDumper/Tests/Dumper/CliDumperTest.php b/src/Symfony/Component/VarDumper/Tests/Dumper/CliDumperTest.php
@@ -293,9 +293,6 @@ public function testFlags()
         putenv('DUMP_STRING_LENGTH=');
     }
 
-    /**
-     * @requires function Twig\Template::getSourceContext
-     */
     public function testThrowingCaster()
     {
         $out = fopen('php://memory', 'r+');
diff --git a/src/Symfony/Component/Workflow/Tests/MarkingStore/MethodMarkingStoreTest.php b/src/Symfony/Component/Workflow/Tests/MarkingStore/MethodMarkingStoreTest.php
@@ -87,9 +87,6 @@ public function testGetMarkingWithValueObject()
         $this->assertSame('first_place', (string) $subject->getMarking());
     }
 
-    /**
-     * @requires PHP 7.4
-     */
     public function testGetMarkingWithUninitializedProperty()
     {
         $subject = new SubjectWithType();
@@ -102,9 +99,6 @@ public function testGetMarkingWithUninitializedProperty()
         $this->assertCount(0, $marking->getPlaces());
     }
 
-    /**
-     * @requires PHP 7.4
-     */
     public function testGetMarkingWithUninitializedProperty2()
     {
         $subject = new SubjectWithType();

Original file line number	Diff line number	Diff line change
`@@ -29,9 +29,6 @@ public function testUlidCanBeGenerated()`
`29`	`29`	`$this->assertTrue(Ulid::isValid($ulid));`
`30`	`30`	`}`
`31`	`31`
`32`		`- /**`
`33`		`- * @requires function \Symfony\Component\Uid\Factory\UlidFactory::create`
`34`		`- */`
`35`	`32`	`public function testUlidFactory()`
`36`	`33`	`{`
`37`	`34`	`$ulid = new Ulid('00000000000000000000000000');`
Original file line number	Diff line number	Diff line change
`@@ -18,9 +18,6 @@`
`18`	`18`	`use Symfony\Component\Uid\UuidV4;`
`19`	`19`	`use Symfony\Component\Uid\UuidV6;`
`20`	`20`
`21`		`-/**`
`22`		`- * @requires function \Symfony\Component\Uid\Factory\UuidFactory::create`
`23`		`- */`
`24`	`21`	`class UuidGeneratorTest extends TestCase`
`25`	`22`	`{`
`26`	`23`	`public function testUuidCanBeGenerated()`
Original file line number	Diff line number	Diff line change
`@@ -26,9 +26,6 @@`
`26`	`26`	`use Symfony\Component\Messenger\Envelope;`
`27`	`27`	`use Symfony\Component\Messenger\MessageBusInterface;`
`28`	`28`
`29`		`-/**`
`30`		`- * @requires function Symfony\Component\DependencyInjection\ReverseContainer::__construct`
`31`		`- */`
`32`	`29`	`class EarlyExpirationDispatcherTest extends TestCase`
`33`	`30`	`{`
`34`	`31`	`public static function tearDownAfterClass(): void`
Original file line number	Diff line number	Diff line change
`@@ -21,9 +21,6 @@`
`21`	`21`	`use Symfony\Component\DependencyInjection\ServiceLocator;`
`22`	`22`	`use Symfony\Component\Filesystem\Filesystem;`
`23`	`23`
`24`		`-/**`
`25`		`- * @requires function Symfony\Component\DependencyInjection\ReverseContainer::__construct`
`26`		`- */`
`27`	`24`	`class EarlyExpirationHandlerTest extends TestCase`
`28`	`25`	`{`
`29`	`26`	`public static function tearDownAfterClass(): void`
Original file line number	Diff line number	Diff line change
`@@ -19,9 +19,6 @@`
`19`	`19`	`use Symfony\Component\DependencyInjection\ReverseContainer;`
`20`	`20`	`use Symfony\Component\DependencyInjection\ServiceLocator;`
`21`	`21`
`22`		`-/**`
`23`		`- * @requires function Symfony\Component\DependencyInjection\ReverseContainer::__construct`
`24`		`- */`
`25`	`22`	`class EarlyExpirationMessageTest extends TestCase`
`26`	`23`	`{`
`27`	`24`	`public function testCreate()`
Original file line number	Diff line number	Diff line change
`@@ -714,9 +714,6 @@ public function testNoInteraction()`
`714`	`714`	`$this->assertEquals('not yet', $dialog->ask($this->createStreamableInputInterfaceMock(null, false), $this->createOutputInterface(), $question));`
`715`	`715`	`}`
`716`	`716`
`717`		`- /**`
`718`		`- * @requires function mb_strwidth`
`719`		`- */`
`720`	`717`	`public function testChoiceOutputFormattingQuestionForUtf8Keys()`
`721`	`718`	`{`
`722`	`719`	`$question = 'Lorem ipsum?';`
Original file line number	Diff line number	Diff line change
`@@ -759,9 +759,6 @@ function ($object, $key, $value) {`
`759`	`759`	`$this->assertFlatViewWithAttr($view);`
`760`	`760`	`}`
`761`	`761`
`762`		`- /**`
`763`		`- * @requires function Symfony\Component\Translation\TranslatableMessage::__construct`
`764`		`- */`
`765`	`762`	`public function testPassTranslatableMessageAsLabelDoesntCastItToString()`
`766`	`763`	`{`
`767`	`764`	`$view = $this->factory->createView(`
Original file line number	Diff line number	Diff line change
`@@ -907,9 +907,6 @@ public function testGetDynamicPublicPropertyWithMagicGetterAllow()`
`907`	`907`	`$this->assertSame($value, $this->propertyAccessor->getValue($object, $path));`
`908`	`908`	`}`
`909`	`909`
`910`		`- /**`
`911`		`- * @requires PHP 7.4`
`912`		`- */`
`913`	`910`	`public function testSetValueWrongTypeShouldThrowWrappedException()`
`914`	`911`	`{`
`915`	`912`	`$object = new TestClassTypedProperty();`
Original file line number	Diff line number	Diff line change
`@@ -184,9 +184,6 @@ public function testHtmlDump()`
`184`	`184`	`$this->assertStringMatchesFormat($expectedDump, $dump);`
`185`	`185`	`}`
`186`	`186`
`187`		`- /**`
`188`		`- * @requires function Twig\Template::getSourceContext`
`189`		`- */`
`190`	`187`	`public function testFrameWithTwig()`
`191`	`188`	`{`
`192`	`189`	`require_once \dirname(__DIR__).'/Fixtures/Twig.php';`
Original file line number	Diff line number	Diff line change
`@@ -293,9 +293,6 @@ public function testFlags()`
`293`	`293`	`putenv('DUMP_STRING_LENGTH=');`
`294`	`294`	`}`
`295`	`295`
`296`		`- /**`
`297`		`- * @requires function Twig\Template::getSourceContext`
`298`		`- */`
`299`	`296`	`public function testThrowingCaster()`
`300`	`297`	`{`
`301`	`298`	`$out = fopen('php://memory', 'r+');`