theorchard / monolog-cascade

Configure multiple loggers and handlers in the blink of an eye
MIT License
145 stars 62 forks source link

Fix the fixture files to account for processors. Should get back the … #27

Closed rantonmattei closed 9 years ago

rantonmattei commented 9 years ago

…coverage to 100%

@OrCharles A contributor kindly added support for Logger processors. See https://github.com/theorchard/monolog-cascade/pull/25 This makes sure the tests are set up properly to actually test a entire config parsing that includes a processor. This would fix the code coverage decrease.

Note that I picked TagProcessor class for the fixture but this is not relevant for the test. I just needed some Processor. Processors are tested in Monolog. Here, we just test that the Processor has been pushed to the Logger instance when parsing the config file.

OrCharles commented 9 years ago

Makes sense, looks good.

R+