for testing and deploying your application
for finding and fixing issues
for empowering human code reviews
<?php
namespace App\Containers\User\UI\API\Tests\Functional;
use App\Containers\User\Models\User;
use App\Containers\User\Tests\TestCase;
/**
* Class RefreshUserTest.
*
* @author Mahmoud Zalt <[email protected]>
*/
class RefreshUserTest extends TestCase
{
protected $endpoint = 'post@users/{id}/refresh';
protected $access = [
'roles' => '',
'permissions' => '',
];
public function testRefreshUserById_()
$user = $this->getTestingUser();
// send the HTTP request
$response = $this->injectId($user->id)->makeCall();
// assert response status is correct
$this->assertEquals('200', $response->getStatusCode());
}
public function testRefreshAnotherUserById_()
$user
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
$anotherUser = factory(User::class)->create();
$response = $this->injectId($anotherUser->id)->makeCall();
$this->assertEquals('500', $response->getStatusCode());
Porto-SAP /
Hello-API
This check looks for variable assignements that are either overwritten by other assignments or where the variable is not used subsequently.
Both the
$myVarassignment in line 1 and the$higherassignment in line 2 are dead. The first because$myVaris never used and the second because$higheris always overwritten for every possible time line.