Skip to content

Commit

Permalink
minor #35190 [PhpUnitBridge] Add test case for @expectedDeprecation a…
Browse files Browse the repository at this point in the history
…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
  • Loading branch information
nicolas-grekas committed Jan 4, 2020
2 parents 8e9aafc + dba1804 commit 28c9463
Showing 1 changed file with 43 additions and 0 deletions.
@@ -0,0 +1,43 @@
<?php

/*
* This file is part of the Symfony package.
*
* (c) Fabien Potencier <fabien@symfony.com>
*
* For the full copyright and license information, please view the LICENSE
* file that was distributed with this source code.
*/

namespace Symfony\Bridge\PhpUnit\Tests;

use PHPUnit\Framework\TestCase;

final class ExpectedDeprecationAnnotationTest extends TestCase
{
/**
* Do not remove this test in the next major versions.
*
* @group legacy
*
* @expectedDeprecation foo
*/
public function testOne()
{
@trigger_error('foo', E_USER_DEPRECATED);
}

/**
* Do not remove this test in the next major versions.
*
* @group legacy
*
* @expectedDeprecation foo
* @expectedDeprecation bar
*/
public function testMany()
{
@trigger_error('foo', E_USER_DEPRECATED);
@trigger_error('bar', E_USER_DEPRECATED);
}
}

0 comments on commit 28c9463

Please sign in to comment.