wmde / hamcrest-html-matchers

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/ClassMatcher.php

Spacing +10 added lines, -10 removed lines

@@ -13,17 +13,17 @@ 
 	 */
 	private $classMatcher;
 
-	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);
 	}
 
 	/**
@@ -32,12 +32,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;
 			}
 		}

src/AttributeMatcher.php

Spacing +17 added lines, -17 removed lines

@@ -18,15 +18,15 @@ 
 	 */
 	private $valueMatcher;
 
-	public static function withAttribute( $attributeName ) {
-		return new static( Util::wrapValueWithIsEqual( $attributeName ) );
+	public static function withAttribute($attributeName) {
+		return new static(Util::wrapValueWithIsEqual($attributeName));
 	}
 
 	/**
 	 * AttributeMatcher constructor.
 	 * @param \Hamcrest\Matcher $attributeNameMatcher
 	 */
-	public function __construct( Matcher $attributeNameMatcher ) {
+	public function __construct(Matcher $attributeNameMatcher) {
 		parent::__construct();
 
 		$this->attributeNameMatcher = $attributeNameMatcher;
@@ -36,20 +36,20 @@ 
 	 * @param Matcher|mixed $value
 	 * @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);
 		}
 	}
 
@@ -59,18 +59,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/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 mixed $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/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 array New queue
 	 */
-	private function addElementsToQueue( array $queue, \DOMNodeList $nodeList ) {
+	private function addElementsToQueue(array $queue, \DOMNodeList $nodeList) {
 		/** @var \DOMElement $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/TextContentsMatcher.php

Spacing +7 added lines, -7 removed lines

@@ -13,17 +13,17 @@ 
 	 */
 	private $matcher;
 
-	public static function havingTextContents( $text ) {
-		return new static( Util::wrapValueWithIsEqual( $text ) );
+	public static function havingTextContents($text) {
+		return new static(Util::wrapValueWithIsEqual($text));
 	}
 
-	public function __construct( Matcher $matcher ) {
+	public function __construct(Matcher $matcher) {
 		parent::__construct();
 		$this->matcher = $matcher;
 	}
 
-	public function describeTo( Description $description ) {
-		$description->appendText( 'having text contents ' )->appendDescriptionOf( $this->matcher );
+	public function describeTo(Description $description) {
+		$description->appendText('having text contents ')->appendDescriptionOf($this->matcher);
 	}
 
 	/**
@@ -32,8 +32,8 @@ 
 	 *
 	 * @return bool
 	 */
-	protected function matchesSafelyWithDiagnosticDescription( $item, Description $mismatchDescription ) {
-		return $this->matcher->matches( $item->textContent );
+	protected function matchesSafelyWithDiagnosticDescription($item, Description $mismatchDescription) {
+		return $this->matcher->matches($item->textContent);
 	}
 
 }

src/TagNameMatcher.php

Spacing +10 added lines, -10 removed lines

@@ -13,18 +13,18 @@ 
 	 */
 	private $tagNameMatcher;
 
-	public static function withTagName( $tagName ) {
-		return new static( Util::wrapValueWithIsEqual( $tagName ) );
+	public static function withTagName($tagName) {
+		return new static(Util::wrapValueWithIsEqual($tagName));
 	}
 
-	public function __construct( Matcher $tagNameMatcher ) {
+	public function __construct(Matcher $tagNameMatcher) {
 		parent::__construct();
 		$this->tagNameMatcher = $tagNameMatcher;
 	}
 
-	public function describeTo( Description $description ) {
-		$description->appendText( 'with tag name ' )
-			->appendDescriptionOf( $this->tagNameMatcher );
+	public function describeTo(Description $description) {
+		$description->appendText('with tag name ')
+			->appendDescriptionOf($this->tagNameMatcher);
 	}
 
 	/**
@@ -33,13 +33,13 @@ 
 	 *
 	 * @return bool
 	 */
-	protected function matchesSafelyWithDiagnosticDescription( $item, Description $mismatchDescription ) {
-		if ( $this->tagNameMatcher->matches( $item->tagName ) ) {
+	protected function matchesSafelyWithDiagnosticDescription($item, Description $mismatchDescription) {
+		if ($this->tagNameMatcher->matches($item->tagName)) {
 			return true;
 		}
 
-		$mismatchDescription->appendText( 'tag name ' );
-		$this->tagNameMatcher->describeMismatch( $item->tagName, $mismatchDescription );
+		$mismatchDescription->appendText('tag name ');
+		$this->tagNameMatcher->describeMismatch($item->tagName, $mismatchDescription);
 		return false;
 	}
 

src/HtmlMatcher.php

Spacing +30 added lines, -30 removed lines

@@ -24,66 +24,66 @@ 
 	 *
 	 * @return HtmlMatcher
 	 */
-	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 ) {
+		if (!$result) {
 			return false;
 		}
-		$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;
 	}
 
-	private function isUnknownTagError( \LibXMLError $error ) {
+	private function isUnknownTagError(\LibXMLError $error) {
 		return $error->code === self::XML_UNKNOWN_TAG_ERROR_CODE;
 	}
 
@@ -92,10 +92,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/RootElementMatcher.php

Spacing +19 added lines, -19 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,25 +40,25 @@ 
 	 *
 	 * @return bool
 	 */
-	protected function matchesSafelyWithDiagnosticDescription( $item, Description $mismatchDescription ) {
-		$DOMNodeList = $item->documentElement->childNodes->item( 0 )->childNodes;
-		if ( $DOMNodeList->length > 1 ) {
+	protected function matchesSafelyWithDiagnosticDescription($item, Description $mismatchDescription) {
+		$DOMNodeList = $item->documentElement->childNodes->item(0)->childNodes;
+		if ($DOMNodeList->length > 1) {
 			// TODO Test this description
-			$mismatchDescription->appendText( 'having ' . $DOMNodeList->length . ' root elements ' );
+			$mismatchDescription->appendText('having ' . $DOMNodeList->length . ' root elements ');
 			return false;
 		}
 
-		$target = $DOMNodeList->item( 0 );
-		if ( !$target ) {
+		$target = $DOMNodeList->item(0);
+		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 true;