Skip to content

Commit

Permalink
workspace list card
Browse files Browse the repository at this point in the history
Signed-off-by: Hailong Cui <[email protected]>

recnetly viewed workspace

Signed-off-by: Hailong Cui <[email protected]>

recent

Signed-off-by: Hailong Cui <[email protected]>
  • Loading branch information
Hailong-am committed Jul 19, 2024
1 parent eaf5fea commit 70de058
Show file tree
Hide file tree
Showing 14 changed files with 535 additions and 5 deletions.
6 changes: 5 additions & 1 deletion src/core/public/chrome/index.ts
Original file line number Diff line number Diff line change
Expand Up @@ -46,7 +46,11 @@ export {
} from './ui/header/header_help_menu';
export { NavType, RightNavigationButton, RightNavigationButtonProps } from './ui';
export { ChromeNavLink, ChromeNavLinks, ChromeNavLinkUpdateableFields } from './nav_links';
export { ChromeRecentlyAccessed, ChromeRecentlyAccessedHistoryItem } from './recently_accessed';
export {
ChromeRecentlyAccessed,
ChromeRecentlyAccessedHistoryItem,
PersistedLog,
} from './recently_accessed';
export { ChromeNavControl, ChromeNavControls } from './nav_controls';
export { ChromeDocTitle } from './doc_title';
export { RightNavigationOrder } from './constants';
Expand Down
2 changes: 2 additions & 0 deletions src/core/public/chrome/recently_accessed/index.ts
Original file line number Diff line number Diff line change
Expand Up @@ -33,3 +33,5 @@ export {
ChromeRecentlyAccessedHistoryItem,
RecentlyAccessedService,
} from './recently_accessed_service';

export { PersistedLog } from './persisted_log';
2 changes: 2 additions & 0 deletions src/core/public/index.ts
Original file line number Diff line number Diff line change
Expand Up @@ -74,6 +74,7 @@ import {
ChromeRegistrationNavLink,
ChromeNavGroupUpdater,
NavGroupItemInMap,
PersistedLog,
} from './chrome';
import { FatalErrorsSetup, FatalErrorsStart, FatalErrorInfo } from './fatal_errors';
import { HttpSetup, HttpStart } from './http';
Expand Down Expand Up @@ -375,6 +376,7 @@ export {
ChromeRegistrationNavLink,
ChromeNavGroupUpdater,
NavGroupItemInMap,
PersistedLog,
};

export { __osdBootstrap__ } from './osd_bootstrap';
Expand Down
1 change: 1 addition & 0 deletions src/core/types/workspace.ts
Original file line number Diff line number Diff line change
Expand Up @@ -14,6 +14,7 @@ export interface WorkspaceAttribute {
icon?: string;
reserved?: boolean;
uiSettings?: Record<string, any>;
lastUpdatedTime?: string;
}

export interface WorkspaceAttributeWithPermission extends WorkspaceAttribute {
Expand Down
2 changes: 2 additions & 0 deletions src/plugins/workspace/common/constants.ts
Original file line number Diff line number Diff line change
Expand Up @@ -184,3 +184,5 @@ export const WORKSPACE_USE_CASES = Object.freeze({
});

export const CURRENT_USER_PLACEHOLDER = '%me%';
export const MAX_WORKSPACE_NAME_LENGTH = 25;
export const RECENT_WORKSPACES_KEY = 'recentWorkspaces';
2 changes: 1 addition & 1 deletion src/plugins/workspace/opensearch_dashboards.json
Original file line number Diff line number Diff line change
Expand Up @@ -7,6 +7,6 @@
"savedObjects",
"opensearchDashboardsReact"
],
"optionalPlugins": ["savedObjectsManagement","management","dataSourceManagement"],
"optionalPlugins": ["savedObjectsManagement","management","dataSourceManagement","contentManagement"],
"requiredBundles": ["opensearchDashboardsReact"]
}

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

6 changes: 6 additions & 0 deletions src/plugins/workspace/public/components/service_card/index.ts
Original file line number Diff line number Diff line change
@@ -0,0 +1,6 @@
/*
* Copyright OpenSearch Contributors
* SPDX-License-Identifier: Apache-2.0
*/

export { WorkspaceListCard } from './workspace_list_card';
Original file line number Diff line number Diff line change
@@ -0,0 +1,66 @@
/*
* Copyright OpenSearch Contributors
* SPDX-License-Identifier: Apache-2.0
*/

import React from 'react';
import { coreMock } from '../../../../../core/public/mocks';
import { fireEvent, render, waitFor } from '@testing-library/react';
import { WorkspaceListCard } from './workspace_list_card';
import { recentWorkspaceManager } from '../../recent_workspace_manager';

describe('workspace list card render normally', () => {
const coreStart = coreMock.createStart();

beforeAll(() => {
const workspaceList = [
{
id: 'ws-1',
name: 'foo',
lastUpdatedTime: new Date().toISOString(),
},
{
id: 'ws-2',
name: 'bar',
lastUpdatedTime: new Date().toISOString(),
},
];
coreStart.workspaces.workspaceList$.next(workspaceList);
});

it('should show workspace list card correctly', () => {
const { container } = render(<WorkspaceListCard core={coreStart} />);
expect(container).toMatchSnapshot();
});

it('should show empty state if no recently viewed workspace', () => {
const { getByTestId, getByText } = render(<WorkspaceListCard core={coreStart} />);
expect(getByTestId('workspace_filter')).toHaveDisplayValue('Recently viewed');

// empty statue for recently viewed
expect(getByText('Workspaces you have recently viewed will appear here.')).toBeInTheDocument();
});

it('should show default filter as recently viewed', () => {
recentWorkspaceManager.addRecentWorkspace('foo');
const { getByTestId, getByText } = render(<WorkspaceListCard core={coreStart} />);
expect(getByTestId('workspace_filter')).toHaveDisplayValue('Recently viewed');

waitFor(() => {
expect(getByText('foo')).toBeInTheDocument();
});
});

it('should show updated filter correctly', () => {
const { getByTestId, getByText } = render(<WorkspaceListCard core={coreStart} />);
expect(getByTestId('workspace_filter')).toHaveDisplayValue('Recently viewed');

const filterSelector = getByTestId('workspace_filter');
fireEvent.change(filterSelector, { target: { value: 'updated' } });
expect(getByTestId('workspace_filter')).toHaveDisplayValue('Recently updated');

// workspace list
expect(getByText('foo')).toBeInTheDocument();
expect(getByText('bar')).toBeInTheDocument();
});
});
Loading

0 comments on commit 70de058

Please sign in to comment.