Maciej Szulik added the comment:

Milan one more thing to change in your patch, in all your TCs you're modifying 
client state, I mean this part:

# lets cheat a bit here:
client.state = 'SELECTED'

It's OK to do that on the mocked server, but this is not acceptable doing on 
the tested library. It's better to call client.login() to change the state to 
'AUTH' properly and only after that, call the actual method under test.

----------

_______________________________________
Python tracker <rep...@bugs.python.org>
<http://bugs.python.org/issue22137>
_______________________________________
_______________________________________________
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

Reply via email to