diff options
author | Stefan Metzmacher <metze@samba.org> | 2019-02-08 10:57:13 +0100 |
---|---|---|
committer | Ralph Boehme <slow@samba.org> | 2019-02-11 11:41:21 +0100 |
commit | 09c4e78fefd55d33f9fc2d3c3e1ba092a8cc490a (patch) | |
tree | b4dbc6589b82bef376ae051244281c25a3655be5 /source4 | |
parent | 162219de1df60573d0e42b73162fd5d086a812ad (diff) | |
download | samba-09c4e78fefd55d33f9fc2d3c3e1ba092a8cc490a.tar.gz |
dsdb/tests/vlv: use only one toplevel dn that is correctly cleaned up
Before "OU=vlvtestou2,%s" % (self.base_dn) was left behind after the
test.
Signed-off-by: Stefan Metzmacher <metze@samba.org>
Reviewed-by: Ralph Boehme <slow@samba.org>
Reviewed-by: Andrew Bartlett <abartlet@samba.org>
Diffstat (limited to 'source4')
-rw-r--r-- | source4/dsdb/tests/python/vlv.py | 18 |
1 files changed, 9 insertions, 9 deletions
diff --git a/source4/dsdb/tests/python/vlv.py b/source4/dsdb/tests/python/vlv.py index 8550a38e287..29cc1718c78 100644 --- a/source4/dsdb/tests/python/vlv.py +++ b/source4/dsdb/tests/python/vlv.py @@ -153,12 +153,16 @@ class TestsWithUserOU(samba.tests.TestCase): session_info=system_session(lp), lp=lp) self.base_dn = self.ldb.domain_dn() - self.ou = "ou=vlv,%s" % self.base_dn + self.tree_dn = "ou=vlvtesttree,%s" % self.base_dn + self.ou = "ou=vlvou,%s" % self.tree_dn if opts.delete_in_setup: try: - self.ldb.delete(self.ou, ['tree_delete:1']) + self.ldb.delete(self.tree_dn, ['tree_delete:1']) except ldb.LdbError as e: - print("tried deleting %s, got error %s" % (self.ou, e)) + print("tried deleting %s, got error %s" % (self.tree_dn, e)) + self.ldb.add({ + "dn": self.tree_dn, + "objectclass": "organizationalUnit"}) self.ldb.add({ "dn": self.ou, "objectclass": "organizationalUnit"}) @@ -191,7 +195,7 @@ class TestsWithUserOU(samba.tests.TestCase): def tearDown(self): super(TestsWithUserOU, self).tearDown() if not opts.delete_in_setup: - self.ldb.delete(self.ou, ['tree_delete:1']) + self.ldb.delete(self.tree_dn, ['tree_delete:1']) class VLVTests(TestsWithUserOU): @@ -1368,11 +1372,7 @@ class PagedResultsTests(TestsWithUserOU): def test_paged_modify_object_scope(self): expr = "(objectClass=*)" - ou2 = "OU=vlvtestou2,%s" % (self.base_dn) - try: - self.ldb.delete(ou2, ['tree_delete:1']) - except ldb.LdbError: - pass + ou2 = "OU=vlvtestou2,%s" % (self.tree_dn) self.ldb.add({"dn": ou2, "objectclass": "organizationalUnit"}) # Do a separate, full search to get all results |