[MINOR] Enable logging for successful tests behind argv param

TestArmada / magellan

Large Scale Automated Testing for Mocha, Nightwatch, Appium, Nodejs, etc

MIT License

287 stars 76 forks source link

[MINOR] Enable logging for successful tests behind argv param #268

Closed mukundgoel closed 5 years ago

mukundgoel commented 5 years ago

Adding option to enable logging if tests are successful. Use case is some of the successful tests are taking a long time to execute on Sauce Labs and we are trying to figure out which commands are taking longer so we can optimize them.

Verified on local with the Checkout app that it works and doesn't break if user didn't pass enablePassedTestsLogging in their magellan.json.

CLAassistant commented 5 years ago

All committers have signed the CLA.

mukundgoel commented 5 years ago

Thanks @chiahrens @StevenUpForever