SMartins\PassportMultiauth\PassportMultiauth

Complexity

Total Complexity

9

Size/Duplication

Total Lines	68
Duplicated Lines	0 %

Test Coverage

Coverage

85%

Importance

Changes	3
Bugs	0	Features	0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	ignoreMigrations()	0	5	1
A	userActing()	0	4	3
A	actingAs()	0	27	5

<?php

namespace SMartins\PassportMultiauth;

use Exception;
use Illuminate\Support\Facades\App;
use Laravel\Passport\HasApiTokens;
use Laravel\Passport\Token;
use Mockery;
use SMartins\PassportMultiauth\Config\AuthConfigHelper;

class PassportMultiauth
{
    /**
     * Indicates if MultiAuth migrations will be run.
     *
     * @var bool
     */
    public static $runsMigrations = true;

    /**
     * Configure MultiAuth to not register its migrations.
     *
     * @return static
     */
    public static function ignoreMigrations()
    {
        static::$runsMigrations = false;

        return new static;
    }

    /**
     * Set the current user for the application with the given scopes.
     *
     * @param \Illuminate\Contracts\Auth\Authenticatable $user
     * @param array $scopes
     * @return \Illuminate\Contracts\Auth\Authenticatable
     * @throws Exception
     */
    public static function actingAs($user, $scopes = [])
    {
        $token = Mockery::mock(Token::class)->shouldIgnoreMissing(false);

        foreach ($scopes as $scope) {
            $token->shouldReceive('can')->with($scope)->andReturn(true);
        }

        $uses = array_flip(class_uses_recursive($user));

        if (! isset($uses[HasApiTokens::class])) {
            throw new Exception('The model ['.get_class($user).'] must uses the trait '.HasApiTokens::class);
        }

        $user->withAccessToken($token);

The method withAccessToken() does not exist on Illuminate\Contracts\Auth\Authenticatable. It seems like you code against a sub-type of Illuminate\Contracts\Auth\Authenticatable such as Illuminate\Foundation\Auth\User.

        if (isset($user->wasRecentlyCreated) && $user->wasRecentlyCreated) {

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

            $user->wasRecentlyCreated = false;
        }

        $guard = AuthConfigHelper::getUserGuard($user);

        app('auth')->guard($guard)->setUser($user);

        app('auth')->shouldUse($guard);

        return $user;
    }

    /**
     * If running unit test and try authenticate an user with actingAs($user)
     * check the guards on request to authenticate or not the user.
     *
     * @return \Illuminate\Contracts\Auth\Authenticatable|null
     */
    public static function userActing()
    {
        if (App::runningUnitTests() && $user = app('auth')->user()) {
            return $user;
        }
    }
}