On 10/18/2013 04:21 PM, Ana Krivokapic wrote:
On 09/30/2013 05:05 PM, Petr Viktorin wrote:
Hello,
This patch introduces an user "template" with the result of a default
user add/show. The template is then customized and used in each test.
This makes the tests shorter, and highlights the "non-default"
(interesting) pieces of the result instead of presenting a wall of text.
Also, when a new default attribute is added to user results (as is the
case in my upcoming ACI patches), there's now only one place to change.
_______________________________________________
Freeipa-devel mailing list
Freeipa-devel@redhat.com
https://www.redhat.com/mailman/listinfo/freeipa-devel
ACK with two tiny nitpicks:
+ Attributes named in ``omit`` are removed from the result; any additional
+ or non-default values can be specified in``overrides``.
^^^^^ missing space
+ """
+ # sn can be None; this should only used from `get_admin_result`
... this should only *be* used ...
Thanks for your attention to detail!
Fixed & pushed to master: 756b997a7d2a4c23b41469ee272e412d7f8ca19f
--
PetrĀ³
_______________________________________________
Freeipa-devel mailing list
Freeipa-devel@redhat.com
https://www.redhat.com/mailman/listinfo/freeipa-devel