fix(gsd): reset db-open attempted flag on close (#4024)

This commit is contained in:
mastertyko 2026-04-13 12:51:59 +02:00 committed by GitHub
parent 92ac0e3a7d
commit 416be1e169
2 changed files with 4 additions and 5 deletions

View file

@ -857,6 +857,7 @@ export function closeDatabase(): void {
currentDb = null;
currentPath = null;
currentPid = 0;
_dbOpenAttempted = false;
}
}

View file

@ -377,15 +377,13 @@ describe('gsd-db', () => {
assert.deepStrictEqual(ar, [], 'getActiveRequirements returns [] when DB closed');
});
test('gsd-db: wasDbOpenAttempted tracks openDatabase calls', () => {
// wasDbOpenAttempted should return true once openDatabase has been called
// (previous tests in this suite already called openDatabase, so the flag is set)
test('gsd-db: closeDatabase resets wasDbOpenAttempted after an intentional close', () => {
openDatabase(':memory:');
assert.ok(wasDbOpenAttempted(), 'wasDbOpenAttempted should be true after openDatabase was called');
// Verify the flag persists even after closeDatabase
closeDatabase();
assert.ok(!isDbAvailable(), 'DB should not be available after close');
assert.ok(wasDbOpenAttempted(), 'wasDbOpenAttempted should remain true after closeDatabase');
assert.ok(!wasDbOpenAttempted(), 'wasDbOpenAttempted should reset after closeDatabase');
});
// ─── Final Report ──────────────────────────────────────────────────────────