Browse files

MDL-42517 testing Changed events test to use assertDebuggingCalled()

  • Loading branch information...
micaherne authored and danpoltawski committed Oct 24, 2013
1 parent 609aa78 commit 04be3fce3e59c2c50a0ce21f5683cc3f3e69d209
Showing with 1 addition and 4 deletions.
  1. +1 −4 lib/tests/eventslib_test.php
@@ -142,11 +142,8 @@ public function test_events_trigger__failed_instant() {
$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');

0 comments on commit 04be3fc

Please sign in to comment.