Skip to content

Commit 60ed58d

Browse files
[2.3] Remove useless tests skips
1 parent a7bcad3 commit 60ed58d

8 files changed

+0
-78
lines changed

Tests/ContainerBuilderTest.php

Lines changed: 0 additions & 24 deletions
Original file line numberDiff line numberDiff line change
@@ -498,10 +498,6 @@ public function testFindDefinition()
498498
*/
499499
public function testAddObjectResource()
500500
{
501-
if (!class_exists('Symfony\Component\Config\Resource\FileResource')) {
502-
$this->markTestSkipped('The "Config" component is not available');
503-
}
504-
505501
$container = new ContainerBuilder();
506502

507503
$container->setResourceTracking(false);
@@ -528,10 +524,6 @@ public function testAddObjectResource()
528524
*/
529525
public function testAddClassResource()
530526
{
531-
if (!class_exists('Symfony\Component\Config\Resource\FileResource')) {
532-
$this->markTestSkipped('The "Config" component is not available');
533-
}
534-
535527
$container = new ContainerBuilder();
536528

537529
$container->setResourceTracking(false);
@@ -558,10 +550,6 @@ public function testAddClassResource()
558550
*/
559551
public function testCompilesClassDefinitionsOfLazyServices()
560552
{
561-
if (!class_exists('Symfony\Component\Config\Resource\FileResource')) {
562-
$this->markTestSkipped('The "Config" component is not available');
563-
}
564-
565553
$container = new ContainerBuilder();
566554

567555
$this->assertEmpty($container->getResources(), 'No resources get registered without resource tracking');
@@ -588,10 +576,6 @@ function (ResourceInterface $resource) use ($classesPath) {
588576
*/
589577
public function testResources()
590578
{
591-
if (!class_exists('Symfony\Component\Config\Resource\FileResource')) {
592-
$this->markTestSkipped('The "Config" component is not available');
593-
}
594-
595579
$container = new ContainerBuilder();
596580
$container->addResource($a = new FileResource(__DIR__.'/Fixtures/xml/services1.xml'));
597581
$container->addResource($b = new FileResource(__DIR__.'/Fixtures/xml/services2.xml'));
@@ -682,21 +666,13 @@ public function testThrowsExceptionWhenSetServiceOnAFrozenContainer()
682666
*/
683667
public function testThrowsExceptionWhenAddServiceOnAFrozenContainer()
684668
{
685-
if (!class_exists('Symfony\Component\Config\Resource\FileResource')) {
686-
$this->markTestSkipped('The "Config" component is not available');
687-
}
688-
689669
$container = new ContainerBuilder();
690670
$container->compile();
691671
$container->set('a', new \stdClass());
692672
}
693673

694674
public function testNoExceptionWhenSetSyntheticServiceOnAFrozenContainer()
695675
{
696-
if (!class_exists('Symfony\Component\Config\Resource\FileResource')) {
697-
$this->markTestSkipped('The "Config" component is not available');
698-
}
699-
700676
$container = new ContainerBuilder();
701677
$def = new Definition('stdClass');
702678
$def->setSynthetic(true);

Tests/CrossCheckTest.php

Lines changed: 0 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -18,13 +18,6 @@ class CrossCheckTest extends \PHPUnit_Framework_TestCase
1818
{
1919
protected static $fixturesPath;
2020

21-
protected function setUp()
22-
{
23-
if (!class_exists('Symfony\Component\Config\Loader\Loader')) {
24-
$this->markTestSkipped('The "Config" component is not available');
25-
}
26-
}
27-
2821
public static function setUpBeforeClass()
2922
{
3023
self::$fixturesPath = __DIR__.'/Fixtures/';

Tests/Dumper/YamlDumperTest.php

Lines changed: 0 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -18,13 +18,6 @@ class YamlDumperTest extends \PHPUnit_Framework_TestCase
1818
{
1919
protected static $fixturesPath;
2020

21-
protected function setUp()
22-
{
23-
if (!class_exists('Symfony\Component\Yaml\Yaml')) {
24-
$this->markTestSkipped('The "Yaml" component is not available');
25-
}
26-
}
27-
2821
public static function setUpBeforeClass()
2922
{
3023
self::$fixturesPath = realpath(__DIR__.'/../Fixtures/');

Tests/Loader/ClosureLoaderTest.php

Lines changed: 0 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -16,13 +16,6 @@
1616

1717
class ClosureLoaderTest extends \PHPUnit_Framework_TestCase
1818
{
19-
protected function setUp()
20-
{
21-
if (!class_exists('Symfony\Component\Config\Loader\Loader')) {
22-
$this->markTestSkipped('The "Config" component is not available');
23-
}
24-
}
25-
2619
/**
2720
* @covers Symfony\Component\DependencyInjection\Loader\ClosureLoader::supports
2821
*/

Tests/Loader/IniFileLoaderTest.php

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -29,10 +29,6 @@ public static function setUpBeforeClass()
2929

3030
protected function setUp()
3131
{
32-
if (!class_exists('Symfony\Component\Config\Loader\Loader')) {
33-
$this->markTestSkipped('The "Config" component is not available');
34-
}
35-
3632
$this->container = new ContainerBuilder();
3733
$this->loader = new IniFileLoader($this->container, new FileLocator(self::$fixturesPath.'/ini'));
3834
}

Tests/Loader/PhpFileLoaderTest.php

Lines changed: 0 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -18,13 +18,6 @@
1818

1919
class PhpFileLoaderTest extends \PHPUnit_Framework_TestCase
2020
{
21-
protected function setUp()
22-
{
23-
if (!class_exists('Symfony\Component\Config\Loader\Loader')) {
24-
$this->markTestSkipped('The "Config" component is not available');
25-
}
26-
}
27-
2821
/**
2922
* @covers Symfony\Component\DependencyInjection\Loader\PhpFileLoader::supports
3023
*/

Tests/Loader/XmlFileLoaderTest.php

Lines changed: 0 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -25,13 +25,6 @@ class XmlFileLoaderTest extends \PHPUnit_Framework_TestCase
2525
{
2626
protected static $fixturesPath;
2727

28-
protected function setUp()
29-
{
30-
if (!class_exists('Symfony\Component\Config\Loader\Loader')) {
31-
$this->markTestSkipped('The "Config" component is not available');
32-
}
33-
}
34-
3528
public static function setUpBeforeClass()
3629
{
3730
self::$fixturesPath = realpath(__DIR__.'/../Fixtures/');
@@ -104,10 +97,6 @@ public function testLoadParameters()
10497

10598
public function testLoadImports()
10699
{
107-
if (!class_exists('Symfony\Component\Yaml\Yaml')) {
108-
$this->markTestSkipped('The "Yaml" component is not available');
109-
}
110-
111100
$container = new ContainerBuilder();
112101
$resolver = new LoaderResolver(array(
113102
new IniFileLoader($container, new FileLocator(self::$fixturesPath.'/xml')),

Tests/Loader/YamlFileLoaderTest.php

Lines changed: 0 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -25,17 +25,6 @@ class YamlFileLoaderTest extends \PHPUnit_Framework_TestCase
2525
{
2626
protected static $fixturesPath;
2727

28-
protected function setUp()
29-
{
30-
if (!class_exists('Symfony\Component\Config\Loader\Loader')) {
31-
$this->markTestSkipped('The "Config" component is not available');
32-
}
33-
34-
if (!class_exists('Symfony\Component\Yaml\Yaml')) {
35-
$this->markTestSkipped('The "Yaml" component is not available');
36-
}
37-
}
38-
3928
public static function setUpBeforeClass()
4029
{
4130
self::$fixturesPath = realpath(__DIR__.'/../Fixtures/');

0 commit comments

Comments
 (0)