[tests] Use @requires annotation when possible

Author: nicolas-grekas

Tests/Constraints/LengthValidatorTest.php

@@ -89,12 +89,11 @@ public function getFiveOrMoreCharacters()
         );
     }
 
+    /**
+     * @requires extension mbstring
+     */
     public function getOneCharset()
     {
-        if (!function_exists('iconv') && !function_exists('mb_convert_encoding')) {
-            $this->markTestSkipped('Mbstring or iconv is required for this test.');
-        }
-
         return array(
             array('é', 'utf8', true),
             array("\xE9", 'CP1252', true),

Tests/Mapping/Cache/ApcCacheTest.php

@@ -17,8 +17,8 @@ class ApcCacheTest extends \PHPUnit_Framework_TestCase
 {
     protected function setUp()
     {
-        if (!extension_loaded('apc') || !ini_get('apc.enable_cli')) {
-            $this->markTestSkipped('APC is not loaded.');
+        if (!ini_get('apc.enable') || !ini_get('apc.enable_cli')) {
+            $this->markTestSkipped('APC is not enabled.');
         }
     }
 

Tests/Mapping/Loader/StaticMethodLoaderTest.php

@@ -94,10 +94,6 @@ public function testLoadClassMetadataIgnoresAbstractMethods()
         // strict standards error
         error_reporting(0);
 
-        if (0 !== error_reporting()) {
-            $this->markTestSkipped('Could not disable error reporting');
-        }
-
         $metadata = new ClassMetadata(__NAMESPACE__.'\AbstractStaticMethodLoader');
 
         $loader = new StaticMethodLoader('loadMetadata');