SMartins\PassportMultiauth\Testing\MultiauthActions

Complexity

Total Complexity

5

Size/Duplication

Total Lines	64
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	0

Test Coverage

Coverage

0%

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	setUp()	0	6	1
B	multiauthActingAs()	0	34	4

<?php

namespace SMartins\PassportMultiauth\Testing;

use Laravel\Passport\Client;

trait MultiauthActions
{
    /**
     * The route to generate the access token. The default value is the standard
     * route from Laravel\Passport.
     *
     * @var string
     */
    protected $oauthTokenRoute = 'oauth/token';

    public function setUp()
    {
        parent::setUp();

        $this->artisan('passport:install');

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

    }

    /**
     * Set the the Authorization header with an access token created using
     * Laravel Passport.
     *
     * @todo Change way to issue token from $this->json() to creating accessing
     *       AccessTokenController@issueToken directly.
     * @todo Pass this method to PassportMultiauth::actingAs().
     *
     * @param  \Illuminate\Contracts\Auth\Authenticatable  $user
     * @param  string $scope
     * @return $this
     */
    public function multiauthActingAs($user, $scope = '')
    {
        $client = Client::where('personal_access_client', false)
                        ->where('revoked', false)
                        ->first();

        $provider = '';
        foreach (config('auth.providers') as $p => $config) {
            if ($user instanceof $config['model']) {
                $provider = $p;
            }
        }

        $params = [
            'grant_type' => 'password',
            'client_id' => $client->id,
            'client_secret' => $client->secret,
            'username' => $user->email,

Accessing email on the interface Illuminate\Contracts\Auth\Authenticatable suggest that you code against a concrete implementation. How about adding an instanceof check?

            'password' => 'secret',
            'scope' => $scope,
        ];

        // If model to be authenticated don't has the default provider
        if (config('auth.guards.api.provider') !== $provider) {
            $params = array_merge($params, ['provider' => $provider]);
        }

        $response = $this->json('POST', $this->oauthTokenRoute, $params);

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

        $accessToken = json_decode($response->original)->access_token;

        $this->withHeader('Authorization', 'Bearer '.$accessToken);

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

        return $this;
    }
}