Skip to content

Commit ebb794c

Browse files
committed
fix failing test - testGenerateDump
Cake standards expect true, false and null to be lowercase. The code was being generated correctly (lowercase), but was being asserted against an incorrect example (which used uppercase NULL) Signed-off-by: Joshua Paling <joshua.paling@gmail.com>
1 parent 14cdc86 commit ebb794c

File tree

1 file changed

+17
-17
lines changed

1 file changed

+17
-17
lines changed

Test/Fixture/test_migration.txt

Lines changed: 17 additions & 17 deletions
Original file line numberDiff line numberDiff line change
@@ -2,37 +2,37 @@
22
'up' => array(
33
'create_table' => array(
44
'articles' => array(
5-
'id' => array('type' => 'integer', 'null' => false, 'default' => NULL, 'key' => 'primary'),
6-
'user_id' => array('type' => 'integer', 'null' => true, 'default' => NULL),
7-
'title' => array('type' => 'string', 'null' => true, 'default' => NULL, 'collate' => 'utf8_general_ci', 'charset' => 'utf8'),
8-
'body' => array('type' => 'text', 'null' => true, 'default' => NULL, 'collate' => 'utf8_general_ci', 'charset' => 'utf8'),
5+
'id' => array('type' => 'integer', 'null' => false, 'default' => null, 'key' => 'primary'),
6+
'user_id' => array('type' => 'integer', 'null' => true, 'default' => null),
7+
'title' => array('type' => 'string', 'null' => true, 'default' => null, 'collate' => 'utf8_general_ci', 'charset' => 'utf8'),
8+
'body' => array('type' => 'text', 'null' => true, 'default' => null, 'collate' => 'utf8_general_ci', 'charset' => 'utf8'),
99
'published' => array('type' => 'string', 'null' => true, 'default' => 'N', 'length' => 1, 'collate' => 'utf8_general_ci', 'charset' => 'utf8'),
10-
'created' => array('type' => 'datetime', 'null' => true, 'default' => NULL),
11-
'updated' => array('type' => 'datetime', 'null' => true, 'default' => NULL),
10+
'created' => array('type' => 'datetime', 'null' => true, 'default' => null),
11+
'updated' => array('type' => 'datetime', 'null' => true, 'default' => null),
1212
'indexes' => array(
1313
'PRIMARY' => array('column' => 'id', 'unique' => 1),
1414
),
1515
'tableParameters' => array('charset' => 'utf8', 'collate' => 'utf8_general_ci', 'engine' => 'InnoDB'),
1616
),
1717
'posts' => array(
18-
'id' => array('type' => 'integer', 'null' => false, 'default' => NULL, 'key' => 'primary'),
19-
'author_id' => array('type' => 'integer', 'null' => false, 'default' => NULL),
20-
'title' => array('type' => 'string', 'null' => false, 'default' => NULL, 'collate' => 'utf8_general_ci', 'charset' => 'utf8'),
21-
'body' => array('type' => 'text', 'null' => true, 'default' => NULL, 'collate' => 'utf8_general_ci', 'charset' => 'utf8'),
18+
'id' => array('type' => 'integer', 'null' => false, 'default' => null, 'key' => 'primary'),
19+
'author_id' => array('type' => 'integer', 'null' => false, 'default' => null),
20+
'title' => array('type' => 'string', 'null' => false, 'default' => null, 'collate' => 'utf8_general_ci', 'charset' => 'utf8'),
21+
'body' => array('type' => 'text', 'null' => true, 'default' => null, 'collate' => 'utf8_general_ci', 'charset' => 'utf8'),
2222
'published' => array('type' => 'string', 'null' => true, 'default' => 'N', 'length' => 1, 'collate' => 'utf8_general_ci', 'charset' => 'utf8'),
23-
'created' => array('type' => 'datetime', 'null' => true, 'default' => NULL),
24-
'updated' => array('type' => 'datetime', 'null' => true, 'default' => NULL),
23+
'created' => array('type' => 'datetime', 'null' => true, 'default' => null),
24+
'updated' => array('type' => 'datetime', 'null' => true, 'default' => null),
2525
'indexes' => array(
2626
'PRIMARY' => array('column' => 'id', 'unique' => 1),
2727
),
2828
'tableParameters' => array('charset' => 'utf8', 'collate' => 'utf8_general_ci', 'engine' => 'InnoDB'),
2929
),
3030
'users' => array(
31-
'id' => array('type' => 'integer', 'null' => false, 'default' => NULL, 'key' => 'primary'),
32-
'user' => array('type' => 'string', 'null' => true, 'default' => NULL, 'collate' => 'utf8_general_ci', 'charset' => 'utf8'),
33-
'password' => array('type' => 'string', 'null' => true, 'default' => NULL, 'collate' => 'utf8_general_ci', 'charset' => 'utf8'),
34-
'created' => array('type' => 'datetime', 'null' => true, 'default' => NULL),
35-
'updated' => array('type' => 'datetime', 'null' => true, 'default' => NULL),
31+
'id' => array('type' => 'integer', 'null' => false, 'default' => null, 'key' => 'primary'),
32+
'user' => array('type' => 'string', 'null' => true, 'default' => null, 'collate' => 'utf8_general_ci', 'charset' => 'utf8'),
33+
'password' => array('type' => 'string', 'null' => true, 'default' => null, 'collate' => 'utf8_general_ci', 'charset' => 'utf8'),
34+
'created' => array('type' => 'datetime', 'null' => true, 'default' => null),
35+
'updated' => array('type' => 'datetime', 'null' => true, 'default' => null),
3636
'indexes' => array(
3737
'PRIMARY' => array('column' => 'id', 'unique' => 1),
3838
),

0 commit comments

Comments
 (0)