@remit/backend 0.0.80 → 0.0.81

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@remit/backend",
-  "version": "0.0.80",
+  "version": "0.0.81",
   "description": "Remit Mail Inspector API backend",
   "license": "MIT",
   "author": "",

package/src/handlers/outbox.test.ts

@@ -109,6 +109,15 @@ const createInMemoryOutboxRepository = (): IOutboxMessageRepository => {
 			outboxMessageId: string,
 			status: OutboxMessageItem["status"],
 		) => repository.update(accountConfigId, outboxMessageId, { status }),
+		updateIfStatus: async (
+			accountConfigId: string,
+			outboxMessageId: string,
+			expected: OutboxMessageItem["status"],
+			input: UpdateOutboxMessageInput,
+		) => {
+			if (mustGet(outboxMessageId).status !== expected) return null;
+			return repository.update(accountConfigId, outboxMessageId, input);
+		},
 		markSent: async (
 			accountConfigId: string,
 			outboxMessageId: string,
@@ -453,12 +462,17 @@ describe("a send whose enqueue fails (#845.8)", () => {
 		assert.equal(response.statusCode, 500);
 	});
 
-	it("leaves the row a draft, not stranded at queued", async () => {
+	it("settles the row at failed, not stranded at queued and not back at draft", async () => {
 		const { outboxMessageId } = await draftAgainstQueue(
 			refusingSqsClient().client,
 		);
 
-		assert.equal(await statusOf(outboxMessageId), OutboxMessageStatus.draft);
+		// `queued` is the dead end this suite exists for. `draft` is the other
+		// wrong answer: it is inside the SMTP worker's send fence, and an error
+		// from SQS does not prove the event was refused — one that landed anyway
+		// would send a row the user can still send by hand. `failed` is outside
+		// the fence and, since #933, still editable and sendable.
+		assert.equal(await statusOf(outboxMessageId), OutboxMessageStatus.failed);
 	});
 
 	it("still lets the user discard what could not be queued", async () => {