Check client existance and provisioning app in setup_test() #52
Conversation
| @@ -26,6 +27,32 @@ def setup_test(): | |||
| If exception is raised then the testcase execution is aborted and smashbox terminates with non-zero exit code, | |||
|
|
|||
| """ | |||
| cmd = config.oc_sync_cmd | |||
| if cmd[0 - len(' --trust'):] == ' --trust': | |||
| cmd = cmd[:0 - len(' --trust')] | |||
There was a problem hiding this comment.
why? --trust and --version cannot be specified together?
There was a problem hiding this comment.
Yes, i reported it to the client team, but for now I check for it and remove it. Basically it can only have one option atm
There was a problem hiding this comment.
Please have a look on test_manager pull request. What about creating a class called test_manager, instead of creating just a raw function? It will enable to add much more plugins to the smashbox, just adding another class to the Test_Manager class itself. Including the checks which are performed in the following changes
|
Haven't forgotten about this. But it will need to wait some time more, sorry. I hope this does not prevent you from working on your branch. |
|
Yes no problem at the moment.just helps to spot errors earlier but if it works its all fine, so no need to rush |
Those checks are needed (tests can't work without them, no user creation etc.) so failing and exit(1)
@moscicki
Fix owncloud#126