Empty <coverage> element in XML configuration after migrating configuration

[arogachev]

Q	A
PHPUnit version	10.5.2
PHP version	8.2.12
Installation Method	Composer / PHAR

Summary

There is empty <coverage/> tag after migrating configuration which seems to be useless.

Current behavior

Empty tag is present. New config.

<?xml version="1.0" encoding="UTF-8"?>
<phpunit xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" bootstrap="vendor/autoload.php" colors="true" failOnRisky="true" failOnWarning="true" stopOnFailure="false" executionOrder="random" resolveDependencies="true" xsi:noNamespaceSchemaLocation="https://schema.phpunit.de/10.5/phpunit.xsd" cacheDirectory=".phpunit.cache">
  <php>
    <ini name="error_reporting" value="-1"/>
  </php>
  <testsuites>
    <testsuite name="Yii Role-Based Access Control tests">
      <directory>./tests</directory>
    </testsuite>
  </testsuites>
  <coverage/> <!-- Here it is.  -->
  <source>
    <include>
      <directory>./src</directory>
    </include>
  </source>
</phpunit>

How to reproduce

Old config (phpunit.xml.dist):

<?xml version="1.0" encoding="UTF-8"?>

<phpunit
    bootstrap="vendor/autoload.php"
    colors="true"
    verbose="true"
    failOnRisky="true"
    failOnWarning="true"
    convertErrorsToExceptions="true"
    convertNoticesToExceptions="true"
    convertWarningsToExceptions="true"
    stopOnFailure="false"
    executionOrder="random"
    resolveDependencies="true"
>
    <php>
        <ini name="error_reporting" value="-1"/>
    </php>

    <testsuites>
        <testsuite name="Yii Role-Based Access Control tests">
            <directory>./tests</directory>
        </testsuite>
    </testsuites>

    <coverage>
        <include>
            <directory>./src</directory>
        </include>
    </coverage>
</phpunit>

vendor/bin/phpunit --migrate-configuration

Expected behavior

No empty tag.

[sebastianbergmann]

I would not consider this a bug, but yes, this can be improved.

[marac19901990]

I made a pull request for this issue:
#5626