Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly
Appearance settings

Commit 3850658

Browse filesBrowse files
nicolas-grekasnicolas-grekas
committed
minor #51776 Replace usages of SkippedTestSuiteError with markTestSkipped() call (digilist)
This PR was merged into the 6.4 branch. Discussion ---------- Replace usages of SkippedTestSuiteError with markTestSkipped() call | Q | A | ------------- | --- | Branch? | 6.4 | Bug fix? | no | New feature? | no | Deprecations? | no | Tickets | n/a | License | MIT Just a simple clean up to replace the usages of `SkippedTestSuiteError` with a call to `markTestSkipped()`. This is a follow up for #51775. Commits ------- 41eb669 Replace usages of SkippedTestSuiteError with markTestSkipped() call
2 parents 1b0aed0 + 41eb669 commit 3850658
Copy full SHA for 3850658

29 files changed

+50
-88
lines changed

‎src/Symfony/Component/Cache/Tests/Adapter/AbstractRedisAdapterTestCase.php

Copy file name to clipboardExpand all lines: src/Symfony/Component/Cache/Tests/Adapter/AbstractRedisAdapterTestCase.php
+2-3Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -11,7 +11,6 @@
1111

1212
namespace Symfony\Component\Cache\Tests\Adapter;
1313

14-
use PHPUnit\Framework\SkippedTestSuiteError;
1514
use Psr\Cache\CacheItemPoolInterface;
1615
use Relay\Relay;
1716
use Symfony\Component\Cache\Adapter\RedisAdapter;
@@ -34,12 +33,12 @@ public function createCachePool(int $defaultLifetime = 0, string $testMethod = n
3433
public static function setUpBeforeClass(): void
3534
{
3635
if (!\extension_loaded('redis')) {
37-
throw new SkippedTestSuiteError('Extension redis required.');
36+
self::markTestSkipped('Extension redis required.');
3837
}
3938
try {
4039
(new \Redis())->connect(...explode(':', getenv('REDIS_HOST')));
4140
} catch (\Exception $e) {
42-
throw new SkippedTestSuiteError(getenv('REDIS_HOST').': '.$e->getMessage());
41+
self::markTestSkipped(getenv('REDIS_HOST').': '.$e->getMessage());
4342
}
4443
}
4544

‎src/Symfony/Component/Cache/Tests/Adapter/CouchbaseBucketAdapterTest.php

Copy file name to clipboardExpand all lines: src/Symfony/Component/Cache/Tests/Adapter/CouchbaseBucketAdapterTest.php
+1-2Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -11,7 +11,6 @@
1111

1212
namespace Symfony\Component\Cache\Tests\Adapter;
1313

14-
use PHPUnit\Framework\SkippedTestSuiteError;
1514
use Psr\Cache\CacheItemPoolInterface;
1615
use Symfony\Component\Cache\Adapter\AbstractAdapter;
1716
use Symfony\Component\Cache\Adapter\CouchbaseBucketAdapter;
@@ -35,7 +34,7 @@ class CouchbaseBucketAdapterTest extends AdapterTestCase
3534
public static function setupBeforeClass(): void
3635
{
3736
if (!CouchbaseBucketAdapter::isSupported()) {
38-
throw new SkippedTestSuiteError('Couchbase >= 2.6.0 < 3.0.0 is required.');
37+
self::markTestSkipped('Couchbase >= 2.6.0 < 3.0.0 is required.');
3938
}
4039

4140
self::$client = AbstractAdapter::createConnection('couchbase://'.getenv('COUCHBASE_HOST').'/cache',

‎src/Symfony/Component/Cache/Tests/Adapter/DoctrineDbalAdapterTest.php

Copy file name to clipboardExpand all lines: src/Symfony/Component/Cache/Tests/Adapter/DoctrineDbalAdapterTest.php
+1-2Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -18,7 +18,6 @@
1818
use Doctrine\DBAL\DriverManager;
1919
use Doctrine\DBAL\Schema\DefaultSchemaManagerFactory;
2020
use Doctrine\DBAL\Schema\Schema;
21-
use PHPUnit\Framework\SkippedTestSuiteError;
2221
use Psr\Cache\CacheItemPoolInterface;
2322
use Symfony\Component\Cache\Adapter\DoctrineDbalAdapter;
2423
use Symfony\Component\Cache\Tests\Fixtures\DriverWrapper;
@@ -33,7 +32,7 @@ class DoctrineDbalAdapterTest extends AdapterTestCase
3332
public static function setUpBeforeClass(): void
3433
{
3534
if (!\extension_loaded('pdo_sqlite')) {
36-
throw new SkippedTestSuiteError('Extension pdo_sqlite required.');
35+
self::markTestSkipped('Extension pdo_sqlite required.');
3736
}
3837

3938
self::$dbFile = tempnam(sys_get_temp_dir(), 'sf_sqlite_cache');

‎src/Symfony/Component/Cache/Tests/Adapter/MemcachedAdapterTest.php

Copy file name to clipboardExpand all lines: src/Symfony/Component/Cache/Tests/Adapter/MemcachedAdapterTest.php
+2-3Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -11,7 +11,6 @@
1111

1212
namespace Symfony\Component\Cache\Tests\Adapter;
1313

14-
use PHPUnit\Framework\SkippedTestSuiteError;
1514
use Psr\Cache\CacheItemPoolInterface;
1615
use Symfony\Component\Cache\Adapter\AbstractAdapter;
1716
use Symfony\Component\Cache\Adapter\MemcachedAdapter;
@@ -33,14 +32,14 @@ class MemcachedAdapterTest extends AdapterTestCase
3332
public static function setUpBeforeClass(): void
3433
{
3534
if (!MemcachedAdapter::isSupported()) {
36-
throw new SkippedTestSuiteError('Extension memcached > 3.1.5 required.');
35+
self::markTestSkipped('Extension memcached > 3.1.5 required.');
3736
}
3837
self::$client = AbstractAdapter::createConnection('memcached://'.getenv('MEMCACHED_HOST'), ['binary_protocol' => false]);
3938
self::$client->get('foo');
4039
$code = self::$client->getResultCode();
4140

4241
if (\Memcached::RES_SUCCESS !== $code && \Memcached::RES_NOTFOUND !== $code) {
43-
throw new SkippedTestSuiteError('Memcached error: '.strtolower(self::$client->getResultMessage()));
42+
self::markTestSkipped('Memcached error: '.strtolower(self::$client->getResultMessage()));
4443
}
4544
}
4645

‎src/Symfony/Component/Cache/Tests/Adapter/PdoAdapterTest.php

Copy file name to clipboardExpand all lines: src/Symfony/Component/Cache/Tests/Adapter/PdoAdapterTest.php
+1-2Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -11,7 +11,6 @@
1111

1212
namespace Symfony\Component\Cache\Tests\Adapter;
1313

14-
use PHPUnit\Framework\SkippedTestSuiteError;
1514
use Psr\Cache\CacheItemPoolInterface;
1615
use Symfony\Component\Cache\Adapter\PdoAdapter;
1716

@@ -25,7 +24,7 @@ class PdoAdapterTest extends AdapterTestCase
2524
public static function setUpBeforeClass(): void
2625
{
2726
if (!\extension_loaded('pdo_sqlite')) {
28-
throw new SkippedTestSuiteError('Extension pdo_sqlite required.');
27+
self::markTestSkipped('Extension pdo_sqlite required.');
2928
}
3029

3130
self::$dbFile = tempnam(sys_get_temp_dir(), 'sf_sqlite_cache');

‎src/Symfony/Component/Cache/Tests/Adapter/PredisAdapterSentinelTest.php

Copy file name to clipboardExpand all lines: src/Symfony/Component/Cache/Tests/Adapter/PredisAdapterSentinelTest.php
+3-4Lines changed: 3 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -11,7 +11,6 @@
1111

1212
namespace Symfony\Component\Cache\Tests\Adapter;
1313

14-
use PHPUnit\Framework\SkippedTestSuiteError;
1514
use Symfony\Component\Cache\Adapter\AbstractAdapter;
1615

1716
/**
@@ -22,13 +21,13 @@ class PredisAdapterSentinelTest extends AbstractRedisAdapterTestCase
2221
public static function setUpBeforeClass(): void
2322
{
2423
if (!class_exists(\Predis\Client::class)) {
25-
throw new SkippedTestSuiteError('The Predis\Client class is required.');
24+
self::markTestSkipped('The Predis\Client class is required.');
2625
}
2726
if (!$hosts = getenv('REDIS_SENTINEL_HOSTS')) {
28-
throw new SkippedTestSuiteError('REDIS_SENTINEL_HOSTS env var is not defined.');
27+
self::markTestSkipped('REDIS_SENTINEL_HOSTS env var is not defined.');
2928
}
3029
if (!$service = getenv('REDIS_SENTINEL_SERVICE')) {
31-
throw new SkippedTestSuiteError('REDIS_SENTINEL_SERVICE env var is not defined.');
30+
self::markTestSkipped('REDIS_SENTINEL_SERVICE env var is not defined.');
3231
}
3332

3433
self::$redis = AbstractAdapter::createConnection('redis:?host['.str_replace(' ', ']&host[', $hosts).']', ['redis_sentinel' => $service, 'class' => \Predis\Client::class]);

‎src/Symfony/Component/Cache/Tests/Adapter/PredisRedisClusterAdapterTest.php

Copy file name to clipboardExpand all lines: src/Symfony/Component/Cache/Tests/Adapter/PredisRedisClusterAdapterTest.php
+1-2Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -11,7 +11,6 @@
1111

1212
namespace Symfony\Component\Cache\Tests\Adapter;
1313

14-
use PHPUnit\Framework\SkippedTestSuiteError;
1514
use Symfony\Component\Cache\Adapter\RedisAdapter;
1615

1716
/**
@@ -22,7 +21,7 @@ class PredisRedisClusterAdapterTest extends AbstractRedisAdapterTestCase
2221
public static function setUpBeforeClass(): void
2322
{
2423
if (!$hosts = getenv('REDIS_CLUSTER_HOSTS')) {
25-
throw new SkippedTestSuiteError('REDIS_CLUSTER_HOSTS env var is not defined.');
24+
self::markTestSkipped('REDIS_CLUSTER_HOSTS env var is not defined.');
2625
}
2726

2827
self::$redis = RedisAdapter::createConnection('redis:?host['.str_replace(' ', ']&host[', $hosts).']', ['class' => \Predis\Client::class, 'redis_cluster' => true, 'prefix' => 'prefix_']);

‎src/Symfony/Component/Cache/Tests/Adapter/RedisAdapterSentinelTest.php

Copy file name to clipboardExpand all lines: src/Symfony/Component/Cache/Tests/Adapter/RedisAdapterSentinelTest.php
+3-4Lines changed: 3 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -11,7 +11,6 @@
1111

1212
namespace Symfony\Component\Cache\Tests\Adapter;
1313

14-
use PHPUnit\Framework\SkippedTestSuiteError;
1514
use Symfony\Component\Cache\Adapter\AbstractAdapter;
1615
use Symfony\Component\Cache\Adapter\RedisAdapter;
1716
use Symfony\Component\Cache\Exception\InvalidArgumentException;
@@ -24,13 +23,13 @@ class RedisAdapterSentinelTest extends AbstractRedisAdapterTestCase
2423
public static function setUpBeforeClass(): void
2524
{
2625
if (!class_exists(\RedisSentinel::class)) {
27-
throw new SkippedTestSuiteError('The RedisSentinel class is required.');
26+
self::markTestSkipped('The RedisSentinel class is required.');
2827
}
2928
if (!$hosts = getenv('REDIS_SENTINEL_HOSTS')) {
30-
throw new SkippedTestSuiteError('REDIS_SENTINEL_HOSTS env var is not defined.');
29+
self::markTestSkipped('REDIS_SENTINEL_HOSTS env var is not defined.');
3130
}
3231
if (!$service = getenv('REDIS_SENTINEL_SERVICE')) {
33-
throw new SkippedTestSuiteError('REDIS_SENTINEL_SERVICE env var is not defined.');
32+
self::markTestSkipped('REDIS_SENTINEL_SERVICE env var is not defined.');
3433
}
3534

3635
self::$redis = AbstractAdapter::createConnection('redis:?host['.str_replace(' ', ']&host[', $hosts).']', ['redis_sentinel' => $service, 'prefix' => 'prefix_']);

‎src/Symfony/Component/Cache/Tests/Adapter/RedisArrayAdapterTest.php

Copy file name to clipboardExpand all lines: src/Symfony/Component/Cache/Tests/Adapter/RedisArrayAdapterTest.php
+1-3Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -11,8 +11,6 @@
1111

1212
namespace Symfony\Component\Cache\Tests\Adapter;
1313

14-
use PHPUnit\Framework\SkippedTestSuiteError;
15-
1614
/**
1715
* @group integration
1816
*/
@@ -22,7 +20,7 @@ public static function setUpBeforeClass(): void
2220
{
2321
parent::setupBeforeClass();
2422
if (!class_exists(\RedisArray::class)) {
25-
throw new SkippedTestSuiteError('The RedisArray class is required.');
23+
self::markTestSkipped('The RedisArray class is required.');
2624
}
2725
self::$redis = new \RedisArray([getenv('REDIS_HOST')], ['lazy_connect' => true]);
2826
self::$redis->setOption(\Redis::OPT_PREFIX, 'prefix_');

‎src/Symfony/Component/Cache/Tests/Adapter/RedisClusterAdapterTest.php

Copy file name to clipboardExpand all lines: src/Symfony/Component/Cache/Tests/Adapter/RedisClusterAdapterTest.php
+2-3Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -11,7 +11,6 @@
1111

1212
namespace Symfony\Component\Cache\Tests\Adapter;
1313

14-
use PHPUnit\Framework\SkippedTestSuiteError;
1514
use Psr\Cache\CacheItemPoolInterface;
1615
use Symfony\Component\Cache\Adapter\AbstractAdapter;
1716
use Symfony\Component\Cache\Adapter\RedisAdapter;
@@ -26,10 +25,10 @@ class RedisClusterAdapterTest extends AbstractRedisAdapterTestCase
2625
public static function setUpBeforeClass(): void
2726
{
2827
if (!class_exists(\RedisCluster::class)) {
29-
throw new SkippedTestSuiteError('The RedisCluster class is required.');
28+
self::markTestSkipped('The RedisCluster class is required.');
3029
}
3130
if (!$hosts = getenv('REDIS_CLUSTER_HOSTS')) {
32-
throw new SkippedTestSuiteError('REDIS_CLUSTER_HOSTS env var is not defined.');
31+
self::markTestSkipped('REDIS_CLUSTER_HOSTS env var is not defined.');
3332
}
3433

3534
self::$redis = AbstractAdapter::createConnection('redis:?host['.str_replace(' ', ']&host[', $hosts).']', ['lazy' => true, 'redis_cluster' => true]);

‎src/Symfony/Component/Cache/Tests/Adapter/RelayAdapterSentinelTest.php

Copy file name to clipboardExpand all lines: src/Symfony/Component/Cache/Tests/Adapter/RelayAdapterSentinelTest.php
+3-4Lines changed: 3 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -11,7 +11,6 @@
1111

1212
namespace Symfony\Component\Cache\Tests\Adapter;
1313

14-
use PHPUnit\Framework\SkippedTestSuiteError;
1514
use Relay\Relay;
1615
use Relay\Sentinel;
1716
use Symfony\Component\Cache\Adapter\AbstractAdapter;
@@ -24,13 +23,13 @@ class RelayAdapterSentinelTest extends AbstractRedisAdapterTestCase
2423
public static function setUpBeforeClass(): void
2524
{
2625
if (!class_exists(Sentinel::class)) {
27-
throw new SkippedTestSuiteError('The Relay\Sentinel class is required.');
26+
self::markTestSkipped('The Relay\Sentinel class is required.');
2827
}
2928
if (!$hosts = getenv('REDIS_SENTINEL_HOSTS')) {
30-
throw new SkippedTestSuiteError('REDIS_SENTINEL_HOSTS env var is not defined.');
29+
self::markTestSkipped('REDIS_SENTINEL_HOSTS env var is not defined.');
3130
}
3231
if (!$service = getenv('REDIS_SENTINEL_SERVICE')) {
33-
throw new SkippedTestSuiteError('REDIS_SENTINEL_SERVICE env var is not defined.');
32+
self::markTestSkipped('REDIS_SENTINEL_SERVICE env var is not defined.');
3433
}
3534

3635
self::$redis = AbstractAdapter::createConnection(

‎src/Symfony/Component/Cache/Tests/Adapter/RelayAdapterTest.php

Copy file name to clipboardExpand all lines: src/Symfony/Component/Cache/Tests/Adapter/RelayAdapterTest.php
+1-2Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -11,7 +11,6 @@
1111

1212
namespace Symfony\Component\Cache\Tests\Adapter;
1313

14-
use PHPUnit\Framework\SkippedTestSuiteError;
1514
use Relay\Relay;
1615
use Symfony\Component\Cache\Adapter\AbstractAdapter;
1716
use Symfony\Component\Cache\Adapter\RedisAdapter;
@@ -30,7 +29,7 @@ public static function setUpBeforeClass(): void
3029
try {
3130
new Relay(...explode(':', getenv('REDIS_HOST')));
3231
} catch (\Relay\Exception $e) {
33-
throw new SkippedTestSuiteError(getenv('REDIS_HOST').': '.$e->getMessage());
32+
self::markTestSkipped(getenv('REDIS_HOST').': '.$e->getMessage());
3433
}
3534
self::$redis = AbstractAdapter::createConnection('redis://'.getenv('REDIS_HOST'), ['lazy' => true, 'class' => Relay::class]);
3635
self::assertInstanceOf(RelayProxy::class, self::$redis);

‎src/Symfony/Component/Cache/Tests/Traits/RedisTraitTest.php

Copy file name to clipboardExpand all lines: src/Symfony/Component/Cache/Tests/Traits/RedisTraitTest.php
+3-4Lines changed: 3 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -11,7 +11,6 @@
1111

1212
namespace Symfony\Component\Cache\Tests\Traits;
1313

14-
use PHPUnit\Framework\SkippedTestSuiteError;
1514
use PHPUnit\Framework\TestCase;
1615
use Symfony\Component\Cache\Traits\RedisTrait;
1716

@@ -20,7 +19,7 @@ class RedisTraitTest extends TestCase
2019
public static function setUpBeforeClass(): void
2120
{
2221
if (!getenv('REDIS_CLUSTER_HOSTS')) {
23-
throw new SkippedTestSuiteError('REDIS_CLUSTER_HOSTS env var is not defined.');
22+
self::markTestSkipped('REDIS_CLUSTER_HOSTS env var is not defined.');
2423
}
2524
}
2625

@@ -30,10 +29,10 @@ public static function setUpBeforeClass(): void
3029
public function testCreateConnection(string $dsn, string $expectedClass)
3130
{
3231
if (!class_exists($expectedClass)) {
33-
throw new SkippedTestSuiteError(sprintf('The "%s" class is required.', $expectedClass));
32+
self::markTestSkipped(sprintf('The "%s" class is required.', $expectedClass));
3433
}
3534
if (!getenv('REDIS_CLUSTER_HOSTS')) {
36-
throw new SkippedTestSuiteError('REDIS_CLUSTER_HOSTS env var is not defined.');
35+
self::markTestSkipped('REDIS_CLUSTER_HOSTS env var is not defined.');
3736
}
3837

3938
$mock = self::getObjectForTrait(RedisTrait::class);

‎src/Symfony/Component/HttpClient/Tests/HttpClientTestCase.php

Copy file name to clipboardExpand all lines: src/Symfony/Component/HttpClient/Tests/HttpClientTestCase.php
+4-5Lines changed: 4 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -11,7 +11,6 @@
1111

1212
namespace Symfony\Component\HttpClient\Tests;
1313

14-
use PHPUnit\Framework\SkippedTestSuiteError;
1514
use Symfony\Component\HttpClient\Exception\ClientException;
1615
use Symfony\Component\HttpClient\Exception\TransportException;
1716
use Symfony\Component\HttpClient\Internal\ClientState;
@@ -318,7 +317,7 @@ private static function startVulcain(HttpClientInterface $client)
318317
}
319318

320319
if ('\\' === \DIRECTORY_SEPARATOR) {
321-
throw new SkippedTestSuiteError('Testing with the "vulcain" is not supported on Windows.');
320+
self::markTestSkipped('Testing with the "vulcain" is not supported on Windows.');
322321
}
323322

324323
$process = new Process(['vulcain'], null, [
@@ -335,14 +334,14 @@ private static function startVulcain(HttpClientInterface $client)
335334

336335
if (!$process->isRunning()) {
337336
if ('\\' !== \DIRECTORY_SEPARATOR && 127 === $process->getExitCode()) {
338-
throw new SkippedTestSuiteError('vulcain binary is missing');
337+
self::markTestSkipped('vulcain binary is missing');
339338
}
340339

341340
if ('\\' !== \DIRECTORY_SEPARATOR && 126 === $process->getExitCode()) {
342-
throw new SkippedTestSuiteError('vulcain binary is not executable');
341+
self::markTestSkipped('vulcain binary is not executable');
343342
}
344343

345-
throw new SkippedTestSuiteError((new ProcessFailedException($process))->getMessage());
344+
self::markTestSkipped((new ProcessFailedException($process))->getMessage());
346345
}
347346

348347
self::$vulcainStarted = true;

‎src/Symfony/Component/HttpFoundation/Tests/ResponseFunctionalTest.php

Copy file name to clipboardExpand all lines: src/Symfony/Component/HttpFoundation/Tests/ResponseFunctionalTest.php
+1-2Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -11,7 +11,6 @@
1111

1212
namespace Symfony\Component\HttpFoundation\Tests;
1313

14-
use PHPUnit\Framework\SkippedTestSuiteError;
1514
use PHPUnit\Framework\TestCase;
1615

1716
class ResponseFunctionalTest extends TestCase
@@ -26,7 +25,7 @@ public static function setUpBeforeClass(): void
2625
2 => ['file', '/dev/null', 'w'],
2726
];
2827
if (!self::$server = @proc_open('exec '.\PHP_BINARY.' -S localhost:8054', $spec, $pipes, __DIR__.'/Fixtures/response-functional')) {
29-
throw new SkippedTestSuiteError('PHP server unable to start.');
28+
self::markTestSkipped('PHP server unable to start.');
3029
}
3130
sleep(1);
3231
}

‎src/Symfony/Component/HttpFoundation/Tests/Session/Storage/Handler/AbstractSessionHandlerTest.php

Copy file name to clipboardExpand all lines: src/Symfony/Component/HttpFoundation/Tests/Session/Storage/Handler/AbstractSessionHandlerTest.php
+1-2Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -11,7 +11,6 @@
1111

1212
namespace Symfony\Component\HttpFoundation\Tests\Session\Storage\Handler;
1313

14-
use PHPUnit\Framework\SkippedTestSuiteError;
1514
use PHPUnit\Framework\TestCase;
1615

1716
class AbstractSessionHandlerTest extends TestCase
@@ -26,7 +25,7 @@ public static function setUpBeforeClass(): void
2625
2 => ['file', '/dev/null', 'w'],
2726
];
2827
if (!self::$server = @proc_open('exec '.\PHP_BINARY.' -S localhost:8053', $spec, $pipes, __DIR__.'/Fixtures')) {
29-
throw new SkippedTestSuiteError('PHP server unable to start.');
28+
self::markTestSkipped('PHP server unable to start.');
3029
}
3130
sleep(1);
3231
}

‎src/Symfony/Component/HttpFoundation/Tests/Session/Storage/Handler/RedisClusterSessionHandlerTest.php

Copy file name to clipboardExpand all lines: src/Symfony/Component/HttpFoundation/Tests/Session/Storage/Handler/RedisClusterSessionHandlerTest.php
+2-4Lines changed: 2 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -11,8 +11,6 @@
1111

1212
namespace Symfony\Component\HttpFoundation\Tests\Session\Storage\Handler;
1313

14-
use PHPUnit\Framework\SkippedTestSuiteError;
15-
1614
/**
1715
* @group integration
1816
*/
@@ -21,11 +19,11 @@ class RedisClusterSessionHandlerTest extends AbstractRedisSessionHandlerTestCase
2119
public static function setUpBeforeClass(): void
2220
{
2321
if (!class_exists(\RedisCluster::class)) {
24-
throw new SkippedTestSuiteError('The RedisCluster class is required.');
22+
self::markTestSkipped('The RedisCluster class is required.');
2523
}
2624

2725
if (!$hosts = getenv('REDIS_CLUSTER_HOSTS')) {
28-
throw new SkippedTestSuiteError('REDIS_CLUSTER_HOSTS env var is not defined.');
26+
self::markTestSkipped('REDIS_CLUSTER_HOSTS env var is not defined.');
2927
}
3028
}
3129

‎src/Symfony/Component/Lock/Tests/Store/MemcachedStoreTest.php

Copy file name to clipboardExpand all lines: src/Symfony/Component/Lock/Tests/Store/MemcachedStoreTest.php
+2-3Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -11,7 +11,6 @@
1111

1212
namespace Symfony\Component\Lock\Tests\Store;
1313

14-
use PHPUnit\Framework\SkippedTestSuiteError;
1514
use Symfony\Component\Lock\Exception\InvalidTtlException;
1615
use Symfony\Component\Lock\Key;
1716
use Symfony\Component\Lock\PersistingStoreInterface;
@@ -31,7 +30,7 @@ class MemcachedStoreTest extends AbstractStoreTestCase
3130
public static function setUpBeforeClass(): void
3231
{
3332
if (version_compare(phpversion('memcached'), '3.1.6', '<')) {
34-
throw new SkippedTestSuiteError('Extension memcached > 3.1.5 required.');
33+
self::markTestSkipped('Extension memcached > 3.1.5 required.');
3534
}
3635

3736
$memcached = new \Memcached();
@@ -40,7 +39,7 @@ public static function setUpBeforeClass(): void
4039
$code = $memcached->getResultCode();
4140

4241
if (\Memcached::RES_SUCCESS !== $code && \Memcached::RES_NOTFOUND !== $code) {
43-
throw new SkippedTestSuiteError('Unable to connect to the memcache host');
42+
self::markTestSkipped('Unable to connect to the memcache host');
4443
}
4544
}
4645

0 commit comments

Comments
0 (0)
Morty Proxy This is a proxified and sanitized view of the page, visit original site.