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

minor #9830 [WCM][Tests] Improve tests running with specific requirements (except intl icu) (cordoval)

Browse Source 
This PR was submitted for the 2.4-dev branch but it was merged into the 2.4 branch instead (closes #9830).

Discussion
----------

[WCM][Tests] Improve tests running with specific requirements (except intl icu)

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

 - [x] go over all skipped tests, take note and check they are reasonable
 - [x] reenable memcache, mongodb, and memcached

We are keeping the icu intl related tests skipped because setting up icu 51.2 is totally time consuming in travis and it would require a custom distro box on travis because there are no ppa's available for the ubuntu version. I tried hard but it does not seem worth it. Same for plugging beta of memcached with pecl, it is just not reasonable to be running beta versions on travis. This then does not address #9797 but at least now we are aware.

This PR now can be merged as is as it improves tests that before were not ran. Not all but more than before. 👶

Commits
-------

6e0b2dc added condition to avoid skipping tests on JSON_PRETTY support
This commit is contained in:
Fabien Potencier 2013-12-22 16:57:41 +01:00
commit d1ace04580
1 changed files with 3 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
src/Symfony/Bundle/FrameworkBundle/Tests/Console/Descriptor/JsonDescriptorTest.php
View File

@ -17,7 +17,9 @@ class JsonDescriptorTest extends AbstractDescriptorTest
{
protected function setUp()
{
$this->markTestSkipped('Test skipped on PHP 5.3 as JSON_PRETTY_PRINT does not exist.');
if (version_compare(phpversion(), '5.4.0', '<')) {
$this->markTestSkipped('Test skipped on PHP 5.3 as JSON_PRETTY_PRINT does not exist.');
}
}
protected function getDescriptor()