From 85e7a92e5448b1e72698c25f6347e133831e0f77 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 aefa4c2108b..6a23e95c0fb 100644 --- a/lib/tests/eventslib_test.php +++ b/lib/tests/eventslib_test.php @@ -132,11 +132,8 @@ class core_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');