use symfony bridge to avoid deprecations

doctrine · Apr 5, 2015 · 25ecf64 · 25ecf64
1 parent 8b61902
commit 25ecf64
Show file tree

Hide file tree

Showing 3 changed files with 14 additions and 1 deletion.
diff --git a/composer.json b/composer.json
@@ -20,6 +20,7 @@
     },
     "require-dev": {
         "symfony/yaml": "~2.0",
+        "symfony/phpunit-bridge": "2.7.*@dev",
         "liip/rmt": "dev-master"
     },
     "suggest":{

diff --git a/lib/Doctrine/ODM/PHPCR/Mapping/Driver/YamlDriver.php b/lib/Doctrine/ODM/PHPCR/Mapping/Driver/YamlDriver.php
@@ -21,6 +21,7 @@
 
 use Doctrine\Common\Persistence\Mapping\Driver\FileDriver;
 use Doctrine\Common\Persistence\Mapping\ClassMetadata;
+use Doctrine\ODM\PHPCR\Exception\InvalidArgumentException;
 use Doctrine\ODM\PHPCR\Mapping\MappingException;
 use Doctrine\Common\Persistence\Mapping\MappingException as DoctrineMappingException;
 use Symfony\Component\Yaml\Yaml;
@@ -270,7 +271,10 @@ private function addMappingFromReference(ClassMetadata $class, $fieldName, $refe
      */
     protected function loadMappingFile($file)
     {
-        return Yaml::parse($file);
+        if (!is_file($file)) {
+            throw new InvalidArgumentException(sprintf('File "%s" not found', $file));
+        }
+        return Yaml::parse(file_get_contents($file));
     }
 
     /**

diff --git a/tests/bootstrap.php b/tests/bootstrap.php
@@ -7,6 +7,14 @@
     throw new RuntimeException('Install dependencies to run test suite.');
 }
 
+$files = array_filter(array(
+    __DIR__.'/../vendor/symfony/symfony/src/Symfony/Bridge/PhpUnit/bootstrap.php',
+    __DIR__.'/../vendor/symfony/phpunit-bridge/bootstrap.php',
+), 'file_exists');
+if ($files) {
+    require_once current($files);
+}
+
 use Doctrine\Common\Annotations\AnnotationRegistry;
 AnnotationRegistry::registerLoader(array($autoload, 'loadClass'));
 AnnotationRegistry::registerFile(__DIR__.'/../lib/Doctrine/ODM/PHPCR/Mapping/Annotations/DoctrineAnnotations.php');