note

Author: TomasVotruba

src/Analyzer/DuplicatedScenarioTitlesAnalyzer.php

@@ -4,6 +4,7 @@
 
 namespace Rector\Behastan\Analyzer;
 
+use Behat\Gherkin\Node\FeatureNode;
 use Entropy\Attributes\RelatedTest;
 use Rector\Behastan\Gherkin\GherkinParser;
 use Rector\Behastan\Tests\Analyzer\DuplicatedScenarioNamesAnalyzer\DuplicatedScenarioTitlesAnalyzerTest;
@@ -28,6 +29,11 @@ public function analyze(array $featureFiles): array
         foreach ($featureFiles as $featureFile) {
             $featureGherkin = $this->gherkinParser->parseFile($featureFile->getRealPath());
 
+            // @todo test and improve here
+            if (! $featureGherkin instanceof FeatureNode) {
+                continue;
+            }
+
             foreach ($featureGherkin->getScenarios() as $scenario) {
                 $scenarioNamesToFiles[$scenario->getTitle()][] = $featureFile->getRealPath();
             }

src/Gherkin/GherkinParser.php

@@ -35,13 +35,10 @@ public function __construct()
         $this->parser = new Parser(new Lexer($arrayKeywords));
     }
 
-    public function parseFile(string $filePath): FeatureNode
+    public function parseFile(string $filePath): ?FeatureNode
     {
         $fileContents = FileSystem::read($filePath);
 
-        $featureNode = $this->parser->parse($fileContents);
-        Assert::isInstanceOf($featureNode, FeatureNode::class);
-
-        return $featureNode;
+        return $this->parser->parse($fileContents);
     }
 }