Replace annotations with attributes

[TavoNiievez]

No description provided.

[Naktibalda]

Now we need test for annotations :)

[Naktibalda]

This docblock was autogenerated, generator still doesn't generate any comments with self return type -

Codeception/src/Codeception/Lib/Generator/Actor.php

Lines 93 to 103 in 522c7f0

	$returnType = 'void';
	if ($method->name == 'haveFriend') {
	$returnType = Friend::class;
	}
	$params = $this->getParamsString($method);
	$inherited[] = (new Template($this->inheritedMethodTemplate))
	->place('method', $method->name)
	->place('params', $params)
	->place('return', $returnType)
	->produce();
	}

[DavertMik]

@TavoNiievez thank you! This one is really important to get merged!
Please check my PR #6487 as it also adds support for repeatable attributes so we could have multiple #[Group] attributes

[Naktibalda]

Are we dropping support for annotations now?
If we are keeping support for annotations, we must have some tests using or (even better) testing them.

[Naktibalda]

Codeception DataProvider attribute does not work in Unit tests.

  [ArgumentCountError] Too few arguments to function Codeception\Util\PathResolverTest::testGetRelativeDir(), 0 passed in /.../Codeception/vendor/phpunit/phpunit/src/Framework/TestCase.php on line 1545 and exactly 4 expected


#1  /.../Codeception/tests/unit/Codeception/Util/PathResolverTest.php:15

[DavertMik]

Are we dropping support for annotations now?
If we are keeping support for annotations, we must have some tests using or (even better) testing them.

No, we are not dropping, so I agree that some tests with annotation should also be kept

[Naktibalda]

I resolved conflicts, fixed failing tests and merged to 5.0

See 128727f for tests where it was necessary to keep @dataProvider annotation.

[TavoNiievez]

Thank you @Naktibalda

[Naktibalda]

and d432509