URL: https://github.com/freeipa/freeipa/pull/240
Author: mirielka
 Title: #240: Document make_delete_command method in UserTracker
Action: opened

PR body:
"""
https://fedorahosted.org/freeipa/ticket/6485
"""

To pull the PR as Git branch:
git remote add ghfreeipa https://github.com/freeipa/freeipa
git fetch ghfreeipa pull/240/head:pr240
git checkout pr240
From 26b0211443b1239c8d4f5dee9d9029049f24861c Mon Sep 17 00:00:00 2001
From: Lenka Doudova <ldoud...@redhat.com>
Date: Tue, 15 Nov 2016 07:22:18 +0100
Subject: [PATCH] Document make_delete_command method in UserTracker

https://fedorahosted.org/freeipa/ticket/6485
---
 ipatests/test_xmlrpc/tracker/user_plugin.py | 16 +++++++++++++---
 1 file changed, 13 insertions(+), 3 deletions(-)

diff --git a/ipatests/test_xmlrpc/tracker/user_plugin.py b/ipatests/test_xmlrpc/tracker/user_plugin.py
index 4485fd9..1b35a5c 100644
--- a/ipatests/test_xmlrpc/tracker/user_plugin.py
+++ b/ipatests/test_xmlrpc/tracker/user_plugin.py
@@ -80,11 +80,21 @@ def make_create_command(self):
             )
 
     def make_delete_command(self, no_preserve=True, preserve=False):
-        """ Make function that deletes a user using user-del """
+        """ Make function that deletes a user using user-del
+
+        Arguments 'preserve' and 'no_preserve' represent implemented
+        options --preserve and --no-preserve of user-del command,
+        which are mutually exclusive.
+        If --preserve=True and --no-preserve=False, the user is moved
+        to deleted container.
+        If --preserve=True and --no-preserve=True, an error is raised.
+        If --preserve=False and --no-preserver=True, user is deleted.
+        """
 
         if preserve and not no_preserve:
-            # necessary to change some user attributes due to moving
-            # to different container
+            # --preserve=True and --no-preserve=False - user is moved to
+            # another container, hence it is necessary to change some user
+            # attributes
             self.attrs[u'dn'] = DN(
                 ('uid', self.uid),
                 api.env.container_deleteuser,
-- 
Manage your subscription for the Freeipa-devel mailing list:
https://www.redhat.com/mailman/listinfo/freeipa-devel
Contribute to FreeIPA: http://www.freeipa.org/page/Contribute/Code

Reply via email to