Doctrine\Tests\Annotations\Metadata\MetadataCollectionTest::testCollectionInterface()

Complexity

Conditions	1
Paths	1

Size

Total Lines	32
Code Lines	22

Duplication

Lines	0
Ratio	0 %

Importance

Changes

0

<?php
    public function testCollectionInterface() : void
    {
        $foo = $this->createDummyMetadata('Foo');
        $bar = $this->createDummyMetadata('Bar');
        $baz = $this->createDummyMetadata('Baz');

        $collection = new MetadataCollection();

        self::assertCount(0, $collection);
        self::assertSame([], iterator_to_array($collection));

        $collection->add($foo, $bar);

        self::assertCount(2, $collection);
        self::assertArrayHasKey('Foo', $collection);
        self::assertSame($foo, $collection['Foo']);
        self::assertArrayHasKey('Bar', $collection);
        self::assertSame($bar, $collection['Bar']);
        self::assertSame([$foo, $bar], iterator_to_array($collection));

$collection of type ArrayAccess|array is incompatible with the type Traversable expected by parameter $iterator of iterator_to_array().

        $collection[] = $baz;

        self::assertCount(3, $collection);

It seems like $collection can also be of type ArrayAccess; however, parameter $haystack of PHPUnit\Framework\Assert::assertCount() does only seem to accept Countable|iterable, maybe add an additional type check?

        self::assertArrayHasKey('Baz', $collection);
        self::assertSame($baz, $collection['Baz']);
        self::assertSame([$foo, $bar, $baz], iterator_to_array($collection));

        unset($collection['Bar']);

        self::assertCount(2, $collection);
        self::assertArrayNotHasKey('Bar', $collection);
        self::assertSame([$foo, $baz], iterator_to_array($collection));
    }