ldb:tests:api.py uses .disconnect before rmdir

super.tearDown() was removing the tmpdir, but because self.ldb had
the file open, the directory was not cleared.

Signed-off-by: Douglas Bagnall <douglas.bagnall@catalyst.net.nz>
Reviewed-by: Andreas Schneider <asn@samba.org>

diff --git a/lib/ldb/tests/python/api.py b/lib/ldb/tests/python/api.py
index ce9366dbc5e0cc3c05bce8306e5581f742baac99..e40009561a09bd11fdc7400f81a7d1ca0e0da117 100755 (executable)
--- a/lib/ldb/tests/python/api.py
+++ b/lib/ldb/tests/python/api.py
@@ -106,10 +106,10 @@ class SimpleLdb(LdbBaseTest):
             pass
 
     def tearDown(self):
+        self.ldb.disconnect()
         shutil.rmtree(self.testdir)
         super().tearDown()
         # Ensure the LDB is closed now, so we close the FD
-        del(self.ldb)
 
     def test_connect(self):
         ldb.Ldb(self.url(), flags=self.flags())