From eb17e8ecd58e51dfbd6fde85ec339af0f7c7a1a3 Mon Sep 17 00:00:00 2001 From: Russell Smith Date: Wed, 26 Jun 2013 20:37:42 +1000 Subject: [PATCH] MDL-39725 database: Include stats collections in unit tests Conflicts: lib/ddl/tests/ddl_test.php --- lib/ddl/tests/ddl_test.php | 11 ++++++++--- 1 file changed, 8 insertions(+), 3 deletions(-) diff --git a/lib/ddl/tests/ddl_test.php b/lib/ddl/tests/ddl_test.php index 77b60864fc9..068aa403745 100644 --- a/lib/ddl/tests/ddl_test.php +++ b/lib/ddl/tests/ddl_test.php @@ -1516,7 +1516,10 @@ class ddl_testcase extends database_driver_testcase { $this->assertEquals($records[1]->secondname, $this->records['test_table1'][0]->secondname); $this->assertEquals($records[2]->intro, $this->records['test_table1'][1]->intro); - // Drop table1 + // Collect statistics about the data in the temp table. + $DB->update_temp_table_stats(); + + // Drop table1. $dbman->drop_table($table1); $this->assertFalse($dbman->table_exists('test_table1')); @@ -1529,8 +1532,10 @@ class ddl_testcase extends database_driver_testcase { $this->assertTrue($e instanceof ddl_table_missing_exception); } - // Fill/modify/delete a few table0 records - // TODO: that's + // Collect statistics about the data in the temp table with less tables. + $DB->update_temp_table_stats(); + + // Fill/modify/delete a few table0 records. // Drop table0 $dbman->drop_table($table0);