Skip to content

Commit a99835a

Browse files
committed
test: remove unnecessary BlockContext wrapper from BottomBar 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, 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 Completes test cleanup for BEM migration.
1 parent 188e356 commit a99835a

File tree

1 file changed

+5
-26
lines changed

1 file changed

+5
-26
lines changed

web/libs/editor/src/components/BottomBar/__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)