whm\Smoke\Rules\Html\CssSelectorExistsRule

Complexity

Total Complexity

5

Size/Duplication

Total Lines	40
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	4

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	init()	0	4	1
B	doValidation()	0	28	4

<?php

namespace whm\Smoke\Rules\Html;

use whm\Smoke\Http\Response;
use whm\Smoke\Rules\StandardRule;
use Symfony\Component\CssSelector\CssSelectorConverter;
use whm\Smoke\Rules\ValidationFailedException;

/**
 * This rule checks if xpath is found in a html document.
 */
class CssSelectorExistsRule extends StandardRule
{
    protected $contentTypes = ['text/html'];

    private $cssSelectors;

    public function init(array $cssSelectors)
    {
        $this->cssSelectors = $cssSelectors;
    }

    public function doValidation(Response $response)
    {
        $domDocument = new \DOMDocument();
        @$domDocument->loadHTML((string) $response->getBody());

It seems like you do not handle an error condition here. This can introduce security issues, and is generally not recommended.

        $domXPath = new \DOMXPath($domDocument);

        $error = false;
        $snotFoundSelectors = array();

        foreach ($this->cssSelectors as $selector) {
            $converter = new CssSelectorConverter();
            $selectorAsXPath = $converter->toXPath($selector['pattern']);

            $count = $domXPath->query($selectorAsXPath)->length;

            if ($count == 0) {
                $error = true;
                $snotFoundSelectors[] = $selector['pattern'];
            }
        }

        if ($error === true) {
            $allNotFoundSelectors = implode('", "', $snotFoundSelectors);

            throw new ValidationFailedException('CSS Selector "' . $allNotFoundSelectors .'" not found in DOM.');
        }
    }
}