david raistrick
c1d982b4a4
WS adapter had no reconnect. WS dies (idle/error/close) → wsReady=null, subscribers dead forever, display frozen until full reload. Changes (src/storage/ws.js): - onClose: schedule reconnect via setTimeout(500ms), ensureWs re-arms. Guard: disposed flag stops reconnect after dispose. - onOpen: resubscribe all existing doc/coll subscribers (backend state may have changed). Re-fetch current values on RECONNECT only (skip first connect — initial REST fetch in subscribe* already did). Added everConnected flag to distinguish first vs reconnect. - reconnectTimer unref'd (Node) to avoid hanging event loop. - dispose(cb): set disposed, clear timer, close ws, then cb. Also fixed test teardown leaks: - server/index.js close(): terminate all wss.clients before wss.close(). Reconnect test spawned new ws to server; old close hung on live conn. - both ws test factories: port 0 (OS picks free) instead of module-local nextPort counter. Parallel jest workers collided on EADDRINUSE. Tests: ws-reconnect GREEN (1.7s), ws-contract 23 GREEN. No regression. server suite 24/24. shared 90/90.
35 lines
1.5 KiB
JavaScript
35 lines
1.5 KiB
JavaScript
// Layer 2 test: exercise ws.js storage adapter against a LIVE backend.
|
|
// Complements Layer 1 (App + firebase mock) which proves App call shape but
|
|
// never touches ws.js. This catches translation bugs in the adapter.
|
|
//
|
|
// Runs the shared storage contract (same spec memory/firebase satisfy) against
|
|
// createWsStorage pointed at an ephemeral backend instance. A FRESH backend is
|
|
// spun up per test to guarantee isolation (backend has no reset endpoint yet).
|
|
|
|
'use strict';
|
|
|
|
const path = require('path');
|
|
const os = require('os');
|
|
const { createServer } = require('../index');
|
|
const { createWsStorage } = require('../../src/storage/ws');
|
|
const { runStorageContract } = require('../../src/storage/contract');
|
|
|
|
// Factory: fresh backend (unique sqlite file) + storage pointed at it.
|
|
// Disposing the storage closes the backend so each test is fully isolated.
|
|
async function makeStorage() {
|
|
const dbPath = path.join(os.tmpdir(), `ws-contract-${Date.now()}-${Math.random().toString(36).slice(2)}.sqlite`);
|
|
const handle = createServer({ dbPath, port: 0 });
|
|
await new Promise((resolve, reject) => {
|
|
handle.server.on('error', reject);
|
|
handle.server.listen(0, resolve);
|
|
});
|
|
const port = handle.server.address().port;
|
|
const baseUrl = `http://127.0.0.1:${port}`;
|
|
const wsUrl = `ws://127.0.0.1:${port}/ws`;
|
|
const storage = createWsStorage({ baseUrl, wsUrl });
|
|
storage.dispose = (done) => handle.close(done);
|
|
return storage;
|
|
}
|
|
|
|
runStorageContract('ws (live backend)', makeStorage);
|