PaymentInstrumentClient::get() - Code Metrics - Inspection of "add tests to disable payment instrument" - brucepc/sum-up - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Push — master ( 519b06...806fd9 )

by Bruce Pinheiro de

created 2018-03-04 21:45 UTC

PaymentInstrumentClient::get() A

↳ Parent: PaymentInstrumentClient

Complexity

Conditions	2
Paths	2

Size

Total Lines	11
Code Lines	6

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	7
CRAP Score	2

Importance

Changes

Metric	Value
cc	2
eloc	6
nc	2
nop	0
dl	0
loc	11
ccs	7
cts	7
cp	1
crap	2
rs	9.4285
c	0
b	0
f	0

<?php

namespace BPCI\SumUp\Customer\PaymentInstrument;

use BPCI\SumUp\ContextInterface;
use BPCI\SumUp\Customer\CustomerInterface;
use BPCI\SumUp\OAuth\AccessToken;
use BPCI\SumUp\SumUpClientInterface;
use BPCI\SumUp\Traits\Client;
use BPCI\SumUp\Traits\ClientInterface;
use GuzzleHttp\Psr7\Response;
use Psr\Http\Message\ResponseInterface;


class PaymentInstrumentClient implements PaymentInstrumentClientInterface, ClientInterface
{
    use Client;

    protected $context;
    protected $options = [];
    protected $token;
    /**
     * @var CustomerInterface
     */
    protected $customer;
    /**
     * @var Response
     */
    protected $lastResponse;

    static function getScopes(): array
    {
        return [
            'payment_instruments'
        ];
    }

	/**
     * Retrieve a paymentInstrument from server and fill the $paymentInstrument Object with response.
	 *
	 */
    public function get(): array
    {
        $response = [];
        if ($this->request('get')) {
            $response = \GuzzleHttp\json_decode(
                $this->lastResponse->getBody(),
                true
            );
        };

        return $response;
	}

	/**
     * Delete an paymentInstrument from server.
	 *
     * @param PaymentInstrumentInterface $paymentInstrument
     * @return bool
	 */
    public function disable(PaymentInstrumentInterface $paymentInstrument):?bool
    {
        $uri = self::getEndPoint().'/'.$paymentInstrument->getToken();
        $uri = self::/** @scrutinizer ignore-call */ getEndPoint().'/'.$paymentInstrument->getToken();

        return $this->request('delete', $paymentInstrument, $uri);
	}

	/**
	 * CheckoutClientInterface constructor.
	 * @param ContextInterface $context
     * @param array $options
     */
    public function __construct(ContextInterface $context, ?array $options = [])
	{
        $this->context = $context;
        $this->options = $options;
	}

	/**
	 * Return last response of client
	 * @return ResponseInterface
	 */
	function getLastResponse(): ResponseInterface

	{
        return $this->lastResponse;
	}

	/**
	 * return the context used to created the client.
	 * @return ContextInterface
	 */
	function getContext(): ContextInterface

	{
        return $this->context;
	}

    /**
     * @param PaymentInstrumentInterface $object
     * @param string|null $type
     * @return mixed
     */
    static function getBody($object, string $type = null)

    {
        return null;
    }

    /**
     * @param ResponseInterface $response
     * @return SumUpClientInterface
     */
    function setLastResponse(ResponseInterface $response): SumUpClientInterface

    {
        $this->lastResponse = $response;
class Alien {}

class Dalek extends Alien {}

class Plot
{
    /** @var  Dalek */
    public $villain;
}

$alien = new Alien();
$plot = new Plot();
if ($alien instanceof Dalek) {
    $plot->villain = $alien;
}

        return $this;
    }

    /**
     * @return array
     */
    function getOptions(): array

    {
        return $this->options;

    }

    /**
     * @return string
     */
    function getEndPoint(): string

    {
        return 'customers/'.$this->customer->getCustomerId().'/payment-instruments';
    }

    /**
     * @param AccessToken $token
     * @return SumUpClientInterface
     */
    function setToken(AccessToken $token): SumUpClientInterface

    {
        $this->token = $token;

        return $this;
    }

    /**
     * @return AccessToken
     */
    function getToken():? AccessToken

    {
        return $this->token;
    }

    /**
     * @param CustomerInterface $customer
     * @return PaymentInstrumentClientInterface
     */
    public function setCustomer(CustomerInterface $customer): PaymentInstrumentClientInterface
    {
        $this->customer = $customer;

        return $this;
    }

    /**
     * @return CustomerInterface
     */
    public function getCustomer(): CustomerInterface
    {
        return $this->customer;
	}
}


brucepc / sum-up

Push — master ( 519b06...806fd9 )

PaymentInstrumentClient::get() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like