App\Containers\Payments\Tests\Unit\ChargeUsersTest

Complexity

Total Complexity

2

Size/Duplication

Total Lines	54
Duplicated Lines	100 %

Coupling/Cohesion

Components	1
Dependencies	4

Importance

Changes	1
Bugs	0	Features	0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	testChargeWithStripe()	23	23	1
B	testChargeWithPaypal()	25	25	1

<?php

namespace App\Containers\Payments\Tests\Unit;

use App\Containers\Paypal\Actions\CreatePaypalAccountAction;
use App\Containers\Paypal\Services\ChargeWithPaypalService;
use App\Containers\Stripe\Actions\CreateStripeAccountAction;
use App\Containers\Stripe\Services\ChargeWithStripeService;
use App\Port\Tests\PHPUnit\Abstracts\TestCase;
use App\Containers\Payments\Services\PaymentsFactory;
use Illuminate\Support\Facades\App;

/**
 * Class ChargeUsersTest.
 *
 * @author Mahmoud Zalt <mahmoud@zalt.me>
 */
class ChargeUsersTest extends TestCase

This class seems to be duplicated in your project.

{

    public function testChargeWithStripe()
    {
        // get the logged in user (create one if no one is logged in)
        $user = $this->registerAndLoginTestingUser();

        // create stripe account for this user
        $createStripeAccountAction = App::make(CreateStripeAccountAction::class);
        $stripeAccount = $createStripeAccountAction->run($user, 'cus_8mBD5S1SoyD4zL', 'card_18Uck6KFvMcBUkvQorbBkYhR', 'credit', '4242', 'WsNM4K8puHbdS2VP');

$stripeAccount is not used, you could remove the assignment.

        $payId = 'ch_z8WDARKFvzcBUkvQzrbBvfhz';

        // mock the ChargeWithStripeService external API call
        $this->mock(ChargeWithStripeService::class)->shouldReceive('charge')->andReturn([
            'payment_method' => 'stripe',
            'description'    => $payId
        ]);

        $paymentsFactory = new PaymentsFactory();
        $result = $paymentsFactory->charge($user, 1000, 'USD');

        $this->assertEquals($result['payment_method'], 'stripe');
        $this->assertEquals($result['description'], $payId);
    }

    public function testChargeWithPaypal()
    {
        // get the logged in user (create one if no one is logged in)
        $user = $this->registerAndLoginTestingUser();

        // create stripe account for this user
        $createPaypalAccountAction = App::make(CreatePaypalAccountAction::class);
        $paypalAccount = $createPaypalAccountAction->run($user, '8mBD5S1SoyD4zL');

$paypalAccount is not used, you could remove the assignment.

        $payId = 'PAY-04797768K5905283VK6DGEMZ';

        // mock the ChargeWithPaypalService external API call
        $this->mock(ChargeWithPaypalService::class)->shouldReceive('charge')->andReturn([
            'payment_method' => 'paypal',
            'description'    => $payId
        ]);

        // TODO: comment out the pocking part above and test the real API call

        $paymentsFactory = new PaymentsFactory();
        $result = $paymentsFactory->charge($user, 1000, 'USD');

        $this->assertEquals($result['payment_method'], 'paypal');
        $this->assertEquals($result['description'], $payId);
    }

}