From 2c0d968f411ef39f37a435cd8a42b6fc81432bba Mon Sep 17 00:00:00 2001 From: Alex Rudenko Date: Mon, 23 Oct 2023 10:02:56 +0200 Subject: [PATCH] test: fix lazy frame tests and expectations (#11218) --- test/TestExpectations.json | 130 ++++++++++++++---------------- test/assets/lazy-oopif-frame.html | 2 +- 2 files changed, 63 insertions(+), 69 deletions(-) diff --git a/test/TestExpectations.json b/test/TestExpectations.json index b06287b7..be4a8abc 100644 --- a/test/TestExpectations.json +++ b/test/TestExpectations.json @@ -905,12 +905,6 @@ "parameters": ["cdp", "firefox"], "expectations": ["SKIP"] }, - { - "testIdPattern": "[oopif.spec] OOPIF-debug OOPIF should support lazy OOP frames", - "platforms": ["darwin", "linux", "win32"], - "parameters": ["chrome"], - "expectations": ["FAIL", "PASS"] - }, { "testIdPattern": "[page.spec] Page BrowserContext.overridePermissions should be prompt by default", "platforms": ["darwin", "linux", "win32"], @@ -2831,12 +2825,30 @@ "parameters": ["chrome", "webDriverBiDi"], "expectations": ["PASS"] }, + { + "testIdPattern": "[oopif.spec] OOPIF clickablePoint, boundingBox, boxModel should work for elements inside OOPIFs", + "platforms": ["darwin", "linux", "win32"], + "parameters": ["chrome", "webDriverBiDi"], + "expectations": ["PASS"] + }, { "testIdPattern": "[oopif.spec] OOPIF should provide access to elements", "platforms": ["darwin", "linux", "win32"], "parameters": ["chrome", "webDriverBiDi"], "expectations": ["PASS"] }, + { + "testIdPattern": "[oopif.spec] OOPIF should provide access to elements", + "platforms": ["darwin", "linux", "win32"], + "parameters": ["chrome", "webDriverBiDi"], + "expectations": ["PASS"] + }, + { + "testIdPattern": "[oopif.spec] OOPIF should support evaluating in oop iframes", + "platforms": ["darwin", "linux", "win32"], + "parameters": ["chrome", "webDriverBiDi"], + "expectations": ["PASS"] + }, { "testIdPattern": "[oopif.spec] OOPIF should support evaluating in oop iframes", "platforms": ["darwin", "linux", "win32"], @@ -2849,12 +2861,42 @@ "parameters": ["chrome", "webDriverBiDi"], "expectations": ["PASS"] }, + { + "testIdPattern": "[oopif.spec] OOPIF should support frames within OOP frames", + "platforms": ["darwin", "linux", "win32"], + "parameters": ["chrome", "webDriverBiDi"], + "expectations": ["PASS"] + }, { "testIdPattern": "[oopif.spec] OOPIF should support frames within OOP iframes", "platforms": ["darwin", "linux", "win32"], "parameters": ["chrome", "webDriverBiDi"], "expectations": ["PASS"] }, + { + "testIdPattern": "[oopif.spec] OOPIF should support frames within OOP iframes", + "platforms": ["darwin", "linux", "win32"], + "parameters": ["chrome", "webDriverBiDi"], + "expectations": ["PASS"] + }, + { + "testIdPattern": "[oopif.spec] OOPIF should support lazy OOP frames", + "platforms": ["darwin", "linux", "win32"], + "parameters": ["firefox", "webDriverBiDi"], + "expectations": ["FAIL", "PASS"] + }, + { + "testIdPattern": "[oopif.spec] OOPIF should support wait for navigation for transitions from local to OOPIF", + "platforms": ["darwin", "linux", "win32"], + "parameters": ["cdp", "chrome"], + "expectations": ["PASS", "TIMEOUT"] + }, + { + "testIdPattern": "[oopif.spec] OOPIF should track navigations within OOP iframes", + "platforms": ["darwin", "linux", "win32"], + "parameters": ["chrome", "webDriverBiDi"], + "expectations": ["PASS"] + }, { "testIdPattern": "[oopif.spec] OOPIF should track navigations within OOP iframes", "platforms": ["darwin", "linux", "win32"], @@ -2868,55 +2910,13 @@ "expectations": ["PASS"] }, { - "testIdPattern": "[oopif.spec] OOPIF-debug OOPIF clickablePoint, boundingBox, boxModel should work for elements inside OOPIFs", + "testIdPattern": "[oopif.spec] OOPIF should treat OOP iframes and normal iframes the same", "platforms": ["darwin", "linux", "win32"], "parameters": ["chrome", "webDriverBiDi"], "expectations": ["PASS"] }, { - "testIdPattern": "[oopif.spec] OOPIF-debug OOPIF should provide access to elements", - "platforms": ["darwin", "linux", "win32"], - "parameters": ["chrome", "webDriverBiDi"], - "expectations": ["PASS"] - }, - { - "testIdPattern": "[oopif.spec] OOPIF-debug OOPIF should support evaluating in oop iframes", - "platforms": ["darwin", "linux", "win32"], - "parameters": ["chrome", "webDriverBiDi"], - "expectations": ["PASS"] - }, - { - "testIdPattern": "[oopif.spec] OOPIF-debug OOPIF should support frames within OOP frames", - "platforms": ["darwin", "linux", "win32"], - "parameters": ["chrome", "webDriverBiDi"], - "expectations": ["PASS"] - }, - { - "testIdPattern": "[oopif.spec] OOPIF-debug OOPIF should support frames within OOP iframes", - "platforms": ["darwin", "linux", "win32"], - "parameters": ["chrome", "webDriverBiDi"], - "expectations": ["PASS"] - }, - { - "testIdPattern": "[oopif.spec] OOPIF-debug OOPIF should support wait for navigation for transitions from local to OOPIF", - "platforms": ["darwin", "linux", "win32"], - "parameters": ["cdp", "chrome"], - "expectations": ["PASS", "TIMEOUT"] - }, - { - "testIdPattern": "[oopif.spec] OOPIF-debug OOPIF should track navigations within OOP iframes", - "platforms": ["darwin", "linux", "win32"], - "parameters": ["chrome", "webDriverBiDi"], - "expectations": ["PASS"] - }, - { - "testIdPattern": "[oopif.spec] OOPIF-debug OOPIF should treat OOP iframes and normal iframes the same", - "platforms": ["darwin", "linux", "win32"], - "parameters": ["chrome", "webDriverBiDi"], - "expectations": ["PASS"] - }, - { - "testIdPattern": "[oopif.spec] OOPIF-debug OOPIF waitForFrame should resolve immediately if the frame already exists", + "testIdPattern": "[oopif.spec] OOPIF waitForFrame should resolve immediately if the frame already exists", "platforms": ["darwin", "linux", "win32"], "parameters": ["chrome", "webDriverBiDi"], "expectations": ["PASS"] @@ -3353,12 +3353,24 @@ "parameters": ["firefox", "headful"], "expectations": ["PASS", "TIMEOUT"] }, + { + "testIdPattern": "[prerender.spec] Prerender can navigate to a prerendered page via Puppeteer", + "platforms": ["darwin", "linux", "win32"], + "parameters": ["firefox", "webDriverBiDi"], + "expectations": ["PASS"] + }, { "testIdPattern": "[prerender.spec] Prerender can screencast", "platforms": ["darwin", "linux", "win32"], "parameters": ["chrome", "webDriverBiDi"], "expectations": ["PASS"] }, + { + "testIdPattern": "[prerender.spec] Prerender via frame can navigate to a prerendered page via Puppeteer", + "platforms": ["darwin", "linux", "win32"], + "parameters": ["firefox", "webDriverBiDi"], + "expectations": ["PASS"] + }, { "testIdPattern": "[proxy.spec] request proxy in incognito browser context should proxy requests when configured at browser level", "platforms": ["darwin", "linux", "win32"], @@ -3863,24 +3875,6 @@ "parameters": ["cdp", "chrome", "headful"], "expectations": ["FAIL", "PASS"] }, - { - "testIdPattern": "[oopif.spec] OOPIF-debug OOPIF should support lazy OOP frames", - "platforms": ["darwin", "linux", "win32"], - "parameters": ["firefox", "headless", "webDriverBiDi"], - "expectations": ["PASS"] - }, - { - "testIdPattern": "[prerender.spec] Prerender can navigate to a prerendered page via Puppeteer", - "platforms": ["darwin", "linux", "win32"], - "parameters": ["firefox", "headless", "webDriverBiDi"], - "expectations": ["PASS"] - }, - { - "testIdPattern": "[prerender.spec] Prerender via frame can navigate to a prerendered page via Puppeteer", - "platforms": ["darwin", "linux", "win32"], - "parameters": ["firefox", "headless", "webDriverBiDi"], - "expectations": ["PASS"] - }, { "testIdPattern": "[requestinterception.spec] request interception Page.setRequestInterception should be abortable", "platforms": ["darwin", "linux", "win32"], diff --git a/test/assets/lazy-oopif-frame.html b/test/assets/lazy-oopif-frame.html index 7c259b66..83a420d0 100644 --- a/test/assets/lazy-oopif-frame.html +++ b/test/assets/lazy-oopif-frame.html @@ -1,3 +1,3 @@
- \ No newline at end of file +