ChargeUsersTest::testChargeWithStripeThroughPaymentProxy() - Code Metrics - Inspection of "add payment proxy feature" - Porto-SAP/Hello-API - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( 4306e3...de53c0 )

by Mahmoud

created 2017-02-22 15:49 UTC

testChargeWithStripeThroughPaymentProxy() A

↳ Parent: ChargeUsersTest

Complexity

Conditions	1
Paths	1

Size

Total Lines	23
Code Lines	11

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
dl	0
loc	23
rs	9.0856
c	0
b	0
f	0
cc	1
eloc	11
nc	1
nop	0

<?php

namespace App\Containers\Stripe\Tests\Unit;

use App\Containers\Stripe\Tasks\CreateStripeAccountObjectTask;
use App\Containers\Stripe\Tasks\ChargeWithStripeTask;
use App\Containers\Stripe\Tests\TestCase;
use App\Ship\Features\Payment\Proxies\PaymentsProxy;
use Illuminate\Support\Facades\App;

/**
 * Class ChargeUsersTest.
 *
 * @author Mahmoud Zalt <[email protected]>
 */
class ChargeUsersTest extends TestCase
{

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

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

if (rand(1, 6) > 3) {
    $higher = true;
} else {
    $higher = false;
}

        $payId = 'ch_z8WDARKFvzcBUkvQzrbBvfhz';

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

        $stripe = App::make(ChargeWithStripeTask::class);
        $result = $stripe->charge($user, 1000, 'USD');

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

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

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

if (rand(1, 6) > 3) {
    $higher = true;
} else {
    $higher = false;
}

        $payId = 'ch_z8WDARKFvzcBUkvQzrbBvfhz';

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

        // the proxy payment method
        $result = $user->charge(1000, 'USD');

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

}


1			<?php
2
3			namespace App\Containers\Stripe\Tests\Unit;
4
5			use App\Containers\Stripe\Tasks\CreateStripeAccountObjectTask;
6			use App\Containers\Stripe\Tasks\ChargeWithStripeTask;
7			use App\Containers\Stripe\Tests\TestCase;
8			use App\Ship\Features\Payment\Proxies\PaymentsProxy;
9			use Illuminate\Support\Facades\App;
10
11			/**
12			* Class ChargeUsersTest.
13			*
14			* @author Mahmoud Zalt <[email protected]>
15			*/
16			class ChargeUsersTest extends TestCase
17			{
18
19			public function testChargeWithStripe()
20			{
21			// get the logged in user (create one if no one is logged in)
22			$user = $this->createTestingUser();
23
24			// create stripe account for this user
25			$createStripeAccountAction = App::make(CreateStripeAccountObjectTask::class);
26			$stripeAccount = $createStripeAccountAction->run($user, 'cus_8mBD5S1SoyD4zL', 'card_18Uck6KFvMcBUkvQorbBkYhR', 'credit', '4242', 'WsNM4K8puHbdS2VP');
			0 ignored issues – show Unused Code introduced 2017-01-31 01:09 UTC by Report Bug Copy Issue Report `$stripeAccount` is not used, you could remove the assignment. This check looks for variable assignements that are either overwritten by other assignments or where the variable is not used subsequently. $myVar = 'Value'; $higher = false; if (rand(1, 6) > 3) { $higher = true; } else { $higher = false; } Both the `$myVar` assignment in line 1 and the `$higher` assignment in line 2 are dead. The first because `$myVar` is never used and the second because `$higher` is always overwritten for every possible time line. Loading history...
27
28			$payId = 'ch_z8WDARKFvzcBUkvQzrbBvfhz';
29
30			// mock the ChargeWithStripeTask external API call
31			$this->mock(ChargeWithStripeTask::class)->shouldReceive('charge')->andReturn([
32			'payment_method' => 'stripe',
33			'description' => $payId
34			]);
35
36			$stripe = App::make(ChargeWithStripeTask::class);
37			$result = $stripe->charge($user, 1000, 'USD');
38
39			$this->assertEquals($result['payment_method'], 'stripe');
40			$this->assertEquals($result['description'], $payId);
41			}
42
43			public function testChargeWithStripeThroughPaymentProxy()
44			{
45			// get the logged in user (create one if no one is logged in)
46			$user = $this->createTestingUser();
47
48			// create stripe account for this user
49			$createStripeAccountAction = App::make(CreateStripeAccountObjectTask::class);
50			$stripeAccount = $createStripeAccountAction->run($user, 'cus_8mBD5S1SoyD4zL', 'card_18Uck6KFvMcBUkvQorbBkYhR', 'credit', '4242', 'WsNM4K8puHbdS2VP');
			0 ignored issues – show Unused Code introduced 2017-02-22 15:53 UTC by Report Bug Copy Issue Report `$stripeAccount` is not used, you could remove the assignment. This check looks for variable assignements that are either overwritten by other assignments or where the variable is not used subsequently. $myVar = 'Value'; $higher = false; if (rand(1, 6) > 3) { $higher = true; } else { $higher = false; } Both the `$myVar` assignment in line 1 and the `$higher` assignment in line 2 are dead. The first because `$myVar` is never used and the second because `$higher` is always overwritten for every possible time line. Loading history...
51
52			$payId = 'ch_z8WDARKFvzcBUkvQzrbBvfhz';
53
54			// mock the ChargeWithStripeTask external API call
55			$this->mock(ChargeWithStripeTask::class)->shouldReceive('run')->andReturn([
56			'payment_method' => 'stripe',
57			'description' => $payId
58			]);
59
60			// the proxy payment method
61			$result = $user->charge(1000, 'USD');
62
63			$this->assertEquals($result['payment_method'], 'stripe');
64			$this->assertEquals($result['description'], $payId);
65			}
66
67			}
68

Porto-SAP / Hello-API

Push — master ( 4306e3...de53c0 )

testChargeWithStripeThroughPaymentProxy() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like