Update tests.

The old tests were relying on the fixed inherit bug. Instead check a specific permission which will result in a deny. Refs #8114
cakephp · Feb 19, 2016 · 4389c79 · 4389c79
1 parent b2509ea
commit 4389c79
Show file tree

Hide file tree

Showing 2 changed files with 1 addition and 2 deletions.
diff --git a/lib/Cake/Model/Permission.php b/lib/Cake/Model/Permission.php
@@ -129,7 +129,6 @@ public function check($aro, $aco, $action = '*') {
 			$perms = Hash::extract($perms, '{n}.' . $this->alias);
 			foreach ($perms as $perm) {
 				if ($action === '*') {
-
 					foreach ($permKeys as $key) {
 						if (!empty($perm)) {
 							if ($perm[$key] == -1) {

diff --git a/lib/Cake/Test/Case/Console/Command/AclShellTest.php b/lib/Cake/Test/Case/Console/Command/AclShellTest.php
@@ -253,7 +253,7 @@ public function testCheck() {
 		$this->Task->args = array('AuthUser.2', 'ROOT/Controller1', 'create');
 		$this->Task->check();
 
-		$this->Task->args = array('AuthUser.2', 'ROOT/Controller1', '*');
+		$this->Task->args = array('AuthUser.2', 'ROOT/Controller1', 'delete');
 		$this->Task->check();
 	}