python.sites tests: remove excessive transaction management

These are atomic anyway.

Signed-off-by: Douglas Bagnall <douglas.bagnall@catalyst.net.nz>
Reviewed-by: Andrew Bartlett <abartlet@samba.org>
Reviewed-by: Garming Sam <garming@catalyst.net.nz>

diff --git a/source4/dsdb/tests/python/sites.py b/source4/dsdb/tests/python/sites.py
index 71725bffa450b68b8a01b956fef3569ef48e8eb6..f42e7bf7e3ecd8c658e98d00d91c88c5b45d1bff 100755 (executable)
--- a/source4/dsdb/tests/python/sites.py
+++ b/source4/dsdb/tests/python/sites.py
@@ -82,20 +82,16 @@ class SimpleSitesTests(SitesBaseTests):
     def test_create_and_delete(self):
         """test creation and deletion of 1 site"""
 
-        self.ldb.transaction_start()
         sites.create_site(self.ldb, self.ldb.get_config_basedn(),
                           "testsamba")
-        self.ldb.transaction_commit()
 
         self.assertRaises(sites.SiteAlreadyExistsException,
                           sites.create_site, self.ldb,
                           self.ldb.get_config_basedn(),
                           "testsamba")
 
-        self.ldb.transaction_start()
         sites.delete_site(self.ldb, self.ldb.get_config_basedn(),
                           "testsamba")
-        self.ldb.transaction_commit()
 
         self.assertRaises(sites.SiteNotFoundException,
                           sites.delete_site, self.ldb,