From 538764c842a53325b6721df3f3db75de5b35030f Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Alberto=20Fern=C3=A1ndez-Capel?= Date: Fri, 17 Nov 2023 10:04:10 +0000 Subject: [PATCH] Clean up oudated comments & fixtures --- src/tests/fixtures/page_refresh.html | 4 ---- src/tests/functional/page_refresh_tests.js | 4 +--- 2 files changed, 1 insertion(+), 7 deletions(-) diff --git a/src/tests/fixtures/page_refresh.html b/src/tests/fixtures/page_refresh.html index b483ac96d..f43d5adb3 100644 --- a/src/tests/fixtures/page_refresh.html +++ b/src/tests/fixtures/page_refresh.html @@ -40,10 +40,6 @@

Frame to be preserved

-
-

Element with Stimulus controller

-
-

Link to another page

diff --git a/src/tests/functional/page_refresh_tests.js b/src/tests/functional/page_refresh_tests.js index 931cc7b63..2c2c2f683 100644 --- a/src/tests/functional/page_refresh_tests.js +++ b/src/tests/functional/page_refresh_tests.js @@ -40,7 +40,6 @@ test("uses morphing to update remote frames", async ({ page }) => { await nextEventNamed(page, "turbo:render", { renderMethod: "morph" }) await nextBeat() - // Only the frame marked with refresh="morph" uses morphing expect(await nextEventOnTarget(page, "remote-frame", "turbo:before-frame-morph")).toBeTruthy() await expect(page.locator("#remote-frame")).toHaveText("Loaded morphed frame") }) @@ -52,8 +51,7 @@ test("don't refresh frames contained in [data-turbo-permanent] elements", async await nextEventNamed(page, "turbo:render", { renderMethod: "morph" }) await nextBeat() - // Only the frame marked with refresh="morph" uses morphing - expect(await noNextEventOnTarget(page, "refresh-reload", "turbo:before-frame-morph")).toBeTruthy() + expect(await noNextEventOnTarget(page, "remote-permanent-frame", "turbo:before-frame-morph")).toBeTruthy() }) test("remote frames are excluded from full page morphing", async ({ page }) => {