Skip to content

Commit 7f8fe40

Browse files
authored
Merge pull request #511 from magento/MQE-1774
MQE-1774: Review community PR #343
2 parents f101b0e + de5bed6 commit 7f8fe40

File tree

1 file changed

+8
-5
lines changed

1 file changed

+8
-5
lines changed

src/Magento/FunctionalTestingFramework/Module/MagentoWebDriver.php

Lines changed: 8 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -517,11 +517,13 @@ public function scrollToTopOfPage()
517517
*/
518518
public function magentoCLI($command, $arguments = null)
519519
{
520-
try {
521-
return $this->shellExecMagentoCLI($command, $arguments);
522-
} catch (\Exception $exception) {
523-
return $this->curlExecMagentoCLI($command, $arguments);
524-
}
520+
return $this->curlExecMagentoCLI($command, $arguments);
521+
//TODO: calling bin/magento from pipeline is timing out, needs investigation (ref: MQE-1774)
522+
// try {
523+
// return $this->shellExecMagentoCLI($command, $arguments);
524+
// } catch (\Exception $exception) {
525+
// return $this->curlExecMagentoCLI($command, $arguments);
526+
// }
525527
}
526528

527529
/**
@@ -837,6 +839,7 @@ public function makeScreenshot($name = null)
837839
*
838840
* @throws \RuntimeException
839841
* @return string
842+
* @SuppressWarnings(PHPMD.UnusedPrivateMethod)
840843
*/
841844
private function shellExecMagentoCLI($command, $arguments): string
842845
{

0 commit comments

Comments
 (0)