Create MutationExecutionResult instance with real path always, not …

…mixed (#1652)

Fixes #1650

Previously, we could have a mix of real paths and relative paths in `MutationExecutionResult` instances. This broke HTML logger, which assumes that all paths are real (absolute), to find the base common path (`Path::getLongestCommonBasePath`).

src/Logger/Html/StrykerHtmlReportBuilder.php

@@ -166,6 +166,9 @@ private function getFiles(): ArrayObject
 
         if ($this->metricsCalculator->getTotalMutantsCount() !== 0) {
             $resultsByPath = $this->retrieveResultsByPath();
+
+            Assert::minCount($resultsByPath, 1, 'There must be at least one result to build HTML report.');
+
             $basePath = Path::getLongestCommonBasePath(array_keys($resultsByPath));
 
             Assert::string($basePath, '$basePath must be a string');

src/Mutation/FileMutationGenerator.php

@@ -97,14 +97,12 @@ public function generate(
             return;
         }
 
-        $fileInfo = $trace->getSourceFileInfo();
-
-        $initialStatements = $this->parser->parse($fileInfo);
+        $initialStatements = $this->parser->parse($trace->getSourceFileInfo());
 
         $mutationCollectorVisitor = new MutationCollectorVisitor(
             new NodeMutationGenerator(
                 $mutators,
-                $fileInfo->getPathname(),
+                $trace->getRealPath(),
                 $initialStatements,
                 $trace,
                 $onlyCovered,

tests/phpunit/Mutation/FileMutationGeneratorTest.php

@@ -114,6 +114,11 @@ public function test_it_generates_mutations_for_a_given_file(): void
             ->method('getAllTestsForMutation')
             ->willReturn([])
         ;
+        $traceMock
+            ->expects($this->once())
+            ->method('getRealPath')
+            ->willReturn(self::FIXTURES_DIR . '/Mutation/OneFile/OneFile.php')
+        ;
 
         $mutationGenerator = SingletonContainer::getContainer()->getFileMutationGenerator();