Skip to content

Commit

Permalink
fixed expected and result order in asserts, also added extra test
Browse files Browse the repository at this point in the history
  • Loading branch information
ceeram authored and markstory committed Feb 23, 2012
1 parent 836f913 commit 6d3c659
Showing 1 changed file with 8 additions and 1 deletion.
9 changes: 8 additions & 1 deletion lib/Cake/Test/Case/Network/CakeRequestTest.php
Expand Up @@ -172,7 +172,7 @@ public function testPostParsing() {
'Article' => array('title')
));
$request = new CakeRequest('some/path');
$this->assertEquals($request->data, $_POST['data']);
$this->assertEquals($_POST['data'], $request->data);

$_POST = array('one' => 1, 'two' => 'three');
$request = new CakeRequest('some/path');
Expand All @@ -197,6 +197,13 @@ public function testPostParsing() {
));
$request = new CakeRequest('some/path');
$this->assertEquals($request->data, $_POST['data']);

$_POST = array('data' => array(
'Article' => array('title' => 'some title'),
'Tag' => array('Tag' => array(1, 2))
));
$request = new CakeRequest('some/path');
$this->assertEquals($request->data, $_POST['data']);
}

/**
Expand Down

1 comment on commit 6d3c659

@ceeram
Copy link
Contributor Author

@ceeram ceeram commented on 6d3c659 Feb 23, 2012

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@markstory the branch which fixed the issue had 4 commits, you only merged 2 of them to 2.0 causing a fail

Please sign in to comment.