fix: new conversation opening using keyboard is not focusing on the l…

…ast message (#15681)

* fix: pass current history state while calling history param when showing a conversation

* test: add tests

src/script/router/Router.test.ts

@@ -19,7 +19,7 @@
 
 import {waitFor} from '@testing-library/react';
 
-import {configureRoutes, navigate} from './Router';
+import {configureRoutes, navigate, setHistoryParam} from './Router';
 
 describe('Router', () => {
   afterEach(() => {
@@ -77,4 +77,34 @@ describe('Router', () => {
       });
     });
   });
+
+  describe('setHistoryParam', () => {
+    // Mock window.history.replaceState before each test
+    beforeEach(() => {
+      global.history.replaceState = jest.fn();
+    });
+
+    // Restore the original method after each test
+    afterEach(() => {
+      (global.history.replaceState as jest.Mock).mockRestore();
+    });
+
+    it('throws an error if history.state is not empty and stateObj is not provided', () => {
+      Object.defineProperty(window.history, 'state', {value: {}, writable: true});
+
+      expect(() => setHistoryParam('/path')).toThrow('stateObj must be provided when history.state is not empty.');
+    });
+
+    it('does not throw an error if history.state is empty and stateObj is not provided', () => {
+      Object.defineProperty(window.history, 'state', {value: null, writable: true});
+
+      expect(() => setHistoryParam('/path')).not.toThrow();
+    });
+
+    it('does not throw an error if history.state is not empty and stateObj is provided', () => {
+      Object.defineProperty(window.history, 'state', {value: {}, writable: true}); // Temporarily override history.state for this test
+
+      expect(() => setHistoryParam('/path', {})).not.toThrow();
+    });
+  });
 });

src/script/router/Router.ts

@@ -46,5 +46,8 @@ export const navigate = (path: string, stateObj?: {}) => {
 };
 
 export const setHistoryParam = (path: string, stateObj?: {}) => {
+  if (window.history.state && !stateObj) {
+    throw new Error('stateObj must be provided when history.state is not empty.');
+  }
   window.history.replaceState(stateObj, '', `#${path}`);
 };

src/script/view_model/ContentViewModel.ts

@@ -218,6 +218,7 @@ export class ContentViewModel {
       this.previousConversation = this.conversationState.activeConversation();
       setHistoryParam(
         generateConversationUrl({id: conversationEntity?.id ?? '', domain: conversationEntity?.domain ?? ''}),
+        history.state,
       );
 
       if (openNotificationSettings) {