Overwatch\UserBundle\Tests\E2E\MyAccountTest

Complexity

Total Complexity

6

Size/Duplication

Total Lines	92
Duplicated Lines	17.39 %

Coupling/Cohesion

Components	1
Dependencies	5

Importance

Changes	3
Bugs	0	Features	0

6 Methods

Rating	Name	Duplication	Size	Complexity
A	setUp()	16	16	1
A	testProfileDetails()	0	23	1
A	testApiKeyHidden()	0	7	1
A	testApiKeyCanBeReset()	0	14	1
A	getApiKeyField()	0	6	1
A	testApiKeyVisibilityCanBeToggled()	0	18	1

<?php

namespace Overwatch\UserBundle\Tests\E2E;

use Facebook\WebDriver\WebDriverBy;
use Overwatch\UserBundle\DataFixtures\ORM\UserFixtures;
use Overwatch\UserBundle\Tests\Base\WebDriverTestCase;

/**
 * MyAccountTest
 *
 * @author Zac Sturgess <zac.sturgess@wearetwogether.com>
 */
class MyAccountTest extends WebDriverTestCase
{
    public function setUp()

This method seems to be duplicated in your project.

    {
        parent::setUp();

        $this->logInAsUser('user-1');
        $this->waitForLoadingAnimation();

        $this->webDriver->findElement(
            WebDriverBy::cssSelector('.header-nav .nav-profile')
        )->click();

        $this->webDriver->findElement(
            WebDriverBy::cssSelector('.header-nav .nav-profile .sub-menu li.first a')
        )->click();
        $this->waitForLoadingAnimation();
    }

    public function testProfileDetails()
    {
        $newDetails = [
            'telephoneNumber' => '+4401628813588'
        ];

        $profileItems = $this->webDriver->findElements(
            WebDriverBy::cssSelector('.profile-details input')
        );

        $this->assertEquals(
            UserFixtures::$users['user-1']->getTelephoneNumber(),
            $profileItems[0]->getAttribute('value')
        );

        $profileItems[0]->clear();
        $profileItems[0]->sendKeys($newDetails['telephoneNumber']);

        $profileItems[count($profileItems) - 1]->click();
        $this->waitForLoadingAnimation();

        $this->assertEquals($newDetails['telephoneNumber'], $profileItems[0]->getAttribute('value'));
    }

    public function testApiKeyHidden()
    {
        $this->assertEquals(
            'password',
            $this->getApiKeyField()->getAttribute('type')
        );
    }

    public function testApiKeyVisibilityCanBeToggled()
    {
        $this->webDriver->findElement(
            WebDriverBy::cssSelector(".password-toggle .toggle-btn")

The string literal .password-toggle .toggle-btn does not require double quotes, as per coding-style, please use single quotes.

        )->click();
        $this->assertEquals(
            'text',
            $this->getApiKeyField()->getAttribute('type')
        );

        $this->webDriver->findElement(
            WebDriverBy::cssSelector(".password-toggle .toggle-btn")

The string literal .password-toggle .toggle-btn does not require double quotes, as per coding-style, please use single quotes.

        )->click();
        $this->assertEquals(
            'password',
            $this->getApiKeyField()->getAttribute('type')
        );
    }

    public function testApiKeyCanBeReset()
    {
        $value = $this->getApiKeyField()->getAttribute('value');

        $this->webDriver->findElement(
            WebDriverBy::cssSelector(".reset-api .btn-warning")

The string literal .reset-api .btn-warning does not require double quotes, as per coding-style, please use single quotes.

        )->click();
        $this->waitForLoadingAnimation();

        $this->assertNotEquals(
            $this->getApiKeyField()->getAttribute('value'),
            $value
        );
    }

    private function getApiKeyField()
    {
        return $this->webDriver->findElement(
            WebDriverBy::id('api-key')
        );
    }
}