fix test suite

Author: kkaefer

src/database.cc

@@ -58,18 +58,19 @@ void Database::Process() {
         return;
     }
 
-
     while (open && (!locked || pending == 0) && !queue.empty()) {
         Call* call = queue.front();
 
         if (call->exclusive && pending > 0) {
             break;
         }
 
+        queue.pop();
         locked = call->exclusive;
         call->callback(call->baton);
-        queue.pop();
         delete call;
+
+        if (locked) break;
     }
 }
 
@@ -201,7 +202,9 @@ Handle<Value> Database::Close(const Arguments& args) {
 }
 
 void Database::EIO_BeginClose(Baton* baton) {
+    assert(baton->db->locked);
     assert(baton->db->open);
+    assert(baton->db->handle);
     assert(baton->db->pending == 0);
     eio_custom(EIO_Close, EIO_PRI_DEFAULT, EIO_AfterClose, baton);
 }
@@ -247,6 +250,11 @@ int Database::EIO_AfterClose(eio_req *req) {
         EMIT_EVENT(db->handle_, 2, args);
     }
 
+    assert(baton->db->locked);
+    assert(!baton->db->open);
+    assert(!baton->db->handle);
+    assert(baton->db->pending == 0);
+
     if (!db->open) {
         Local<Value> args[] = { String::NewSymbol("close"), argv[0] };
         EMIT_EVENT(db->handle_, 1, args);

test/constants.test.js

@@ -0,0 +1,36 @@
+var sqlite3 = require('sqlite3');
+var assert = require('assert');
+
+exports['test constants'] = function() {
+    assert.ok(sqlite3.OPEN_READONLY === 1);
+    assert.ok(sqlite3.OPEN_READWRITE === 2);
+    assert.ok(sqlite3.OPEN_CREATE === 4);
+
+    assert.ok(sqlite3.OK === 0);
+    assert.ok(sqlite3.ERROR === 1);
+    assert.ok(sqlite3.INTERNAL === 2);
+    assert.ok(sqlite3.PERM === 3);
+    assert.ok(sqlite3.ABORT === 4);
+    assert.ok(sqlite3.BUSY === 5);
+    assert.ok(sqlite3.LOCKED === 6);
+    assert.ok(sqlite3.NOMEM === 7);
+    assert.ok(sqlite3.READONLY === 8);
+    assert.ok(sqlite3.INTERRUPT === 9);
+    assert.ok(sqlite3.IOERR === 10);
+    assert.ok(sqlite3.CORRUPT === 11);
+    assert.ok(sqlite3.NOTFOUND === 12);
+    assert.ok(sqlite3.FULL === 13);
+    assert.ok(sqlite3.CANTOPEN === 14);
+    assert.ok(sqlite3.PROTOCOL === 15);
+    assert.ok(sqlite3.EMPTY === 16);
+    assert.ok(sqlite3.SCHEMA === 17);
+    assert.ok(sqlite3.TOOBIG === 18);
+    assert.ok(sqlite3.CONSTRAINT === 19);
+    assert.ok(sqlite3.MISMATCH === 20);
+    assert.ok(sqlite3.MISUSE === 21);
+    assert.ok(sqlite3.NOLFS === 22);
+    assert.ok(sqlite3.AUTH === 23);
+    assert.ok(sqlite3.FORMAT === 24);
+    assert.ok(sqlite3.RANGE === 25);
+    assert.ok(sqlite3.NOTADB === 26);
+};

test/open_close.test.js

@@ -3,40 +3,6 @@ var assert = require('assert');
 var fs = require('fs');
 var helper = require('./support/helper');
 
-exports['constants'] = function() {
-    assert.ok(sqlite3.OPEN_READONLY === 1);
-    assert.ok(sqlite3.OPEN_READWRITE === 2);
-    assert.ok(sqlite3.OPEN_CREATE === 4);
-
-    assert.ok(sqlite3.OK === 0);
-    assert.ok(sqlite3.ERROR === 1);
-    assert.ok(sqlite3.INTERNAL === 2);
-    assert.ok(sqlite3.PERM === 3);
-    assert.ok(sqlite3.ABORT === 4);
-    assert.ok(sqlite3.BUSY === 5);
-    assert.ok(sqlite3.LOCKED === 6);
-    assert.ok(sqlite3.NOMEM === 7);
-    assert.ok(sqlite3.READONLY === 8);
-    assert.ok(sqlite3.INTERRUPT === 9);
-    assert.ok(sqlite3.IOERR === 10);
-    assert.ok(sqlite3.CORRUPT === 11);
-    assert.ok(sqlite3.NOTFOUND === 12);
-    assert.ok(sqlite3.FULL === 13);
-    assert.ok(sqlite3.CANTOPEN === 14);
-    assert.ok(sqlite3.PROTOCOL === 15);
-    assert.ok(sqlite3.EMPTY === 16);
-    assert.ok(sqlite3.SCHEMA === 17);
-    assert.ok(sqlite3.TOOBIG === 18);
-    assert.ok(sqlite3.CONSTRAINT === 19);
-    assert.ok(sqlite3.MISMATCH === 20);
-    assert.ok(sqlite3.MISUSE === 21);
-    assert.ok(sqlite3.NOLFS === 22);
-    assert.ok(sqlite3.AUTH === 23);
-    assert.ok(sqlite3.FORMAT === 24);
-    assert.ok(sqlite3.RANGE === 25);
-    assert.ok(sqlite3.NOTADB === 26);
-};
-
 exports['open and close non-existent database'] = function(beforeExit) {
     var opened, closed;