Merge branch '44711-28-r2' of git://github.com/samhemelryk/moodle
authorDan Poltawski <dan@moodle.com>
Tue, 20 May 2014 12:32:14 +0000 (20:32 +0800)
committerDan Poltawski <dan@moodle.com>
Tue, 20 May 2014 12:32:14 +0000 (20:32 +0800)
lib/phpunit/classes/util.php
question/engine/tests/questionusage_autosave_test.php

index 0eab428..af30367 100644 (file)
@@ -112,6 +112,9 @@ class phpunit_util extends testing_util {
         // Stop any message redirection.
         phpunit_util::stop_event_redirection();
 
+        // We used to call gc_collect_cycles here to ensure desctructors were called between tests.
+        // This accounted for 25% of the total time running phpunit - so we removed it.
+
         // Show any unhandled debugging messages, the runbare() could already reset it.
         self::display_debugging_messages();
         self::reset_debugging();
index 1431540..e846bc5 100644 (file)
@@ -685,4 +685,10 @@ class question_usage_autosave_test extends qbehaviour_walkthrough_test_base {
 
         $this->delete_quba();
     }
+
+    protected function tearDown() {
+        // This test relies on the destructor for the second DB connection being called before running the next test.
+        // Without this change - there will be unit test failures on "some" DBs (MySQL).
+        gc_collect_cycles();
+    }
 }