Skip to content

Commit

Permalink
Add additional test for f23d811
Browse files Browse the repository at this point in the history
I neglected to put a negative test to ensure validatePost fails when the
URL differs.
  • Loading branch information
markstory committed Apr 26, 2014
1 parent 5b46eb7 commit a28158d
Showing 1 changed file with 33 additions and 1 deletion.
Expand Up @@ -1067,7 +1067,7 @@ public function testFormDisabledFields() {
*
* @return void
*/
public function testRadio() {
public function testValidatePostRadio() {
$this->Controller->Security->startup($this->Controller);
$key = $this->Controller->request->params['_Token']['key'];
$fields = '3be63770e7953c6d2119f5377a9303372040f66f%3An%3A0%3A%7B%7D';
Expand Down Expand Up @@ -1101,6 +1101,38 @@ public function testRadio() {
$this->assertTrue($result);
}

/**
* test validatePost uses here() as a hash input.
*
* @return void
*/
public function testValidatePostUrlAsHashInput() {
$this->Controller->Security->startup($this->Controller);

$key = $this->Controller->request->params['_Token']['key'];
$fields = '5415d31b4483c1e09ddb58d2a91ba9650b12aa83%3A';
$unlocked = '';

$this->Controller->request->data = array(
'Model' => array('username' => '', 'password' => ''),
'_Token' => compact('key', 'fields', 'unlocked')
);
$this->assertTrue($this->Controller->Security->validatePost($this->Controller));

$request = $this->getMock('CakeRequest', array('here'), array('articles/edit/1', false));
$request->expects($this->at(0))
->method('here')
->will($this->returnValue('/posts/index?page=1'));
$request->expects($this->at(1))
->method('here')
->will($this->returnValue('/posts/edit/1'));

$this->Controller->Security->request = $request;
$this->assertFalse($this->Controller->Security->validatePost($this->Controller));
$this->assertFalse($this->Controller->Security->validatePost($this->Controller));
}


/**
* test that a requestAction's controller will have the _Token appended to
* the params.
Expand Down

0 comments on commit a28158d

Please sign in to comment.