Skip to content

Commit

Permalink
Adding return checks, and changing mock creation.
Browse files Browse the repository at this point in the history
  • Loading branch information
markstory committed Aug 11, 2010
1 parent fac9773 commit 54132cb
Showing 1 changed file with 4 additions and 4 deletions.
8 changes: 4 additions & 4 deletions cake/tests/cases/libs/view/helper_collection.test.php
Expand Up @@ -134,7 +134,7 @@ function testTrigger() {
$this->Helpers->TriggerMockForm->expects($this->once())->method('beforeRender')
->with('one', 'two');

$this->Helpers->trigger('beforeRender', array('one', 'two'));
$this->asserTrue($this->Helpers->trigger('beforeRender', array('one', 'two')));
}

/**
Expand All @@ -144,8 +144,8 @@ function testTrigger() {
*/
function testTriggerWithDisabledHelpers() {
if (!class_exists('TriggerMockHtmlHelper')) {
$this->getMock('HtmlHelper', array(), array($this->View), 'TriggerMockHtmlHelper');
$this->getMock('FormHelper', array(), array($this->View), 'TriggerMockFormHelper');
$this->getMock('HtmlHelper', array(), array(), 'TriggerMockHtmlHelper', false);
$this->getMock('FormHelper', array(), array(), 'TriggerMockFormHelper', false);
}

$this->Helpers->load('TriggerMockHtml');
Expand All @@ -157,7 +157,7 @@ function testTriggerWithDisabledHelpers() {

$this->Helpers->disable('TriggerMockForm');

$this->Helpers->trigger('beforeRender', array('one', 'two'));
$this->asserTrue($this->Helpers->trigger('beforeRender', array('one', 'two')));
}

/**
Expand Down

0 comments on commit 54132cb

Please sign in to comment.