diogo/symfony
Archived
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

minor #18532 [DependencyInjection] Remove check for YAML class in CrossCheckTest (JhonnyL)

Browse Source 
This PR was merged into the 2.3 branch.

Discussion
----------

[DependencyInjection] Remove check for YAML class in CrossCheckTest

| Q             | A
| ------------- | ---
| Branch?       | 2.3
| Bug fix?      | no
| New feature?  | no
| BC breaks?    |  no
| Deprecations? |  no
| Tests pass?   | yes
| Fixed tickets |
| License       | MIT
| Doc PR        |

I suppose it's a leftover from when YAML component only was added in suggest.

Commits
-------

af92275 [DependencyInjection] Remove YAML check in CrossCheckTest
This commit is contained in:
Fabien Potencier 2016-04-13 07:04:16 -07:00
commit 8c89a3a0a3
1 changed files with 6 additions and 13 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

19
src/Symfony/Component/DependencyInjection/Tests/CrossCheckTest.php
View File

@ -73,24 +73,17 @@ class CrossCheckTest extends \PHPUnit_Framework_TestCase
public function crossCheckLoadersDumpers() public function crossCheckLoadersDumpers()
{ {
$tests = array( return array(
array('services1.xml', 'xml'), array('services1.xml', 'xml'),
array('services2.xml', 'xml'), array('services2.xml', 'xml'),
array('services6.xml', 'xml'), array('services6.xml', 'xml'),
array('services8.xml', 'xml'), array('services8.xml', 'xml'),
array('services9.xml', 'xml'), array('services9.xml', 'xml'),
array('services1.yml', 'yaml'),
array('services2.yml', 'yaml'),
array('services6.yml', 'yaml'),
array('services8.yml', 'yaml'),
array('services9.yml', 'yaml'),
); );
if (class_exists('Symfony\Component\Yaml\Yaml')) {
$tests = array_merge($tests, array(
array('services1.yml', 'yaml'),
array('services2.yml', 'yaml'),
array('services6.yml', 'yaml'),
array('services8.yml', 'yaml'),
array('services9.yml', 'yaml'),
));
}
return $tests;
} }
} }