Add --logger-html='file.html' CLI option for HTML report

infection · Jan 7, 2022 · e46b4b2 · e46b4b2
1 parent 544efe4
commit e46b4b2
Show file tree

Hide file tree

Showing 7 changed files with 165 additions and 2 deletions.
diff --git a/src/Command/RunCommand.php b/src/Command/RunCommand.php
@@ -118,6 +118,8 @@ final class RunCommand extends BaseCommand
     /** @var string */
     private const OPTION_LOGGER_GITHUB = 'logger-github';
 
+    private const OPTION_LOGGER_HTML = 'logger-html';
+
     private const OPTION_USE_NOOP_MUTATORS = 'noop';
 
     private const OPTION_EXECUTE_ONLY_COVERING_TEST_CASES = 'only-covering-test-cases';
@@ -264,6 +266,12 @@ protected function configure(): void
                 InputOption::VALUE_NONE,
                 'Log escaped Mutants as GitHub Annotations.',
             )
+            ->addOption(
+                self::OPTION_LOGGER_HTML,
+                null,
+                InputOption::VALUE_REQUIRED,
+                'Path to HTML report file, similar to PHPUnit HTML report.',
+            )
             ->addOption(
                 self::OPTION_USE_NOOP_MUTATORS,
                 null,
@@ -388,6 +396,7 @@ private function createContainer(IO $io, LoggerInterface $logger): Container
         $testFramework = trim((string) $input->getOption(self::OPTION_TEST_FRAMEWORK));
         $testFrameworkExtraOptions = trim((string) $input->getOption(self::OPTION_TEST_FRAMEWORK_OPTIONS));
         $initialTestsPhpOptions = trim((string) $input->getOption(self::OPTION_INITIAL_TESTS_PHP_OPTIONS));
+        $htmlFileLogPath = trim((string) $input->getOption(self::OPTION_LOGGER_HTML));
 
         /** @var string|null $minMsi */
         $minMsi = $input->getOption(self::OPTION_MIN_MSI);
@@ -473,6 +482,7 @@ private function createContainer(IO $io, LoggerInterface $logger): Container
             $isForGitDiffLines,
             $gitDiffBase,
             (bool) $input->getOption(self::OPTION_LOGGER_GITHUB),
+            $htmlFileLogPath === '' ? Container::DEFAULT_HTML_LOGGER_PATH : $htmlFileLogPath,
             (bool) $input->getOption(self::OPTION_USE_NOOP_MUTATORS),
             (bool) $input->getOption(self::OPTION_EXECUTE_ONLY_COVERING_TEST_CASES)
         );

diff --git a/src/Configuration/ConfigurationFactory.php b/src/Configuration/ConfigurationFactory.php
@@ -119,6 +119,7 @@ public function create(
         bool $isForGitDiffLines,
         ?string $gitDiffBase,
         bool $useGitHubLogger,
+        ?string $htmlLogFilePath,
         bool $useNoopMutators,
         bool $executeOnlyCoveringTestCases
     ): Configuration {
@@ -150,7 +151,7 @@ public function create(
             ),
             $this->retrieveFilter($filter, $gitDiffFilter, $isForGitDiffLines, $gitDiffBase),
             $schema->getSource()->getExcludes(),
-            $this->retrieveLogs($schema->getLogs(), $useGitHubLogger),
+            $this->retrieveLogs($schema->getLogs(), $useGitHubLogger, $htmlLogFilePath),
             $logVerbosity,
             $namespacedTmpDir,
             $this->retrievePhpUnit($schema, $configDir),
@@ -315,12 +316,16 @@ private function retrieveFilter(string $filter, ?string $gitDiffFilter, bool $is
         return $this->gitDiffFileProvider->provide($gitDiffFilter, $baseBranch);
     }
 
-    private function retrieveLogs(Logs $logs, bool $useGitHubLogger): Logs
+    private function retrieveLogs(Logs $logs, bool $useGitHubLogger, ?string $htmlLogFilePath): Logs
     {
         if ($useGitHubLogger) {
             $logs->setUseGitHubAnnotationsLogger($useGitHubLogger);
         }
 
+        if ($htmlLogFilePath !== null) {
+            $logs->setHtmlLogFilePath($htmlLogFilePath);
+        }
+
         return $logs;
     }
 }
diff --git a/src/Configuration/Entry/Logs.php b/src/Configuration/Entry/Logs.php
@@ -94,6 +94,11 @@ public function getHtmlLogFilePath(): ?string
         return $this->htmlLogFilePath;
     }
 
+    public function setHtmlLogFilePath(string $htmlLogFilePath): void
+    {
+        $this->htmlLogFilePath = $htmlLogFilePath;
+    }
+
     public function getSummaryLogFilePath(): ?string
     {
         return $this->summaryLogFilePath;

diff --git a/src/Container.php b/src/Container.php
@@ -170,6 +170,7 @@ final class Container
     public const DEFAULT_GIT_DIFF_LINES = false;
     public const DEFAULT_GIT_DIFF_BASE = null;
     public const DEFAULT_USE_GITHUB_LOGGER = false;
+    public const DEFAULT_HTML_LOGGER_PATH = null;
     public const DEFAULT_USE_NOOP_MUTATORS = false;
     public const DEFAULT_EXECUTE_ONLY_COVERING_TEST_CASES = false;
     public const DEFAULT_NO_PROGRESS = false;
@@ -710,6 +711,7 @@ public static function create(): self
             self::DEFAULT_GIT_DIFF_LINES,
             self::DEFAULT_GIT_DIFF_BASE,
             self::DEFAULT_USE_GITHUB_LOGGER,
+            self::DEFAULT_HTML_LOGGER_PATH,
             self::DEFAULT_USE_NOOP_MUTATORS,
             self::DEFAULT_EXECUTE_ONLY_COVERING_TEST_CASES
         );
@@ -743,6 +745,7 @@ public function withValues(
         bool $isForGitDiffLines,
         ?string $gitDiffBase,
         bool $useGitHubLogger,
+        ?string $htmlLogFilePath,
         bool $useNoopMutators,
         bool $executeOnlyCoveringTestCases
     ): self {
@@ -820,6 +823,7 @@ static function (self $container) use (
                 $isForGitDiffLines,
                 $gitDiffBase,
                 $useGitHubLogger,
+                $htmlLogFilePath,
                 $useNoopMutators,
                 $executeOnlyCoveringTestCases
             ): Configuration {
@@ -847,6 +851,7 @@ static function (self $container) use (
                     $isForGitDiffLines,
                     $gitDiffBase,
                     $useGitHubLogger,
+                    $htmlLogFilePath,
                     $useNoopMutators,
                     $executeOnlyCoveringTestCases
                 );

diff --git a/tests/benchmark/Tracing/provide-traces-closure.php b/tests/benchmark/Tracing/provide-traces-closure.php
@@ -71,6 +71,7 @@
     Container::DEFAULT_GIT_DIFF_LINES,
     Container::DEFAULT_GIT_DIFF_BASE,
     Container::DEFAULT_USE_GITHUB_LOGGER,
+    Container::DEFAULT_HTML_LOGGER_PATH,
     true,
     Container::DEFAULT_EXECUTE_ONLY_COVERING_TEST_CASES
 );