Merge pull request #428 from ably/integration-test-history-operation

Test operation fields in chat history
ably · Dec 4, 2024 · 6a4f924 · 6a4f924
2 parents 9de6a77 + 11dc56a
commit 6a4f924
Showing 1 changed file with 4 additions and 4 deletions.
diff --git a/test/core/messages.integration.test.ts b/test/core/messages.integration.test.ts
@@ -284,8 +284,8 @@ describe('messages integration', { timeout: 10000 }, () => {
     expect(history.items[0]?.isUpdated).toBe(false);
     expect(history.items[0]?.isDeleted).toBe(true);
     expect(history.items[0]?.deletedAt).toEqual(deletedMessage1.deletedAt);
-    // todo: uncomment when operation is returned correctly in history endpoint
-    // expect(history.items[0]?.deletedBy).toEqual(deletedMessage1.deletedBy);
+    expect(history.items[0]?.deletedBy).toEqual(deletedMessage1.deletedBy);
+    expect(history.items[0]?.operation?.description).toEqual('Deleted message');
 
     // We shouldn't have a "next" link in the response
     expect(history.hasNext()).toBe(false);
@@ -325,8 +325,8 @@ describe('messages integration', { timeout: 10000 }, () => {
     expect(history.items[0]?.isUpdated).toBe(true);
     expect(history.items[0]?.isDeleted).toBe(false);
     expect(history.items[0]?.updatedAt).toEqual(updatedMessage1.updatedAt);
-    // todo: uncomment when operation is returned correctly in history endpoint
-    // expect(history.items[0]?.updatedBy).toEqual(updatedMessage1.updatedBy);
+    expect(history.items[0]?.updatedBy).toEqual(updatedMessage1.updatedBy);
+    expect(history.items[0]?.operation?.description).toEqual('updated message');
 
     // We shouldn't have a "next" link in the response
     expect(history.hasNext()).toBe(false);