Update to PhpUnit 12, remove error if no methods declared

[tacman]

Description

Since you've updated to Symfony 8, I figured you'd want to have composer outdated show everything as being up today.

So I updated to PHPUnit 12, and got errors related to not declaring methods on the controllers.

            $methods = $route->getMethods();
            if (!$methods) {
                @trigger_error(\sprintf('Route %s has no configured HTTP methods. It is recommended that you set at least one HTTP method for your route in its configuration.', $routeName), \E_USER_DEPRECATED);

                $methods[] = 'GET';
            }

I find this "error" completely incorrect, since at best it's a recommendation but not even the Symfony maker bundle generates it with a method.

I asked pierstoval about it, he created an environment variable to silence them, but for this package I think we should just skip it.

Pierstoval/SmokeTesting#9

I'll submit a PR.

Example

No response

[tacman]

Fixed in #206

[COil]

Agree, this is not an official “best practice”. I've turned the deprecation into a comment :

            $methods = $route->getMethods();
            if (!$methods) {
                // If we get there, it is because your route doesn't have methods requirment.
                // You can add one by adding ` methods: ['GET']` to your Route attribute.
                $methods[] = 'GET';
            }