feat(ChromeTestConductor): execute tests in hidden documents

ph-fritsche · ph-fritsche · commit 669bb61c9020 · 2025-01-14T19:32:49.000Z
diff --git a/src/conductor/ChromeTestConductor.ts b/src/conductor/ChromeTestConductor.ts
@@ -22,7 +22,10 @@ export class ChromeTestConductor extends TestConductor {
         }),
     ) {
         super(title, setupFiles, coverageVar)
+
+        this.defaultPage = browser.then(b => b.newPage())
     }
+    protected defaultPage
 
     readonly reporterServer = new HttpReporterServer()
 
@@ -92,6 +95,8 @@ await new TestRunner(
 )
         `
 
+        await (await this.defaultPage).bringToFront()
+
         await page.setContent(`<html><head><script type="module">${childCode}</script>`)
 
         await donePromise
diff --git a/test/conductor/ChromeTestConductor.ts b/test/conductor/ChromeTestConductor.ts
@@ -47,7 +47,7 @@ test('conduct test', async () => {
     expect(getTestFunction(suite, 2)).toHaveProperty('title', 'failing test')
     expect(getTestFunction(suite, 2).result.get()).toHaveProperty('type', 'fail')
 
-    expect((await (await conductor.browser).pages()).length).toBe(0)
+    expect((await (await conductor.browser).pages()).length).toBe(1)
 }, generousTimeout)
 
 test('abort test', async () => {
@@ -73,3 +73,19 @@ test('abort test', async () => {
     expect(getTestFunction(suite, 1).result.get()).toHaveProperty('type', 'success')
     expect(getTestFunction(suite, 2).result.get()).toBe(undefined)
 }, generousTimeout)
+
+test('document is hidden', async () => {
+    const {conductor, fileServer} = await setupConductor()
+    fileServer.provider.files.set('checkVisibility.js', Promise.resolve({content: `
+        if (document.visibilityState !== 'hidden') {
+            throw new Error('document.visibilityState is: ' + document.visibilityState)
+        }
+    `}))
+    const suiteUrl = String(await fileServer.url) + 'checkVisibility.js'
+    const run = createTestRun([conductor], [{url: suiteUrl, title: 'visibilityState'}])
+    const suite = run.runs.get(conductor)!.suites.get(suiteUrl)!
+
+    await suite.exec().catch(() => void 0)
+
+    expect(suite.errors[Symbol.iterator]().next().value?.message).toBe(undefined)
+})

Original file line number	Diff line number	Diff line change
`@@ -22,7 +22,10 @@ export class ChromeTestConductor extends TestConductor {`
`22`	`22`	`}),`
`23`	`23`	`) {`
`24`	`24`	`super(title, setupFiles, coverageVar)`
	`25`	`+`
	`26`	`+ this.defaultPage = browser.then(b => b.newPage())`
`25`	`27`	`}`
	`28`	`+ protected defaultPage`
`26`	`29`
`27`	`30`	`readonly reporterServer = new HttpReporterServer()`
`28`	`31`
`@@ -92,6 +95,8 @@ await new TestRunner(`
`92`	`95`	`)`
`93`	`96`	`
`94`	`97`
	`98`	`+ await (await this.defaultPage).bringToFront()`
	`99`	`+`
`95`	`100`	await page.setContent(`<html><head><script type="module">${childCode}</script>`)
`96`	`101`
`97`	`102`	`await donePromise`