wmde / hamcrest-html-matchers

src/ChildElementMatcher.php

Spacing +18 added lines, -18 removed lines

@@ -13,19 +13,19 @@ 
 	 */
 	private $matcher;
 
-	public static function havingChild( Matcher $elementMatcher = null ) {
-		return new static( $elementMatcher );
+	public static function havingChild(Matcher $elementMatcher = null) {
+		return new static($elementMatcher);
 	}
 
-	public function __construct( Matcher $matcher = null ) {
-		parent::__construct( \DOMNode::class );
+	public function __construct(Matcher $matcher = null) {
+		parent::__construct(\DOMNode::class);
 		$this->matcher = $matcher;
 	}
 
-	public function describeTo( Description $description ) {
-		$description->appendText( 'having child ' );
-		if ( $this->matcher ) {
-			$description->appendDescriptionOf( $this->matcher );
+	public function describeTo(Description $description) {
+		$description->appendText('having child ');
+		if ($this->matcher) {
+			$description->appendDescriptionOf($this->matcher);
 		}
 	}
 
@@ -35,32 +35,32 @@ 
 	 *
 	 * @return bool
 	 */
-	protected function matchesSafelyWithDiagnosticDescription( $item, Description $mismatchDescription ) {
-		if ( $item instanceof \DOMDocument ) {
-			$directChildren = iterator_to_array( $item->documentElement->childNodes );
+	protected function matchesSafelyWithDiagnosticDescription($item, Description $mismatchDescription) {
+		if ($item instanceof \DOMDocument) {
+			$directChildren = iterator_to_array($item->documentElement->childNodes);
 
-			$body = array_shift( $directChildren );
+			$body = array_shift($directChildren);
 			$directChildren = $body->childNodes;
 		} else {
 			$directChildren = $item->childNodes;
 		}
 
-		if ( $directChildren->length === 0 ) {
-			$mismatchDescription->appendText( 'having no children' );
+		if ($directChildren->length === 0) {
+			$mismatchDescription->appendText('having no children');
 			return false;
 		}
 
-		if ( !$this->matcher ) {
+		if (!$this->matcher) {
 			return $directChildren->length > 0;
 		}
 
-		foreach ( new XmlNodeRecursiveIterator( $directChildren ) as $child ) {
-			if ( $this->matcher && $this->matcher->matches( $child ) ) {
+		foreach (new XmlNodeRecursiveIterator($directChildren) as $child) {
+			if ($this->matcher && $this->matcher->matches($child)) {
 				return true;
 			}
 		}
 
-		$mismatchDescription->appendText( 'having no children ' )->appendDescriptionOf( $this->matcher );
+		$mismatchDescription->appendText('having no children ')->appendDescriptionOf($this->matcher);
 		return false;
 	}
 

src/TagMatcher.php

Spacing +1 added lines, -1 removed lines

@@ -7,7 +7,7 @@
 abstract class TagMatcher extends TypeSafeDiagnosingMatcher {
 
 	public function __construct() {
-		parent::__construct( self::TYPE_OBJECT, \DOMElement::class );
+		parent::__construct(self::TYPE_OBJECT, \DOMElement::class);
 	}
 
 }

src/RootElementMatcher.php

Spacing +21 added lines, -21 removed lines

@@ -18,19 +18,19 @@ 
 	 *
 	 * @return static
 	 */
-	public static function havingRootElement( Matcher $tagMatcher = null ) {
-		return new static( $tagMatcher );
+	public static function havingRootElement(Matcher $tagMatcher = null) {
+		return new static($tagMatcher);
 	}
 
-	public function __construct( Matcher $tagMatcher = null ) {
-		parent::__construct( self::TYPE_OBJECT, \DOMDocument::class );
+	public function __construct(Matcher $tagMatcher = null) {
+		parent::__construct(self::TYPE_OBJECT, \DOMDocument::class);
 		$this->tagMatcher = $tagMatcher;
 	}
 
-	public function describeTo( Description $description ) {
-		$description->appendText( 'having root element ' );
-		if ( $this->tagMatcher ) {
-			$description->appendDescriptionOf( $this->tagMatcher );
+	public function describeTo(Description $description) {
+		$description->appendText('having root element ');
+		if ($this->tagMatcher) {
+			$description->appendDescriptionOf($this->tagMatcher);
 		}
 	}
 
@@ -40,27 +40,27 @@ 
 	 *
 	 * @return bool
 	 */
-	protected function matchesSafelyWithDiagnosticDescription( $item, Description $mismatchDescription ) {
-		$DOMNodeList = iterator_to_array( $item->documentElement->childNodes );
+	protected function matchesSafelyWithDiagnosticDescription($item, Description $mismatchDescription) {
+		$DOMNodeList = iterator_to_array($item->documentElement->childNodes);
 
-		$body = array_shift( $DOMNodeList );
-		$DOMNodeList = iterator_to_array( $body->childNodes );
-		if ( count( $DOMNodeList ) > 1 ) {
+		$body = array_shift($DOMNodeList);
+		$DOMNodeList = iterator_to_array($body->childNodes);
+		if (count($DOMNodeList) > 1) {
 			// TODO Test this description
-			$mismatchDescription->appendText( 'having ' . count( $DOMNodeList ) . ' root elements ' );
+			$mismatchDescription->appendText('having ' . count($DOMNodeList) . ' root elements ');
 			return false;
 		}
 
-		$target = array_shift( $DOMNodeList );
-		if ( !$target ) {
+		$target = array_shift($DOMNodeList);
+		if (!$target) {
 			// TODO Reproduce?
-			$mismatchDescription->appendText( 'having no root elements ' );
+			$mismatchDescription->appendText('having no root elements ');
 			return false;
 		}
-		if ( $this->tagMatcher ) {
-			$mismatchDescription->appendText( 'root element ' );
-			$this->tagMatcher->describeMismatch( $target, $mismatchDescription );
-			return $this->tagMatcher->matches( $target );
+		if ($this->tagMatcher) {
+			$mismatchDescription->appendText('root element ');
+			$this->tagMatcher->describeMismatch($target, $mismatchDescription);
+			return $this->tagMatcher->matches($target);
 		}
 
 		return (bool)$target;

src/DirectChildElementMatcher.php

Spacing +20 added lines, -20 removed lines

@@ -13,19 +13,19 @@ 
 	 */
 	private $matcher;
 
-	public static function havingDirectChild( Matcher $elementMatcher = null ) {
-		return new static( $elementMatcher );
+	public static function havingDirectChild(Matcher $elementMatcher = null) {
+		return new static($elementMatcher);
 	}
 
-	public function __construct( Matcher $matcher = null ) {
-		parent::__construct( \DOMNode::class );
+	public function __construct(Matcher $matcher = null) {
+		parent::__construct(\DOMNode::class);
 		$this->matcher = $matcher;
 	}
 
-	public function describeTo( Description $description ) {
-		$description->appendText( 'having direct child ' );
-		if ( $this->matcher ) {
-			$description->appendDescriptionOf( $this->matcher );
+	public function describeTo(Description $description) {
+		$description->appendText('having direct child ');
+		if ($this->matcher) {
+			$description->appendDescriptionOf($this->matcher);
 		}
 	}
 
@@ -35,36 +35,36 @@ 
 	 *
 	 * @return bool
 	 */
-	protected function matchesSafelyWithDiagnosticDescription( $item, Description $mismatchDescription ) {
-		if ( $item instanceof \DOMDocument ) {
-			$directChildren = iterator_to_array( $item->documentElement->childNodes );
+	protected function matchesSafelyWithDiagnosticDescription($item, Description $mismatchDescription) {
+		if ($item instanceof \DOMDocument) {
+			$directChildren = iterator_to_array($item->documentElement->childNodes);
 
-			$body = array_shift( $directChildren );
+			$body = array_shift($directChildren);
 			$directChildren = $body->childNodes;
 		} else {
 			$directChildren = $item->childNodes;
 		}
 
-		if ( $directChildren->length === 0 ) {
-			$mismatchDescription->appendText( 'with no direct children' );
+		if ($directChildren->length === 0) {
+			$mismatchDescription->appendText('with no direct children');
 			return false;
 		}
 
 		$childWord = $directChildren->length === 1 ? 'child' : 'children';
 
-		$mismatchDescription->appendText( "with direct {$childWord} " );
+		$mismatchDescription->appendText("with direct {$childWord} ");
 
-		if ( !$this->matcher ) {
-			return count( $directChildren ) !== 0;
+		if (!$this->matcher) {
+			return count($directChildren) !== 0;
 		}
 
-		foreach ( $directChildren as $child ) {
-			if ( $this->matcher && $this->matcher->matches( $child ) ) {
+		foreach ($directChildren as $child) {
+			if ($this->matcher && $this->matcher->matches($child)) {
 				return true;
 			}
 		}
 
-		$this->matcher->describeMismatch( $child, $mismatchDescription );
+		$this->matcher->describeMismatch($child, $mismatchDescription);
 
 		return false;
 	}

src/XmlNodeRecursiveIterator.php

Spacing +7 added lines, -7 removed lines

@@ -4,9 +4,9 @@ 
 
 class XmlNodeRecursiveIterator extends \ArrayIterator {
 
-	public function __construct( \DOMNodeList $nodeList ) {
-		$queue = $this->addElementsToQueue( [], $nodeList );
-		parent::__construct( $queue );
+	public function __construct(\DOMNodeList $nodeList) {
+		$queue = $this->addElementsToQueue([], $nodeList);
+		parent::__construct($queue);
 	}
 
 	/**
@@ -15,12 +15,12 @@ 
 	 *
 	 * @return \DOMNode[] New queue
 	 */
-	private function addElementsToQueue( array $queue, \DOMNodeList $nodeList ) {
+	private function addElementsToQueue(array $queue, \DOMNodeList $nodeList) {
 		/** @var \DOMNode $node */
-		foreach ( $nodeList as $node ) {
+		foreach ($nodeList as $node) {
 			$queue[] = $node;
-			if ( $node->childNodes !== null ) {
-				$queue = $this->addElementsToQueue( $queue, $node->childNodes );
+			if ($node->childNodes !== null) {
+				$queue = $this->addElementsToQueue($queue, $node->childNodes);
 			}
 		}
 

src/HtmlMatcher.php

Spacing +30 added lines, -30 removed lines

@@ -24,61 +24,61 @@ 
 	 *
 	 * @return self
 	 */
-	public static function htmlPiece( Matcher $elementMatcher = null ) {
-		return new static( $elementMatcher );
+	public static function htmlPiece(Matcher $elementMatcher = null) {
+		return new static($elementMatcher);
 	}
 
-	private function __construct( Matcher $elementMatcher = null ) {
+	private function __construct(Matcher $elementMatcher = null) {
 		$this->elementMatcher = $elementMatcher;
 	}
 
-	public function describeTo( Description $description ) {
-		$description->appendText( 'valid html piece ' );
-		if ( $this->elementMatcher ) {
-			$description->appendDescriptionOf( $this->elementMatcher );
+	public function describeTo(Description $description) {
+		$description->appendText('valid html piece ');
+		if ($this->elementMatcher) {
+			$description->appendDescriptionOf($this->elementMatcher);
 		}
 	}
 
-	protected function matchesWithDiagnosticDescription( $html, Description $mismatchDescription ) {
-		$internalErrors = libxml_use_internal_errors( true );
+	protected function matchesWithDiagnosticDescription($html, Description $mismatchDescription) {
+		$internalErrors = libxml_use_internal_errors(true);
 		$document = new \DOMDocument();
 
-		$html = $this->escapeScriptTagContents( $html );
+		$html = $this->escapeScriptTagContents($html);
 
-		if ( !@$document->loadHTML( mb_convert_encoding( $html, 'HTML-ENTITIES', 'UTF-8' ) ) ) {
-			$mismatchDescription->appendText( 'there was some parsing error' );
+		if (!@$document->loadHTML(mb_convert_encoding($html, 'HTML-ENTITIES', 'UTF-8'))) {
+			$mismatchDescription->appendText('there was some parsing error');
 			return false;
 		}
 
 		$errors = libxml_get_errors();
 		libxml_clear_errors();
-		libxml_use_internal_errors( $internalErrors );
+		libxml_use_internal_errors($internalErrors);
 
 		$result = true;
 		/** @var \LibXMLError $error */
-		foreach ( $errors as $error ) {
-			if ( $this->isUnknownTagError( $error ) ) {
+		foreach ($errors as $error) {
+			if ($this->isUnknownTagError($error)) {
 				continue;
 			}
 
-			$mismatchDescription->appendText( 'there was parsing error: ' )
-				->appendText( trim( $error->message ) )
-				->appendText( ' on line ' )
-				->appendText( $error->line );
+			$mismatchDescription->appendText('there was parsing error: ')
+				->appendText(trim($error->message))
+				->appendText(' on line ')
+				->appendText($error->line);
 			$result = false;
 		}
 
-		if ( $result === false ) {
+		if ($result === false) {
 			return $result;
 		}
-		$mismatchDescription->appendText( 'valid html piece ' );
+		$mismatchDescription->appendText('valid html piece ');
 
-		if ( $this->elementMatcher ) {
-			$result = $this->elementMatcher->matches( $document );
-			$this->elementMatcher->describeMismatch( $document, $mismatchDescription );
+		if ($this->elementMatcher) {
+			$result = $this->elementMatcher->matches($document);
+			$this->elementMatcher->describeMismatch($document, $mismatchDescription);
 		}
 
-		$mismatchDescription->appendText( "\nActual html:\n" )->appendText( $html );
+		$mismatchDescription->appendText("\nActual html:\n")->appendText($html);
 
 		return $result;
 	}
@@ -88,7 +88,7 @@ 
 	 *
 	 * @return bool
 	 */
-	private function isUnknownTagError( \LibXMLError $error ) {
+	private function isUnknownTagError(\LibXMLError $error) {
 		return $error->code === self::XML_UNKNOWN_TAG_ERROR_CODE;
 	}
 
@@ -97,10 +97,10 @@ 
 	 *
 	 * @return string HTML
 	 */
-	private function escapeScriptTagContents( $html ) {
-		return preg_replace_callback( '#(<script.*>)(.*)(</script>)#isU', function ( $matches ) {
-			return $matches[1] . str_replace( '</', '<\/', $matches[2] ) . $matches[3];
-		}, $html );
+	private function escapeScriptTagContents($html) {
+		return preg_replace_callback('#(<script.*>)(.*)(</script>)#isU', function($matches) {
+			return $matches[1] . str_replace('</', '<\/', $matches[2]) . $matches[3];
+		}, $html);
 	}
 
 }

src/functions.php

Spacing +27 added lines, -27 removed lines

@@ -2,88 +2,88 @@
 
 use Hamcrest\Matcher;
 
-if ( !function_exists( 'htmlPiece' ) ) {
+if (!function_exists('htmlPiece')) {
 	/**
 	 * @param Matcher|null $elementMatcher
 	 *
 	 * @return \WMDE\HamcrestHtml\HtmlMatcher
 	 */
-	function htmlPiece( Matcher $elementMatcher = null ) {
-		return \WMDE\HamcrestHtml\HtmlMatcher::htmlPiece( $elementMatcher );
+	function htmlPiece(Matcher $elementMatcher = null) {
+		return \WMDE\HamcrestHtml\HtmlMatcher::htmlPiece($elementMatcher);
 	}
 }
 
-if ( !function_exists( 'havingRootElement' ) ) {
-	function havingRootElement( Matcher $matcher = null ) {
-		return \WMDE\HamcrestHtml\RootElementMatcher::havingRootElement( $matcher );
+if (!function_exists('havingRootElement')) {
+	function havingRootElement(Matcher $matcher = null) {
+		return \WMDE\HamcrestHtml\RootElementMatcher::havingRootElement($matcher);
 	}
 }
 
-if ( !function_exists( 'havingDirectChild' ) ) {
-	function havingDirectChild( Matcher $elementMatcher = null ) {
-		return \WMDE\HamcrestHtml\DirectChildElementMatcher::havingDirectChild( $elementMatcher );
+if (!function_exists('havingDirectChild')) {
+	function havingDirectChild(Matcher $elementMatcher = null) {
+		return \WMDE\HamcrestHtml\DirectChildElementMatcher::havingDirectChild($elementMatcher);
 	}
 }
 
-if ( !function_exists( 'havingChild' ) ) {
-	function havingChild( Matcher $elementMatcher = null ) {
-		return \WMDE\HamcrestHtml\ChildElementMatcher::havingChild( $elementMatcher );
+if (!function_exists('havingChild')) {
+	function havingChild(Matcher $elementMatcher = null) {
+		return \WMDE\HamcrestHtml\ChildElementMatcher::havingChild($elementMatcher);
 	}
 }
 
-if ( !function_exists( 'withTagName' ) ) {
+if (!function_exists('withTagName')) {
 	/**
 	 * @param Matcher|string $tagName
 	 *
 	 * @return \WMDE\HamcrestHtml\TagNameMatcher
 	 */
-	function withTagName( $tagName ) {
-		return \WMDE\HamcrestHtml\TagNameMatcher::withTagName( $tagName );
+	function withTagName($tagName) {
+		return \WMDE\HamcrestHtml\TagNameMatcher::withTagName($tagName);
 	}
 }
 
-if ( !function_exists( 'withAttribute' ) ) {
+if (!function_exists('withAttribute')) {
 	/**
 	 * @param Matcher|string $attributeName
 	 *
 	 * @return \WMDE\HamcrestHtml\AttributeMatcher
 	 */
-	function withAttribute( $attributeName ) {
-		return \WMDE\HamcrestHtml\AttributeMatcher::withAttribute( $attributeName );
+	function withAttribute($attributeName) {
+		return \WMDE\HamcrestHtml\AttributeMatcher::withAttribute($attributeName);
 	}
 }
 
-if ( !function_exists( 'withClass' ) ) {
+if (!function_exists('withClass')) {
 	/**
 	 * @param Matcher|string $class
 	 *
 	 * @return \WMDE\HamcrestHtml\ClassMatcher
 	 */
-	function withClass( $class ) {
+	function withClass($class) {
 		// TODO don't allow to call with empty string
 
-		return \WMDE\HamcrestHtml\ClassMatcher::withClass( $class );
+		return \WMDE\HamcrestHtml\ClassMatcher::withClass($class);
 	}
 }
 
-if ( !function_exists( 'havingTextContents' ) ) {
+if (!function_exists('havingTextContents')) {
 	/**
 	 * @param Matcher|string $text
 	 *
 	 * @return \WMDE\HamcrestHtml\TextContentsMatcher
 	 */
-	function havingTextContents( $text ) {
-		return \WMDE\HamcrestHtml\TextContentsMatcher::havingTextContents( $text );
+	function havingTextContents($text) {
+		return \WMDE\HamcrestHtml\TextContentsMatcher::havingTextContents($text);
 	}
 }
 
-if ( !function_exists( 'tagMatchingOutline' ) ) {
+if (!function_exists('tagMatchingOutline')) {
 	/**
 	 * @param string $htmlOutline
 	 *
 	 * @return \WMDE\HamcrestHtml\ComplexTagMatcher
 	 */
-	function tagMatchingOutline( $htmlOutline ) {
-		return \WMDE\HamcrestHtml\ComplexTagMatcher::tagMatchingOutline( $htmlOutline );
+	function tagMatchingOutline($htmlOutline) {
+		return \WMDE\HamcrestHtml\ComplexTagMatcher::tagMatchingOutline($htmlOutline);
 	}
 }

src/AttributeMatcher.php

Spacing +17 added lines, -17 removed lines

@@ -23,14 +23,14 @@ 
 	 *
 	 * @return self
 	 */
-	public static function withAttribute( $attributeName ) {
-		return new static( Util::wrapValueWithIsEqual( $attributeName ) );
+	public static function withAttribute($attributeName) {
+		return new static(Util::wrapValueWithIsEqual($attributeName));
 	}
 
 	/**
 	 * @param Matcher $attributeNameMatcher
 	 */
-	public function __construct( Matcher $attributeNameMatcher ) {
+	public function __construct(Matcher $attributeNameMatcher) {
 		parent::__construct();
 
 		$this->attributeNameMatcher = $attributeNameMatcher;
@@ -41,20 +41,20 @@ 
 	 *
 	 * @return AttributeMatcher
 	 */
-	public function havingValue( $value ) {
+	public function havingValue($value) {
 		// TODO: Throw exception if value is set
 		$result = clone $this;
-		$result->valueMatcher = Util::wrapValueWithIsEqual( $value );
+		$result->valueMatcher = Util::wrapValueWithIsEqual($value);
 
 		return $result;
 	}
 
-	public function describeTo( Description $description ) {
-		$description->appendText( 'with attribute ' )
-			->appendDescriptionOf( $this->attributeNameMatcher );
-		if ( $this->valueMatcher ) {
-			$description->appendText( ' having value ' )
-				->appendDescriptionOf( $this->valueMatcher );
+	public function describeTo(Description $description) {
+		$description->appendText('with attribute ')
+			->appendDescriptionOf($this->attributeNameMatcher);
+		if ($this->valueMatcher) {
+			$description->appendText(' having value ')
+				->appendDescriptionOf($this->valueMatcher);
 		}
 	}
 
@@ -64,18 +64,18 @@ 
 	 *
 	 * @return bool
 	 */
-	protected function matchesSafelyWithDiagnosticDescription( $item, Description $mismatchDescription ) {
+	protected function matchesSafelyWithDiagnosticDescription($item, Description $mismatchDescription) {
 		/** @var \DOMAttr $attribute */
-		foreach ( $item->attributes as $attribute ) {
-			if ( $this->valueMatcher ) {
+		foreach ($item->attributes as $attribute) {
+			if ($this->valueMatcher) {
 				if (
-					$this->attributeNameMatcher->matches( $attribute->name )
-					&& $this->valueMatcher->matches( $attribute->value )
+					$this->attributeNameMatcher->matches($attribute->name)
+					&& $this->valueMatcher->matches($attribute->value)
 				) {
 					return true;
 				}
 			} else {
-				if ( $this->attributeNameMatcher->matches( $attribute->name ) ) {
+				if ($this->attributeNameMatcher->matches($attribute->name)) {
 					return true;
 				}
 			}

src/ClassMatcher.php

Spacing +10 added lines, -10 removed lines

@@ -18,17 +18,17 @@ 
 	 *
 	 * @return self
 	 */
-	public static function withClass( $class ) {
-		return new static( Util::wrapValueWithIsEqual( $class ) );
+	public static function withClass($class) {
+		return new static(Util::wrapValueWithIsEqual($class));
 	}
 
-	public function __construct( Matcher $class ) {
+	public function __construct(Matcher $class) {
 		parent::__construct();
 		$this->classMatcher = $class;
 	}
 
-	public function describeTo( Description $description ) {
-		$description->appendText( 'with class ' )->appendDescriptionOf( $this->classMatcher );
+	public function describeTo(Description $description) {
+		$description->appendText('with class ')->appendDescriptionOf($this->classMatcher);
 	}
 
 	/**
@@ -37,12 +37,12 @@ 
 	 *
 	 * @return bool
 	 */
-	protected function matchesSafelyWithDiagnosticDescription( $item, Description $mismatchDescription ) {
-		$classAttribute = $item->getAttribute( 'class' );
+	protected function matchesSafelyWithDiagnosticDescription($item, Description $mismatchDescription) {
+		$classAttribute = $item->getAttribute('class');
 
-		$classes = preg_split( '/\s+/u', $classAttribute );
-		foreach ( $classes as $class ) {
-			if ( $this->classMatcher->matches( $class ) ) {
+		$classes = preg_split('/\s+/u', $classAttribute);
+		foreach ($classes as $class) {
+			if ($this->classMatcher->matches($class)) {
 				return true;
 			}
 		}