MQE-751: Screenshot/source capture firing after failed hooks

imeron2433 · imeron2433 · commit 6e84829b6812 · 2018-02-05T16:20:44.000-06:00
- change usage of $fail param to appease code sniff
diff --git a/src/Magento/FunctionalTestingFramework/Module/MagentoWebDriver.php b/src/Magento/FunctionalTestingFramework/Module/MagentoWebDriver.php
@@ -514,6 +514,10 @@ public function _failed(TestInterface $test, $fail)
 
         $this->addAttachment($this->pngReport, $test->getMetadata()->getName() . '.png', 'image/png');
         $this->addAttachment($this->htmlReport, $test->getMetadata()->getName() . '.html', 'text/html');
+
+        $this->debug("Failure due to : {$fail->getMessage()}");
+        $this->debug("Screenshot saved to {$this->pngReport}");
+        $this->debug("Html saved to {$this->htmlReport}");
     }
 
     /**

Original file line number	Diff line number	Diff line change
`@@ -514,6 +514,10 @@ public function _failed(TestInterface $test, $fail)`
`514`	`514`
`515`	`515`	`$this->addAttachment($this->pngReport, $test->getMetadata()->getName() . '.png', 'image/png');`
`516`	`516`	`$this->addAttachment($this->htmlReport, $test->getMetadata()->getName() . '.html', 'text/html');`
	`517`	`+`
	`518`	`+ $this->debug("Failure due to : {$fail->getMessage()}");`
	`519`	`+ $this->debug("Screenshot saved to {$this->pngReport}");`
	`520`	`+ $this->debug("Html saved to {$this->htmlReport}");`
`517`	`521`	`}`
`518`	`522`
`519`	`523`	`/**`