MQE-1510

[KevinBKozan]

Contribution checklist

• Pull request has a meaningful description of its purpose
• All commits are accompanied by meaningful commit messages
• All new or changed code is covered with unit/verification tests (if applicable)
• All automated tests passed successfully (all builds on Travis CI are green)
• Changes to Framework doesn't have backward incompatible changes for tests or have related Pull Request with fixes to tests

[coveralls]

Coverage increased (+0.7%) to 53.546% when pulling 6ed2be4 on MQE-1510 into 17082a1 on develop.

[coveralls]

Coverage increased (+0.3%) to 53.394% when pulling d21fa29 on MQE-1510 into 02902f9 on develop.

[soumyau]

escapeshellcmd usage seems to skip the command run in its entirety if there are escape sequences involved. AC + implementation needs to be discussed/modified based on whether we intend to fail the test with a specific http response code or find a way to execute just the original magento command.

[KevinBKozan]

@soumyau I added a check to see if escapeshellcmd made any difference to the command. That should give faster feedback on if the command is. I'm running builds to ensure this doesn't break existing tests.

That said, I'm still going to continue trying to figure out why exactly symfony is just hiding the error.

[soumyau]

Looks good. Invalid formats in cli commands throw 500 error now.