TestMonitor\DevOps\AccessToken

Complexity

Total Complexity

4

Size/Duplication

Total Lines	69
Duplicated Lines	0 %

Coupling/Cohesion

Components	2
Dependencies	1

Test Coverage

Coverage

100%

Importance

Changes

0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	6	1
A	fromDevOps()	0	8	1
A	expired()	0	4	1
A	toArray()	0	8	1

<?php

namespace TestMonitor\DevOps;

use League\OAuth2\Client\Token\AccessToken as VSTSAccessToken;

class AccessToken
{
    /**
     * @var string
     */
    public $accessToken;

    /**
     * @var string
     */
    public $refreshToken;

    /**
     * @var string
     */
    public $expiresIn;

    /**
     * Token constructor.
     *
     * @param string $accessToken
     * @param string $refreshToken
     * @param int $expiresIn
     */
    public function __construct(string $accessToken = '', string $refreshToken = '', int $expiresIn = 0)
    {
        $this->accessToken = $accessToken;
        $this->refreshToken = $refreshToken;
        $this->expiresIn = $expiresIn;

The property $expiresIn was declared of type string, but $expiresIn is of type integer. Maybe add a type cast?

    }

    /**
     * @param \League\OAuth2\Client\Token\AccessToken $token
     *
     * @return \TestMonitor\DevOps\AccessToken
     */
    public static function fromDevOps(VSTSAccessToken $token)
    {
        return new self(
            $token->getToken(),
            $token->getRefreshToken(),
            $token->getExpires()
        );
    }

    /**
     * Determines if the access token has expired.
     *
     * @return bool
     */
    public function expired()
    {
        return ($this->expiresIn - time()) < 60;
    }

    /**
     * Returns the token as an array.
     *
     * @return array
     */
    public function toArray()
    {
        return [
            'access_token' => $this->accessToken,
            'refresh_token' => $this->refreshToken,
            'expires_in' => $this->expiresIn,
        ];
    }
}