Added test cases

ls1intum · Oct 29, 2024 · 9979342 · 9979342
1 parent 9726517
commit 9979342
Show file tree

Hide file tree

Showing 2 changed files with 22 additions and 2 deletions.
diff --git a/...view/course-conversations/layout/conversation-messages/conversation-messages.component.ts b/...view/course-conversations/layout/conversation-messages/conversation-messages.component.ts
@@ -39,7 +39,7 @@ import { CustomBreakpointNames } from 'app/shared/breakpoints/breakpoints.servic
 })
 export class ConversationMessagesComponent implements OnInit, AfterViewInit, OnDestroy {
     private ngUnsubscribe = new Subject<void>();
-    private sessionStorageKey = 'conversationId.scrollPosition.';
+    sessionStorageKey = 'conversationId.scrollPosition.';
 
     readonly PageType = PageType;
     readonly ButtonType = ButtonType;
@@ -282,7 +282,7 @@ export class ConversationMessagesComponent implements OnInit, AfterViewInit, OnD
         });
     }
 
-    private saveScrollPosition = () => {
+    saveScrollPosition = () => {
         this.scrollSubject.next(this.content.nativeElement.scrollTop);
     };
 

diff --git a/...course-conversations/layout/conversation-messages/conversation-messages.component.spec.ts b/...course-conversations/layout/conversation-messages/conversation-messages.component.spec.ts
@@ -131,6 +131,26 @@ examples.forEach((activeConversation) => {
             expect(getFilteredPostSpy).toHaveBeenCalledOnce();
         }));
 
+        it('should save the scroll position in sessionStorage', fakeAsync(() => {
+            // Überwache sessionStorage.setItem
+            const setItemSpy = jest.spyOn(sessionStorage, 'setItem');
+            component.ngOnInit();
+            component.content.nativeElement.scrollTop = 200;
+            component.saveScrollPosition();
+            tick(100);
+            const expectedKey = `${component.sessionStorageKey}${component._activeConversation?.id}`;
+            const expectedValue = '200';
+            expect(setItemSpy).toHaveBeenCalledWith(expectedKey, expectedValue);
+        }));
+
+        it('should scroll to the bottom when a new message is created', fakeAsync(() => {
+            component.content.nativeElement.scrollTop = 100;
+            fixture.detectChanges();
+            component.handleNewMessageCreated();
+            tick();
+            expect(component.content.nativeElement.scrollTop).toBe(component.content.nativeElement.scrollHeight);
+        }));
+
         it('should create empty post with the correct conversation type', fakeAsync(() => {
             const createEmptyPostForContextSpy = jest.spyOn(metisService, 'createEmptyPostForContext').mockReturnValue(new Post());
             component.createEmptyPost();