MQE-2143: Cleanup mftf.log

Author: soumyau

src/Magento/FunctionalTestingFramework/Util/Logger/MftfLogger.php

@@ -7,28 +7,51 @@
 namespace Magento\FunctionalTestingFramework\Util\Logger;
 
 use Magento\FunctionalTestingFramework\Config\MftfApplicationConfig;
-use Monolog\Handler\StreamHandler;
+use Magento\FunctionalTestingFramework\Exceptions\TestFrameworkException;
+use Monolog\Handler\HandlerInterface;
 use Monolog\Logger;
 
 class MftfLogger extends Logger
 {
     /**
-     * Prints a deprecation warning, as well as adds a log at the WARNING level during test generation.
+     * MFTF execution phase
+     *
+     * @var string
+     */
+    private $phase;
+
+    /**
+     * MftfLogger constructor.
+     *
+     * @param string $name The logging channel
+     * @param HandlerInterface[] $handlers Optional stack of handlers, the first one in the array is called first, etc.
+     * @param callable[] $processors Optional array of processors
+     * @throws TestFrameworkException
+     */
+    public function __construct($name, array $handlers = array(), array $processors = array())
+    {
+        parent::__construct($name, $handlers, $processors);
+        $this->phase = MftfApplicationConfig::getConfig()->getPhase();
+    }
+
+    /**
+     * Prints a deprecation warning, as well as adds a log at the WARNING level.
+     * Suppresses logging during execution phase.
      *
      * @param string  $message The log message.
      * @param array   $context The log context.
      * @param boolean $verbose
      * @return void
-     * @throws \Magento\FunctionalTestingFramework\Exceptions\TestFrameworkException
      */
     public function deprecation($message, array $context = [], $verbose = false)
     {
-        // Suppress print during unit testing
-        if (MftfApplicationConfig::getConfig()->getPhase() === MftfApplicationConfig::GENERATION_PHASE) {
-            $message = "DEPRECATION: " . $message;
-            if ($verbose) {
-                print ($message . json_encode($context) . "\n");
-            }
+        $message = "DEPRECATION: " . $message;
+        // print during test generation
+        if ($this->phase === MftfApplicationConfig::GENERATION_PHASE && $verbose) {
+            print ($message . json_encode($context) . "\n");
+        }
+        // suppress logging during test execution
+        if ($this->phase !== MftfApplicationConfig::EXECUTION_PHASE) {
             parent::warning($message, $context);
         }
     }
@@ -40,35 +63,35 @@ public function deprecation($message, array $context = [], $verbose = false)
      * @param array   $context The log context.
      * @param boolean $verbose
      * @return void
-     * @throws \Magento\FunctionalTestingFramework\Exceptions\TestFrameworkException
      */
     public function criticalFailure($message, array $context = [], $verbose = false)
     {
         $message = "FAILURE: " . $message;
         // Suppress print during unit testing
-        if (MftfApplicationConfig::getConfig()->getPhase() !== MftfApplicationConfig::UNIT_TEST_PHASE && $verbose) {
+        if ($this->phase !== MftfApplicationConfig::UNIT_TEST_PHASE && $verbose) {
             print ($message . implode("\n", $context) . "\n");
         }
         parent::critical($message, $context);
     }
 
     /**
-     * Adds a log record at the NOTICE level during test generation.
+     * Adds a log record at the NOTICE level.
+     * Suppresses logging during execution phase.
      *
      * @param string  $message
      * @param array   $context
      * @param boolean $verbose
      * @return void
-     * @throws \Magento\FunctionalTestingFramework\Exceptions\TestFrameworkException
      */
     public function notification($message, array $context = [], $verbose = false)
     {
-        // Print during generation phase
-        if (MftfApplicationConfig::getConfig()->getPhase() === MftfApplicationConfig::GENERATION_PHASE) {
-            $message = "NOTICE: " . $message;
-            if ($verbose) {
-                print ($message . implode("\n", $context) . "\n");
-            }
+        $message = "NOTICE: " . $message;
+        // print during test generation
+        if ($this->phase === MftfApplicationConfig::GENERATION_PHASE && $verbose) {
+            print ($message . json_encode($context) . "\n");
+        }
+        // suppress logging during test execution
+        if ($this->phase !== MftfApplicationConfig::EXECUTION_PHASE) {
             parent::notice($message, $context);
         }
     }