Catch error if file content could not be parsed

Author: Andreas Frömer

CHANGELOG.md

@@ -2,6 +2,8 @@
 
 ## [Unreleased] - TBA
 ### Fixed
+- Fixed `FileContentProvider` to throw an exception when the file does not exist
+- Added `try/catch` to `FileSymbolProvider` to continue working instead of crashing if a file could not be parsed
 ### Added
 ### Changed
 ### Removed

src/Exception/IOException.php

@@ -12,4 +12,9 @@ public static function unableToOpenHandle(string $path): self
     {
         return new self('Unable to open resource ' . $path);
     }
+
+    public static function fileDoesNotExist(string $path): self
+    {
+        return new self('File does not exist: ' . $path);
+    }
 }

src/Parser/PHP/AbstractCollector.php

@@ -10,7 +10,7 @@
 
 abstract class AbstractCollector extends NodeVisitorAbstract implements SymbolCollectorInterface
 {
-    private ?Closure $includeCallback;
+    private ?Closure $includeCallback = null;
 
     public function setFileIncludeCallback(Closure $includeCallback): void
     {

src/Symbol/Provider/FileSymbolProvider.php

@@ -37,11 +37,16 @@ public function __construct(SymbolNameParserInterface $parser, FileContentProvid
     public function provide(): Generator
     {
         foreach ($this->fileIterator as $file) {
-            $content = $this->fileContentProvider->getContent($file);
-            $this->parser->setCurrentFile($file);
-
-            foreach ($this->parser->parseSymbolNames($content) as $symbolName) {
-                yield $symbolName => new Symbol($symbolName);
+            try {
+                $content = $this->fileContentProvider->getContent($file);
+                $this->parser->setCurrentFile($file);
+
+                foreach ($this->parser->parseSymbolNames($content) as $symbolName) {
+                    yield $symbolName => new Symbol($symbolName);
+                }
+            } catch (IOException $exception) {
+                // TODO add logging
+                continue;
             }
         }