Rossmitchell\Twofactor\Tests\Integration\Abstracts\AbstractTestClass

Complexity

Total Complexity

5

Size/Duplication

Total Lines	42
Duplicated Lines	0 %

Coupling/Cohesion

Components	0
Dependencies	1

Importance

Changes	1
Bugs	1	Features	0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	getFormKey()	0	4	1
A	login()	0	12	2
A	createObject()	0	10	2

<?php
/**
 * A two factor authentication module that protects both the admin and customer logins
 * Copyright (C) 2017  Ross Mitchell
 *
 * This file is part of Rossmitchell/Twofactor.
 *
 * Rossmitchell/Twofactor is free software: you can redistribute it and/or modify
 * it under the terms of the GNU General Public License as published by
 * the Free Software Foundation, either version 3 of the License, or
 * (at your option) any later version.
 *
 * This program is distributed in the hope that it will be useful,
 * but WITHOUT ANY WARRANTY; without even the implied warranty of
 * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
 * GNU General Public License for more details.
 *
 * You should have received a copy of the GNU General Public License
 * along with this program. If not, see <http://www.gnu.org/licenses/>.
 */

namespace Rossmitchell\Twofactor\Tests\Integration\Abstracts;

use Magento\Customer\Model\Customer;
use Magento\Customer\Model\Session;
use Magento\Framework\Data\Form\FormKey;
use Magento\TestFramework\TestCase\AbstractController;

class AbstractTestClass extends AbstractController
{

    public function getFormKey()
    {
        return $this->createObject(FormKey::class)->getFormKey();
    }

    /**
     * Login the user
     *
     * @param string $customerEmail Customer to mark as logged in for the session
     *
     * @param int    $websiteId
     *
     * @return void
     * @throws \Exception
     */
    public function login($customerEmail, $websiteId = 1)
    {
        /** @var Session $session */
        $session = $this->createObject(Session::class, false);
        /** @var Customer $customer */
        $customer = $this->createObject(Customer::class);
        $customer->setWebsiteId($websiteId);
        $customerId = $customer->loadByEmail($customerEmail);
        if($session->loginById($customerId->getId()) === false) {

Expected 1 space after IF keyword; 0 found

Expected "if (...) {\n"; found "if(...) {\n"

            throw new \Exception("Could not log customer in");

Direct throw of Exception is discouraged. Use \Magento\Framework\Exception\LocalizedException instead.

        }
    }

    public function createObject($className, $new = true)
    {
        if ($new === true) {
            return $this->_objectManager->create($className);

The direct use of ObjectManager is discouraged. Inject necessary dependencies via constructor.

        }

        return $this->_objectManager->get($className);

The direct use of ObjectManager is discouraged. Inject necessary dependencies via constructor.

Code must not contain multiple empty lines in a row; found 2 empty lines.

    }

Function closing brace must go on the next line following the body; found 2 blank lines before brace

}