Browse Source

clarify test

euromark 12 years ago
parent
commit
5e2b7715ca
1 changed files with 9 additions and 6 deletions
  1. 9 6
      Test/Case/Model/Behavior/WhoDidItBehaviorTest.php

+ 9 - 6
Test/Case/Model/Behavior/WhoDidItBehaviorTest.php

@@ -31,6 +31,7 @@ class WhoDidItBehaviorTest extends MyCakeTestCase {
 	}
 
 	public function testSaveWithDefaultSettings() {
+		// create (id + name + created + modified)
 		$data = array(
 			'name' => 'Foo'
 		);
@@ -39,6 +40,11 @@ class WhoDidItBehaviorTest extends MyCakeTestCase {
 		$this->assertTrue((bool)$res);
 		$this->assertTrue(count($res['WhoDidItPlayer']) === 4);
 
+		// update (id + name + modified)
+		$res = $this->Model->save($data + array('id' => $this->Model->id));
+		$this->assertTrue((bool)$res);
+		$this->assertTrue(count($res['WhoDidItPlayer']) === 3);
+
 		// create a new one being logged in
 		CakeSession::write('Auth.User.id', '1');
 		$data = array(
@@ -51,13 +57,10 @@ class WhoDidItBehaviorTest extends MyCakeTestCase {
 		$this->assertEquals('1', $res['WhoDidItPlayer']['created_by']);
 		$this->assertEquals('1', $res['WhoDidItPlayer']['modified_by']);
 
-		// now update
-		$data = array(
-			'name' => 'Foo2x'
-		);
-		$res = $this->Model->save($data);
+		// now update being logged in
+		$res = $this->Model->save($data + array('id' => $this->Model->id));
 		$this->assertTrue((bool)$res);
-		$this->assertTrue(count($res['WhoDidItPlayer']) === 3);
+		$this->assertTrue(count($res['WhoDidItPlayer']) === 4);
 		$this->assertEquals('1', $res['WhoDidItPlayer']['modified_by']);
 	}