AdminPageTest::testMakeSurePublicCantAccess() - Code Metrics - t2t2/draft - Measure and Improve Code Quality continuously with Scrutinizer

AdminPageTest::testMakeSurePublicCantAccess() A
last analyzed 2016-09-26 09:48 UTC

↳ Parent: AdminPageTest

Complexity

Conditions	2
Paths	2

Size

Total Lines	16
Code Lines	9

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	9
CRAP Score	2.024

Importance

Changes

Metric	Value
dl	0
loc	16
ccs	9
cts	11
cp	0.8182
rs	9.4285
c	0
b	0
f	0
cc	2
eloc	9
nc	2
nop	1
crap	2.024

<?php

class AdminPageTest extends TestCase {
namespace YourVendor;

class YourClass { }

	/**
	 * Make sure public can't access admin pages
	 *
	 * @dataProvider pubUsers
	 *
	 * @param callable $userCb
	 *
	 * @return void
	 */
	public function testMakeSurePublicCantAccess($userCb) {
		// Set current user
		$user = $userCb();
		$this->flushSession();
		Route::enableFilters();
		if ($user) {
			$this->be($user);
		}

		// get admin homepage
		$this->client->request('GET', '/admin');

		$this->assertTrue($this->client->getResponse()->isRedirection());

		Route::disableFilters();
	}

	/**
	 * Return public users to test with.
	 * Because this is ran before application is created it returns callbacks.
	 *
	 * @return array
	 */
	public function pubUsers() {
		return [
			[function () {
				return null;
			}], // Not logged in
			[function () {
				User::find(2);
			}], // Nonadmin user
		];
	}

	public function testAdminHomepage() {
		/** @type User $user */
		$user = User::find(1);
		$this->be($user);

		$this->client->request('GET', '/admin');

		$this->assertTrue($this->client->getResponse()->isOk());
	}
}


1		<?php
2
3		class AdminPageTest extends TestCase {
		0 ignored issues – show Coding Style Compatibility introduced 2016-04-12 22:00 UTC by Report Bug Copy Issue Report PSR1 recommends that each class must be in a namespace of at least one level to avoid collisions. You can fix this by adding a namespace to your class: namespace YourVendor; class YourClass { } When choosing a vendor namespace, try to pick something that is not too generic to avoid conflicts with other libraries. Loading history...
4
5		/**
6		* Make sure public can't access admin pages
7		*
8		* @dataProvider pubUsers
9		*
10		* @param callable $userCb
11		*
12		* @return void
13		*/
14	2	public function testMakeSurePublicCantAccess($userCb) {
15		// Set current user
16	2	$user = $userCb();
17	2	$this->flushSession();
18	2	Route::enableFilters();
19	2	if ($user) {
20		$this->be($user);
21		}
22
23		// get admin homepage
24	2	$this->client->request('GET', '/admin');
25
26	2	$this->assertTrue($this->client->getResponse()->isRedirection());
27
28	2	Route::disableFilters();
29	2	}
30
31		/**
32		* Return public users to test with.
33		* Because this is ran before application is created it returns callbacks.
34		*
35		* @return array
36		*/
37	1	public function pubUsers() {
38		return [
39		[function () {
40	1	return null;
41		}], // Not logged in
42	1	[function () {
43	1	User::find(2);
44	1	}], // Nonadmin user
45		];
46		}
47
48	1	public function testAdminHomepage() {
49		/** @type User $user */
50	1	$user = User::find(1);
51	1	$this->be($user);
52
53	1	$this->client->request('GET', '/admin');
54
55	1	$this->assertTrue($this->client->getResponse()->isOk());
56	1	}
57		}
58

t2t2 / draft

AdminPageTest::testMakeSurePublicCantAccess() A last analyzed 2016-09-26 09:48 UTC

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like

AdminPageTest::testMakeSurePublicCantAccess() A
last analyzed 2016-09-26 09:48 UTC