PagaMasTarde\SeleniumFormUtils\Step\AbstractStep

Complexity

Total Complexity

8

Size/Duplication

Total Lines	92
Duplicated Lines	0 %

Importance

Changes

0

5 Methods

Rating	Name	Duplication	Size	Complexity
A	waitTobeVisible()	0	6	1
A	moveToIFrame()	0	4	1
A	validateStep()	0	15	3
A	__construct()	0	4	1
A	waitTobeClickAble()	0	6	1

<?php

namespace Pagantis\SeleniumFormUtils\Step;

use Facebook\WebDriver\WebDriver;
use Facebook\WebDriver\WebDriverBy;
use Facebook\WebDriver\WebDriverExpectedCondition;
use Faker\Factory;

/**
 * Class AbstractStep
 *
 * @package Pagantis\SeleniumFormUtils\Step
 */
abstract class AbstractStep implements StepInterface
{
    /**
     * @var WebDriver
     */
    protected $webDriver;

    /**
     * @var Factory
     */
    protected $faker;

    /**
     * AbstractStep constructor.
     *
     * @param WebDriver $webDriver
     */
    public function __construct(WebDriver $webDriver)
    {
        $this->webDriver = $webDriver;
        $this->faker = Factory::create();

It seems like Faker\Factory::create() of type Faker\Generator is incompatible with the declared type Faker\Factory of property $faker.

    }

    /**
     * @param WebDriverBy $webDriverBy
     *
     * @return \Facebook\WebDriver\WebDriverElement
     *
     * @throws \Facebook\WebDriver\Exception\NoSuchElementException
     * @throws \Facebook\WebDriver\Exception\TimeOutException
     */
    public function waitTobeClickAble(WebDriverBy $webDriverBy)
    {
        $condition = WebDriverExpectedCondition::elementToBeClickable($webDriverBy);
        $this->webDriver->wait(5, 200)->until($condition);

        return $this->webDriver->findElement($webDriverBy);
    }

    /**
     * @param WebDriverBy $webDriverBy
     *
     * @return \Facebook\WebDriver\WebDriverElement
     *
     * @throws \Facebook\WebDriver\Exception\NoSuchElementException
     * @throws \Facebook\WebDriver\Exception\TimeOutException
     */
    public function waitTobeVisible(WebDriverBy $webDriverBy)
    {
        $condition = WebDriverExpectedCondition::visibilityOfElementLocated($webDriverBy);
        $this->webDriver->wait(5, 200)->until($condition);

        return $this->webDriver->findElement($webDriverBy);
    }

    /**
     * @param string $step
     *
     * @throws \Exception
     */
    public function validateStep($step)
    {
        $element = WebDriverBy::cssSelector(".Loading .is-disabled");
        $condition = WebDriverExpectedCondition::presenceOfElementLocated($element);
        $this->webDriver->wait(5, 200)->until($condition);

        $path = parse_url($this->webDriver->getCurrentURL(), PHP_URL_PATH);
        $arguments = explode(DIRECTORY_SEPARATOR, $path);
        $tempStep = '';
        for ($i = 2; $i < count($arguments); $i++) {

It seems like you are calling the size function count() as part of the test condition. You might want to compute the size beforehand, and not on each iteration.

            $tempStep .= DIRECTORY_SEPARATOR.$arguments[$i];
        }

        if ($step !== $tempStep) {
            throw new \Exception('Wrong step: ' . $tempStep);
        }
    }

    /**
     * @param $iFrameLocator
     *
     * @throws \Facebook\WebDriver\Exception\NoSuchElementException
     * @throws \Facebook\WebDriver\Exception\TimeOutException
     */
    public function moveToIFrame($iFrameLocator)
    {
        $condition = WebDriverExpectedCondition::frameToBeAvailableAndSwitchToIt($iFrameLocator);
        $this->webDriver->wait(5, 200)->until($condition);
    }

    /**
     * Switch to parent window
     */
    public function moveToParent()
    {
        $handles=$this->webDriver->getWindowHandles();
        $parent = end($handles);
        $this->webDriver->switchTo()->window($parent);
    }
}