More work on merge error handling

Author: sebastianbergmann

src/cli/MergeCommand.php

@@ -29,6 +29,12 @@ public function run(Arguments $arguments): int
             return 1;
         }
 
+        if (!$arguments->reportConfigured()) {
+            print 'No code coverage report configured' . PHP_EOL;
+
+            return 1;
+        }
+
         $files = (new Facade)->getFilesAsArray(
             $arguments->directory(),
             ['.cov']

tests/end-to-end/merge/directory-without-cov-files.phpt

@@ -1,13 +1,15 @@
 --TEST--
-phpcov merge ../../fixture/empty-directory
+phpcov merge --text coverage.txt ../../fixture/empty-directory
 --INI--
 xdebug.overload_var_dump=0
 --FILE--
 <?php declare(strict_types=1);
 require __DIR__ . '/../../../vendor/autoload.php';
 
 $_SERVER['argv'][1] = 'merge';
-$_SERVER['argv'][2] = __DIR__ . '/../../fixture/empty-directory';
+$_SERVER['argv'][2] = '--text';
+$_SERVER['argv'][3] = 'coverage.txt';
+$_SERVER['argv'][4] = __DIR__ . '/../../fixture/empty-directory';
 
 var_dump((new SebastianBergmann\PHPCOV\Application)->run($_SERVER['argv']));
 --EXPECTF--

tests/end-to-end/merge/valid-directory-without-report.phpt

@@ -0,0 +1,17 @@
+--TEST--
+phpcov merge --text /tmp/file ../../fixture/example/coverage
+--INI--
+xdebug.overload_var_dump=0
+--FILE--
+<?php declare(strict_types=1);
+require __DIR__ . '/../../../vendor/autoload.php';
+
+$_SERVER['argv'][1] = 'merge';
+$_SERVER['argv'][2] = __DIR__ . '/../../fixture/example/coverage';
+
+var_dump((new SebastianBergmann\PHPCOV\Application)->run($_SERVER['argv']));
+--EXPECTF--
+phpcov %s by Sebastian Bergmann.
+
+No code coverage report configured
+int(1)