Skip to content

Update URL Preview playwright tests to use new endpoint #32573

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
Half-Shot merged 6 commits into from
Feb 23, 2026
+28 −82
Merged

Update URL Preview playwright tests to use new endpoint #32573

Changes from 1 commit
Commits
Show all changes
6 commits
Select commit Hold shift + click to select a range
5c096c4
Update URL Preview playwright tests to use new endpoint
Half-Shot Feb 19, 2026
ef29986
Drop extra preview test
Half-Shot Feb 19, 2026
b33e55d
Adopt axe, fixup paths, etc for url previews
Half-Shot Feb 19, 2026
46713e3
work dammit
Half-Shot Feb 19, 2026
8febedd
begone unwanted screenshot
Half-Shot Feb 19, 2026
527399f
Make github happy?
t3chguy Feb 19, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
6 changes: 2 additions & 4 deletions playwright/e2e/messages/messages.spec.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -221,8 +221,7 @@ test.describe("Message url previews", () => {
},
});
test("should render a basic preview", { tag: "@screenshot" }, async ({ page, user, app, room }) => {
// TODO: This should be changed to _matrix/client/v1/media/preview_url when the matrix-js-sdk is updated.
await page.route("**/_matrix/media/v3/preview_url**", (route, request) => {
await page.route("**/_matrix/client/v1/media/preview_url**", (route, request) => {
const requestedPage = new URL(request.url()).searchParams.get("url");
expect(requestedPage).toEqual("https://example.org/");
return route.fulfill({
Expand All @@ -238,8 +237,7 @@ test.describe("Message url previews", () => {
});
test("should render a preview with a thumbnail", { tag: "@screenshot" }, async ({ page, user, bot, app, room }) => {
const mxc = (await bot.uploadContent(MEDIA_FILE, { name: "image.png", type: "image/png" })).content_uri;
// TODO: This should be changed to _matrix/client/v1/media/preview_url when the matrix-js-sdk is updated.
await page.route("**/_matrix/media/v3/preview_url**", (route, request) => {
await page.route("**/_matrix/client/v1/media/preview_url**", (route, request) => {
const requestedPage = new URL(request.url()).searchParams.get("url");
expect(requestedPage).toEqual("https://example.org/");
return route.fulfill({
Expand Down
Loading