s4:torture:drs: Fix code spelling
authorAndreas Schneider <asn@samba.org>
Thu, 3 Aug 2023 13:40:42 +0000 (15:40 +0200)
committerAndrew Bartlett <abartlet@samba.org>
Mon, 14 Aug 2023 21:45:30 +0000 (21:45 +0000)
Signed-off-by: Andreas Schneider <asn@samba.org>
Reviewed-by: Andrew Bartlett <abartlet@samba.org>
source4/torture/drs/python/fsmo.py
source4/torture/drs/python/link_conflicts.py
source4/torture/drs/python/repl_move.py
source4/torture/drs/python/repl_rodc.py
source4/torture/drs/python/repl_schema.py
source4/torture/drs/python/repl_secdesc.py
source4/torture/drs/python/replica_sync.py
source4/torture/drs/python/ridalloc_exop.py

index 6021ce4ad703f1c343735e30ea6078e37e9ce604..55805b92af9508649384cc7b67224ae789838f3d 100644 (file)
@@ -83,7 +83,7 @@ class DrsFsmoTestCase(drs_base.DrsBaseTestCase):
         cur_master = ''
         retries = int(self.fsmo_wait_max_time / self.fsmo_wait_sleep_time) + 1
         for i in range(0, retries):
-            # check if master has been transfered
+            # check if master has been transferred
             res = ldb_dc.search(role_dn,
                                 scope=SCOPE_BASE, attrs=["fSMORoleOwner"])
             assert len(res) == 1, "Only one fSMORoleOwner value expected!"
@@ -103,7 +103,7 @@ class DrsFsmoTestCase(drs_base.DrsBaseTestCase):
         print("Testing for %s role transfer from %s to %s" % (role, self.dnsname_dc1, self.dnsname_dc2))
 
         self._net_fsmo_role_transfer(DC=self.dnsname_dc2, role=role)
-        # check if the role is transfered
+        # check if the role is transferred
         (res, master) = self._wait_for_role_transfer(ldb_dc=self.ldb_dc2,
                                                      role_dn=role_dn,
                                                      master=self.dsServiceName_dc2)
@@ -113,7 +113,7 @@ class DrsFsmoTestCase(drs_base.DrsBaseTestCase):
         # dc1 gets back the role from dc2
         print("Testing for %s role transfer from %s to %s" % (role, self.dnsname_dc2, self.dnsname_dc1))
         self._net_fsmo_role_transfer(DC=self.dnsname_dc1, role=role)
-        # check if the role is transfered
+        # check if the role is transferred
         (res, master) = self._wait_for_role_transfer(ldb_dc=self.ldb_dc1,
                                                      role_dn=role_dn,
                                                      master=self.dsServiceName_dc1)
@@ -123,7 +123,7 @@ class DrsFsmoTestCase(drs_base.DrsBaseTestCase):
         # dc1 keeps the role
         print("Testing for no-op %s role transfer from %s to %s" % (role, self.dnsname_dc2, self.dnsname_dc1))
         self._net_fsmo_role_transfer(DC=self.dnsname_dc1, role=role, noop=True)
-        # check if the role is transfered
+        # check if the role is transferred
         (res, master) = self._wait_for_role_transfer(ldb_dc=self.ldb_dc1,
                                                      role_dn=role_dn,
                                                      master=self.dsServiceName_dc1)
index 2c2f9a653fe1ec1ea0a2381ce9ac904dcc8609c4..d344b7e49717b67bae6b9a6fdbaea841bbc7c42a 100644 (file)
@@ -206,7 +206,7 @@ class DrsReplicaLinkConflictTestCase(drs_base.DrsBaseTestCase):
         res2 = self.ldb_dc2.search(base="<GUID=%s>" % src_guid,
                                    scope=SCOPE_BASE, attrs=["managedBy"])
 
-        # check the object has only have one occurence of the single-valued
+        # check the object has only have one occurrence of the single-valued
         # attribute and it matches on both DCs
         self.assert_attrs_match(res1, res2, "managedBy", 1)
 
@@ -253,7 +253,7 @@ class DrsReplicaLinkConflictTestCase(drs_base.DrsBaseTestCase):
         res2 = self.ldb_dc2.search(base="<GUID=%s>" % src_guid,
                                    scope=SCOPE_BASE, attrs=["managedBy"])
 
-        # check the object has only have one occurence of the single-valued
+        # check the object has only have one occurrence of the single-valued
         # attribute and it matches on both DCs
         self.assert_attrs_match(res1, res2, "managedBy", 1)
 
@@ -657,7 +657,7 @@ class DrsReplicaLinkConflictTestCase(drs_base.DrsBaseTestCase):
         res2 = self.ldb_dc2.search(base="<GUID=%s>" % src_guid,
                                    scope=SCOPE_BASE, attrs=["managedBy"])
 
-        # check the object has only have one occurence of the single-valued
+        # check the object has only have one occurrence of the single-valued
         # attribute and it matches on both DCs
         self.assert_attrs_match(res1, res2, "managedBy", 1)
 
@@ -713,7 +713,7 @@ class DrsReplicaLinkConflictTestCase(drs_base.DrsBaseTestCase):
         res2 = self.ldb_dc2.search(base="<GUID=%s>" % src_guid,
                                    scope=SCOPE_BASE, attrs=["managedBy"])
 
-        # check the object has only have one occurence of the single-valued
+        # check the object has only have one occurrence of the single-valued
         # attribute and it matches on both DCs
         self.assert_attrs_match(res1, res2, "managedBy", 1)
 
index c855150ff85bb317a19471264f40bbec92d8fe7b..c206ab818f84972f041dc59d699071c78555b73c 100644 (file)
@@ -2451,7 +2451,7 @@ class DrsMoveBetweenTreeOfObjectTestCase(drs_base.DrsBaseTestCase):
         self._net_drs_replicate(DC=self.dnsname_dc2, fromDC=self.dnsname_dc1, forced=True)
 
     def test_ReplicateAddInConflictOU_time(self):
-        """Verifies how an object is replicated between two DCs, when created in an ambigious location
+        """Verifies how an object is replicated between two DCs, when created in an ambiguous location
            This test should verify that:
             - Without replication, two conflicting objects can be created
             - force the conflict resolution algorithm so we know which copy will win
@@ -2486,7 +2486,7 @@ class DrsMoveBetweenTreeOfObjectTestCase(drs_base.DrsBaseTestCase):
         # object something to be under on both DCs.
 
         # We sleep between the two adds so that DC1 adds second, and
-        # so wins the conflict resoution due to a later creation time
+        # so wins the conflict resolution due to a later creation time
         # (modification timestamp on the name attribute).
         self.ldb_dc2.add(self.ou2)
         time.sleep(1)
@@ -2496,7 +2496,7 @@ class DrsMoveBetweenTreeOfObjectTestCase(drs_base.DrsBaseTestCase):
         new_dn.add_base(self.ou2_dn)
         self.ldb_dc1.rename(user_dn, new_dn)
 
-        # Now that we have renamed the user (and so bumpted the
+        # Now that we have renamed the user (and so bumped the
         # usnChanged), bump the value on the OUs.
         msg = ldb.Message()
         msg.dn = self.ou2_dn
@@ -2531,7 +2531,7 @@ class DrsMoveBetweenTreeOfObjectTestCase(drs_base.DrsBaseTestCase):
         self._net_drs_replicate(DC=self.dnsname_dc2, fromDC=self.dnsname_dc1, forced=True)
 
     def test_ReplicateAddInConflictOU2(self):
-        """Verifies how an object is replicated between two DCs, when created in an ambigious location
+        """Verifies how an object is replicated between two DCs, when created in an ambiguous location
            This test should verify that:
             - Without replication, two conflicting objects can be created
             - force the conflict resolution algorithm so we know which copy will win
@@ -2573,7 +2573,7 @@ class DrsMoveBetweenTreeOfObjectTestCase(drs_base.DrsBaseTestCase):
         new_dn.add_base(self.ou2_dn)
         self.ldb_dc1.rename(user_dn, new_dn)
 
-        # Now that we have renamed the user (and so bumpted the
+        # Now that we have renamed the user (and so bumped the
         # usnChanged), bump the value on the OUs.
         msg = ldb.Message()
         msg.dn = self.ou2_dn
index 8bdb04da048283288ae3a53e45d0b727e631dc19..ab3d6fa19282ed35443878b98a1d83757ec40bf6 100644 (file)
@@ -696,7 +696,7 @@ class DrsRodcTestCase(drs_base.DrsBaseTestCase):
 
             if list_1[i].attid in changed_attributes:
                 # We do the changes too quickly, so unless we put sleeps
-                # inbetween calls, these remain the same. Checking the USNs
+                # in between calls, these remain the same. Checking the USNs
                 # is enough.
                 pass
                 #self.assertTrue(list_1[i].originating_change_time < list_2[i].originating_change_time)
index 44317ac881968c0737ad87b20cf2bb25d291659b..9c039a5bdc3c1190959d5fdf19cd5866c66062a2 100644 (file)
@@ -298,7 +298,7 @@ class DrsReplSchemaTestCase(drs_base.DrsBaseTestCase):
         self._check_object(a_dn)
 
     def test_attribute_on_ou(self):
-        """Simple test having an OU with a custome attribute replicated correctly
+        """Simple test having an OU with a custom attribute replicated correctly
 
         This ensures that the server
         """
index 3c36ba68910e836fa061c651c31fefa5ee67b07d..6fb00196273b364a6f37614e962baa9fa944343f 100644 (file)
@@ -310,7 +310,7 @@ class ReplAclTestCase(drs_base.DrsBaseTestCase):
                       self.sd_utils_dc1.get_sd_as_sddl(sub3_ou_dn))
 
         # Rename new_ou (l2) to under self.ou (this must happen second).  If the
-        # inheritence between l3 and l4 is name-based, this could
+        # inheritance between l3 and l4 is name-based, this could
         # break.
 
         # The tree is now self.ou -> l2 -> l3 -> l4
index b1438b766ba8627f4b2b294bb2f23df4318d5614..f40b16d89d51dc8078c622d654d62d77e68e3851 100644 (file)
@@ -87,7 +87,7 @@ class DrsReplicaSyncTestCase(drs_base.DrsBaseTestCase):
         self._net_drs_replicate(DC=self.dnsname_dc1, fromDC=self.dnsname_dc2, forced=False)
 
     def test_ReplDisabled(self):
-        """Tests we cann't replicate when replication is disabled"""
+        """Tests we can't replicate when replication is disabled"""
         self._disable_inbound_repl(self.dnsname_dc1)
 
         ccache_name = self.get_creds_ccache_name()
index d6d0082eccd7389385a2e09f44c4ed1a1dfa34d9..ecd5cec6f4e59db55b1cb02abdc2615fa0bbbc00 100644 (file)
@@ -327,7 +327,7 @@ class DrsReplicaSyncTestCase(drs_base.DrsBaseTestCase):
         self.assertCmdSuccess(result, out, err)
 
     def test_offline_manual_seized_ridalloc_with_dbcheck(self):
-        """Peform the same actions as test_offline_samba_tool_seized_ridalloc,
+        """Perform the same actions as test_offline_samba_tool_seized_ridalloc,
         but do not create the RID set. Confirm that dbcheck correctly creates
         the RID Set.
 
@@ -379,7 +379,7 @@ class DrsReplicaSyncTestCase(drs_base.DrsBaseTestCase):
             shutil.rmtree(targetdir, ignore_errors=True)
 
     def test_offline_manual_seized_ridalloc_add_user(self):
-        """Peform the same actions as test_offline_samba_tool_seized_ridalloc,
+        """Perform the same actions as test_offline_samba_tool_seized_ridalloc,
         but do not create the RID set. Confirm that user-add correctly creates
         the RID Set."""
         fsmo_dn = ldb.Dn(self.ldb_dc1, "CN=RID Manager$,CN=System," + self.ldb_dc1.domain_dn())
@@ -427,7 +427,7 @@ class DrsReplicaSyncTestCase(drs_base.DrsBaseTestCase):
             shutil.rmtree(targetdir, ignore_errors=True)
 
     def test_offline_manual_seized_ridalloc_add_user_as_admin(self):
-        """Peform the same actions as test_offline_samba_tool_seized_ridalloc,
+        """Perform the same actions as test_offline_samba_tool_seized_ridalloc,
         but do not create the RID set. Confirm that user-add correctly creates
         the RID Set."""
         fsmo_dn = ldb.Dn(self.ldb_dc1, "CN=RID Manager$,CN=System," + self.ldb_dc1.domain_dn())