@proteinjs/db-driver-spanner 1.14.1 → 1.14.3

package/test/SessionPoolExhaustion.test.ts

@@ -1,7 +1,9 @@
 import { Database, SessionPool } from '@google-cloud/spanner';
-import { Db, Record, StringColumn, Table, withRecordColumns } from '@proteinjs/db';
+import { Db, Record, StringColumn, Table, tableByName, withRecordColumns } from '@proteinjs/db';
 import { TransactionContext } from '@proteinjs/db-transaction-context';
 import { SpannerDriver } from '@proteinjs/db-driver-spanner';
+import { registerTestUser, clearTestUser } from '@proteinjs/db/test';
+import { SourceRepository } from '@proteinjs/reflection';
 import { getDropTestTable } from './util/getDropTestTable';
 import { SpannerEmulatorProvisioner } from './util/SpannerEmulatorProvisioner';
 import '../generated/test/index';
@@ -20,8 +22,9 @@ class WedgeEmployeeTestTable extends Table<WedgeEmployee> {
 }
 
 const employeeTable: Table<WedgeEmployee> = new WedgeEmployeeTestTable();
-// Local table — not in any reflection source graph, so thread getTable explicitly.
-const getTable = () => employeeTable;
+// Local table — not in any reflection source graph, so thread getTable explicitly; other
+// lookups (e.g. the delete path's reverse-cascade scan over graph tables) resolve normally.
+const getTable = (tableName: string) => (tableName === employeeTable.name ? employeeTable : tableByName(tableName));
 const spannerConfig = {
   projectId: 'proteinjs-test',
   instanceName: 'proteinjs-test',
@@ -58,6 +61,14 @@ describe('Session pool at max=1 (wedge designed out; real exhaustion documented)
   const txnDb = new Db(spannerDriver, getTable, new TransactionContext());
 
   beforeAll(async () => {
+    // Fail-closed auth needs an explicit identity (the suite predates the flip — n3xa4 side).
+    registerTestUser();
+    // HERMETIC single-table world: the delete path's reverse-cascade scan walks getTables();
+    // scoping the registry to the local table keeps this suite from querying other suites'
+    // graph tables (which don't exist in an isolated run). Object cache wins over the graph.
+    (SourceRepository.get() as unknown as { objectCache: { [key: string]: unknown[] } }).objectCache[
+      '@proteinjs/db/Table'
+    ] = [employeeTable];
     await SpannerEmulatorProvisioner.ensureProvisioned(spannerConfig);
     await spannerDriver.createDbIfNotExists();
     // Setup issues overlapping schema-metadata queries; with fail: true the second acquisition
@@ -73,6 +84,10 @@ describe('Session pool at max=1 (wedge designed out; real exhaustion documented)
     getSessionPool().options.fail = false; // teardown queries queue like setup's
     await dropTable(employeeTable);
     await SpannerEmulatorProvisioner.release();
+    delete (SourceRepository.get() as unknown as { objectCache: { [key: string]: unknown[] } }).objectCache[
+      '@proteinjs/db/Table'
+    ];
+    clearTestUser();
   }, 60000);
 
   test('SpannerConfig.sessionPoolOptions reach the session pool', () => {

package/test/SessionRecordTableQuery.test.ts

@@ -1,6 +1,7 @@
 import moment from 'moment';
 import { SpannerDriver } from '@proteinjs/db-driver-spanner';
 import { Db, DateColumn, QueryBuilderFactory, Record, StringColumn, Table, withRecordColumns } from '@proteinjs/db';
+import { registerTestUser, clearTestUser } from '@proteinjs/db/test';
 import { TransactionContext } from '@proteinjs/db-transaction-context';
 import { getDropTestTable } from './util/getDropTestTable';
 import { SpannerEmulatorProvisioner } from './util/SpannerEmulatorProvisioner';
@@ -77,6 +78,9 @@ describe('Session-shaped record table query (the admin Sessions table path)', ()
   const db = new Db(spannerDriver, getTable, new TransactionContext());
 
   beforeAll(async () => {
+    // The admin Sessions table reads as an admin; UserAuth is fail-closed, so the suite carries
+    // that identity explicitly (the session table defaults to the admin door — no auth block).
+    registerTestUser();
     await SpannerEmulatorProvisioner.ensureProvisioned({
       projectId: 'proteinjs-test',
       instanceName: 'proteinjs-test',
@@ -87,6 +91,7 @@ describe('Session-shaped record table query (the admin Sessions table path)', ()
   }, 60000);
 
   afterAll(async () => {
+    clearTestUser();
     await dropTable(table);
     await SpannerEmulatorProvisioner.release();
   }, 30000);

package/test/TransactionSafety.test.ts

@@ -1,6 +1,8 @@
-import { Db, QueryBuilderFactory, Record, StringColumn, Table, withRecordColumns } from '@proteinjs/db';
+import { Db, QueryBuilderFactory, Record, StringColumn, Table, tableByName, withRecordColumns } from '@proteinjs/db';
 import { TransactionContext } from '@proteinjs/db-transaction-context';
 import { SpannerDriver } from '@proteinjs/db-driver-spanner';
+import { registerTestUser, clearTestUser } from '@proteinjs/db/test';
+import { SourceRepository } from '@proteinjs/reflection';
 import { getDropTestTable } from './util/getDropTestTable';
 import { SpannerEmulatorProvisioner } from './util/SpannerEmulatorProvisioner';
 import '../generated/test/index';
@@ -19,8 +21,9 @@ class SafetyEmployeeTestTable extends Table<SafetyEmployee> {
 }
 
 const employeeTable: Table<SafetyEmployee> = new SafetyEmployeeTestTable();
-// Local table — not in any reflection source graph, so thread getTable explicitly.
-const getTable = () => employeeTable;
+// Local table — not in any reflection source graph, so thread getTable explicitly; other
+// lookups (e.g. the delete path's reverse-cascade scan over graph tables) resolve normally.
+const getTable = (tableName: string) => (tableName === employeeTable.name ? employeeTable : tableByName(tableName));
 const spannerConfig = {
   projectId: 'proteinjs-test',
   instanceName: 'proteinjs-test',
@@ -45,6 +48,14 @@ describe('Transaction safety (stateless contract)', () => {
   const txnDb = new Db(spannerDriver, getTable, new TransactionContext());
 
   beforeAll(async () => {
+    // Fail-closed auth needs an explicit identity (the suite predates the flip — n3xa4 side).
+    registerTestUser();
+    // HERMETIC single-table world: the delete path's reverse-cascade scan walks getTables();
+    // scoping the registry to the local table keeps this suite from querying other suites'
+    // graph tables (which don't exist in an isolated run). Object cache wins over the graph.
+    (SourceRepository.get() as unknown as { objectCache: { [key: string]: unknown[] } }).objectCache[
+      '@proteinjs/db/Table'
+    ] = [employeeTable];
     await SpannerEmulatorProvisioner.ensureProvisioned(spannerConfig);
     await spannerDriver.createDbIfNotExists();
     await spannerDriver.getTableManager().loadTable(employeeTable);
@@ -53,6 +64,10 @@ describe('Transaction safety (stateless contract)', () => {
   afterAll(async () => {
     await dropTable(employeeTable);
     await SpannerEmulatorProvisioner.release();
+    delete (SourceRepository.get() as unknown as { objectCache: { [key: string]: unknown[] } }).objectCache[
+      '@proteinjs/db/Table'
+    ];
+    clearTestUser();
   }, 60000);
 
   test('a PRE-CONSTRUCTED Db rides the ambient transaction: its reads see the txn write, and the txn commits', async () => {

package/test/util/SpannerEmulatorProvisioner.ts

@@ -44,8 +44,17 @@ export class SpannerEmulatorProvisioner {
       SpannerEmulatorProvisioner.swallowAlreadyExists(error);
     }
     try {
-      const [, operation] = await instance.createDatabase(config.databaseName);
+      const [database, operation] = await instance.createDatabase(config.databaseName);
       await operation.promise();
+      // createDatabase constructs a Database handle whose session pool OPENS in its constructor
+      // (database.js: pool_.open()). Ignored, it becomes an orphan: its fill/keepalive machinery
+      // outlives release(), and once the admin client closes, every subsequent pool op emits an
+      // unlistened 'error' on this handle — crashing whichever jest suite happens to be active
+      // (only fresh-emulator runs hit this; ALREADY_EXISTS constructs no handle). Own it through
+      // its death: the listener covers session creates that settle after close, and the close
+      // runs while the admin client can still delete the pool's sessions cleanly.
+      database.on('error', () => undefined);
+      await database.close().catch(() => undefined);
     } catch (error) {
       SpannerEmulatorProvisioner.swallowAlreadyExists(error);
     }

package/LICENSE

@@ -1,21 +0,0 @@
-MIT License
-
-Copyright (c) 2023 Brent Bahry
-
-Permission is hereby granted, free of charge, to any person obtaining a copy
-of this software and associated documentation files (the "Software"), to deal
-in the Software without restriction, including without limitation the rights
-to use, copy, modify, merge, publish, distribute, sublicense, and/or sell
-copies of the Software, and to permit persons to whom the Software is
-furnished to do so, subject to the following conditions:
-
-The above copyright notice and this permission notice shall be included in all
-copies or substantial portions of the Software.
-
-THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
-IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
-FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE
-AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
-LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM,
-OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE
-SOFTWARE.