tests/user_check_password_script: Don't try to delete user after failed add
authorJoseph Sutton <josephsutton@catalyst.net.nz>
Tue, 10 May 2022 04:18:14 +0000 (16:18 +1200)
committerAndrew Bartlett <abartlet@samba.org>
Tue, 10 May 2022 23:05:31 +0000 (23:05 +0000)
The user account should not exist if account creation failed.

Signed-off-by: Joseph Sutton <josephsutton@catalyst.net.nz>
Reviewed-by: Andrew Bartlett <abartlet@samba.org>
python/samba/tests/samba_tool/user_check_password_script.py
selftest/knownfail.d/samba-tool-add-user [deleted file]

index 12d98eb7b6271a5739ff3b97e78a128eb9a91217..60a9add1a19677bd964e083759a45d9331161ecd 100644 (file)
@@ -42,10 +42,6 @@ class UserCheckPwdTestCase(SambaToolCmdTest):
                                             "-H", "ldap://%s" % os.environ["DC_SERVER"],
                                             "-U%s%%%s" % (os.environ["DC_USERNAME"], os.environ["DC_PASSWORD"]))
         self.assertCmdFail(result, "Should fail adding a user with %s password." % desc)
-        (result, out, err) = self.runsubcmd("user", "delete", user["name"],
-                                            "-H", "ldap://%s" % os.environ["DC_SERVER"],
-                                            "-U%s%%%s" % (os.environ["DC_USERNAME"], os.environ["DC_PASSWORD"]))
-        self.assertCmdSuccess(result, out, err, "Should delete user with %s password." % desc)
 
         (result, out, err) = self.runsubcmd("user", "add", user["name"], good_password,
                                             "-H", "ldap://%s" % os.environ["DC_SERVER"],
diff --git a/selftest/knownfail.d/samba-tool-add-user b/selftest/knownfail.d/samba-tool-add-user
deleted file mode 100644 (file)
index e487281..0000000
+++ /dev/null
@@ -1,2 +0,0 @@
-^samba.tests.samba_tool.user_check_password_script.samba.tests.samba_tool.user_check_password_script.UserCheckPwdTestCase.test_checkpassword_unacceptable.chgdcpass:local
-^samba.tests.samba_tool.user_check_password_script.samba.tests.samba_tool.user_check_password_script.UserCheckPwdTestCase.test_checkpassword_username.chgdcpass:local