FirstAvailable::choose() - Code Metrics - magium/MagiumMagento - Measure and Improve Code Quality continuously with Scrutinizer

FirstAvailable::choose() A
last analyzed 2017-11-09 15:32 UTC

↳ Parent: FirstAvailable

Complexity

Conditions	3
Paths	4

Size

Total Lines	22
Code Lines	13

Duplication

Lines	0
Ratio	0 %

Importance

Changes	2
Bugs	0	Features	0

Metric	Value
c	2
b	0
f	0
dl	0
loc	22
rs	9.2
cc	3
eloc	13
nc	4
nop	1

<?php

namespace Magium\Magento\Actions\Checkout\ShippingMethods;

use Facebook\WebDriver\WebDriverBy;
use Magium\AbstractTestCase;
use Magium\Magento\AbstractMagentoTestCase;
use Magium\Magento\Themes\OnePageCheckout\AbstractThemeConfiguration;
use Magium\WebDriver\ExpectedCondition;
use Magium\WebDriver\WebDriver;

class FirstAvailable implements ShippingMethodInterface
{
    const ACTION = 'Checkout\ShippingMethods\FirstAvailable';
    protected $webDriver;
    protected $theme;
    protected $testCase;

    public function __construct(
        WebDriver               $webDriver,
        AbstractThemeConfiguration      $theme,
        AbstractMagentoTestCase $testCase
    ) {
        $this->webDriver        = $webDriver;
        $this->theme            = $theme;
        $this->testCase         = $testCase;
    }

    public function choose($required)
    {
        $this->webDriver->wait()->until(
            ExpectedCondition::elementExists(
function acceptsInteger($int) { }

$x = '123'; // string "123"

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
                $this->theme->getShippingMethodFormXpath(), AbstractTestCase::BY_XPATH
            )
        );

        // Some products, such as virtual products, do not get shipped
        if ($required) {
            $this->testCase->assertElementExists($this->theme->getDefaultShippingXpath(), AbstractTestCase::BY_XPATH);
            $this->testCase->assertElementDisplayed($this->theme->getDefaultShippingXpath(), AbstractTestCase::BY_XPATH);
        }

        if ($this->webDriver->elementDisplayed($this->theme->getDefaultShippingXpath(), AbstractTestCase::BY_XPATH)) {
            $xpath = $this->theme->getDefaultShippingXpath();
            $element = $this->webDriver->byXpath($xpath);
            $this->webDriver->action()->moveToElement($element);
            $this->webDriver->wait()->until(ExpectedCondition::elementToBeClickable(WebDriverBy::xpath($xpath)));
function acceptsInteger($int) { }

$x = '123'; // string "123"

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
            $this->webDriver->byXpath($xpath)->click();
        }
    }

}


1			<?php
2
3			namespace Magium\Magento\Actions\Checkout\ShippingMethods;
4
5			use Facebook\WebDriver\WebDriverBy;
6			use Magium\AbstractTestCase;
7			use Magium\Magento\AbstractMagentoTestCase;
8			use Magium\Magento\Themes\OnePageCheckout\AbstractThemeConfiguration;
9			use Magium\WebDriver\ExpectedCondition;
10			use Magium\WebDriver\WebDriver;
11
12			class FirstAvailable implements ShippingMethodInterface
13			{
14			const ACTION = 'Checkout\ShippingMethods\FirstAvailable';
15			protected $webDriver;
16			protected $theme;
17			protected $testCase;
18
19			public function __construct(
20			WebDriver $webDriver,
21			AbstractThemeConfiguration $theme,
22			AbstractMagentoTestCase $testCase
23			) {
24			$this->webDriver = $webDriver;
25			$this->theme = $theme;
26			$this->testCase = $testCase;
27			}
28
29			public function choose($required)
30			{
31			$this->webDriver->wait()->until(
32			ExpectedCondition::elementExists(
			0 ignored issues – show Documentation introduced 2017-02-17 18:01 UTC by Report Bug Copy Issue Report `\Magium\WebDriver\Expect...ractTestCase::BY_XPATH)` is of type `object<Facebook\WebDrive...riverExpectedCondition>`, but the function expects a `callable`. It seems like the type of the argument is not accepted by the function/method which you are calling. In some cases, in particular if PHP’s automatic type-juggling kicks in this might be fine. In other cases, however this might be a bug. We suggest to add an explicit type cast like in the following example: function acceptsInteger($int) { } $x = '123'; // string "123" // Instead of acceptsInteger($x); // we recommend to use acceptsInteger((integer) $x); Loading history...
33			$this->theme->getShippingMethodFormXpath(), AbstractTestCase::BY_XPATH
34			)
35			);
36
37			// Some products, such as virtual products, do not get shipped
38			if ($required) {
39			$this->testCase->assertElementExists($this->theme->getDefaultShippingXpath(), AbstractTestCase::BY_XPATH);
40			$this->testCase->assertElementDisplayed($this->theme->getDefaultShippingXpath(), AbstractTestCase::BY_XPATH);
41			}
42
43			if ($this->webDriver->elementDisplayed($this->theme->getDefaultShippingXpath(), AbstractTestCase::BY_XPATH)) {
44			$xpath = $this->theme->getDefaultShippingXpath();
45			$element = $this->webDriver->byXpath($xpath);
46			$this->webDriver->action()->moveToElement($element);
47			$this->webDriver->wait()->until(ExpectedCondition::elementToBeClickable(WebDriverBy::xpath($xpath)));
			0 ignored issues – show Documentation introduced 2017-02-21 14:57 UTC by Report Bug Copy Issue Report `\Magium\WebDriver\Expect...riverBy::xpath($xpath))` is of type `object<Magium\WebDriver\ExpectedCondition>`, but the function expects a `callable`. It seems like the type of the argument is not accepted by the function/method which you are calling. In some cases, in particular if PHP’s automatic type-juggling kicks in this might be fine. In other cases, however this might be a bug. We suggest to add an explicit type cast like in the following example: function acceptsInteger($int) { } $x = '123'; // string "123" // Instead of acceptsInteger($x); // we recommend to use acceptsInteger((integer) $x); Loading history...
48			$this->webDriver->byXpath($xpath)->click();
49			}
50			}
51
52			}
53

magium / MagiumMagento

GitHub Access Token became invalid

FirstAvailable::choose() A last analyzed 2017-11-09 15:32 UTC

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like

FirstAvailable::choose() A
last analyzed 2017-11-09 15:32 UTC