Permalink
Browse files

Fixing test method names.

Adding a test for updateAll with empty values.
  • Loading branch information...
1 parent 6cc1688 commit 95d298bf284e81c10425f41afa05d41c8fe30cb6 @markstory markstory committed Mar 26, 2011
Showing with 14 additions and 2 deletions.
  1. +14 −2 cake/tests/cases/libs/model/model_write.test.php
@@ -3891,13 +3891,25 @@ function testFindAllForeignKey() {
}
/**
+ * test updateAll with empty values.
+ *
+ * @return void
+ */
+ function testUpdateAllEmptyValues() {
+ $this->loadFixtures('Author', 'Post');
+ $model = new Author();
+ $result = $model->updateAll(array('user' => '""'));
+ $this->assertTrue($result);
+ }
+
+/**
* testProductUpdateAllWithForeignKey
@ceeram
ceeram Mar 26, 2011 CakePHP member

the docblock should be updated as well

*
* @link http://code.cakephp.org/tickets/view/69
* @access public
* @return void
*/
- function testProductUpdateAll() {
+ function testUpdateAllWithJoins() {
$this->skipIf(
$this->db->config['driver'] == 'postgres',
'%s Currently, there is no way of doing joins in an update statement in postgresql'
@@ -3948,7 +3960,7 @@ function testProductUpdateAll() {
* @access public
* @return void
*/
- function testProductUpdateAllWithoutForeignKey() {
+ function testUpdateAllWithoutForeignKey() {
$this->skipIf(
$this->db->config['driver'] == 'postgres',
'%s Currently, there is no way of doing joins in an update statement in postgresql'

0 comments on commit 95d298b

Please sign in to comment.