matrix/matrix-react-sdk
1
0
Fork 0
mirror of https://github.com/matrix-org/matrix-react-sdk.git synced 2025-07-28 15:22:05 +03:00
Code Activity

Import base64 utils directly from js-sdk (#12871)

Browse Source 
* Import base64 utils directly from js-sdk

See comments in code

* Use the authenticated routes (because the service worker said so)

* Revert "Use the authenticated routes (because the service worker said so)"

This reverts commit 835806d253.

* Use the authenticated routes (because the service worker said so)

* Continue fighting Playwright

* Document who is at fault if the import breaks (it's us)

* Update playwright/e2e/timeline/timeline.spec.ts

Co-authored-by: Robin <robin@robin.town>

---------

Co-authored-by: Robin <robin@robin.town>
This commit is contained in:
Travis Ralston
2024-08-06 22:33:13 -06:00
committed by GitHub
parent 8285283cc3
commit 7a4783f907
2 changed files with 18 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

12
playwright/e2e/timeline/timeline.spec.ts
View File

@ -720,11 +720,16 @@ test.describe("Timeline", () => {
).toBeVisible();
});
test("should render url previews", async ({ page, app, room, axe, checkA11y }) => {
test("should render url previews", async ({ page, app, room, axe, checkA11y, context }) => {
axe.disableRules("color-contrast");
await page.route(
"**/_matrix/media/v3/thumbnail/matrix.org/2022-08-16_yaiSVSRIsNFfxDnV?*",
// Element Web uses a Service Worker to rewrite unauthenticated media requests to authenticated ones, but
// the page can't see this happening. We intercept the route at the BrowserContext to ensure we get it
// post-worker, but we can't waitForResponse on that, so the page context is still used there. Because
// the page doesn't see the rewrite, it waits for the unauthenticated route. This is only confusing until
// the js-sdk (and thus the app as a whole) switches to using authenticated endpoints by default, hopefully.
await context.route(
"**/_matrix/client/v1/media/thumbnail/matrix.org/2022-08-16_yaiSVSRIsNFfxDnV?*",
async (route) => {
await route.fulfill({
path: "playwright/sample-files/riot.png",
@ -750,6 +755,7 @@ test.describe("Timeline", () => {
const requestPromises: Promise<any>[] = [
page.waitForResponse("**/_matrix/media/v3/preview_url?url=https%3A%2F%2Fcall.element.io%2F&ts=*"),
// see context.route above for why we listen for the unauthenticated endpoint
page.waitForResponse("**/_matrix/media/v3/thumbnail/matrix.org/2022-08-16_yaiSVSRIsNFfxDnV?*"),
];