[Workspace]fix: recent item links are not correctly constructed (#9275)

* fix: recent item links are not correctly constructed

Signed-off-by: SuZhou-Joe <[email protected]>

* Changeset file for PR #9275 created/updated

---------

Signed-off-by: SuZhou-Joe <[email protected]>
Co-authored-by: opensearch-changeset-bot[bot] <154024398+opensearch-changeset-bot[bot]@users.noreply.github.com>

changelogs/fragments/9275.yml

@@ -0,0 +1,2 @@
+fix:
+- Recent item links are not correctly constructed ([#9275](https://github.com/opensearch-project/OpenSearch-Dashboards/pull/9275))

src/core/public/chrome/ui/header/header.tsx

@@ -502,6 +502,7 @@ export function Header({
         renderBreadcrumbs={renderBreadcrumbs(true, true)}
         buttonSize={useApplicationHeader ? 's' : 'xs'}
         loadingCount$={observables.loadingCount$}
+        workspaceEnabled={application.capabilities.workspaces.enabled}
       />
     </EuiHeaderSectionItem>
   );

src/core/public/chrome/ui/header/recent_items.test.tsx

@@ -9,6 +9,7 @@ import { BehaviorSubject } from 'rxjs';
 import { applicationServiceMock, httpServiceMock } from '../../../mocks';
 import { SavedObjectWithMetadata } from './recent_items';
 import { RecentItems } from './recent_items';
+import { createRecentNavLink } from './nav_link';
 
 jest.mock('./nav_link', () => ({
   createRecentNavLink: jest.fn().mockImplementation(() => {
@@ -171,4 +172,26 @@ describe('Recent items', () => {
     render(<RecentItems {...defaultMockProps} recentlyAccessed$={mockRecentlyAccessed$} />);
     expect(screen.queryByText('visualizeMock')).not.toBeInTheDocument();
   });
+
+  it('workspace feature flag should be passed to createRecentNavLink correctly', async () => {
+    jest.clearAllMocks();
+    const { getByTestId, findByText, getByText } = render(
+      <RecentItems
+        {...defaultMockProps}
+        workspaceEnabled
+        recentlyAccessed$={mockRecentlyAccessed$}
+        workspaceList$={mockWorkspaceList$}
+      />
+    );
+    fireEvent.click(getByTestId('recentItemsSectionButton'));
+    await findByText('Recent assets');
+    fireEvent.click(getByText('visualizeMock'));
+    expect(createRecentNavLink).toBeCalledWith(
+      expect.anything(),
+      expect.anything(),
+      expect.anything(),
+      expect.anything(),
+      true
+    );
+  });
 });

src/core/public/chrome/ui/header/recent_items.tsx

@@ -45,6 +45,7 @@ export interface Props {
   buttonSize?: EuiHeaderSectionItemButtonProps['size'];
   http: HttpStart;
   loadingCount$: Rx.Observable<number>;
+  workspaceEnabled?: boolean;
 }
 
 interface SavedObjectMetadata {
@@ -112,6 +113,7 @@ export const RecentItems = ({
   buttonSize = 's',
   http,
   loadingCount$,
+  workspaceEnabled,
 }: Props) => {
   const [isPopoverOpen, setIsPopoverOpen] = useState(false);
   const [isPreferencesPopoverOpen, setIsPreferencesPopoverOpen] = useState(false);
@@ -143,6 +145,9 @@ export const RecentItems = ({
           }}
           iconType="managementApp"
           data-test-subj="preferencesSettingButton"
+          aria-label={i18n.translate('core.header.recent.preferences.buttonAriaLabel', {
+            defaultMessage: 'Preferences setting',
+          })}
         />
       }
       isOpen={isPreferencesPopoverOpen}
@@ -296,7 +301,8 @@ export const RecentItems = ({
                       item,
                       navLinks.filter((link) => !link.hidden),
                       basePath,
-                      navigateToUrl
+                      navigateToUrl,
+                      !!workspaceEnabled
                     ).href
                   )
                 }