WMDE\HamcrestHtml\ChildElementMatcher

Complexity

Total Complexity

13

Size/Duplication

Total Lines	74
Duplicated Lines	14.86 %

Coupling/Cohesion

Components	1
Dependencies	4

Test Coverage

Coverage

71.43%

Importance

Changes

0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	havingChild()	0	3	1
A	__construct()	0	4	1
A	describeTo()	10	10	3
C	matchesSafelyWithDiagnosticDescription()	0	31	8

<?php

namespace WMDE\HamcrestHtml;

use Hamcrest\Description;
use Hamcrest\Matcher;
use Hamcrest\TypeSafeDiagnosingMatcher;

class ChildElementMatcher extends TypeSafeDiagnosingMatcher {

	/**
	 * @var Matcher|string|null
	 */
	private $matcher;

	/**
	 * @param Matcher|string|null $elementMatcher
	 *
	 * @return self
	 */
	public static function havingChild( $elementMatcher = null ) {
		return new static( $elementMatcher );
	}

	/**
	 * @param Matcher|string|null $matcher
	 */
	public function __construct( $matcher = null ) {
		parent::__construct( \DOMNode::class );
		$this->matcher = $matcher;
	}

	public function describeTo( Description $description ) {

This method seems to be duplicated in your project.

		$description->appendText( 'having child ' );
		if ( $this->matcher ) {
			if ( $this->matcher instanceof Matcher ) {
				$description->appendDescriptionOf( $this->matcher );
			} else {
				$description->appendValue( $this->matcher );
			}
		}
	}

	/**
	 * @param \DOMDocument|\DOMNode $item
	 * @param Description $mismatchDescription
	 *
	 * @return bool
	 */
	protected function matchesSafelyWithDiagnosticDescription( $item, Description $mismatchDescription ) {
		if ( $item instanceof \DOMDocument ) {
			$item = $item->documentElement->childNodes->item( 0 );
			/** @var \DOMElement $item */
		}
		$directChildren = $item->childNodes;

		if ( $directChildren->length === 0 ) {
			$mismatchDescription->appendText( 'having no children' );
			return false;
		}

		if ( !$this->matcher ) {
			return true;
		}

		if ( $this->matcher instanceof Matcher ) {
			foreach ( new XmlNodeRecursiveIterator( $directChildren ) as $child ) {
				if ( $this->matcher->matches( $child ) ) {
					return true;
				}
			}
		} else {
			if ( $item->getElementsByTagName( $this->matcher )->length !== 0 ) {
				return true;
			}
		}

		$mismatchDescription->appendText( 'having no children ' )->appendDescriptionOf( $this->matcher );

It seems like $this->matcher can also be of type string; however, Hamcrest\Description::appendDescriptionOf() does only seem to accept object<Hamcrest\SelfDescribing>, maybe add an additional type check?

		return false;
	}

}