SimpleSAML\Test\XML\SerializableElementTestTrait::testSerialization()

Complexity

Conditions	3
Paths	3

Size

Total Lines	18
Code Lines	13

Duplication

Lines	0
Ratio	0 %

Importance

Changes

0

<?php
    public function testSerialization(): void
    {
        if (!class_exists($this->testedClass)) {
            $this->markTestSkipped(

It seems like markTestSkipped() must be provided by classes using this trait. How about adding it as abstract method to this trait?

                'Unable to run ' . self::class . '::testSerialization(). Please set ' . self::class
                . ':$testedClass to a class-string representing the XML-class being tested',
            );
        } elseif (empty($this->xmlRepresentation)) {
            $this->markTestSkipped(
                'Unable to run ' . self::class . '::testSerialization(). Please set ' . self::class
                . ':$xmlRepresentation to a DOMDocument representing the XML-class being tested',
            );
        } else {
            /** @psalm-var \DOMElement */
            $xmlRepresentationDocument = $this->xmlRepresentation->documentElement;
            $this->assertEquals(

It seems like assertEquals() must be provided by classes using this trait. How about adding it as abstract method to this trait?

                $this->xmlRepresentation->saveXML($xmlRepresentationDocument),
                strval(unserialize(serialize($this->testedClass::fromXML($xmlRepresentationDocument)))),
            );