From cb9f855c2b36d5517edd73db31fcc690d8e5ef0d Mon Sep 17 00:00:00 2001 From: teknium1 <127238744+teknium1@users.noreply.github.com> Date: Sun, 28 Jun 2026 00:51:05 -0700 Subject: [PATCH] test(whatsapp-bridge): drop structural send-queue integration test MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit The .integration.test.mjs greps bridge.js source text for the queue wiring — a change-detector that breaks on any benign refactor of the same code. The behavioral unit test (bridge.sendqueue.test.mjs) already covers FIFO ordering, error isolation, timeout propagation, and single-consumer concurrency, which is the contract that matters. --- .../bridge.sendqueue.integration.test.mjs | 29 ------------------- 1 file changed, 29 deletions(-) delete mode 100644 scripts/whatsapp-bridge/bridge.sendqueue.integration.test.mjs diff --git a/scripts/whatsapp-bridge/bridge.sendqueue.integration.test.mjs b/scripts/whatsapp-bridge/bridge.sendqueue.integration.test.mjs deleted file mode 100644 index ad4d2b890b4..00000000000 --- a/scripts/whatsapp-bridge/bridge.sendqueue.integration.test.mjs +++ /dev/null @@ -1,29 +0,0 @@ -/** - * Integration test: verify bridge.js has the send queue wired in. - * This reads the source file and asserts the required patterns exist. - */ -import { strict as assert } from 'node:assert'; -import { readFileSync } from 'node:fs'; -import { resolve, dirname } from 'node:path'; -import { fileURLToPath } from 'node:url'; - -const __dirname = dirname(fileURLToPath(import.meta.url)); -const src = readFileSync(resolve(__dirname, 'bridge.js'), 'utf-8'); - -// 1. Queue primitives exist -assert.ok(src.includes('let _sendQueue = Promise.resolve()'), - 'bridge.js must define _sendQueue'); -assert.ok(src.includes('function enqueueSend(fn)'), - 'bridge.js must define enqueueSend'); - -// 2. sendWithTimeout is wrapped -assert.ok(src.includes('return enqueueSend(() =>'), - 'sendWithTimeout must call enqueueSend'); - -// 3. The queue is before sendWithTimeout in the file -const queueIdx = src.indexOf('let _sendQueue'); -const sendTimeoutIdx = src.indexOf('function sendWithTimeout'); -assert.ok(queueIdx < sendTimeoutIdx, - '_sendQueue must be defined before sendWithTimeout'); - -console.log('✅ bridge.js send queue integration check passed.');