[Freeipa-devel] [freeipa PR#275][comment] Enhance __repr__ method of Principal

2016-11-30 Thread martbab
URL: https://github.com/freeipa/freeipa/pull/275 Title: #275: Enhance __repr__ method of Principal martbab commented: """ Fixed upstream master: https://fedorahosted.org/freeipa/changeset/38cc40ddb5bf965801500bb4f66fd965b12e3c88 """ See the full comment at

[Freeipa-devel] [freeipa PR#275][comment] Enhance __repr__ method of Principal

2016-11-29 Thread martbab
URL: https://github.com/freeipa/freeipa/pull/275 Title: #275: Enhance __repr__ method of Principal martbab commented: """ That sound like a good idea. Added such assert to the unit tests. """ See the full comment at https://github.com/freeipa/freeipa/pull/275#issuecomment-263638134 -- Manage

[Freeipa-devel] [freeipa PR#275][comment] Enhance __repr__ method of Principal

2016-11-29 Thread tiran
URL: https://github.com/freeipa/freeipa/pull/275 Title: #275: Enhance __repr__ method of Principal tiran commented: """ Can you please add a test to```ipatests/test_ipapython/test_kerberos.py``` ```test_principals```? Something along the line ```assert repr(princ) ==

[Freeipa-devel] [freeipa PR#275][comment] Enhance __repr__ method of Principal

2016-11-29 Thread martbab
URL: https://github.com/freeipa/freeipa/pull/275 Title: #275: Enhance __repr__ method of Principal martbab commented: """ Sorry I somehow botched that, but it worked nevertheless. I have re-worked the PR according to your comments. ``` In [1]: import ipapython.kerberos In [2]: p =