App\Port\Test\PHPUnit\Traits\TestingTrait

Complexity

Total Complexity

54

Size/Duplication

Total Lines	378
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	6

Importance

Changes

0

21 Methods

Rating	Name	Duplication	Size	Complexity
A	registerAndLoginTestingAdmin()	0	8	1
A	registerAndLoginTestingDeveloper()	0	11	2
A	assertResponseContainKeys()	0	10	3
A	assertResponseContainValues()	0	10	3
A	assertResponseContainKeyValue()	0	12	2
A	migrateDatabase()	0	4	1
A	responseToArray()	0	12	3
A	formatToKeyValueToString()	0	14	3
A	mock()	0	7	1
A	getResponseObject()	0	4	1
A	injectEndpointId()	0	12	3
A	overrideSubDomain()	0	15	3
D	apiCall()	0	28	10
A	getTestingFile()	0	6	1
A	getTestingImage()	0	4	1
A	assertValidationErrorContain()	0	8	2
A	getTestingUser()	0	8	2
A	getTestingAdmin()	0	7	1
A	getLoggedInTestingUserToken()	0	4	1
A	createTestingUser()	0	18	4
B	setupTestingUserAccess()	0	13	6

<?php

namespace App\Port\Test\PHPUnit\Traits;

use App;
use App\Containers\Authorization\Tasks\AssignRoleTask;
use App\Containers\User\Actions\CreateUserAction;
use App\Containers\User\Models\User;
use Artisan;
use Dingo\Api\Http\Response as DingoAPIResponse;
use Illuminate\Http\Response;
use Illuminate\Http\UploadedFile;
use Illuminate\Support\Arr as LaravelArr;
use Illuminate\Support\Facades\Config;
use Illuminate\Support\Str as LaravelStr;
use Mockery;
use Symfony\Component\Debug\Exception\UndefinedMethodException;
use Vinkla\Hashids\Facades\Hashids;

/**
 * Class TestingTrait.
 *
 * All the functions in this trait are accessible from all your tests.
 *
 * @author  Mahmoud Zalt <mahmoud@zalt.me>
 */
trait TestingTrait
{

    /**
     * the Logged in user, used for protected routes.
     *
     * @var User
     */
    public $loggedInTestingUser;

    /**
     * @param        $endpoint
     * @param string $verb
     * @param array  $data
     * @param bool   $protected
     * @param array  $headers
     *
     * @return  mixed
     * @throws \Symfony\Component\Debug\Exception\UndefinedMethodException
     */
    public function apiCall($endpoint, $verb = 'get', array $data = [], $protected = true, array $headers = [])
    {
        // if endpoint is protected (requires token to access it's functionality)
        if ($protected && !array_has($headers, 'Authorization')) {
            // append the token to the header
            $headers['Authorization'] = 'Bearer ' . $this->getLoggedInTestingUserToken();
        }

        switch ($verb) {
            case 'get':
                $endpoint = $data ? $endpoint . '?' . http_build_query($data) : $endpoint;
                $response = $this->get($endpoint, $headers)->response;

It seems like get() must be provided by classes using this trait. How about adding it as abstract method to this trait?

                break;
            case 'post':
            case 'put':
            case 'patch':
            case 'delete':
                $response = $this->{$verb}($endpoint, $data, $headers)->response;
                break;
            case 'json:post':
                $response = $this->json('post', $endpoint, $data, $headers)->response;

It seems like json() must be provided by classes using this trait. How about adding it as abstract method to this trait?

                break;
            default:
                throw new UndefinedMethodException('Undefined HTTP Verb (' . $verb . ').');

The call to UndefinedMethodException::__construct() misses a required argument $previous.

        }

        return $response;
    }

    /**
     * @param        $fileName
     * @param        $stubDirPath
     * @param string $mimeType
     * @param null   $size
     *
     * @return  \Illuminate\Http\UploadedFile
     */
    public function getTestingFile($fileName, $stubDirPath, $mimeType = 'text/plain', $size = null)
    {
        $file = $stubDirPath . $fileName;

        return new UploadedFile($file, $fileName, $mimeType, $size, null, true); // null = null | $testMode = true

36% of this comment could be valid code. Did you maybe forget this after debugging?

    }

    /**
     * @param        $imageName
     * @param        $stubDirPath
     * @param string $mimeType
     * @param null   $size
     *
     * @return  \Illuminate\Http\UploadedFile
     */
    public function getTestingImage($imageName, $stubDirPath, $mimeType = 'image/jpeg', $size = null)
    {
        return $this->getTestingFile($imageName, $stubDirPath, $mimeType, $size);
    }

    /**
     * @param \Dingo\Api\Http\Response $response
     * @param array                    $messages
     */
    public function assertValidationErrorContain(DingoAPIResponse $response, array $messages)
    {
        $arrayResponse = json_decode($response->getContent());

        foreach ($messages as $key => $value) {
            $this->assertEquals($arrayResponse->errors->{$key}[0], $value);

It seems like assertEquals() must be provided by classes using this trait. How about adding it as abstract method to this trait?

        }
    }

    /**
     * get teh current logged in user OR create new one if no one exist
     *
     * @param null $access
     *
     * @return  \App\Containers\User\Models\User|mixed
     */
    public function getTestingUser($access = null)
    {
        if (!$user = $this->loggedInTestingUser) {
            $user = $this->createTestingUser($access);
        }

        return $user;
    }

    /**
     * @param null $permissions
     *
     * @return  \App\Containers\User\Models\User|mixed
     */
    public function getTestingAdmin($permissions = null)
    {
        return $this->getTestingUser([
            'roles'        => 'admin',
            '$permissions' => $permissions,
        ]);
    }

    /**
     * get teh current logged in user token.
     *
     * @return string
     */
    public function getLoggedInTestingUserToken()
    {
        return $this->getTestingUser()->token;
    }

    /**
     * @param null $access
     * @param null $userDetails
     *
     * @return  mixed
     */
    public function createTestingUser($access = null, $userDetails = null)
    {
        // if no user detail provided, use the default details.
        $userDetails = $userDetails ? : [
            'name'     => 'Mahmoud Zalt',
            'email'    => 'testing@hello.dev',
            'password' => 'secret.Pass7',
        ];

        // create new user and login (true)

36% of this comment could be valid code. Did you maybe forget this after debugging?

        $user = App::make(CreateUserAction::class)->run($userDetails['email'], $userDetails['password'],
            $userDetails['name'], null, null, true
        );

        $user = $this->setupTestingUserAccess($user, $access ? : (isset($this->access) ? $this->access : null));

The property access does not exist. Did you maybe forget to declare it?

        return $this->loggedInTestingUser = $user;
    }

    /**
     * @param $user
     * @param $access
     *
     * @return  mixed
     */
    private function setupTestingUserAccess($user, $access)
    {
        if (isset($access['permissions']) && !empty($access['permissions'])) {
            $user->givePermissionTo($access['permissions']);
        }
        if (isset($access['roles']) && !empty($access['roles'])) {
            if (!$user->hasRole($access['roles'])) {
                $user->assignRole($access['roles']);
            }
        }

        return $user;
    }

    /**
     * @param null $userDetails
     *
     * @return  mixed
     */
    public function registerAndLoginTestingAdmin($userDetails = null)
    {
        $user = $this->createTestingUser($userDetails);

        $user = $this->makeAdmin($user);

It seems like makeAdmin() must be provided by classes using this trait. How about adding it as abstract method to this trait?

        return $user;
    }

    /**
     * Normal user with Developer Role
     *
     * @param null $userDetails
     *
     * @return  \App\Containers\User\Models\User|mixed
     */
    public function registerAndLoginTestingDeveloper($userDetails = null)
    {
        $user = $this->getTestingUser(null, $userDetails);

The call to TestingTrait::getTestingUser() has too many arguments starting with $userDetails.

        // Give Developer Role to this User if he doesn't have it already
        if (!$user->hasRole('developer')) {
            App::make(AssignRoleTask::class)->run($user, ['developer']);
        }

        return $user;
    }

    /**
     * @param $keys
     * @param $response
     */
    public function assertResponseContainKeys($keys, $response)
    {
        if (!is_array($keys)) {
            $keys = (array)$keys;
        }

        foreach ($keys as $key) {
            $this->assertTrue(array_key_exists($key, $this->responseToArray($response)));

It seems like assertTrue() must be provided by classes using this trait. How about adding it as abstract method to this trait?

        }
    }

    /**
     * @param $values
     * @param $response
     */
    public function assertResponseContainValues($values, $response)
    {
        if (!is_array($values)) {
            $values = (array)$values;
        }

        foreach ($values as $value) {
            $this->assertTrue(in_array($value, $this->responseToArray($response)));

It seems like assertTrue() must be provided by classes using this trait. How about adding it as abstract method to this trait?

        }
    }

    /**
     * @param $data
     * @param $response
     */
    public function assertResponseContainKeyValue($data, $response)
    {
        $response = json_encode(LaravelArr::sortRecursive(
            (array)$this->responseToArray($response)
        ));

        foreach (LaravelArr::sortRecursive($data) as $key => $value) {
            $expected = $this->formatToExpectedJson($key, $value);

It seems like formatToExpectedJson() must be provided by classes using this trait. How about adding it as abstract method to this trait?

            $this->assertTrue(LaravelStr::contains($response, $expected),

It seems like assertTrue() must be provided by classes using this trait. How about adding it as abstract method to this trait?

                "The JSON fragment [ {$expected} ] does not exist in the response [ {$response} ].");
        }
    }

    /**
     * Migrate the database.
     */
    public function migrateDatabase()
    {
        Artisan::call('migrate');
    }

    /**
     * @param $response
     *
     * @return  mixed
     */
    private function responseToArray($response)
    {
        if ($response instanceof \Illuminate\Http\Response) {
            $response = json_decode($response->getContent(), true);
        }

        if (array_key_exists('data', $response)) {
            $response = $response['data'];
        }

        return $response;
    }

    /**
     * Format the given key and value into a JSON string for expectation checks.
     *
     * @param string $key
     * @param mixed  $value
     *
     * @return string
     */
    private function formatToKeyValueToString($key, $value)
    {
        $expected = json_encode([$key => $value]);

        if (LaravelStr::startsWith($expected, '{')) {
            $expected = substr($expected, 1);
        }

        if (LaravelStr::endsWith($expected, '}')) {
            $expected = substr($expected, 0, -1);
        }

        return $expected;
    }

    /**
     * Mocking helper
     *
     * @param $class
     *
     * @return  \Mockery\MockInterface
     */
    public function mock($class)
    {
        $mock = Mockery::mock($class);
        App::instance($class, $mock);

        return $mock;
    }

    /**
     * get response object, get the string content from it and convert it to an std object
     * making it easier to read
     *
     * @param $response
     *
     * @return  mixed
     */
    public function getResponseObject(Response $response)
    {
        return json_decode($response->getContent());
    }

    /**
     * Inject the ID in the Endpoint URI
     *
     * Example: you give it ('users/{id}/stores', 100) it returns 'users/100/stores'
     *
     * @param      $endpoint
     * @param      $id
     * @param bool $skipEncoding
     *
     * @return  mixed
     */
    public function injectEndpointId($endpoint, $id, $skipEncoding = false)
    {
        // In case Hash ID is enabled it will encode the ID first
        if (Config::get('hello.hash-id')) {

            if (!$skipEncoding) {
                $id = Hashids::encode($id);
            }
        }

        return str_replace("{id}", $id, $endpoint);
    }

    /**
     * override default URL subDomain in case you want to change it for some tests
     *
     * @param      $subDomain
     * @param null $url
     */
    public function overrideSubDomain($subDomain, $url = null)
    {
        $url = ($url) ? : $this->baseUrl;

The property baseUrl does not exist. Did you maybe forget to declare it?

        $info = parse_url($url);

        $array = explode('.', $info['host']);

        $withoutDomain = (array_key_exists(count($array) - 2,
                $array) ? $array[count($array) - 2] : '') . '.' . $array[count($array) - 1];

        $newSubDomain = $info['scheme'] . '://' . $subDomain . '.' . $withoutDomain;

        $this->baseUrl = $newSubDomain;
    }

}