From 05fec3ef4bdd39d865a1946d2d0165126730138c Mon Sep 17 00:00:00 2001 From: Stefan Metzmacher Date: Wed, 1 Jun 2016 00:18:05 +0200 Subject: [PATCH] s4:dsdb/tests: use more useful userAccountControl/pwdLastSet values in the urgent_replication test Using UF_SMARDCARD_REQUIRED has some side effects, so we better use UF_DONT_EXPIRE_PASSWD which doesn't trigger additional actions. Setting pwdLastSet to "1" is not allowed, only "-1" is able to change an existing value of "0". BUG: https://bugzilla.samba.org/show_bug.cgi?id=9654 Signed-off-by: Stefan Metzmacher Reviewed-by: Andrew Bartlett --- source4/dsdb/tests/python/urgent_replication.py | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/source4/dsdb/tests/python/urgent_replication.py b/source4/dsdb/tests/python/urgent_replication.py index ec2f6d13c2b..81f8b1cbb57 100755 --- a/source4/dsdb/tests/python/urgent_replication.py +++ b/source4/dsdb/tests/python/urgent_replication.py @@ -296,7 +296,7 @@ rIDAvailablePool: 133001-1073741823""", ["relax:0"]) # urgent replication should be enabled when modifying userAccountControl m = Message() m.dn = Dn(self.ldb, "cn=user UrgAttr test,cn=users," + self.base_dn) - m["userAccountControl"] = MessageElement(str(dsdb.UF_NORMAL_ACCOUNT+dsdb.UF_SMARTCARD_REQUIRED), FLAG_MOD_REPLACE, + m["userAccountControl"] = MessageElement(str(dsdb.UF_NORMAL_ACCOUNT+dsdb.UF_DONT_EXPIRE_PASSWD), FLAG_MOD_REPLACE, "userAccountControl") self.ldb.modify(m) res = self.ldb.load_partition_usn(self.base_dn) @@ -314,7 +314,7 @@ rIDAvailablePool: 133001-1073741823""", ["relax:0"]) # urgent replication should be enabled when modifying pwdLastSet m = Message() m.dn = Dn(self.ldb, "cn=user UrgAttr test,cn=users," + self.base_dn) - m["pwdLastSet"] = MessageElement("1", FLAG_MOD_REPLACE, + m["pwdLastSet"] = MessageElement("-1", FLAG_MOD_REPLACE, "pwdLastSet") self.ldb.modify(m) res = self.ldb.load_partition_usn(self.base_dn) -- 2.47.2