is the @uses annotation for tests correct?

[swiffer]

Description

the docs state that "The @uses annotation specifies code which will be executed by a test, but is not intended to be covered by the test."

https://docs.phpunit.de/en/10.5/annotations.html#uses

currently bake adds @uses ControllerName to a controller test case. Shouldn't the @covers attribute be used instead as the primary use case for a controller test is testing the controller itself?

Bake Version

3.2.0

PHP Version

8.4

[ADmad]

Docblock annotations are deprecated in phpunit 11 and the attribute for coverage can only be used at class level, not method level, so we should simply drop those annotations.

[swiffer]

The @uses annotation is added at classlevel (for controllers) as well currently. Why not replacing with attributes (which exist for classes and functions)?

[ADmad]

I guess we can use the CoversClass attribute then.

[dereuromark]

See also dereuromark/cakephp-ide-helper#351

[dereuromark]

The uses annotation allows bidirectional IDE lookup, so basically to jump via uses detection from controller to test and the other way around.
So those are not directly related to coverage IMO.

[swiffer]

I see, so the primary use case was IDE, not coverage intended, and makes sense.