Skip to content
This repository has been archived by the owner on Jan 30, 2020. It is now read-only.

Commit

Permalink
Change name of invokable to firephp instead of firebug.
Browse files Browse the repository at this point in the history
Added test case to check invokable.
  • Loading branch information
Show file tree
Hide file tree
Showing 2 changed files with 12 additions and 1 deletion.
2 changes: 1 addition & 1 deletion src/WriterPluginManager.php
Original file line number Diff line number Diff line change
Expand Up @@ -26,7 +26,7 @@ class WriterPluginManager extends AbstractPluginManager
*/
protected $invokableClasses = array(
'db' => 'Zend\Log\Writer\Db',
'firebug' => 'Zend\Log\Writer\Firebug',
'firephp' => 'Zend\Log\Writer\FirePhp',
'mail' => 'Zend\Log\Writer\Mail',
'mock' => 'Zend\Log\Writer\Mock',
'null' => 'Zend\Log\Writer\Null',
Expand Down
11 changes: 11 additions & 0 deletions test/WriterPluginManagerTest.php
Original file line number Diff line number Diff line change
Expand Up @@ -20,6 +20,11 @@
*/
class WriterPluginManagerTest extends \PHPUnit_Framework_TestCase
{
/**
* @var WriterPluginManager
*/
protected $plugins;

public function setUp()
{
$this->plugins = new WriterPluginManager();
Expand All @@ -30,4 +35,10 @@ public function testRegisteringInvalidWriterRaisesException()
$this->setExpectedException('Zend\Log\Exception\InvalidArgumentException', 'must implement');
$this->plugins->setService('test', $this);
}

public function testInvokableClassFirephp()
{
$firephp = $this->plugins->get('firephp');
$this->assertInstanceOf('Zend\Log\Writer\Firephp', $firephp);
}
}

0 comments on commit f6430c3

Please sign in to comment.