Iterate

Signed-off-by: Michael Telatynski <7t3chguy@gmail.com>

test/components/views/elements/SearchWarning-test.tsx

@@ -32,10 +32,10 @@ describe("<SearchWarning />", () => {
         });
 
         it("renders with a logo by default", () => {
-            const { asFragment, queryByRole } = render(
+            const { asFragment, getByRole } = render(
                 <SearchWarning isRoomEncrypted={true} kind={WarningKind.Search} />,
             );
-            expect(queryByRole("img")).toBeInTheDocument();
+            expect(getByRole("presentation")).toHaveAttribute("src", "https://logo");
             expect(asFragment()).toMatchSnapshot();
         });
 

test/components/views/right_panel/PinnedMessagesCard-test.tsx

@@ -87,8 +87,7 @@ describe("<PinnedMessagesCard />", () => {
     };
 
     const mountPins = async (room: Room): Promise<RenderResult> => {
-        let pins!: RenderResult;
-        pins = render(
+        const pins = render(
             <MatrixClientContext.Provider value={cli}>
                 <PinnedMessagesCard
                     room={room}

test/components/views/rooms/MemberList-test.tsx

@@ -16,7 +16,7 @@ limitations under the License.
 */
 
 import React from "react";
-import { act, fireEvent, render, RenderResult, screen, waitFor } from "@testing-library/react";
+import { act, fireEvent, render, RenderResult, screen } from "@testing-library/react";
 import { Room, MatrixClient, RoomState, RoomMember, User, MatrixEvent } from "matrix-js-sdk/src/matrix";
 import { KnownMembership } from "matrix-js-sdk/src/types";
 import { mocked, MockedObject } from "jest-mock";

test/hooks/useUserOnboardingTasks-test.tsx

@@ -80,9 +80,11 @@ describe("useUserOnboardingTasks", () => {
         });
         const { result, rerender } = renderHook(() => useUserOnboardingTasks(context.result.current));
         expect(result.current[4].id).toBe("permission-notifications");
-        await waitFor(() => expect(result.current[4].completed).toBe(false));
+        expect(result.current[4].completed).toBe(false);
         result.current[4].action!.onClick!({ type: "click" } as any);
-        rerender();
-        await waitFor(() => expect(result.current[4].completed).toBe(true));
+        await waitFor(() => {
+            rerender();
+            expect(result.current[4].completed).toBe(true);
+        });
     });
 });

test/jsdom.ts

@@ -0,0 +1,9 @@
+import TestEnvironment from "@jest/environment-jsdom-abstract";
+
+type Params = TestEnvironment["constructor"]["arguments"];
+
+export default class JSDOMEnv extends TestEnvironment {
+    constructor(config: Params[0], context: Params[1]) {
+        super(config, context, require("jsdom"));
+    }
+}