Skip to content

[PhpUnitBridge] Add test case for @expectedDeprecation annotation #35190

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

Conversation

fancyweb
Copy link
Contributor

@fancyweb fancyweb commented Jan 3, 2020

Q A
Branch? 3.4
Bug fix? no
New feature? no
Deprecations? no
Tickets -
License MIT
Doc PR -

Useful twice because it also tests the fact that a test that expects a deprecation and that does not perform any assertion is not considered risky.

@fancyweb fancyweb force-pushed the phpunit-bridge-add-expected-deprecation-annotation-test branch from 3d152ab to dba1804 Compare January 3, 2020 11:34
@nicolas-grekas nicolas-grekas added this to the 3.4 milestone Jan 4, 2020
@nicolas-grekas
Copy link
Member

Thank you @fancyweb.

nicolas-grekas added a commit that referenced this pull request Jan 4, 2020
…nnotation (fancyweb)

This PR was merged into the 3.4 branch.

Discussion
----------

[PhpUnitBridge] Add test case for @expectedDeprecation annotation

| Q             | A
| ------------- | ---
| Branch?       | 3.4
| Bug fix?      | no
| New feature?  | no
| Deprecations? | no
| Tickets       | -
| License       | MIT
| Doc PR        | -

Useful twice because it also tests the fact that a test that expects a deprecation and that does not perform any assertion is not considered risky.

Commits
-------

dba1804 [PhpUnitBridge] Add test case for @expectedDeprecation annotation
@nicolas-grekas nicolas-grekas merged commit dba1804 into symfony:3.4 Jan 4, 2020
@fancyweb fancyweb deleted the phpunit-bridge-add-expected-deprecation-annotation-test branch January 4, 2020 11:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants