make tests finish

database.uninitialize must be called to drop the connection

src/test/apps-test.js

@@ -149,7 +149,10 @@ describe('Apps', function () {
     });
 
     after(function (done) {
-        database._clear(done);
+        async.series([
+            database._clear,
+            database.uninitialize
+        ], done);
     });
 
     describe('validateHostname', function () {

src/test/apptask-test.js

@@ -109,7 +109,10 @@ describe('apptask', function () {
     });
 
     after(function (done) {
-        database._clear(done);
+        async.series([
+            database._clear,
+            database.uninitialize
+        ], done);
     });
 
     it('initializes succesfully', function (done) {

src/test/certificates-test.js

@@ -22,7 +22,10 @@ function setup(done) {
 }
 
 function cleanup(done) {
-    database._clear(done);
+    async.series([
+        database._clear,
+        database.uninitialize
+    ], done);
 }
 
 describe('Certificates', function () {

src/test/cloudron-test.js

@@ -13,13 +13,16 @@ var async = require('async'),
 
 function setup(done) {
     async.series([
-        database.initialize.bind(null),
-        database._clear.bind(null)
+        database.initialize,
+        database._clear
     ], done);
 }
 
 function cleanup(done) {
-    database._clear(done);
+    async.series([
+        database._clear,
+        database.uninitialize
+    ], done);
 }
 
 describe('Cloudron', function () {

src/test/database-test.js

@@ -78,7 +78,10 @@ describe('database', function () {
     });
 
     after(function (done) {
-        database._clear(done);
+        async.series([
+            database._clear,
+            database.uninitialize
+        ], done);
     });
 
     describe('domains', function () {

src/test/digest-test.js

@@ -53,16 +53,6 @@ function checkMails(number, email, done) {
 }
 
 describe('digest', function () {
-    function cleanup(done) {
-        mailer._clearMailQueue();
-        safe.fs.unlinkSync(paths.UPDATE_CHECKER_FILE);
-
-        async.series([
-            settings.uninitialize,
-            database._clear
-        ], done);
-    }
-
     before(function (done) {
         config._reset();
         config.set('fqdn', 'domain.com');
@@ -83,7 +73,16 @@ describe('digest', function () {
         ], done);
     });
 
-    after(cleanup);
+    after(function (done) {
+        mailer._clearMailQueue();
+        safe.fs.unlinkSync(paths.UPDATE_CHECKER_FILE);
+
+        async.series([
+            settings.uninitialize,
+            database._clear,
+            database.uninitialize
+        ], done);
+    });
 
     describe('disabled', function () {
         before(function (done) {

src/test/dns-test.js

@@ -37,7 +37,10 @@ describe('dns provider', function () {
     });
 
     after(function (done) {
-        database._clear(done);
+        async.series([
+            database._clear,
+            database.uninitialize
+        ], done);
     });
 
     describe('noop', function () {

src/test/eventlog-test.js

@@ -6,10 +6,11 @@
 
 'use strict';
 
-var database = require('../database.js'),
-    expect = require('expect.js'),
+var async = require('async'),
+    database = require('../database.js'),
     eventlog = require('../eventlog.js'),
-    EventLogError = eventlog.EventLogError;
+    EventLogError = eventlog.EventLogError,
+    expect = require('expect.js');
 
 function setup(done) {
     // ensure data/config/mount paths
@@ -20,7 +21,10 @@ function setup(done) {
 }
 
 function cleanup(done) {
-    database._clear(done);
+    async.series([
+        database._clear,
+        database.uninitialize
+    ], done);
 }
 
 describe('Eventlog', function () {

src/test/groups-test.js

@@ -67,7 +67,10 @@ function setup(done) {
 }
 
 function cleanup(done) {
-    database._clear(done);
+    async.series([
+        database._clear,
+        database.uninitialize
+    ], done);
 }
 
 describe('Groups', function () {

src/test/janitor-test.js

@@ -55,7 +55,10 @@ describe('janitor', function () {
     });
 
     after(function (done) {
-        database._clear(done);
+        async.series([
+            database._clear,
+            database.uninitialize
+        ], done);
     });
 
     it('can cleanupTokens', function (done) {

src/test/ldap-test.js

@@ -182,7 +182,8 @@ function setup(done) {
 function cleanup(done) {
     async.series([
         ldapServer.stop,
-        database._clear
+        database._clear,
+        database.uninitialize
     ], function () {
         dockerProxy.close(function () { done(); }); // some strange error
     });

src/test/settings-test.js

@@ -45,7 +45,8 @@ function cleanup(done) {
 
     async.series([
         settings.uninitialize,
-        database._clear
+        database._clear,
+        database.uninitialize
     ], done);
 }
 

src/test/updatechecker-test.js

@@ -53,7 +53,8 @@ function cleanup(done) {
 
     async.series([
         settings.uninitialize,
-        database._clear
+        database._clear,
+        database.uninitialize
     ], done);
 }
 

src/test/user-test.js

@@ -80,7 +80,10 @@ function setup(done) {
 function cleanup(done) {
     mailer._clearMailQueue();
 
-    database._clear(done);
+    async.series([
+        database._clear,
+        database.uninitialize
+    ], done);
 }
 
 function checkMails(number, options, callback) {