for testing and deploying your application
for finding and fixing issues
for empowering human code reviews
<?php
namespace App\Containers\Stripe\Tests\Unit;
use App\Containers\Stripe\Tasks\CreateStripeAccountObjectTask;
use App\Containers\Stripe\Tests\TestCase;
use Illuminate\Support\Facades\App;
/**
* Class CreateStripeAccountObjectTest.
*
* @author Mahmoud Zalt <[email protected]>
*/
class CreateStripeAccountObjectTest extends TestCase
{
public function testCreateStripeAccountObject()
// get the logged in user (create one if no one is logged in)
$user = $this->getTestingUser();
// create stripe account for this user
$createStripeAccountAction = App::make(CreateStripeAccountObjectTask::class);
$stripeAccount = $createStripeAccountAction->run($user, 'cus_8mBD5S1SoyD4zL', 'card_18Uck6KFvMcBUkvQorbBkYhR', 'credit', '4242', 'WsNM4K8puHbdS2VP');
$stripeAccount
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.
$myVar
$higher
$this->seeInDatabase('stripe_accounts', ['user_id' => $user->id]);
}
This check looks for variable assignements that are either overwritten by other assignments or where the variable is not used subsequently.
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.