Skip to content

Commit 1ee35e8

Browse files
NicolappsConvex, Inc.
authored andcommitted
Use fn to handle component events in Storybook (#42066)
GitOrigin-RevId: 4d8b2ab7421ba19b2c6117fc8c8ca2a9f3d999ad
1 parent 4170945 commit 1ee35e8

File tree

16 files changed

+79
-64
lines changed

16 files changed

+79
-64
lines changed

npm-packages/common/config/rush/pnpm-lock.yaml

Lines changed: 35 additions & 32 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

npm-packages/dashboard-common/src/elements/DateRangePicker.stories.tsx

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,13 +1,14 @@
11
import { Meta, StoryObj } from "@storybook/nextjs";
22
import { DateRangePicker } from "@common/elements/DateRangePicker";
3+
import { fn } from "storybook/test";
34

45
export const Primary: Story = {
56
args: {
67
date: {
78
from: new Date(Date.now() - 7 * 24 * 60 * 60 * 1000),
89
to: new Date(Date.now() + 7 * 24 * 60 * 60 * 1000),
910
},
10-
setDate: () => {},
11+
setDate: fn(),
1112
},
1213
};
1314

npm-packages/dashboard-common/src/elements/ObjectEditor/ObjectEditor.stories.tsx

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -5,6 +5,7 @@ import { ObjectEditor } from "@common/elements/ObjectEditor/ObjectEditor";
55
import { mockConvexReactClient } from "@common/lib/mockConvexReactClient";
66
import { DeploymentInfoContext } from "@common/lib/deploymentContext";
77
import { mockDeploymentInfo } from "@common/lib/mockDeploymentInfo";
8+
import { fn } from "storybook/test";
89

910
const mockClient = mockConvexReactClient()
1011
.registerQueryFake(udfs.listById.default, ({ ids }) => ids.map(() => null))
@@ -30,8 +31,8 @@ type Story = StoryObj<typeof meta>;
3031
export const Primary: Story = {
3132
args: {
3233
defaultValue: null,
33-
onChange: () => {},
34-
onError: () => {},
34+
onChange: fn(),
35+
onError: fn(),
3536
path: "document",
3637
mode: "editField",
3738
},

npm-packages/dashboard-common/src/elements/Sidebar.stories.tsx

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -8,6 +8,7 @@ import {
88
Cross1Icon,
99
} from "@radix-ui/react-icons";
1010
import { Sidebar } from "@common/elements/Sidebar";
11+
import { fn } from "storybook/test";
1112

1213
const meta = {
1314
component: Sidebar,
@@ -17,7 +18,7 @@ const meta = {
1718
</div>
1819
),
1920
args: {
20-
setCollapsed: () => {},
21+
setCollapsed: fn(),
2122
},
2223
} satisfies Meta<typeof Sidebar>;
2324

npm-packages/dashboard-common/src/features/data/components/ContextMenu.stories.tsx

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -3,12 +3,13 @@ import { useRef, useState } from "react";
33
import { ExternalLinkIcon, StarIcon, TrashIcon } from "@radix-ui/react-icons";
44
import { ContextMenu } from "@common/features/data/components/ContextMenu";
55
import { useContextMenuTrigger } from "@common/features/data/lib/useContextMenuTrigger";
6+
import { fn } from "storybook/test";
67

78
const meta = {
89
component: ContextMenu,
910
args: {
1011
target: null,
11-
onClose: () => {},
12+
onClose: fn(),
1213
},
1314
argTypes: {
1415
target: {

npm-packages/dashboard-common/src/features/data/components/DataFilters/DataFilters.stories.tsx

Lines changed: 4 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -9,6 +9,7 @@ import {
99
DeploymentInfoContext,
1010
} from "@common/lib/deploymentContext";
1111
import { mockDeploymentInfo } from "@common/lib/mockDeploymentInfo";
12+
import { fn } from "storybook/test";
1213

1314
// @ts-expect-error
1415
const deployment: ConnectedDeployment = {};
@@ -51,12 +52,9 @@ export const Default: Story = {
5152
tableName: "myTable",
5253
defaultDocument: { myColumn: 0 },
5354
filters: { clauses: [] },
54-
onFiltersChange: () => {
55-
// eslint-disable-next-line no-alert
56-
alert("Filters applied");
57-
},
58-
setDraftFilters: () => {},
59-
setShowFilters: () => {},
55+
onFiltersChange: fn(),
56+
setDraftFilters: fn(),
57+
setShowFilters: fn(),
6058
tableFields: ["myColumn"],
6159
componentId: "myComponent",
6260
activeSchema: null,

npm-packages/dashboard-common/src/features/data/components/DataFilters/IndexFilters.stories.tsx

Lines changed: 7 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -5,6 +5,7 @@ import { mockConvexReactClient } from "@common/lib/mockConvexReactClient";
55
import { ConvexProvider } from "convex/react";
66
import udfs from "@common/udfs";
77
import { SearchIndexFilter } from "system-udfs/convex/_system/frontend/lib/filters";
8+
import { fn } from "storybook/test";
89
import { IndexFilters } from "./IndexFilters";
910

1011
const mockClient = mockConvexReactClient()
@@ -109,12 +110,12 @@ const meta: Meta<typeof IndexFilters> = {
109110
],
110111
},
111112
getValidatorForField: () => ({ type: "string" }),
112-
onFiltersChange: () => {},
113-
applyFiltersWithHistory: async () => {},
114-
setDraftFilters: () => {},
115-
onChangeOrder: () => {},
116-
onChangeIndexFilter: () => {},
117-
onError: () => {},
113+
onFiltersChange: fn(),
114+
applyFiltersWithHistory: fn(),
115+
setDraftFilters: fn(),
116+
onChangeOrder: fn(),
117+
onChangeIndexFilter: fn(),
118+
onError: fn(),
118119
hasInvalidFilters: false,
119120
invalidFilters: {},
120121
},

npm-packages/dashboard-common/src/features/data/components/FilterEditor/DateTimePicker.stories.tsx

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,10 +1,11 @@
11
import { Meta, StoryObj } from "@storybook/nextjs";
22
import { DateTimePicker } from "@common/features/data/components/FilterEditor/DateTimePicker";
3+
import { fn } from "storybook/test";
34

45
export const Primary: Story = {
56
args: {
67
date: new Date(),
7-
onChange: () => {},
8+
onChange: fn(),
89
},
910
};
1011

npm-packages/dashboard-common/src/features/data/components/FilterEditor/FilterEditor.stories.tsx

Lines changed: 5 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -5,6 +5,7 @@ import { FilterEditor } from "@common/features/data/components/FilterEditor/Filt
55
import { mockConvexReactClient } from "@common/lib/mockConvexReactClient";
66
import { DeploymentInfoContext } from "@common/lib/deploymentContext";
77
import { mockDeploymentInfo } from "@common/lib/mockDeploymentInfo";
8+
import { fn } from "storybook/test";
89

910
const mockClient = mockConvexReactClient()
1011
.registerQueryFake(udfs.listById.default, ({ ids }) => ids.map(() => null))
@@ -30,9 +31,9 @@ export const Editor: Story = {
3031
args: {
3132
fields: ["_id", "_creationTime", "myColumn"],
3233
defaultDocument: { myColumn: 0 },
33-
onChange: () => {},
34-
onDelete: () => {},
35-
onApplyFilters: () => {},
36-
onError: () => {},
34+
onChange: fn(),
35+
onDelete: fn(),
36+
onApplyFilters: fn(),
37+
onError: fn(),
3738
},
3839
};

npm-packages/dashboard-common/src/features/data/components/Table/EditDocumentPanel/EditDocumentPanel.stories.tsx

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -7,6 +7,7 @@ import { mockConvexReactClient } from "@common/lib/mockConvexReactClient";
77
import { Panel, PanelGroup } from "react-resizable-panels";
88
import { DeploymentInfoContext } from "@common/lib/deploymentContext";
99
import { mockDeploymentInfo } from "@common/lib/mockDeploymentInfo";
10+
import { fn } from "storybook/test";
1011

1112
const mockClient = mockConvexReactClient()
1213
.registerQueryFake(udfs.listById.default, ({ ids }) => ids.map(() => null))
@@ -17,8 +18,8 @@ const meta = {
1718
component: EditDocumentPanel,
1819
args: {
1920
tableName: "users",
20-
onClose: () => {},
21-
onSave: async () => {},
21+
onClose: fn(),
22+
onSave: fn(),
2223
},
2324
render: (args) => (
2425
<ConvexProvider client={mockClient}>

0 commit comments

Comments
 (0)