test: remove act in favour of async timers

Signed-off-by: Peter Makowski <me+git@petermakowski.io>
canonical · May 14, 2024 · d037a49 · d037a49
1 parent 407add1
commit d037a49
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 4 deletions.
diff --git a/src/components/ActionButton/ActionButton.test.tsx b/src/components/ActionButton/ActionButton.test.tsx
@@ -1,12 +1,11 @@
-import React from "react";
+import React, { act } from "react";
 import { render, screen } from "@testing-library/react";
 
 import ActionButton, {
   Label,
   LOADER_MIN_DURATION,
   SUCCESS_DURATION,
 } from "./ActionButton";
-import { act } from "react-dom/test-utils";
 
 describe("ActionButton", () => {
   jest.useFakeTimers();

diff --git a/src/components/NotificationProvider/NotificationProvider.test.tsx b/src/components/NotificationProvider/NotificationProvider.test.tsx
@@ -8,7 +8,6 @@ import {
   useNotify,
 } from "./NotificationProvider";
 import Button from "../Button";
-import { act } from "react-dom/test-utils";
 
 describe("NotificationProvider", () => {
   it("stores and renders notifications", async () => {
@@ -54,7 +53,7 @@ describe("NotificationProvider", () => {
     );
 
     const clickBtn = async (testId: string) =>
-      await act(async () => await userEvent.click(screen.getByTestId(testId)));
+      await userEvent.click(screen.getByTestId(testId));
 
     expect(screen.getByTestId("notification-consumer")).toBeEmptyDOMElement();