From: Jule Anger <ja@sernet.de>
Date: Thu, 27 Aug 2020 07:29:51 +0000 (+0200)
Subject: samba-tool tests: rename 'computer create' to 'computer add'
X-Git-Tag: talloc-2.3.2~349
X-Git-Url: http://git.ipfire.org/gitweb.cgi?a=commitdiff_plain;h=1f61ef204d9ddfc09296dbf075b5d65006b98ae5;p=thirdparty%2Fsamba.git

samba-tool tests: rename 'computer create' to 'computer add'

Signed-off-by: Jule Anger <ja@sernet.de>
Reviewed-by: Björn Baumbach <bb@samba.org>
Reviewed-by: Douglas Bagnall <douglas.bagnall@catalyst.net.nz>
---

diff --git a/python/samba/tests/samba_tool/computer.py b/python/samba/tests/samba_tool/computer.py
index de101868ad1..27815adbe0a 100644
--- a/python/samba/tests/samba_tool/computer.py
+++ b/python/samba/tests/samba_tool/computer.py
@@ -67,7 +67,7 @@ class ComputerCmdTestCase(SambaToolCmdTest):
             self.assertCmdSuccess(result, out, err)
             self.assertNotIn(
                 "ERROR", err, "There shouldn't be any error message")
-            self.assertIn("Computer '%s' created successfully" %
+            self.assertIn("Computer '%s' added successfully" %
                           computer["name"], out)
 
             found = self._find_computer(computer["name"])
@@ -129,14 +129,14 @@ class ComputerCmdTestCase(SambaToolCmdTest):
                 self.assertFalse(found)
 
     def test_newcomputer(self):
-        """This tests the "computer create" and "computer delete" commands"""
+        """This tests the "computer add" and "computer delete" commands"""
         # try to create all the computers again, this should fail
         for computer in self.computers:
             (result, out, err) = self._create_computer(computer)
-            self.assertCmdFail(result, "Succeeded to create existing computer")
+            self.assertCmdFail(result, "Succeeded to add existing computer")
             self.assertIn("already exists", err)
 
-        # try to delete all the computers we just created
+        # try to delete all the computers we just added
         for computer in self.computers:
             (result, out, err) = self.runsubcmd("computer", "delete", "%s" %
                                                 computer["name"])
@@ -151,12 +151,12 @@ class ComputerCmdTestCase(SambaToolCmdTest):
         # test creating computers
         for computer in self.computers:
             (result, out, err) = self.runsubcmd(
-                "computer", "create", "%s" % computer["name"],
+                "computer", "add", "%s" % computer["name"],
                 "--description=%s" % computer["description"])
 
             self.assertCmdSuccess(result, out, err)
             self.assertEqual(err, "", "There shouldn't be any error message")
-            self.assertIn("Computer '%s' created successfully" %
+            self.assertIn("Computer '%s' added successfully" %
                           computer["name"], out)
 
             found = self._find_computer(computer["name"])
@@ -305,7 +305,7 @@ class ComputerCmdTestCase(SambaToolCmdTest):
 
         args = args.split()
 
-        return self.runsubcmd('computer', 'create', *args)
+        return self.runsubcmd('computer', 'add', *args)
 
     def _create_ou(self, ou):
         return self.runsubcmd("ou", "add", "OU=%s" % ou["name"],