Merge pull request #11 from mzk/service-map

catch php warning: XmlContainer

Author: lookyman

src/Exception/XmlContainerNotExistsException.php

@@ -0,0 +1,10 @@
+<?php
+
+declare(strict_types = 1);
+
+namespace Lookyman\PHPStan\Symfony\Exception;
+
+final class XmlContainerNotExistsException extends \InvalidArgumentException
+{
+
+}

src/ServiceMap.php

@@ -4,6 +4,7 @@
 
 namespace Lookyman\PHPStan\Symfony;
 
+use Lookyman\PHPStan\Symfony\Exception\XmlContainerNotExistsException;
 use PhpParser\Node;
 use PhpParser\Node\Expr\ClassConstFetch;
 use PhpParser\Node\Name;
@@ -21,7 +22,11 @@ public function __construct(string $containerXml)
 	{
 		$this->services = $aliases = [];
 		/** @var \SimpleXMLElement $def */
-		foreach (\simplexml_load_file($containerXml)->services->service as $def) {
+		$xml = @\simplexml_load_file($containerXml);
+		if ($xml === false) {
+			throw new XmlContainerNotExistsException(\sprintf('Container %s not exists', $containerXml));
+		}
+		foreach ($xml->services->service as $def) {
 			$attrs = $def->attributes();
 			if (!isset($attrs->id)) {
 				continue;

tests/ServiceMapTest.php

@@ -24,6 +24,14 @@ public function testGetServiceFromNode(array $service)
 		self::assertEquals($service, $serviceMap->getServiceFromNode(new String_($service['id'])));
 	}
 
+	/**
+	 * @expectedException \Lookyman\PHPStan\Symfony\Exception\XmlContainerNotExistsException
+	 */
+	public function testFileNotExists()
+	{
+		new ServiceMap(__DIR__ . '/foo.xml');
+	}
+
 	public function getServiceFromNodeProvider(): array
 	{
 		return [