wmde / hamcrest-html-matchers

src/ClassMatcher.php

Spacing +15 added lines, -15 removed lines

@@ -17,24 +17,24 @@ 
 	 *
 	 * @return self
 	 */
-	public static function withClass( $class ) {
-		return new static( $class );
+	public static function withClass($class) {
+		return new static($class);
 	}
 
 	/**
 	 * @param Matcher|string $class
 	 */
-	public function __construct( $class ) {
+	public function __construct($class) {
 		parent::__construct();
 		$this->classMatcher = $class;
 	}
 
-	public function describeTo( Description $description ) {
-		$description->appendText( 'with class ' );
-		if ( $this->classMatcher instanceof Matcher ) {
-			$description->appendDescriptionOf( $this->classMatcher );
+	public function describeTo(Description $description) {
+		$description->appendText('with class ');
+		if ($this->classMatcher instanceof Matcher) {
+			$description->appendDescriptionOf($this->classMatcher);
 		} else {
-			$description->appendValue( $this->classMatcher );
+			$description->appendValue($this->classMatcher);
 		}
 	}
 
@@ -44,19 +44,19 @@ 
 	 *
 	 * @return bool
 	 */
-	protected function matchesSafelyWithDiagnosticDescription( $item, Description $mismatchDescription ) {
-		$classAttribute = $item->getAttribute( 'class' );
+	protected function matchesSafelyWithDiagnosticDescription($item, Description $mismatchDescription) {
+		$classAttribute = $item->getAttribute('class');
 
-		if ( $this->classMatcher instanceof Matcher ) {
-			$classes = preg_split( '/\s+/u', $classAttribute );
-			foreach ( $classes as $class ) {
-				if ( $this->classMatcher->matches( $class ) ) {
+		if ($this->classMatcher instanceof Matcher) {
+			$classes = preg_split('/\s+/u', $classAttribute);
+			foreach ($classes as $class) {
+				if ($this->classMatcher->matches($class)) {
 					return true;
 				}
 			}
 			return false;
 		} else {
-			return (bool)preg_match( '/(^|\s)' . preg_quote( $this->classMatcher, '/' ) . '(\s|$)/u', $classAttribute );
+			return (bool)preg_match('/(^|\s)' . preg_quote($this->classMatcher, '/') . '(\s|$)/u', $classAttribute);
 		}
 	}
 

src/AttributeMatcher.php

Spacing +31 added lines, -31 removed lines

@@ -22,14 +22,14 @@ 
 	 *
 	 * @return self
 	 */
-	public static function withAttribute( $attributeName ) {
-		return new static( $attributeName );
+	public static function withAttribute($attributeName) {
+		return new static($attributeName);
 	}
 
 	/**
 	 * @param Matcher|string $attributeNameMatcher
 	 */
-	public function __construct( $attributeNameMatcher ) {
+	public function __construct($attributeNameMatcher) {
 		parent::__construct();
 
 		$this->attributeNameMatcher = $attributeNameMatcher;
@@ -40,7 +40,7 @@ 
 	 *
 	 * @return AttributeMatcher
 	 */
-	public function havingValue( $value ) {
+	public function havingValue($value) {
 		// TODO: Throw exception if value is set
 		$result = clone $this;
 		$result->valueMatcher = $value;
@@ -48,19 +48,19 @@ 
 		return $result;
 	}
 
-	public function describeTo( Description $description ) {
-		$description->appendText( 'with attribute ' );
-		if ( $this->attributeNameMatcher instanceof Matcher ) {
-			$description->appendDescriptionOf( $this->attributeNameMatcher );
+	public function describeTo(Description $description) {
+		$description->appendText('with attribute ');
+		if ($this->attributeNameMatcher instanceof Matcher) {
+			$description->appendDescriptionOf($this->attributeNameMatcher);
 		} else {
-			$description->appendValue( $this->attributeNameMatcher );
+			$description->appendValue($this->attributeNameMatcher);
 		}
-		if ( $this->valueMatcher !== null ) {
-			$description->appendText( ' having value ' );
-			if ( $this->attributeNameMatcher instanceof Matcher ) {
-				$description->appendDescriptionOf( $this->valueMatcher );
+		if ($this->valueMatcher !== null) {
+			$description->appendText(' having value ');
+			if ($this->attributeNameMatcher instanceof Matcher) {
+				$description->appendDescriptionOf($this->valueMatcher);
 			} else {
-				$description->appendValue( $this->valueMatcher );
+				$description->appendValue($this->valueMatcher);
 			}
 		}
 	}
@@ -71,33 +71,33 @@ 
 	 *
 	 * @return bool
 	 */
-	protected function matchesSafelyWithDiagnosticDescription( $item, Description $mismatchDescription ) {
-		if ( $this->valueMatcher === null ) {
-			if ( $this->attributeNameMatcher instanceof Matcher ) {
+	protected function matchesSafelyWithDiagnosticDescription($item, Description $mismatchDescription) {
+		if ($this->valueMatcher === null) {
+			if ($this->attributeNameMatcher instanceof Matcher) {
 				/** @var \DOMAttr $attribute */
-				foreach ( $item->attributes as $attribute ) {
-					if ( $this->attributeNameMatcher->matches( $attribute->name ) ) {
+				foreach ($item->attributes as $attribute) {
+					if ($this->attributeNameMatcher->matches($attribute->name)) {
 						return true;
 					}
 				}
 			} else {
-				return $item->hasAttribute( $this->attributeNameMatcher );
+				return $item->hasAttribute($this->attributeNameMatcher);
 			}
 		} else {
-			if ( $this->attributeNameMatcher instanceof Matcher ) {
-				if ( $this->valueMatcher instanceof Matcher ) {
+			if ($this->attributeNameMatcher instanceof Matcher) {
+				if ($this->valueMatcher instanceof Matcher) {
 					/** @var \DOMAttr $attribute */
-					foreach ( $item->attributes as $attribute ) {
-						if ( $this->attributeNameMatcher->matches( $attribute->name )
-							&& $this->valueMatcher->matches( $attribute->value )
+					foreach ($item->attributes as $attribute) {
+						if ($this->attributeNameMatcher->matches($attribute->name)
+							&& $this->valueMatcher->matches($attribute->value)
 						) {
 							return true;
 						}
 					}
 				} else {
 					/** @var \DOMAttr $attribute */
-					foreach ( $item->attributes as $attribute ) {
-						if ( $this->attributeNameMatcher->matches( $attribute->name )
+					foreach ($item->attributes as $attribute) {
+						if ($this->attributeNameMatcher->matches($attribute->name)
 							&& $attribute->value === $this->valueMatcher
 						) {
 							return true;
@@ -105,11 +105,11 @@ 
 					}
 				}
 			} else {
-				if ( $this->valueMatcher instanceof Matcher ) {
-					return $item->hasAttribute( $this->attributeNameMatcher )
-						&& $this->valueMatcher->matches( $item->getAttribute( $this->attributeNameMatcher ) );
+				if ($this->valueMatcher instanceof Matcher) {
+					return $item->hasAttribute($this->attributeNameMatcher)
+						&& $this->valueMatcher->matches($item->getAttribute($this->attributeNameMatcher));
 				} else {
-					return $item->getAttribute( $this->attributeNameMatcher ) === $this->valueMatcher;
+					return $item->getAttribute($this->attributeNameMatcher) === $this->valueMatcher;
 				}
 			}
 		}

src/TagNameMatcher.php

Spacing +16 added lines, -16 removed lines

@@ -17,24 +17,24 @@ 
 	 *
 	 * @return self
 	 */
-	public static function withTagName( $tagName ) {
-		return new static( $tagName );
+	public static function withTagName($tagName) {
+		return new static($tagName);
 	}
 
 	/**
 	 * @param Matcher|string $tagNameMatcher
 	 */
-	public function __construct( $tagNameMatcher ) {
+	public function __construct($tagNameMatcher) {
 		parent::__construct();
 		$this->tagNameMatcher = $tagNameMatcher;
 	}
 
-	public function describeTo( Description $description ) {
-		$description->appendText( 'with tag name ' );
-		if ( $this->tagNameMatcher instanceof Matcher ) {
-			$description->appendDescriptionOf( $this->tagNameMatcher );
+	public function describeTo(Description $description) {
+		$description->appendText('with tag name ');
+		if ($this->tagNameMatcher instanceof Matcher) {
+			$description->appendDescriptionOf($this->tagNameMatcher);
 		} else {
-			$description->appendValue( $this->tagNameMatcher );
+			$description->appendValue($this->tagNameMatcher);
 		}
 	}
 
@@ -44,22 +44,22 @@ 
 	 *
 	 * @return bool
 	 */
-	protected function matchesSafelyWithDiagnosticDescription( $item, Description $mismatchDescription ) {
-		if ( $this->tagNameMatcher instanceof Matcher ) {
-			if ( $this->tagNameMatcher->matches( $item->tagName ) ) {
+	protected function matchesSafelyWithDiagnosticDescription($item, Description $mismatchDescription) {
+		if ($this->tagNameMatcher instanceof Matcher) {
+			if ($this->tagNameMatcher->matches($item->tagName)) {
 				return true;
 			}
 		} else {
-			if ( $item->tagName === $this->tagNameMatcher ) {
+			if ($item->tagName === $this->tagNameMatcher) {
 				return true;
 			}
 		}
 
-		$mismatchDescription->appendText( 'tag name ' );
-		if ( $this->tagNameMatcher instanceof Matcher ) {
-			$this->tagNameMatcher->describeMismatch( $item->tagName, $mismatchDescription );
+		$mismatchDescription->appendText('tag name ');
+		if ($this->tagNameMatcher instanceof Matcher) {
+			$this->tagNameMatcher->describeMismatch($item->tagName, $mismatchDescription);
 		} else {
-			$mismatchDescription->appendText( 'was ' )->appendValue( $item->tagName );
+			$mismatchDescription->appendText('was ')->appendValue($item->tagName);
 		}
 		return false;
 	}

src/ComplexTagMatcher.php

Spacing +61 added lines, -61 removed lines

@@ -30,24 +30,24 @@ 
 	 *
 	 * @return self
 	 */
-	public static function tagMatchingOutline( $htmlOutline ) {
-		return new self( $htmlOutline );
+	public static function tagMatchingOutline($htmlOutline) {
+		return new self($htmlOutline);
 	}
 
 	/**
 	 * @param string $tagHtmlRepresentation
 	 */
-	public function __construct( $tagHtmlRepresentation ) {
+	public function __construct($tagHtmlRepresentation) {
 		parent::__construct();
 
 		$this->tagHtmlOutline = $tagHtmlRepresentation;
-		$this->matcher = $this->createMatcherFromHtml( $tagHtmlRepresentation );
+		$this->matcher = $this->createMatcherFromHtml($tagHtmlRepresentation);
 	}
 
-	public function describeTo( Description $description ) {
-		$description->appendText( 'tag matching outline `' )
-			->appendText( $this->tagHtmlOutline )
-			->appendText( '` ' );
+	public function describeTo(Description $description) {
+		$description->appendText('tag matching outline `')
+			->appendText($this->tagHtmlOutline)
+			->appendText('` ');
 	}
 
 	/**
@@ -56,12 +56,12 @@ 
 	 *
 	 * @return bool
 	 */
-	protected function matchesSafelyWithDiagnosticDescription( $item, Description $mismatchDescription ) {
-		$result = $this->matcher->matches( $item );
-		if ( !$result ) {
-			$mismatchDescription->appendText( 'was `' )
-				->appendText( $this->elementToString( $item ) )
-				->appendText( '`' );
+	protected function matchesSafelyWithDiagnosticDescription($item, Description $mismatchDescription) {
+		$result = $this->matcher->matches($item);
+		if (!$result) {
+			$mismatchDescription->appendText('was `')
+				->appendText($this->elementToString($item))
+				->appendText('`');
 		}
 		return $result;
 	}
@@ -71,23 +71,23 @@ 
 	 *
 	 * @return Matcher
 	 */
-	private function createMatcherFromHtml( $htmlOutline ) {
-		$document = $this->parseHtml( $htmlOutline );
-		$targetTag = $this->getSingleTagFromThe( $document );
+	private function createMatcherFromHtml($htmlOutline) {
+		$document = $this->parseHtml($htmlOutline);
+		$targetTag = $this->getSingleTagFromThe($document);
 
-		$this->assertTagDoesNotContainChildren( $targetTag );
+		$this->assertTagDoesNotContainChildren($targetTag);
 
-		$attributeMatchers = $this->createAttributeMatchers( $htmlOutline, $targetTag );
-		$classMatchers = $this->createClassMatchers( $targetTag );
+		$attributeMatchers = $this->createAttributeMatchers($htmlOutline, $targetTag);
+		$classMatchers = $this->createClassMatchers($targetTag);
 
 		return AllOf::allOf(
-			TagNameMatcher::withTagName( $targetTag->tagName ),
-			count( $attributeMatchers ) === 1
+			TagNameMatcher::withTagName($targetTag->tagName),
+			count($attributeMatchers) === 1
 				? $attributeMatchers[0]
-				: call_user_func_array( [ AllOf::class, 'allOf' ], $attributeMatchers ),
-			count( $classMatchers ) === 1
+				: call_user_func_array([AllOf::class, 'allOf'], $attributeMatchers),
+			count($classMatchers) === 1
 				? $classMatchers[0]
-				: call_user_func_array( [ AllOf::class, 'allOf' ], $classMatchers )
+				: call_user_func_array([AllOf::class, 'allOf'], $classMatchers)
 		);
 	}
 
@@ -96,7 +96,7 @@ 
 	 *
 	 * @return bool
 	 */
-	private function isUnknownTagError( \LibXMLError $error ) {
+	private function isUnknownTagError(\LibXMLError $error) {
 		return $error->code === self::XML_UNKNOWN_TAG_ERROR_CODE;
 	}
 
@@ -106,10 +106,10 @@ 
 	 *
 	 * @return bool
 	 */
-	private function isBooleanAttribute( $inputHtml, $attributeName ) {
-		$quotedName = preg_quote( $attributeName, '/' );
+	private function isBooleanAttribute($inputHtml, $attributeName) {
+		$quotedName = preg_quote($attributeName, '/');
 
-		$attributeHasValueAssigned = preg_match( "/\b{$quotedName}\s*=/ui", $inputHtml );
+		$attributeHasValueAssigned = preg_match("/\b{$quotedName}\s*=/ui", $inputHtml);
 		return !$attributeHasValueAssigned;
 	}
 
@@ -119,26 +119,26 @@ 
 	 * @return \DOMDocument
 	 * @throws \InvalidArgumentException
 	 */
-	private function parseHtml( $html ) {
-		$internalErrors = libxml_use_internal_errors( true );
+	private function parseHtml($html) {
+		$internalErrors = libxml_use_internal_errors(true);
 		$document = new \DOMDocument();
 
-		if ( !@$document->loadHTML( $html ) ) {
-			throw new \InvalidArgumentException( "There was some parsing error of `$html`" );
+		if (!@$document->loadHTML($html)) {
+			throw new \InvalidArgumentException("There was some parsing error of `$html`");
 		}
 
 		$errors = libxml_get_errors();
 		libxml_clear_errors();
-		libxml_use_internal_errors( $internalErrors );
+		libxml_use_internal_errors($internalErrors);
 
 		/** @var \LibXMLError $error */
-		foreach ( $errors as $error ) {
-			if ( $this->isUnknownTagError( $error ) ) {
+		foreach ($errors as $error) {
+			if ($this->isUnknownTagError($error)) {
 				continue;
 			}
 
 			throw new \InvalidArgumentException(
-				'There was parsing error: ' . trim( $error->message ) . ' on line ' . $error->line
+				'There was parsing error: ' . trim($error->message) . ' on line ' . $error->line
 			);
 		}
 
@@ -151,22 +151,22 @@ 
 	 * @return \DOMElement
 	 * @throws \InvalidArgumentException
 	 */
-	private function getSingleTagFromThe( \DOMDocument $document ) {
-		$directChildren = iterator_to_array( $document->documentElement->childNodes );
+	private function getSingleTagFromThe(\DOMDocument $document) {
+		$directChildren = iterator_to_array($document->documentElement->childNodes);
 
-		$body = array_shift( $directChildren );
-		$directChildren = iterator_to_array( $body->childNodes );
+		$body = array_shift($directChildren);
+		$directChildren = iterator_to_array($body->childNodes);
 
-		if ( count( $directChildren ) !== 1 ) {
-			throw new InvalidArgumentException( 'Expected exactly 1 tag description, got ' . count( $directChildren ) );
+		if (count($directChildren) !== 1) {
+			throw new InvalidArgumentException('Expected exactly 1 tag description, got ' . count($directChildren));
 		}
 
 		return $directChildren[0];
 	}
 
-	private function assertTagDoesNotContainChildren( \DOMElement $targetTag ) {
-		if ( $targetTag->childNodes->length > 0 ) {
-			throw new InvalidArgumentException( 'Nested elements are not allowed' );
+	private function assertTagDoesNotContainChildren(\DOMElement $targetTag) {
+		if ($targetTag->childNodes->length > 0) {
+			throw new InvalidArgumentException('Nested elements are not allowed');
 		}
 	}
 
@@ -176,17 +176,17 @@ 
 	 *
 	 * @return AttributeMatcher[]
 	 */
-	private function createAttributeMatchers( $inputHtml, \DOMElement $targetTag ) {
+	private function createAttributeMatchers($inputHtml, \DOMElement $targetTag) {
 		$attributeMatchers = [];
 		/** @var \DOMAttr $attribute */
-		foreach ( $targetTag->attributes as $attribute ) {
-			if ( $attribute->name === 'class' ) {
+		foreach ($targetTag->attributes as $attribute) {
+			if ($attribute->name === 'class') {
 				continue;
 			}
 
-			$attributeMatcher = AttributeMatcher::withAttribute( $attribute->name );
-			if ( !$this->isBooleanAttribute( $inputHtml, $attribute->name ) ) {
-				$attributeMatcher = $attributeMatcher->havingValue( $attribute->value );
+			$attributeMatcher = AttributeMatcher::withAttribute($attribute->name);
+			if (!$this->isBooleanAttribute($inputHtml, $attribute->name)) {
+				$attributeMatcher = $attributeMatcher->havingValue($attribute->value);
 			}
 
 			$attributeMatchers[] = $attributeMatcher;
@@ -199,14 +199,14 @@ 
 	 *
 	 * @return ClassMatcher[]
 	 */
-	private function createClassMatchers( \DOMElement $targetTag ) {
+	private function createClassMatchers(\DOMElement $targetTag) {
 		$classMatchers = [];
-		$classValue = $targetTag->getAttribute( 'class' );
-		foreach ( explode( ' ', $classValue ) as $expectedClass ) {
-			if ( $expectedClass === '' ) {
+		$classValue = $targetTag->getAttribute('class');
+		foreach (explode(' ', $classValue) as $expectedClass) {
+			if ($expectedClass === '') {
 				continue;
 			}
-			$classMatchers[] = ClassMatcher::withClass( $expectedClass );
+			$classMatchers[] = ClassMatcher::withClass($expectedClass);
 		}
 		return $classMatchers;
 	}
@@ -216,11 +216,11 @@ 
 	 *
 	 * @return string
 	 */
-	private function elementToString( \DOMElement $element ) {
+	private function elementToString(\DOMElement $element) {
 		$newDocument = new \DOMDocument();
-		$cloned = $element->cloneNode( true );
-		$newDocument->appendChild( $newDocument->importNode( $cloned, true ) );
-		return trim( $newDocument->saveHTML() );
+		$cloned = $element->cloneNode(true);
+		$newDocument->appendChild($newDocument->importNode($cloned, true));
+		return trim($newDocument->saveHTML());
 	}
 
 }

src/TextContentsMatcher.php

Spacing +11 added lines, -11 removed lines

@@ -17,24 +17,24 @@ 
 	 *
 	 * @return self
 	 */
-	public static function havingTextContents( $text ) {
-		return new static( $text );
+	public static function havingTextContents($text) {
+		return new static($text);
 	}
 
 	/**
 	 * @param Matcher|string $matcher
 	 */
-	public function __construct( $matcher ) {
+	public function __construct($matcher) {
 		parent::__construct();
 		$this->matcher = $matcher;
 	}
 
-	public function describeTo( Description $description ) {
-		$description->appendText( 'having text contents ' );
-		if ( $this->matcher instanceof Matcher ) {
-			$description->appendDescriptionOf( $this->matcher );
+	public function describeTo(Description $description) {
+		$description->appendText('having text contents ');
+		if ($this->matcher instanceof Matcher) {
+			$description->appendDescriptionOf($this->matcher);
 		} else {
-			$description->appendValue( $this->matcher );
+			$description->appendValue($this->matcher);
 		}
 	}
 
@@ -44,9 +44,9 @@ 
 	 *
 	 * @return bool
 	 */
-	protected function matchesSafelyWithDiagnosticDescription( $item, Description $mismatchDescription ) {
-		if ( $this->matcher instanceof Matcher ) {
-			return $this->matcher->matches( $item->textContent );
+	protected function matchesSafelyWithDiagnosticDescription($item, Description $mismatchDescription) {
+		if ($this->matcher instanceof Matcher) {
+			return $this->matcher->matches($item->textContent);
 		} else {
 			return $item->textContent === (string)$this->matcher;
 		}

src/ChildElementMatcher.php

Spacing +21 added lines, -21 removed lines

@@ -18,25 +18,25 @@ 
 	 *
 	 * @return self
 	 */
-	public static function havingChild( $elementMatcher = null ) {
-		return new static( $elementMatcher );
+	public static function havingChild($elementMatcher = null) {
+		return new static($elementMatcher);
 	}
 
 	/**
 	 * @param Matcher|string|null $matcher
 	 */
-	public function __construct( $matcher = null ) {
-		parent::__construct( \DOMNode::class );
+	public function __construct($matcher = null) {
+		parent::__construct(\DOMNode::class);
 		$this->matcher = $matcher;
 	}
 
-	public function describeTo( Description $description ) {
-		$description->appendText( 'having child ' );
-		if ( $this->matcher ) {
-			if ( $this->matcher instanceof Matcher ) {
-				$description->appendDescriptionOf( $this->matcher );
+	public function describeTo(Description $description) {
+		$description->appendText('having child ');
+		if ($this->matcher) {
+			if ($this->matcher instanceof Matcher) {
+				$description->appendDescriptionOf($this->matcher);
 			} else {
-				$description->appendValue( $this->matcher );
+				$description->appendValue($this->matcher);
 			}
 		}
 	}
@@ -47,35 +47,35 @@ 
 	 *
 	 * @return bool
 	 */
-	protected function matchesSafelyWithDiagnosticDescription( $item, Description $mismatchDescription ) {
-		if ( $item instanceof \DOMDocument ) {
-			$item = $item->documentElement->childNodes->item( 0 );
+	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' );
+		if ($directChildren->length === 0) {
+			$mismatchDescription->appendText('having no children');
 			return false;
 		}
 
-		if ( !$this->matcher ) {
+		if (!$this->matcher) {
 			return true;
 		}
 
-		if ( $this->matcher instanceof Matcher ) {
-			foreach ( new XmlNodeRecursiveIterator( $directChildren ) as $child ) {
-				if ( $this->matcher->matches( $child ) ) {
+		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 ) {
+			if ($item->getElementsByTagName($this->matcher)->length !== 0) {
 				return true;
 			}
 		}
 
-		$mismatchDescription->appendText( 'having no children ' )->appendDescriptionOf( $this->matcher );
+		$mismatchDescription->appendText('having no children ')->appendDescriptionOf($this->matcher);
 		return false;
 	}