minor #255 Clean tests about the PSR-3 processor (stof)

This PR was merged into the 3.x-dev branch.

Discussion
----------

Clean tests about the PSR-3 processor

Tests were not doing the right assertions, and so not covering the feature.

This was detected by looking at why PHPUnit was marking the test as risky.

Commits
-------

e9aa4ae Clean tests about the PSR-3 processor

Author: stof

Tests/DependencyInjection/FixtureMonologExtensionTest.php

@@ -194,6 +194,17 @@ public function testChannelParametersResolved()
         );
     }
 
+    public function testPsr3MessageProcessingEnabled()
+    {
+        $container = $this->getContainer('parameterized_handlers');
+
+        $logger = $container->getDefinition('monolog.handler.custom');
+
+        $methodCalls = $logger->getMethodCalls();
+
+        $this->assertContains(array('pushProcessor', array(new Reference('monolog.processor.psr_log_message'))), $methodCalls, 'The PSR-3 processor should not be enabled', false, false);
+    }
+
     public function testPsr3MessageProcessingDisabled()
     {
         $container = $this->getContainer('process_psr_3_messages_disabled');
@@ -202,12 +213,7 @@ public function testPsr3MessageProcessingDisabled()
 
         $methodCalls = $logger->getMethodCalls();
 
-        foreach ($methodCalls as $methodCall) {
-            list($methodName, $params) = $methodCall;
-            if ($methodName === 'pushProcessor') {
-                $this->assertNotEquals(array(new Definition('monolog.processor.psr_log_message')), $params);
-            }
-        }
+        $this->assertNotContains(array('pushProcessor', array(new Reference('monolog.processor.psr_log_message'))), $methodCalls, 'The PSR-3 processor should not be enabled', false, false);
     }
 
     protected function getContainer($fixture)