@remit/account-worker 0.0.29 → 0.0.30

package/package.json

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

package/src/config.ts

@@ -1,6 +1,9 @@
 import { getClient } from "@remit/backend/client";
 import type { GroupedQueueProducer, QueueProducer } from "@remit/data-ports";
-import { createQueueProducer } from "@remit/sqs-client/producer";
+import {
+	createGroupedQueueProducer,
+	createStandardQueueProducer,
+} from "@remit/sqs-client/producer";
 import type { StorageService } from "@remit/storage-service";
 import { createStorageService } from "@remit/storage-service/s3";
 import { env } from "expect-env";
@@ -26,10 +29,10 @@ export interface FanoutQueues {
 }
 
 export const sqsFanoutQueues = (): FanoutQueues => ({
-	accountFinalize: createQueueProducer({
+	accountFinalize: createStandardQueueProducer({
 		queueUrl: getAccountFinalizeQueueUrl(),
 	}),
-	accountPurgeDelete: createQueueProducer({
+	accountPurgeDelete: createGroupedQueueProducer({
 		queueUrl: getAccountPurgeDeleteQueueUrl(),
 	}),
 });

package/src/handlers/account-fanout.test.ts

@@ -6,34 +6,9 @@ import type { CascadeServices } from "../cascade.js";
 import type { AccountDeleteEvent } from "../events.js";
 import { processAccountFanout } from "./account-fanout.js";
 
-// A CascadeServices fake with one account and no data: enough for
-// enumerateCascadeEntities to yield a single Account row (so the stop-signal
-// loop has one accountId to iterate) without standing up a database.
 const buildServices = (): CascadeServices =>
 	({
-		accountConfigService: {
-			get: async () => ({ userId: "user-1" }),
-			describe: async () => ({
-				account: [{ accountId: "acc-1" }],
-				address: [],
-			}),
-		},
-		accountService: {
-			describe: async () => ({ mailbox: [] }),
-		},
-		messageService: {
-			listAllByMailbox: async () => [],
-			describe: async () => ({}),
-		},
-		outboxMessageService: { listByAccount: async () => ({ items: [] }) },
-		mailboxLockService: { listByAccount: async () => [] },
-		messagePlacementMoveService: { listByAccountId: async () => [] },
-		messageFlagPushService: { listByAccountId: async () => [] },
-		threadMessageService: { listAllByAccount: async () => [] },
-		accountSettingService: { listByAccountConfig: async () => [] },
-		filterService: { listByAccountConfig: async () => [] },
-		labelService: { listByAccountConfig: async () => [] },
-		messageLabelService: { listByLabelId: async () => [] },
+		accountConfigService: { get: async () => ({ userId: "user-1" }) },
 	}) as unknown as CascadeServices;
 
 const deleteEvent: AccountDeleteEvent = {
@@ -48,7 +23,7 @@ describe("processAccountFanout on account delete", () => {
 		await processAccountFanout(deleteEvent, noopLogger, {
 			queues: {
 				accountFinalize,
-				accountPurgeDelete: recordingQueue(),
+				accountPurgeDelete: recordingQueue().grouped,
 			},
 			services: buildServices(),
 			signOut: async () => {},

package/src/handlers/account-fanout.ts

@@ -33,11 +33,6 @@ export interface ProcessAccountFanoutDeps {
 	signOut?: DeletionCapabilities["signOut"];
 }
 
-/**
- * Fanout step of the account-deletion cascade. Signs the user out and
- * enqueues the finalize step. Throws on any non-swallowable error so SQS partial batch-failure
- * retries the whole record.
- */
 export const processAccountFanout = async (
 	event: AccountFanoutEvent,
 	log: Logger,

package/src/handlers/account-purge.test.ts

@@ -42,7 +42,7 @@ const baseDeps = (queue: RecordingQueue): ProcessPurgeFanoutDeps => ({
 		{ threadMessageId: "tm-1", messageId: "msg-1", mailboxId: "mbx-1" },
 		{ threadMessageId: "tm-2", messageId: "msg-2", mailboxId: "mbx-1" },
 	]),
-	queue,
+	queue: queue.grouped,
 });
 
 describe("processAccountDataPurge", () => {