Skip to content

Commit 944b8d4

Browse files
authored
feat(content-explorer): add story and visual regression for theming (#3889)
* feat(content-explorer): add story and visual regression for theming * fix: update mock import path
1 parent c51d1f6 commit 944b8d4

File tree

11 files changed

+676
-10
lines changed

11 files changed

+676
-10
lines changed

src/elements/__mocks__/mockRequests.ts renamed to src/elements/common/__mocks__/mockRequests.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
import { DEFAULT_HOSTNAME_API } from '../../constants';
1+
import { DEFAULT_HOSTNAME_API } from '../../../constants';
22

33
const apiV2Path = `${DEFAULT_HOSTNAME_API}/2.0`;
44

src/elements/content-explorer/stories/__mocks__/mockSubFolder.ts renamed to src/elements/common/__mocks__/mockSubfolder.ts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
const mockSubFolder = {
1+
const mockSubfolder = {
22
type: 'folder',
33
id: '73426618530',
44
etag: '3',
@@ -154,4 +154,4 @@ const mockSubFolder = {
154154
},
155155
};
156156

157-
export default mockSubFolder;
157+
export default mockSubfolder;

src/elements/common/__mocks__/mockTheme.ts

Lines changed: 642 additions & 0 deletions
Large diffs are not rendered by default.

src/elements/content-explorer/stories/ContentExplorer.stories.js

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,7 @@
11
// @flow
22

33
import ContentExplorer from '../ContentExplorer';
4+
import mockTheme from '../../common/__mocks__/mockTheme';
45

56
export const basic = {};
67

@@ -25,6 +26,12 @@ export const withSidebar = {
2526
},
2627
};
2728

29+
export const withTheming = {
30+
args: {
31+
theme: mockTheme,
32+
},
33+
};
34+
2835
export default {
2936
title: 'Elements/ContentExplorer',
3037
component: ContentExplorer,

src/elements/content-explorer/stories/tests/ContentExplorer-visual.stories.js

Lines changed: 13 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -2,8 +2,8 @@ import { http, HttpResponse } from 'msw';
22
import { expect, userEvent, waitFor, within, screen } from '@storybook/test';
33

44
import ContentExplorer from '../../ContentExplorer';
5-
import { mockEmptyRootFolder, mockRootFolder } from '../__mocks__/mockRootFolder';
6-
import mockSubFolder from '../__mocks__/mockSubFolder';
5+
import { mockEmptyRootFolder, mockRootFolder } from '../../../common/__mocks__/mockRootFolder';
6+
import mockSubfolder from '../../../common/__mocks__/mockSubfolder';
77

88
import { DEFAULT_HOSTNAME_API } from '../../../../constants';
99

@@ -243,6 +243,16 @@ export const searchEmptyState = {
243243
},
244244
};
245245

246+
export const withTheming = {
247+
play: async ({ canvasElement }) => {
248+
const canvas = within(canvasElement);
249+
250+
await waitFor(async () => {
251+
expect(await canvas.getByText('Preview Test Folder')).toBeInTheDocument();
252+
});
253+
},
254+
};
255+
246256
export default {
247257
title: 'Elements/ContentExplorer/tests/ContentExplorer/visual',
248258
component: ContentExplorer,
@@ -258,7 +268,7 @@ export default {
258268
return HttpResponse.json(mockRootFolder);
259269
}),
260270
http.get(`${DEFAULT_HOSTNAME_API}/2.0/folders/73426618530`, () => {
261-
return HttpResponse.json(mockSubFolder);
271+
return HttpResponse.json(mockSubfolder);
262272
}),
263273
http.get(`${DEFAULT_HOSTNAME_API}/2.0/folders/74729718131`, () => {
264274
return HttpResponse.json(mockEmptyRootFolder);

src/elements/content-picker/stories/tests/ContentPicker-visual.stories.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11
import { expect, userEvent, waitFor, within } from '@storybook/test';
22
import { http, HttpResponse } from 'msw';
33
import ContentPicker from '../../ContentPicker';
4-
import { mockRootFolder } from '../../../content-explorer/stories/__mocks__/mockRootFolder';
4+
import { mockRootFolder } from '../../../common/__mocks__/mockRootFolder';
55

66
import { DEFAULT_HOSTNAME_API } from '../../../../constants';
77

src/elements/content-preview/stories/tests/ContentPreview-visual.stories.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,7 @@ import { expect, userEvent, waitFor, within } from '@storybook/test';
55
import { Notification } from '@box/blueprint-web';
66
import { http, HttpResponse } from 'msw';
77

8-
import { mockEventRequest, mockFileRequest, mockUserRequest } from '../../../__mocks__/mockRequests';
8+
import { mockEventRequest, mockFileRequest, mockUserRequest } from '../../../common/__mocks__/mockRequests';
99
import { DEFAULT_HOSTNAME_API } from '../../../../constants';
1010

1111
import ContentPreview from '../../ContentPreview';

src/elements/content-sidebar/stories/tests/BoxAISidebar-visual.stories.tsx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,7 @@ import { type StoryObj } from '@storybook/react';
33
import { http, HttpResponse } from 'msw';
44
import ContentSidebar from '../../ContentSidebar';
55
import BoxAISidebar from '../../BoxAISidebar';
6-
import { mockFileRequest, mockUserRequest } from '../../../__mocks__/mockRequests';
6+
import { mockFileRequest, mockUserRequest } from '../../../common/__mocks__/mockRequests';
77

88
const mockFeatures = {
99
'boxai.sidebar.enabled': true,

src/elements/content-sidebar/stories/tests/MetadataSidebarRedesign-visual.stories.tsx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -17,7 +17,7 @@ import {
1717
mockGlobalMetadataTemplates,
1818
mockMetadataInstances,
1919
} from '../__mocks__/MetadataSidebarRedesignedMocks';
20-
import { mockUserRequest } from '../../../__mocks__/mockRequests';
20+
import { mockUserRequest } from '../../../common/__mocks__/mockRequests';
2121

2222
const token = global.TOKEN;
2323

0 commit comments

Comments
 (0)