@remit/imap-worker 0.0.30 → 0.0.31

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@remit/imap-worker",
-  "version": "0.0.30",
+  "version": "0.0.31",
   "type": "module",
   "main": "dist/index.js",
   "exports": {

package/src/handlers/flag-push.test.ts

@@ -1,6 +1,6 @@
 import assert from "node:assert/strict";
-import { afterEach, describe, it, mock } from "node:test";
-import { getClient } from "@remit/backend/client";
+import { afterEach, before, describe, it, mock } from "node:test";
+import { getClient, type RemitClient, setClient } from "@remit/backend/client";
 import type { Logger } from "@remit/logger-lambda";
 import type { FlagPushEvent } from "../events.js";
 import {
@@ -58,6 +58,21 @@ describe("handleFlagPush — deleted mailbox is terminal (#287/#289)", () => {
 		flagName,
 	} as FlagPushEvent;
 
+	// The client is supplied by injection (`setClient`), so these tests register
+	// the repositories they then mock rather than reaching a composition.
+	before(() => {
+		setClient({
+			account: { get: async () => undefined },
+			message: { get: async () => undefined },
+			mailbox: { get: async () => undefined },
+			flagPush: {
+				find: async () => undefined,
+				updateState: async () => undefined,
+				delete: async () => undefined,
+			},
+		} as unknown as RemitClient);
+	});
+
 	afterEach(() => mock.restoreAll());
 
 	it("drops the orphaned marker and acks without pushing when the mailbox is gone", async () => {

package/src/handlers/message-move.test.ts

@@ -1,6 +1,6 @@
 import assert from "node:assert";
-import { afterEach, describe, it, mock } from "node:test";
-import { getClient } from "@remit/backend/client";
+import { afterEach, before, describe, it, mock } from "node:test";
+import { getClient, type RemitClient, setClient } from "@remit/backend/client";
 import type { AccountItem, ThreadMessageItem } from "@remit/data-ports";
 import type { Logger } from "@remit/logger-lambda";
 import type { MessageMoveEvent } from "../events.js";
@@ -219,6 +219,17 @@ describe("handleMessageMove — deleted mailbox is terminal (#287/#289)", () =>
 		timestamp: 1700000000000,
 	} as MessageMoveEvent;
 
+	// The client is supplied by injection (`setClient`), so these tests register
+	// the repositories they then mock rather than reaching a composition.
+	before(() => {
+		setClient({
+			account: { get: async () => undefined },
+			mailbox: { get: async () => undefined },
+			message: { updateUid: async () => undefined },
+			secrets: { decrypt: async () => undefined },
+		} as unknown as RemitClient);
+	});
+
 	afterEach(() => mock.restoreAll());
 
 	it("acks without connecting when the source mailbox was deleted", async () => {

package/src/handlers/placement-move-push.test.ts

@@ -1,6 +1,6 @@
 import assert from "node:assert/strict";
-import { afterEach, describe, it, mock } from "node:test";
-import { getClient } from "@remit/backend/client";
+import { afterEach, before, describe, it, mock } from "node:test";
+import { getClient, type RemitClient, setClient } from "@remit/backend/client";
 import type { Logger } from "@remit/logger-lambda";
 import type { IImapConnection } from "@remit/mailbox-service";
 import type { PlacementMovePushEvent } from "../events.js";
@@ -295,6 +295,21 @@ describe("handlePlacementMovePush — deleted mailbox is terminal (#287/#289)",
 		timestamp: 1700000000000,
 	};
 
+	// The client is supplied by injection (`setClient`), so these tests register
+	// the repositories they then mock rather than reaching a composition.
+	before(() => {
+		setClient({
+			account: { get: async () => undefined },
+			message: { get: async () => undefined },
+			mailbox: { get: async () => undefined },
+			placementMove: {
+				find: async () => undefined,
+				updateState: async () => undefined,
+				delete: async () => undefined,
+			},
+		} as unknown as RemitClient);
+	});
+
 	afterEach(() => mock.restoreAll());
 
 	it("drops the marker and acks without pushing when a mailbox is gone", async () => {

package/src/handlers/sync-message-body.test.ts

@@ -221,10 +221,8 @@ describe("syncMessageBody — DLQ propagation (integrated, #1270)", () => {
 	const cappedMailbox = (): MailboxItem =>
 		({ fullPath: "INBOX" }) as unknown as MailboxItem;
 
-	// On the DynamoDB path (main repo, DATA_BACKEND unset) the client is injected
-	// by the composition root; register a client whose repositories these tests
-	// then mock. On the relational path (reader, DATA_BACKEND=postgres) getClient
-	// builds its own client and this injection is inert.
+	// The client is supplied by injection (`setClient`), so these tests register
+	// the repositories they then mock rather than reaching a composition.
 	before(() => {
 		setClient({
 			account: { get: async () => cappedAccount() },