From 93f3b62ec88b2bd9971cdbc9dba98947acf49e6f Mon Sep 17 00:00:00 2001 From: Ian Meron Date: Mon, 23 Apr 2018 15:07:22 -0500 Subject: [PATCH] MQE-977: ErrorLogger expects source to be a log paramater - update logger to validate param existence before using "source" param --- .../FunctionalTestingFramework/Extension/ErrorLogger.php | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/Magento/FunctionalTestingFramework/Extension/ErrorLogger.php b/src/Magento/FunctionalTestingFramework/Extension/ErrorLogger.php index 287884065..822395921 100644 --- a/src/Magento/FunctionalTestingFramework/Extension/ErrorLogger.php +++ b/src/Magento/FunctionalTestingFramework/Extension/ErrorLogger.php @@ -50,7 +50,7 @@ public function logErrors($webDriver, $stepEvent) //Types available should be "server", "browser", "driver". Only care about browser at the moment. $browserLogEntries = $webDriver->manage()->getLog("browser"); foreach ($browserLogEntries as $entry) { - if ($entry["source"] === "javascript") { + if (array_key_exists("source", $entry) && $entry["source"] === "javascript") { $this->logError("javascript", $stepEvent, $entry); } }