added more tests

Signed-off-by: Abhinav Kumar <[email protected]>

Author: abhinavkrin

apps/meteor/client/views/root/MainLayout/LayoutWithSidebar.spec.tsx

@@ -53,6 +53,28 @@ describe('LayoutWithSidebar - First_Channel_After_Login navigation', () => {
 		expect(push).toHaveBeenCalledWith({ name: 'general' });
 	});
 
+	it('does NOT redirect if First_Channel_After_Login starts with "?"', () => {
+		const push = setupChannelRouteMock();
+		mockedUseCurrentRoutePath.mockReturnValue('/');
+
+		render(<LayoutWithSidebar>content</LayoutWithSidebar>, {
+			wrapper: mockAppRoot().withSetting('First_Channel_After_Login', '?general').build(),
+		});
+
+		expect(push).not.toHaveBeenCalled();
+	});
+
+	it('does NOT redirect if First_Channel_After_Login starts with "@@"', () => {
+		const push = setupChannelRouteMock();
+		mockedUseCurrentRoutePath.mockReturnValue('/');
+
+		render(<LayoutWithSidebar>content</LayoutWithSidebar>, {
+			wrapper: mockAppRoot().withSetting('First_Channel_After_Login', '@@general').build(),
+		});
+
+		expect(push).not.toHaveBeenCalled();
+	});
+
 	it('redirects when route is "/home"', () => {
 		const push = setupChannelRouteMock();
 		mockedUseCurrentRoutePath.mockReturnValue('/home');

apps/meteor/client/views/root/MainLayout/LayoutWithSidebarV2.spec.tsx

@@ -62,6 +62,28 @@ describe('LayoutWithSidebarV2 - First_Channel_After_Login navigation', () => {
 		expect(navigate).toHaveBeenCalledWith({ name: '/channel/general' });
 	});
 
+	it('does NOT redirect if First_Channel_After_Login starts with "?"', () => {
+		const navigate = setupRouterMock();
+		mockedUseCurrentRoutePath.mockReturnValue('/');
+
+		render(<LayoutWithSidebarV2>content</LayoutWithSidebarV2>, {
+			wrapper: mockAppRoot().withSetting('First_Channel_After_Login', '?general').build(),
+		});
+
+		expect(navigate).not.toHaveBeenCalled();
+	});
+
+	it('does NOT redirect if First_Channel_After_Login starts with "@@"', () => {
+		const navigate = setupRouterMock();
+		mockedUseCurrentRoutePath.mockReturnValue('/');
+
+		render(<LayoutWithSidebarV2>content</LayoutWithSidebarV2>, {
+			wrapper: mockAppRoot().withSetting('First_Channel_After_Login', '@@general').build(),
+		});
+
+		expect(navigate).not.toHaveBeenCalled();
+	});
+
 	it('redirects when route is "/home"', () => {
 		const navigate = setupRouterMock();
 		mockedUseCurrentRoutePath.mockReturnValue('/home');