From d95889a98d32a7f7cbfbb21ea5cadbe4aaf01eef Mon Sep 17 00:00:00 2001 From: Matt Lehner Date: Fri, 20 Apr 2012 13:25:38 -0400 Subject: [PATCH] code cleanups as suggested by @stof --- tests/Monolog/Formatter/GelfMessageFormatterTest.php | 8 ++++---- tests/Monolog/Handler/GelfHandlerTest.php | 5 ++--- tests/bootstrap.php | 2 +- 3 files changed, 7 insertions(+), 8 deletions(-) diff --git a/tests/Monolog/Formatter/GelfMessageFormatterTest.php b/tests/Monolog/Formatter/GelfMessageFormatterTest.php index d03fc5c1..94092689 100644 --- a/tests/Monolog/Formatter/GelfMessageFormatterTest.php +++ b/tests/Monolog/Formatter/GelfMessageFormatterTest.php @@ -18,7 +18,7 @@ class GelfMessageFormatterTest extends \PHPUnit_Framework_TestCase /** * @covers Monolog\Formatter\GelfMessageFormatter::format */ - function testDefaultFormatter() + public function testDefaultFormatter() { $formatter = new GelfMessageFormatter(); $record = array( @@ -53,7 +53,7 @@ class GelfMessageFormatterTest extends \PHPUnit_Framework_TestCase /** * @covers Monolog\Formatter\GelfMessageFormatter::format */ - function testFormatWithFileAndLine() + public function testFormatWithFileAndLine() { $formatter = new GelfMessageFormatter(); $record = array( @@ -76,7 +76,7 @@ class GelfMessageFormatterTest extends \PHPUnit_Framework_TestCase /** * @covers Monolog\Formatter\GelfMessageFormatter::format */ - function testFormatWithContext() + public function testFormatWithContext() { $formatter = new GelfMessageFormatter(); $record = array( @@ -114,7 +114,7 @@ class GelfMessageFormatterTest extends \PHPUnit_Framework_TestCase /** * @covers Monolog\Formatter\GelfMessageFormatter::format */ - function testFormatWithExtra() + public function testFormatWithExtra() { $formatter = new GelfMessageFormatter(); $record = array( diff --git a/tests/Monolog/Handler/GelfHandlerTest.php b/tests/Monolog/Handler/GelfHandlerTest.php index 5e438d46..a0af4830 100644 --- a/tests/Monolog/Handler/GelfHandlerTest.php +++ b/tests/Monolog/Handler/GelfHandlerTest.php @@ -30,9 +30,8 @@ class GelfHandlerTest extends TestCase { public function setUp() { - if (!class_exists("Gelf\MessagePublisher")) - { - $this->markTestSkipped("https://github.com/mlehner/gelf-php not installed"); + if (!class_exists("Gelf\MessagePublisher")) { + $this->markTestSkipped("mlehner/gelf-php not installed"); } } diff --git a/tests/bootstrap.php b/tests/bootstrap.php index ea4bb20d..b5a19634 100644 --- a/tests/bootstrap.php +++ b/tests/bootstrap.php @@ -9,5 +9,5 @@ * file that was distributed with this source code. */ -require_once __DIR__ . "/../vendor/.composer/autoload.php"; +require_once __DIR__ . "/../vendor/autoload.php"; require_once __DIR__.'/Monolog/TestCase.php';