minor #41596 [FrameworkBundle] Remove duplicate catch block (OskarStark)
This PR was merged into the 4.4 branch.
Discussion
----------
[FrameworkBundle] Remove duplicate catch block
| Q | A
| ------------- | ---
| Branch? | 4.4
| Bug fix? | no
| New feature? | no
| Deprecations? | no
| Tickets | Fix #... <!-- prefix each issue number with "Fix #", no need to create an issue if none exist, explain below instead -->
| License | MIT
| Doc PR | symfony/symfony-docs#... <!-- required for new features -->
Spottet while reviewing https://github.com/symfony/symfony/pull/41576 by `@derrabus`
Commits
-------
32cac1c6f2
Remove duplicate catch block
This commit is contained in:
commit
e658d001e7
@ -39,11 +39,6 @@ class CachePoolsTest extends AbstractWebTestCase
|
||||
throw $e;
|
||||
}
|
||||
$this->markTestSkipped($e->getMessage());
|
||||
} catch (\PHPUnit\Framework\Error\Warning $e) {
|
||||
if (0 !== strpos($e->getMessage(), 'unable to connect to')) {
|
||||
throw $e;
|
||||
}
|
||||
$this->markTestSkipped($e->getMessage());
|
||||
} catch (InvalidArgumentException $e) {
|
||||
if (0 !== strpos($e->getMessage(), 'Redis connection ')) {
|
||||
throw $e;
|
||||
@ -67,11 +62,6 @@ class CachePoolsTest extends AbstractWebTestCase
|
||||
throw $e;
|
||||
}
|
||||
$this->markTestSkipped($e->getMessage());
|
||||
} catch (\PHPUnit\Framework\Error\Warning $e) {
|
||||
if (0 !== strpos($e->getMessage(), 'unable to connect to')) {
|
||||
throw $e;
|
||||
}
|
||||
$this->markTestSkipped($e->getMessage());
|
||||
}
|
||||
}
|
||||
|
||||
|
Reference in New Issue
Block a user