Merge pull request #173 from magento/TurnOffReadinessLogging

MQE-1147: Turn Off Readiness Logging

Author: okolesnyk

src/Magento/FunctionalTestingFramework/Extension/PageReadinessExtension.php

@@ -251,6 +251,7 @@ private function shouldSkipCheck($step)
      * @param string $message
      * @param array  $context
      * @return void
+     * @SuppressWarnings(PHPMD)
      */
     private function logDebug($message, $context = [])
     {
@@ -264,7 +265,8 @@ private function logDebug($message, $context = [])
                 $logContext[$metric->getName() . '.failCount'] = $metric->getFailureCount();
             }
             $context = array_merge($logContext, $context);
-            $this->logger->info($message, $context);
+            //TODO REMOVE THIS LINE, UNCOMMENT LOGGER
+            //$this->logger->info($message, $context);
         }
     }
 }

src/Magento/FunctionalTestingFramework/Extension/ReadinessMetrics/AbstractMetricCheck.php

@@ -248,7 +248,7 @@ private function getCurrentValue($refresh = false)
      */
     public function getStoredValue()
     {
-        return $this->storedValue;
+        return $this->storedValue ?? null;
     }
 
     /**
@@ -306,11 +306,13 @@ public function resetTracker()
      * @param string $message
      * @param array  $context
      * @return void
+     * @SuppressWarnings(PHPMD)
      */
     protected function errorLog($message, $context = [])
     {
         $context = array_merge($this->getLogContext(), $context);
-        $this->logger->error($message, $context);
+        //TODO REMOVE THIS LINE, UNCOMMENT LOGGER
+        //$this->logger->error($message, $context);
     }
 
     /**
@@ -319,11 +321,13 @@ protected function errorLog($message, $context = [])
      * @param string $message
      * @param array  $context
      * @return void
+     * @SuppressWarnings(PHPMD)
      */
     protected function infoLog($message, $context = [])
     {
         $context = array_merge($this->getLogContext(), $context);
-        $this->logger->info($message, $context);
+        //TODO REMOVE THIS LINE, UNCOMMENT LOGGER
+        //$this->logger->info($message, $context);
     }
 
     /**
@@ -332,12 +336,14 @@ protected function infoLog($message, $context = [])
      * @param string $message
      * @param array  $context
      * @return void
+     * @SuppressWarnings(PHPMD)
      */
     protected function debugLog($message, $context = [])
     {
         if ($this->verbose) {
             $context = array_merge($this->getLogContext(), $context);
-            $this->logger->debug($message, $context);
+            //TODO REMOVE THIS LINE, UNCOMMENT LOGGER
+            //$this->logger->debug($message, $context);
         }
     }