Skip to content

Commit 8ef178d

Browse files
committed
Fix Discord typing test hook merge drift
1 parent 6b2ea14 commit 8ef178d

3 files changed

Lines changed: 27 additions & 19 deletions

File tree

extensions/discord/src/monitor/message-handler.queue.test.ts

Lines changed: 9 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -212,7 +212,7 @@ describe("createDiscordMessageHandler queue behavior", () => {
212212

213213
const handler = createDiscordMessageHandler({
214214
...createDiscordHandlerParams(),
215-
__testing: { createReplyTypingFeedback },
215+
testing: { createReplyTypingFeedback },
216216
});
217217
await expect(
218218
handler(createMessageData("m-typing", "dm-1") as never, {} as never),
@@ -248,7 +248,7 @@ describe("createDiscordMessageHandler queue behavior", () => {
248248

249249
const handler = createDiscordMessageHandler({
250250
...createDiscordHandlerParams(),
251-
__testing: { createReplyTypingFeedback },
251+
testing: { createReplyTypingFeedback },
252252
});
253253
await expect(
254254
handler(createMessageData("m-typing-fails", "dm-1") as never, {} as never),
@@ -271,7 +271,7 @@ describe("createDiscordMessageHandler queue behavior", () => {
271271

272272
const handler = createDiscordMessageHandler({
273273
...createDiscordHandlerParams(),
274-
__testing: { createReplyTypingFeedback },
274+
testing: { createReplyTypingFeedback },
275275
});
276276
await expect(
277277
handler(createMessageData("m-rejected", "dm-1") as never, {} as never),
@@ -303,7 +303,7 @@ describe("createDiscordMessageHandler queue behavior", () => {
303303

304304
const handler = createDiscordMessageHandler({
305305
...createDiscordHandlerParams(),
306-
__testing: { createReplyTypingFeedback },
306+
testing: { createReplyTypingFeedback },
307307
});
308308
await expect(
309309
handler(createMessageData("m-never-mode", "dm-1") as never, {} as never),
@@ -338,7 +338,7 @@ describe("createDiscordMessageHandler queue behavior", () => {
338338

339339
const handler = createDiscordMessageHandler({
340340
...createDiscordHandlerParams(),
341-
__testing: { createReplyTypingFeedback },
341+
testing: { createReplyTypingFeedback },
342342
});
343343
await expect(
344344
handler(createMessageData(`m-${typingMode}-mode`, "dm-1") as never, {} as never),
@@ -373,7 +373,7 @@ describe("createDiscordMessageHandler queue behavior", () => {
373373

374374
const handler = createDiscordMessageHandler({
375375
...createDiscordHandlerParams(),
376-
__testing: { createReplyTypingFeedback },
376+
testing: { createReplyTypingFeedback },
377377
});
378378
await expect(
379379
handler(createMessageData("m-guild", "guild-channel") as never, {} as never),
@@ -422,7 +422,7 @@ describe("createDiscordMessageHandler queue behavior", () => {
422422

423423
const handler = createDiscordMessageHandler({
424424
...createDiscordHandlerParams(),
425-
__testing: { createReplyTypingFeedback },
425+
testing: { createReplyTypingFeedback },
426426
});
427427
await expect(
428428
handler(createMessageData("m-guild-instant", "guild-channel") as never, {} as never),
@@ -465,7 +465,7 @@ describe("createDiscordMessageHandler queue behavior", () => {
465465
});
466466
const handler = createDiscordMessageHandler({
467467
...createDiscordHandlerParams(),
468-
__testing: { createReplyTypingFeedback },
468+
testing: { createReplyTypingFeedback },
469469
});
470470

471471
await expect(handler(createMessageData("m-1") as never, {} as never)).resolves.toBeUndefined();
@@ -793,7 +793,7 @@ describe("createDiscordMessageHandler queue behavior", () => {
793793

794794
const handler = createDiscordMessageHandler({
795795
...createDiscordHandlerParams(),
796-
__testing: { createReplyTypingFeedback },
796+
testing: { createReplyTypingFeedback },
797797
});
798798
await expect(handler(createMessageData("m-1") as never, {} as never)).resolves.toBeUndefined();
799799
await flushQueueWork();

extensions/discord/src/monitor/message-handler.ts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -210,7 +210,7 @@ export function createDiscordMessageHandler(
210210
}
211211
startAcceptedTypingFeedback({
212212
ctx,
213-
createFeedback: params.__testing?.createReplyTypingFeedback,
213+
createFeedback: params.testing?.createReplyTypingFeedback,
214214
});
215215
applyImplicitReplyBatchGate(ctx, params.replyToMode, false);
216216
messageRunQueue.enqueue(buildDiscordInboundJob(ctx, { replayKeys }));
@@ -263,7 +263,7 @@ export function createDiscordMessageHandler(
263263
}
264264
startAcceptedTypingFeedback({
265265
ctx,
266-
createFeedback: params.__testing?.createReplyTypingFeedback,
266+
createFeedback: params.testing?.createReplyTypingFeedback,
267267
});
268268
applyImplicitReplyBatchGate(ctx, params.replyToMode, true);
269269
if (entries.length > 1) {

extensions/discord/src/monitor/message-run-queue.ts

Lines changed: 16 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -102,14 +102,22 @@ export function createDiscordMessageRunQueue(
102102

103103
return {
104104
enqueue(job) {
105-
runQueue.enqueue(job.queueKey, async ({ lifecycleSignal }) => {
106-
await processDiscordQueuedMessage({
107-
job,
108-
lifecycleSignal,
109-
replayGuard,
110-
testing: params.testing,
111-
});
112-
});
105+
runQueue.enqueue(
106+
job.queueKey,
107+
async ({ lifecycleSignal }) => {
108+
await processDiscordQueuedMessage({
109+
job,
110+
lifecycleSignal,
111+
replayGuard,
112+
testing: params.testing,
113+
});
114+
},
115+
{
116+
onSkip: () => {
117+
cleanupSkippedDiscordQueuedMessage({ job, replayGuard });
118+
},
119+
},
120+
);
113121
},
114122
deactivate: runQueue.deactivate,
115123
};

0 commit comments

Comments
 (0)