RefreshUserTest::testRefreshAnotherUserById_() - Code Metrics - Inspection of "use the new `check` function from the requests cla..." - Porto-SAP/Hello-API - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( d7caaf...048f0c )

by Mahmoud

created 2017-02-28 19:01 UTC

RefreshUserTest::testRefreshAnotherUserById_() A

↳ Parent: RefreshUserTest

Complexity

Conditions	1
Paths	1

Size

Total Lines	11
Code Lines	5

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
dl	0
loc	11
rs	9.4285
c	0
b	0
f	0
cc	1
eloc	5
nc	1
nop	0

<?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->getTestingUser();
$myVar = 'Value';
$higher = false;

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

        $anotherUser = factory(User::class)->create();
        // send the HTTP request
        $response = $this->injectId($anotherUser->id)->makeCall();

        // assert response status is correct
        $this->assertEquals('500', $response->getStatusCode());
    }

}


1			<?php
2
3			namespace App\Containers\User\UI\API\Tests\Functional;
4
5			use App\Containers\User\Models\User;
6			use App\Containers\User\Tests\TestCase;
7
8			/**
9			* Class RefreshUserTest.
10			*
11			* @author Mahmoud Zalt <[email protected]>
12			*/
13			class RefreshUserTest extends TestCase
14			{
15
16			protected $endpoint = 'post@users/{id}/refresh';
17
18			protected $access = [
19			'roles' => '',
20			'permissions' => '',
21			];
22
23			public function testRefreshUserById_()
24			{
25			$user = $this->getTestingUser();
26
27			// send the HTTP request
28			$response = $this->injectId($user->id)->makeCall();
29
30			// assert response status is correct
31			$this->assertEquals('200', $response->getStatusCode());
32			}
33
34			public function testRefreshAnotherUserById_()
35			{
36			$user = $this->getTestingUser();
			0 ignored issues – show Unused Code introduced 2017-02-28 19:05 UTC by Report Bug Copy Issue Report `$user` 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...
37
38			$anotherUser = factory(User::class)->create();
39			// send the HTTP request
40			$response = $this->injectId($anotherUser->id)->makeCall();
41
42			// assert response status is correct
43			$this->assertEquals('500', $response->getStatusCode());
44			}
45
46			}
47

Porto-SAP / Hello-API

Push — master ( d7caaf...048f0c )

RefreshUserTest::testRefreshAnotherUserById_() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like