ytake / Laravel-FluentLogger

fluent logger for laravel (with Monolog handler for Fluentd)
MIT License
63 stars 23 forks source link

Test 2 fails #14

Closed pravindahal closed 7 years ago

pravindahal commented 8 years ago
$ ./vendor/bin/phpunit
PHPUnit 4.8.30 by Sebastian Bergmann and contributors.
Warning:    The Xdebug extension is not loaded
        No code coverage will be generated.

.E....

Time: 115 ms, Memory: 4.00MB

There was 1 error:

1) FluentHandlerTest::testLogHandler
Invalid argument supplied for foreach()

/Users/.../Laravel-FluentLogger/vendor/monolog/monolog/src/Monolog/Formatter/LineFormatter.php:80
/Users/.../Laravel-FluentLogger/vendor/monolog/monolog/src/Monolog/Handler/AbstractProcessingHandler.php:35
/Users/.../Laravel-FluentLogger/tests/FluentHandlerTest.php:31

FAILURES!
Tests: 6, Assertions: 5, Errors: 1.
pravindahal commented 8 years ago

I have fixed the test along with this PR: https://github.com/ytake/Laravel-FluentLogger/pull/15