testing

Author: cjihrig

test/parallel/test-sqlite-data-types.js

@@ -83,6 +83,7 @@ suite('data binding and mapping', () => {
       text: '',
       buf: new Uint8Array(),
     });
+    db.close();
   });
 
   test('unsupported data types', (t) => {
@@ -116,6 +117,7 @@ suite('data binding and mapping', () => {
       code: 'ERR_INVALID_ARG_TYPE',
       message: /Provided value cannot be bound to SQLite parameter 2/,
     });
+    db.close();
   });
 
   test('throws when binding a BigInt that is too large', (t) => {
@@ -136,6 +138,7 @@ suite('data binding and mapping', () => {
       code: 'ERR_INVALID_ARG_VALUE',
       message: /BigInt value is too large to bind/,
     });
+    db.close();
   });
 
   test('statements are unbound on each call', (t) => {
@@ -157,5 +160,6 @@ suite('data binding and mapping', () => {
       db.prepare('SELECT * FROM data ORDER BY key').all(),
       [{ key: 1, val: 5 }, { key: 2, val: null }],
     );
+    db.close();
   });
 });

test/parallel/test-sqlite-database-sync.js

@@ -67,6 +67,7 @@ suite('DatabaseSync.prototype.open()', () => {
     t.assert.strictEqual(existsSync(dbPath), false);
     t.assert.strictEqual(db.open(), undefined);
     t.assert.strictEqual(existsSync(dbPath), true);
+    db.close();
   });
 
   test('throws if database is already open', (t) => {
@@ -79,6 +80,7 @@ suite('DatabaseSync.prototype.open()', () => {
       code: 'ERR_INVALID_STATE',
       message: /database is already open/,
     });
+    db.close();
   });
 });
 
@@ -106,6 +108,7 @@ suite('DatabaseSync.prototype.prepare()', () => {
     const db = new DatabaseSync(nextDb());
     const stmt = db.prepare('CREATE TABLE webstorage(key TEXT)');
     t.assert.ok(stmt instanceof StatementSync);
+    db.close();
   });
 
   test('throws if database is not open', (t) => {
@@ -128,6 +131,7 @@ suite('DatabaseSync.prototype.prepare()', () => {
       code: 'ERR_INVALID_ARG_TYPE',
       message: /The "sql" argument must be a string/,
     });
+    db.close();
   });
 });
 
@@ -148,6 +152,7 @@ suite('DatabaseSync.prototype.exec()', () => {
       { key: 1, val: 2 },
       { key: 8, val: 9 },
     ]);
+    db.close();
   });
 
   test('reports errors from SQLite', (t) => {
@@ -159,6 +164,7 @@ suite('DatabaseSync.prototype.exec()', () => {
       code: 'ERR_SQLITE_ERROR',
       message: /syntax error/,
     });
+    db.close();
   });
 
   test('throws if database is not open', (t) => {
@@ -181,5 +187,6 @@ suite('DatabaseSync.prototype.exec()', () => {
       code: 'ERR_INVALID_ARG_TYPE',
       message: /The "sql" argument must be a string/,
     });
+    db.close();
   });
 });

test/parallel/test-sqlite-named-parameters.js

@@ -35,6 +35,7 @@ suite('named parameters', () => {
       code: 'ERR_INVALID_STATE',
       message: /Unknown named parameter '\$unknown'/,
     });
+    db.close();
   });
 
   test('bare named parameters are supported', (t) => {
@@ -49,6 +50,7 @@ suite('named parameters', () => {
       db.prepare('SELECT * FROM data').get(),
       { key: 1, val: 9 },
     );
+    db.close();
   });
 
   test('duplicate bare named parameters are supported', (t) => {
@@ -63,6 +65,7 @@ suite('named parameters', () => {
       db.prepare('SELECT * FROM data').get(),
       { key: 1, val: 1 },
     );
+    db.close();
   });
 
   test('bare named parameters throw on ambiguous names', (t) => {
@@ -79,5 +82,6 @@ suite('named parameters', () => {
       message: 'Cannot create bare named parameter \'k\' because of ' +
                'conflicting names \'$k\' and \'@k\'.',
     });
+    db.close();
   });
 });

test/parallel/test-sqlite-statement-sync.js

@@ -38,6 +38,7 @@ suite('StatementSync.prototype.get()', () => {
     t.assert.strictEqual(stmt.get(), undefined);
     stmt = db.prepare('SELECT * FROM storage');
     t.assert.strictEqual(stmt.get(), undefined);
+    db.close();
   });
 
   test('executes a query and returns the first result', (t) => {
@@ -49,6 +50,7 @@ suite('StatementSync.prototype.get()', () => {
     t.assert.strictEqual(stmt.get('key2', 'val2'), undefined);
     stmt = db.prepare('SELECT * FROM storage ORDER BY key');
     t.assert.deepStrictEqual(stmt.get(), { key: 'key1', val: 'val1' });
+    db.close();
   });
 });
 
@@ -57,6 +59,7 @@ suite('StatementSync.prototype.all()', () => {
     const db = new DatabaseSync(nextDb());
     const stmt = db.prepare('CREATE TABLE storage(key TEXT, val TEXT)');
     t.assert.deepStrictEqual(stmt.all(), []);
+    db.close();
   });
 
   test('executes a query and returns all results', (t) => {
@@ -77,6 +80,7 @@ suite('StatementSync.prototype.all()', () => {
       { key: 'key1', val: 'val1' },
       { key: 'key2', val: 'val2' },
     ]);
+    db.close();
   });
 });
 
@@ -90,6 +94,7 @@ suite('StatementSync.prototype.run()', () => {
     t.assert.strictEqual(setup, undefined);
     const stmt = db.prepare('SELECT * FROM storage');
     t.assert.deepStrictEqual(stmt.run(), { changes: 1, lastInsertRowid: 1 });
+    db.close();
   });
 
   test('SQLite throws when trying to bind too many parameters', (t) => {
@@ -107,6 +112,7 @@ suite('StatementSync.prototype.run()', () => {
       errcode: 25,
       errstr: 'column index out of range',
     });
+    db.close();
   });
 
   test('SQLite defaults to NULL for unbound parameters', (t) => {
@@ -124,6 +130,7 @@ suite('StatementSync.prototype.run()', () => {
       errcode: 1299,
       errstr: 'constraint failed',
     });
+    db.close();
   });
 });
 
@@ -137,6 +144,7 @@ suite('StatementSync.prototype.sourceSQL()', () => {
     const sql = 'INSERT INTO types (key, val) VALUES ($k, $v)';
     const stmt = db.prepare(sql);
     t.assert.strictEqual(stmt.sourceSQL(), sql);
+    db.close();
   });
 });
 
@@ -155,6 +163,7 @@ suite('StatementSync.prototype.expandedSQL()', () => {
       { changes: 1, lastInsertRowid: 33 },
     );
     t.assert.strictEqual(stmt.expandedSQL(), expanded);
+    db.close();
   });
 });
 
@@ -189,6 +198,7 @@ suite('StatementSync.prototype.setReadBigInts()', () => {
       insert.run(30),
       { changes: 1, lastInsertRowid: 30 },
     );
+    db.close();
   });
 
   test('throws when input is not a boolean', (t) => {
@@ -204,6 +214,7 @@ suite('StatementSync.prototype.setReadBigInts()', () => {
       code: 'ERR_INVALID_ARG_TYPE',
       message: /The "readBigInts" argument must be a boolean/,
     });
+    db.close();
   });
 
   test('BigInt is required for reading large integers', (t) => {
@@ -220,6 +231,7 @@ suite('StatementSync.prototype.setReadBigInts()', () => {
     t.assert.deepStrictEqual(good.get(), {
       [`${Number.MAX_SAFE_INTEGER} + 1`]: 2n ** 53n,
     });
+    db.close();
   });
 });
 
@@ -247,6 +259,7 @@ suite('StatementSync.prototype.setAllowBareNamedParameters()', () => {
       stmt.run({ k: 3, v: 6 }),
       { changes: 1, lastInsertRowid: 3 },
     );
+    db.close();
   });
 
   test('throws when input is not a boolean', (t) => {
@@ -262,5 +275,6 @@ suite('StatementSync.prototype.setAllowBareNamedParameters()', () => {
       code: 'ERR_INVALID_ARG_TYPE',
       message: /The "allowBareNamedParameters" argument must be a boolean/,
     });
+    db.close();
   });
 });

test/parallel/test-sqlite-transactions.js

@@ -45,6 +45,7 @@ suite('manual transactions', () => {
       db.prepare('SELECT * FROM data').all(),
       [{ key: 100 }],
     );
+    db.close();
   });
 
   test('a transaction is rolled back', (t) => {
@@ -68,5 +69,6 @@ suite('manual transactions', () => {
       { changes: 1, lastInsertRowid: 100 },
     );
     t.assert.deepStrictEqual(db.prepare('SELECT * FROM data').all(), []);
+    db.close();
   });
 });

test/parallel/test-sqlite.js

@@ -66,6 +66,7 @@ test('ERR_SQLITE_ERROR is thrown for errors originating from SQLite', (t) => {
     errcode: 1555,
     errstr: 'constraint failed',
   });
+  db.close();
 });
 
 test('in-memory databases are supported', (t) => {
@@ -101,4 +102,5 @@ test('PRAGMAs are supported', (t) => {
     db.prepare('PRAGMA journal_mode').get(),
     { journal_mode: 'wal' },
   );
+  db.close();
 });