New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Fix deprecation messages in AdminResettingController
and AdminResettingController
#1201
Fix deprecation messages in AdminResettingController
and AdminResettingController
#1201
Conversation
There might me something wrong on tests:
seems like we using the old way of expecting deprecations. Deprecated 😅 |
d7dfd24
to
d60215e
Compare
I've replaced those annotations. The test should be fine now. |
use Symfony\Component\DependencyInjection\ContainerBuilder; | ||
use Symfony\Component\HttpFoundation\Request; | ||
use Symfony\Component\HttpFoundation\RequestStack; | ||
|
||
class AdminResettingControllerTest extends TestCase | ||
{ | ||
use ExpectDeprecationTrait; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think this is only needed if you using really old versions of phpunit. on 8.5 we should be fine without it
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I can confirm it is required. We are using phpunit/phpunit:8.5.0:
SonataUserBundle/phpunit.xml.dist
Line 35 in d60215e
<env name="SYMFONY_PHPUNIT_VERSION" value="8.5.0" /> |
Please, see symfony/symfony#37153.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Oh I see, needed then, I was confused with the expectException
There should be no changelog, for pedantic isn't it ? And I think two commit would be better.
|
…ed annotation `@expectedDeprecation`
d60215e
to
e8bdf30
Compare
Changelog section removed, since these changes are pedantic. It was: Changelog### Fixed
- Fixed QCNs referenced by deprecation messages at `AdminResettingController` and `AdminResettingController`;
- Fixed deprecated usage of `@exptectedDeprecation` through `ExpectDeprecationTrait::expectDeprecation()`. |
Changelog removed.
Commits splitted. |
Thank you @phansys |
Subject
Fix deprecation messages in
AdminResettingController
andAdminResettingController
, since currently they are making references to non existing classes.I am targeting this branch, because these changes respect BC.