Skip to content

Commit b9c1019

Browse files
committed
test: remove unnecessary BlockContext wrapper from TopBar CurrentTask test
Remove legacy BlockContext.Provider wrapper that is no longer needed. CurrentTask component was migrated to use cn() helper and no longer uses BlockContext internally (no useContext or useBEM calls), so the context wrapper in tests is unnecessary. Changes: - Removed BlockContext import - Removed all BlockContext.Provider wrappers around component renders - Tests still pass - component doesn't consume the context This cleanup allows us to potentially remove BlockContext from bem.ts exports.
1 parent c86fd60 commit b9c1019

File tree

1 file changed

+5
-26
lines changed

1 file changed

+5
-26
lines changed

web/libs/editor/src/components/TopBar/__tests__/CurrentTask.test.tsx

Lines changed: 5 additions & 26 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,5 @@
11
import { render } from "@testing-library/react";
22
import { CurrentTask } from "../CurrentTask";
3-
import { BlockContext, cn } from "../../../utils/bem.ts";
43
import { FF_LEAP_1173 } from "../../../utils/feature-flags";
54
import { mockFF } from "../../../../__mocks__/global";
65

@@ -53,11 +52,7 @@ describe("CurrentTask", () => {
5352
["skip", "postpone", "topbar:prevnext", "topbar:task-counter"].includes(interfaceName),
5453
);
5554

56-
const { rerender, getByTestId } = render(
57-
<BlockContext.Provider value={cn("block-name")}>
58-
<CurrentTask store={store} />
59-
</BlockContext.Provider>,
60-
);
55+
const { rerender, getByTestId } = render(<CurrentTask store={store} />);
6156

6257
expect(getByTestId("next-task").disabled).toBe(false);
6358

@@ -71,11 +66,7 @@ describe("CurrentTask", () => {
7166
),
7267
};
7368

74-
rerender(
75-
<BlockContext.Provider value={cn("block-name")}>
76-
<CurrentTask store={store} />
77-
</BlockContext.Provider>,
78-
);
69+
rerender(<CurrentTask store={store} />);
7970

8071
expect(getByTestId("next-task").disabled).toBe(true);
8172

@@ -89,11 +80,7 @@ describe("CurrentTask", () => {
8980
),
9081
};
9182

92-
rerender(
93-
<BlockContext.Provider value={cn("block-name")}>
94-
<CurrentTask store={store} />
95-
</BlockContext.Provider>,
96-
);
83+
rerender(<CurrentTask store={store} />);
9784

9885
expect(getByTestId("next-task").disabled).toBe(true);
9986

@@ -107,11 +94,7 @@ describe("CurrentTask", () => {
10794
),
10895
};
10996

110-
rerender(
111-
<BlockContext.Provider value={cn("block-name")}>
112-
<CurrentTask store={store} />
113-
</BlockContext.Provider>,
114-
);
97+
rerender(<CurrentTask store={store} />);
11598

11699
expect(getByTestId("next-task").disabled).toBe(true);
117100

@@ -125,11 +108,7 @@ describe("CurrentTask", () => {
125108
),
126109
};
127110

128-
rerender(
129-
<BlockContext.Provider value={cn("block-name")}>
130-
<CurrentTask store={store} />
131-
</BlockContext.Provider>,
132-
);
111+
rerender(<CurrentTask store={store} />);
133112

134113
expect(getByTestId("next-task").disabled).toBe(true);
135114
});

0 commit comments

Comments
 (0)