Skip to content

Fix PHPUnit coverage report generation #253

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 4 commits into from
Dec 1, 2025
Merged

Fix PHPUnit coverage report generation #253

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
4 commits
Select commit Hold shift + click to select a range
9407847
Fix PHPUnit coverage report generation
mspirkov Dec 1, 2025
c90a789
Update PHPUnit config
mspirkov Dec 1, 2025
d7bad3f
Fix CS
mspirkov Dec 1, 2025
aede3ab
Fix CS
mspirkov Dec 1, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
23 changes: 10 additions & 13 deletions phpunit.xml.dist
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,28 +1,25 @@
<?xml version="1.0" encoding="utf-8"?>

<phpunit
bootstrap="vendor/autoload.php"
colors="true"
verbose="true"
convertDeprecationsToExceptions="false"
forceCoversAnnotation="true"
>
<coverage>
<include>
<directory suffix=".php">src</directory>
</include>
<report>
<clover outputFile="build/logs/clover.xml" />
<html outputDirectory="build/coverage" lowUpperBound="35" highLowerBound="70" />
</report>
</coverage>
<testsuites>
<testsuite name="unit">
<directory>./tests/unit/</directory>
</testsuite>
</testsuites>
<filter>
<whitelist>
<directory suffix=".php">src</directory>
</whitelist>
</filter>
<logging>
<log type="coverage-html"
target="build/coverage"
lowUpperBound="35"
highLowerBound="70" />
<log type="coverage-clover" target="build/logs/clover.xml"/>
<log type="junit" target="build/logs/junit.xml"/>
<junit outputFile="build/logs/junit.xml" />
</logging>
</phpunit>
63 changes: 0 additions & 63 deletions tests/unit/CollectionResolverTest.php
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -27,20 +27,13 @@
use PHPUnit\Framework\TestCase;
use RuntimeException;

/**
* @coversDefaultClass \phpDocumentor\Reflection\TypeResolver
*/
class CollectionResolverTest extends TestCase
{
/**
* @uses \phpDocumentor\Reflection\Types\Context
* @uses \phpDocumentor\Reflection\Types\Compound
* @uses \phpDocumentor\Reflection\PseudoTypes\Generic
* @uses \phpDocumentor\Reflection\Types\String_
*
* @covers ::resolve
* @covers ::createType
* @covers ::__construct
*/
public function testResolvingCollection(): void
{
Expand All @@ -59,10 +52,6 @@ public function testResolvingCollection(): void
* @uses \phpDocumentor\Reflection\Types\Compound
* @uses \phpDocumentor\Reflection\PseudoTypes\Generic
* @uses \phpDocumentor\Reflection\Types\String_
*
* @covers ::__construct
* @covers ::resolve
* @covers ::createType
*/
public function testResolvingCollectionWithKeyType(): void
{
Expand All @@ -86,10 +75,6 @@ public function testResolvingCollectionWithKeyType(): void
* @uses \phpDocumentor\Reflection\Types\Context
* @uses \phpDocumentor\Reflection\Types\Compound
* @uses \phpDocumentor\Reflection\Types\String_
*
* @covers ::__construct
* @covers ::resolve
* @covers ::createType
*/
public function testResolvingArrayCollection(): void
{
Expand All @@ -112,10 +97,6 @@ public function testResolvingArrayCollection(): void
* @uses \phpDocumentor\Reflection\Types\Context
* @uses \phpDocumentor\Reflection\Types\Compound
* @uses \phpDocumentor\Reflection\Types\String_
*
* @covers ::__construct
* @covers ::resolve
* @covers ::createType
*/
public function testResolvingArrayCollectionWithKey(): void
{
Expand All @@ -138,10 +119,6 @@ public function testResolvingArrayCollectionWithKey(): void
* @uses \phpDocumentor\Reflection\Types\Context
* @uses \phpDocumentor\Reflection\Types\Compound
* @uses \phpDocumentor\Reflection\Types\String_

* @covers ::__construct
* @covers ::resolve
* @covers ::createType
*/
public function testResolvingArrayCollectionWithKeyAndWhitespace(): void
{
Expand All @@ -165,10 +142,6 @@ public function testResolvingArrayCollectionWithKeyAndWhitespace(): void
* @uses \phpDocumentor\Reflection\Types\Compound
* @uses \phpDocumentor\Reflection\PseudoTypes\Generic
* @uses \phpDocumentor\Reflection\Types\String_
*
* @covers ::__construct
* @covers ::resolve
* @covers ::createType
*/
public function testResolvingCollectionOfCollection(): void
{
Expand All @@ -194,11 +167,6 @@ public function testResolvingCollectionOfCollection(): void
$this->assertEquals(new Object_(new Fqsen('\\DateTime')), $nestedGenericTypes[0]);
}

/**
* @covers ::__construct
* @covers ::resolve
* @covers ::createType
*/
public function testGoodArrayCollectionKey(): void
{
$fixture = new TypeResolver();
Expand All @@ -214,11 +182,6 @@ public function testGoodArrayCollectionKey(): void
$this->assertSame('array<class-string,string>', (string) $resolvedType);
}

/**
* @covers ::__construct
* @covers ::resolve
* @covers ::createType
*/
public function testMissingStartCollection(): void
{
$this->expectException(RuntimeException::class);
Expand All @@ -227,11 +190,6 @@ public function testMissingStartCollection(): void
$fixture->resolve('<string>', new Context(''));
}

/**
* @covers ::__construct
* @covers ::resolve
* @covers ::createType
*/
public function testMissingEndCollection(): void
{
$this->expectException(RuntimeException::class);
Expand All @@ -240,11 +198,6 @@ public function testMissingEndCollection(): void
$fixture->resolve('ArrayObject<object|string', new Context(''));
}

/**
* @covers ::__construct
* @covers ::resolve
* @covers ::createType
*/
public function testBadCollectionClass(): void
{
$this->expectException(RuntimeException::class);
Expand All @@ -257,10 +210,6 @@ public function testBadCollectionClass(): void
* @uses \phpDocumentor\Reflection\Types\Context
* @uses \phpDocumentor\Reflection\Types\Compound
* @uses \phpDocumentor\Reflection\Types\String_
*
* @covers ::__construct
* @covers ::resolve
* @covers ::createType
*/
public function testResolvingCollectionAsArray(): void
{
Expand All @@ -282,10 +231,6 @@ public function testResolvingCollectionAsArray(): void
/**
* @uses \phpDocumentor\Reflection\Types\Context
* @uses \phpDocumentor\Reflection\Types\String_
*
* @covers ::__construct
* @covers ::resolve
* @covers ::createType
*/
public function testResolvingList(): void
{
Expand All @@ -307,10 +252,6 @@ public function testResolvingList(): void
/**
* @uses \phpDocumentor\Reflection\Types\Context
* @uses \phpDocumentor\Reflection\Types\String_
*
* @covers ::__construct
* @covers ::resolve
* @covers ::createType
*/
public function testResolvingNonEmptyList(): void
{
Expand All @@ -332,10 +273,6 @@ public function testResolvingNonEmptyList(): void
/**
* @uses \phpDocumentor\Reflection\Types\Context
* @uses \phpDocumentor\Reflection\Types\Nullable
*
* @covers ::__construct
* @covers ::resolve
* @covers ::createType
*/
public function testResolvingNullableArray(): void
{
Expand Down
21 changes: 0 additions & 21 deletions tests/unit/FqsenResolverTest.php
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -17,14 +17,8 @@
use phpDocumentor\Reflection\Types\Context;
use PHPUnit\Framework\TestCase;

/**
* @coversDefaultClass \phpDocumentor\Reflection\FqsenResolver
*/
final class FqsenResolverTest extends TestCase
{
/**
* @covers ::resolve
*/
public function testResolveFqsen(): void
{
$fqsenResolver = new FqsenResolver();
Expand All @@ -35,9 +29,6 @@ public function testResolveFqsen(): void
static::assertSame('\DocBlock', (string) $result);
}

/**
* @covers ::resolve
*/
public function testResolveFqsenWithEmoji(): void
{
$fqsenResolver = new FqsenResolver();
Expand All @@ -48,9 +39,6 @@ public function testResolveFqsenWithEmoji(): void
static::assertSame('\My😁DocBlock', (string) $result);
}

/**
* @covers ::resolve
*/
public function testResolveWithoutContext(): void
{
$fqsenResolver = new FqsenResolver();
Expand All @@ -59,9 +47,6 @@ public function testResolveWithoutContext(): void
static::assertSame('\DocBlock', (string) $result);
}

/**
* @covers ::resolve
*/
public function testResolveFromAlias(): void
{
$fqsenResolver = new FqsenResolver();
Expand All @@ -72,9 +57,6 @@ public function testResolveFromAlias(): void
static::assertSame('\some\other\ns', (string) $result);
}

/**
* @covers ::resolve
*/
public function testResolveFromPartialAlias(): void
{
$fqsenResolver = new FqsenResolver();
Expand All @@ -85,9 +67,6 @@ public function testResolveFromPartialAlias(): void
static::assertSame('\some\other\ns', (string) $result);
}

/**
* @covers ::resolve
*/
public function testResolveThrowsExceptionWhenGarbageInputIsPassed(): void
{
$this->expectException(InvalidArgumentException::class);
Expand Down
27 changes: 0 additions & 27 deletions tests/unit/IntegerRangeResolverTest.php
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -18,19 +18,12 @@
use PHPUnit\Framework\TestCase;
use RuntimeException;

/**
* @coversDefaultClass \phpDocumentor\Reflection\TypeResolver
*/
class IntegerRangeResolverTest extends TestCase
{
/**
* @uses \phpDocumentor\Reflection\Types\Context
* @uses \phpDocumentor\Reflection\Types\Compound
* @uses \phpDocumentor\Reflection\Types\String_
*
* @covers ::__construct
* @covers ::resolve
* @covers ::createType
*/
public function testResolvingIntRange(): void
{
Expand All @@ -52,10 +45,6 @@ public function testResolvingIntRange(): void
* @uses \phpDocumentor\Reflection\Types\Context
* @uses \phpDocumentor\Reflection\Types\Compound
* @uses \phpDocumentor\Reflection\Types\String_
*
* @covers ::__construct
* @covers ::resolve
* @covers ::createType
*/
public function testResolvingIntRangeWithKeywords(): void
{
Expand All @@ -77,10 +66,6 @@ public function testResolvingIntRangeWithKeywords(): void
* @uses \phpDocumentor\Reflection\Types\Context
* @uses \phpDocumentor\Reflection\Types\Compound
* @uses \phpDocumentor\Reflection\Types\String_
*
* @covers ::__construct
* @covers ::resolve
* @covers ::createType
*/
public function testResolvingIntRangeErrorMissingMaxValue(): void
{
Expand All @@ -95,10 +80,6 @@ public function testResolvingIntRangeErrorMissingMaxValue(): void
* @uses \phpDocumentor\Reflection\Types\Context
* @uses \phpDocumentor\Reflection\Types\Compound
* @uses \phpDocumentor\Reflection\Types\String_
*
* @covers ::__construct
* @covers ::resolve
* @covers ::createType
*/
public function testResolvingIntRangeErrorMisingMinValue(): void
{
Expand All @@ -113,10 +94,6 @@ public function testResolvingIntRangeErrorMisingMinValue(): void
* @uses \phpDocumentor\Reflection\Types\Context
* @uses \phpDocumentor\Reflection\Types\Compound
* @uses \phpDocumentor\Reflection\Types\String_
*
* @covers ::__construct
* @covers ::resolve
* @covers ::createType
*/
public function testResolvingIntRangeErrorMisingComma(): void
{
Expand All @@ -131,10 +108,6 @@ public function testResolvingIntRangeErrorMisingComma(): void
* @uses \phpDocumentor\Reflection\Types\Context
* @uses \phpDocumentor\Reflection\Types\Compound
* @uses \phpDocumentor\Reflection\Types\String_
*
* @covers ::__construct
* @covers ::resolve
* @covers ::createType
*/
public function testResolvingIntRangeErrorMissingEnd(): void
{
Expand Down
5 changes: 0 additions & 5 deletions tests/unit/NumericResolverTest.php
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -20,17 +20,12 @@
use phpDocumentor\Reflection\Types\String_;
use PHPUnit\Framework\TestCase;

/**
* @coversDefaultClass \phpDocumentor\Reflection\TypeResolver
*/
class NumericResolverTest extends TestCase
{
/**
* @uses \phpDocumentor\Reflection\Types\Context
* @uses \phpDocumentor\Reflection\Types\Compound
* @uses \phpDocumentor\Reflection\Types\String_
*
* @covers ::__construct
*/
public function testResolvingIntRange(): void
{
Expand Down
9 changes: 0 additions & 9 deletions tests/unit/PseudoTypes/ArrayKeyTest.php
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -17,24 +17,15 @@
use phpDocumentor\Reflection\Types\String_;
use PHPUnit\Framework\TestCase;

/**
* @coversDefaultClass \phpDocumentor\Reflection\PseudoTypes\ArrayKey
*/
final class ArrayKeyTest extends TestCase
{
/**
* @covers ::__construct
* @covers ::__toString
*/
public function testArrayKeyCanBeConstructedAndStringifiedCorrectly(): void
{
$this->assertSame('array-key', (string) (new ArrayKey()));
}

/**
* @uses \phpDocumentor\Reflection\PseudoTypes\ArrayKey::__construct
*
* @covers ::getIterator
*/
public function testArrayKeyCanBeIterated(): void
{
Expand Down
Loading