fix: thread order constant names + log date on threads

ahochsteger · May 29, 2024 · cb6624c · cb6624c
1 parent d1a1a08
commit cb6624c
Show file tree

Hide file tree

Showing 4 changed files with 15 additions and 9 deletions.
diff --git a/docs/docs/reference/enum-types.mdx b/docs/docs/reference/enum-types.mdx
@@ -185,6 +185,6 @@ Represents a thread field to be ordered by for processing.
 
 | Key | Value | Description |
 |-----|-------|-------------|
-| `DATE` | `lastMessageDate` | Order by the date of the last message in the thread. |
+| `FIRST_MESSAGE_SUBJECT` | `firstMessageSubject` | Order by the subject of the first message in the thread. |
 | `ID` | `id` | Order by the ID of the thread. |
-| `SUBJECT` | `firstMessageSubject` | Order by the subject of the first message in the thread. |
+| `LAST_MESSAGE_DATE` | `lastMessageDate` | Order by the date of the last message in the thread. |
diff --git a/src/lib/config/ThreadConfig.ts b/src/lib/config/ThreadConfig.ts
@@ -27,15 +27,15 @@ export enum ThreadOrderField {
   /**
    * Order by the date of the last message in the thread.
    */
-  DATE = "lastMessageDate",
+  LAST_MESSAGE_DATE = "lastMessageDate",
   /**
    * Order by the ID of the thread.
    */
   ID = "id",
   /**
    * Order by the subject of the first message in the thread.
    */
-  SUBJECT = "firstMessageSubject",
+  FIRST_MESSAGE_SUBJECT = "firstMessageSubject",
 }
 
 /**

diff --git a/src/lib/processors/ThreadProcessor.spec.ts b/src/lib/processors/ThreadProcessor.spec.ts
@@ -133,15 +133,21 @@ describe("order threads", () => {
   it("should order threads ascending", () => {
     threads = ThreadProcessor.ordered(
       threads,
-      { orderBy: ThreadOrderField.DATE, orderDirection: OrderDirection.ASC },
+      {
+        orderBy: ThreadOrderField.LAST_MESSAGE_DATE,
+        orderDirection: OrderDirection.ASC,
+      },
       ThreadProcessor.orderRules,
     )
     expect(threads.map((t) => t.getId())).toEqual(["t2", "t1", "t3"])
   })
   it("should order threads ascending", () => {
     threads = ThreadProcessor.ordered(
       threads,
-      { orderBy: ThreadOrderField.DATE, orderDirection: OrderDirection.DESC },
+      {
+        orderBy: ThreadOrderField.LAST_MESSAGE_DATE,
+        orderDirection: OrderDirection.DESC,
+      },
       ThreadProcessor.orderRules,
     )
     expect(threads.map((t) => t.getId())).toEqual(["t3", "t1", "t2"])

diff --git a/src/lib/processors/ThreadProcessor.ts b/src/lib/processors/ThreadProcessor.ts
@@ -359,10 +359,10 @@ export class ThreadProcessor extends BaseProcessor {
   ): number {
     return (
       {
-        [ThreadOrderField.DATE]:
+        [ThreadOrderField.LAST_MESSAGE_DATE]:
           a.getLastMessageDate().getTime() - b.getLastMessageDate().getTime(),
         [ThreadOrderField.ID]: a.getId().localeCompare(b.getId()),
-        [ThreadOrderField.SUBJECT]: a
+        [ThreadOrderField.FIRST_MESSAGE_SUBJECT]: a
           .getFirstMessageSubject()
           .localeCompare(b.getFirstMessageSubject()),
       }[config.orderBy] *
@@ -433,7 +433,7 @@ export class ThreadProcessor extends BaseProcessor {
     const config: RequiredThreadConfig = ctx.thread.config
     ctx.log.trace(ctx, {
       location: "ThreadProcessor.processEntity()",
-      message: `Processing thread id '${ctx.log.redact(ctx, thread.getId())}' (subject:'${ctx.log.redact(ctx, thread.getFirstMessageSubject())}') started ...`,
+      message: `Processing thread id '${ctx.log.redact(ctx, thread.getId())}' (lastMessageDate:${thread.getLastMessageDate().toISOString()}, subject:'${ctx.log.redact(ctx, thread.getFirstMessageSubject())}') started ...`,
     })
     // Execute pre-main actions:
     result = this.executeActions(