Code review comment for lp:~canonical-platform-qa/snappy-ecosystem-tests/minor_tweaks

Revision history for this message
I Ahmad (iahmad) wrote :

LGTM - except that either the get_store_credential should throw the exception or at least print the error message to warn the user that he/she haven't set the environment variables correctly. At the moment it become little difficult to find the the reason for test failing.

« Back to merge proposal