From: Joseph Sutton <josephsutton@catalyst.net.nz>
Date: Tue, 10 May 2022 00:59:59 +0000 (+1200)
Subject: tests/samba-tool user_wdigest: Check command results
X-Git-Tag: talloc-2.3.4~192
X-Git-Url: http://git.ipfire.org/gitweb.cgi?a=commitdiff_plain;h=9b0f25ec498a318111a5f4fdbba3e1ce82bc0124;p=thirdparty%2Fsamba.git

tests/samba-tool user_wdigest: Check command results

Ensure that the commands to create and delete the user execute
successfully.

Signed-off-by: Joseph Sutton <josephsutton@catalyst.net.nz>
Reviewed-by: Andrew Bartlett <abartlet@samba.org>
---

diff --git a/python/samba/tests/samba_tool/user_wdigest.py b/python/samba/tests/samba_tool/user_wdigest.py
index 0d03b51fe3d..874de0d3814 100644
--- a/python/samba/tests/samba_tool/user_wdigest.py
+++ b/python/samba/tests/samba_tool/user_wdigest.py
@@ -61,14 +61,22 @@ class UserCmdWdigestTestCase(SambaToolCmdTest):
             attrs=["nETBIOSName"])
         self.netbios_domain = str(res[0]["nETBIOSName"][0])
         self.password = self.random_password()
-        self.runsubcmd("user",
-                       "create",
-                       USER_NAME,
-                       self.password)
+        result, out, err = self.runsubcmd("user",
+                                          "create",
+                                          USER_NAME,
+                                          self.password)
+        self.assertCmdSuccess(result,
+                              out,
+                              err,
+                              "Ensure user is created")
 
     def tearDown(self):
         super(UserCmdWdigestTestCase, self).tearDown()
-        self.runsubcmd("user", "delete", USER_NAME)
+        result, out, err = self.runsubcmd("user", "delete", USER_NAME)
+        self.assertCmdSuccess(result,
+                              out,
+                              err,
+                              "Ensure user is deleted")
 
     def _testWDigest(self, attribute, expected, missing=False):