for testing and deploying your application
for finding and fixing issues
for empowering human code reviews
Check for unnecessary variable assignments.
These results are based on our legacy PHP analysis, consider migrating to our new PHP analysis engine instead. Learn more
<?php
namespace Postpay\Tests\HttpClients;
use PHPUnit\Framework\TestCase;
use Postpay\Exceptions\PostpayException;
use Postpay\Http\Request;
use Postpay\HttpClients\Curl;
use Postpay\HttpClients\CurlClient;
class CurlClientTest extends TestCase
{
public function testSend()
$curl = $this->createMock(Curl::class);
$curl->method('getInfo')->willReturn(200);
$client = new CurlClient($curl);
$response = $client->send(new Request('GET'));
self::assertSame(200, $response->getStatusCode());
}
public function testSendRequest()
$request = $this->createMock(Request::class);
$url = 'https://postman-echo.com/basic-auth';
$request->method('getUrl')->willReturn($url);
$request->method('getMethod')->willReturn('GET');
$request->method('getHeaders')->willReturn([]);
$request->method('getAuth')->willReturn(['postman', 'password']);
$client = new CurlClient();
$response = $client->send($request);
public function testSendError()
$curl->method('errno')->willReturn(1);
$this->expectException(PostpayException::class);
$response = $client->send(new Request('POST'));
$response
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
public function testGetRequestHeaders()
$curl = new CurlClient();
$request = new Request('GET');
$request->setHeaders(['test' => true]);
$headers = $curl->getRequestHeaders($request);
self::assertEquals('test: 1', $headers[0]);
postpayio /
postpay-php
mongkok
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.