From 04be3fce3e59c2c50a0ce21f5683cc3f3e69d209 Mon Sep 17 00:00:00 2001 From: Michael Aherne Date: Thu, 24 Oct 2013 11:24:26 +0100 Subject: [PATCH] MDL-42517 testing Changed events test to use assertDebuggingCalled() --- lib/tests/eventslib_test.php | 5 +---- 1 file changed, 1 insertion(+), 4 deletions(-) diff --git a/lib/tests/eventslib_test.php b/lib/tests/eventslib_test.php index 870bee93599..94142c1c610 100644 --- a/lib/tests/eventslib_test.php +++ b/lib/tests/eventslib_test.php @@ -142,11 +142,8 @@ class eventslib_testcase extends advanced_testcase { $this->assertEquals(1, events_trigger('test_instant', 'fail'), 'fail first event: %s'); $this->assertEquals(1, events_trigger('test_instant', 'ok'), 'this one should fail too: %s'); - // We disable debugging for this next test. It'll make some noise when it fails to dispatch - // so that problems don't go permanently unnoticed. - $CFG->debug = 0; $this->assertEquals(0, events_cron('test_instant'), 'all events should stay in queue: %s'); - $CFG->debug = $olddebug; + $this->assertDebuggingCalled(); $this->assertEquals(2, events_pending_count('test_instant'), 'two events should in queue: %s'); $this->assertEquals(0, eventslib_sample_function_handler('status'), 'verify no event dispatched yet: %s');