test: Fix mediaView test, ,member is not optional anymore

Updated the test because now name will be the userId instead of default display name
This commit is contained in:
Valere
2025-10-08 08:50:35 +02:00
parent 05e7b5a7ff
commit 13fb46644c

View File

@@ -5,7 +5,7 @@ SPDX-License-Identifier: AGPL-3.0-only OR LicenseRef-Element-Commercial
Please see LICENSE in the repository root for full details. Please see LICENSE in the repository root for full details.
*/ */
import { describe, expect, it, test } from "vitest"; import { describe, expect, it, test, vi } from "vitest";
import { render, screen } from "@testing-library/react"; import { render, screen } from "@testing-library/react";
import { axe } from "vitest-axe"; import { axe } from "vitest-axe";
import { TooltipProvider } from "@vector-im/compound-web"; import { TooltipProvider } from "@vector-im/compound-web";
@@ -16,6 +16,7 @@ import {
import { LocalTrackPublication, Track } from "livekit-client"; import { LocalTrackPublication, Track } from "livekit-client";
import { TrackInfo } from "@livekit/protocol"; import { TrackInfo } from "@livekit/protocol";
import { type ComponentProps } from "react"; import { type ComponentProps } from "react";
import { type RoomMember } from "matrix-js-sdk";
import { MediaView } from "./MediaView"; import { MediaView } from "./MediaView";
import { EncryptionStatus } from "../state/MediaViewModel"; import { EncryptionStatus } from "../state/MediaViewModel";
@@ -45,7 +46,11 @@ describe("MediaView", () => {
mirror: false, mirror: false,
unencryptedWarning: false, unencryptedWarning: false,
video: trackReference, video: trackReference,
member: undefined, member: vi.mocked<RoomMember>({
name: () => "some name",
userId: "@alice:example.com",
getMxcAvatarUrl: vi.fn().mockReturnValue(undefined),
}),
localParticipant: false, localParticipant: false,
focusable: true, focusable: true,
}; };
@@ -59,9 +64,9 @@ describe("MediaView", () => {
test("neither video nor avatar are shown", () => { test("neither video nor avatar are shown", () => {
render(<MediaView {...baseProps} video={trackReferencePlaceholder} />); render(<MediaView {...baseProps} video={trackReferencePlaceholder} />);
expect(screen.queryByTestId("video")).toBeNull(); expect(screen.queryByTestId("video")).toBeNull();
expect(screen.queryAllByRole("img", { name: "some name" }).length).toBe( expect(
0, screen.queryAllByRole("img", { name: "@alice:example.com" }).length,
); ).toBe(0);
}); });
}); });
@@ -70,14 +75,18 @@ describe("MediaView", () => {
render( render(
<MediaView {...baseProps} video={undefined} localParticipant={true} />, <MediaView {...baseProps} video={undefined} localParticipant={true} />,
); );
expect(screen.getByRole("img", { name: "some name" })).toBeVisible(); expect(
screen.getByRole("img", { name: "@alice:example.com" }),
).toBeVisible();
expect(screen.queryAllByText("Waiting for media...").length).toBe(0); expect(screen.queryAllByText("Waiting for media...").length).toBe(0);
}); });
it("shows avatar and label for remote user", () => { it("shows avatar and label for remote user", () => {
render( render(
<MediaView {...baseProps} video={undefined} localParticipant={false} />, <MediaView {...baseProps} video={undefined} localParticipant={false} />,
); );
expect(screen.getByRole("img", { name: "some name" })).toBeVisible(); expect(
screen.getByRole("img", { name: "@alice:example.com" }),
).toBeVisible();
expect(screen.getByText("Waiting for media...")).toBeVisible(); expect(screen.getByText("Waiting for media...")).toBeVisible();
}); });
}); });
@@ -131,7 +140,9 @@ describe("MediaView", () => {
<MediaView {...baseProps} videoEnabled={false} /> <MediaView {...baseProps} videoEnabled={false} />
</TooltipProvider>, </TooltipProvider>,
); );
expect(screen.getByRole("img", { name: "some name" })).toBeVisible(); expect(
screen.getByRole("img", { name: "@alice:example.com" }),
).toBeVisible();
expect(screen.getByTestId("video")).not.toBeVisible(); expect(screen.getByTestId("video")).not.toBeVisible();
}); });
}); });