Open Bug 1669435 Opened 4 years ago Updated 2 months ago

Also run Puppeteer unit tests in headful mode in CI

Categories

(Remote Protocol :: Agent, task, P3)

task

Tracking

(Not tracked)

People

(Reporter: whimboo, Unassigned)

References

(Depends on 1 open bug)

Details

(Whiteboard: [webdriver:backlog])

By default we run all the tests in non-headless mode in CI. But pup jobs are only run in headless mode. So should add a job that runs in non-headless.

Preferable we should wait until we run by changeset on trunk as getting added by bug 1628589.

Blocks: 1713030
No longer blocks: 1713030
Blocks: 1735509
Depends on: 1797744
Summary: Also run Puppeteer unit tests in non-headless mode in CI → Also run Puppeteer unit tests in headful mode in CI

When we want to announce Puppeteer compatibility with Firefox for WebDriver BiDi we probably should have the tests also running in headful mode. Maybe something we can target for early M9 if needed?

Whiteboard: [puppeteer-beta2-mvp] → [webdriver:triage]

Discussed in triage, let's move it to the backlog

Whiteboard: [webdriver:triage] → [webdriver:backlog]

(In reply to Julian Descottes [:jdescottes] from comment #2)

Discussed in triage, let's move it to the backlog

What exactly has been discussed during triage? Shall we add this job to reach the Puppeteer compatibility or aren't we interested in headful tests right now and keep it as P3? That's not clear to me. Julian, maybe you can clarify? Thanks.

Flags: needinfo?(jdescottes)

(In reply to Henrik Skupin [:whimboo][⌚️UTC+2] from comment #3)

(In reply to Julian Descottes [:jdescottes] from comment #2)

Discussed in triage, let's move it to the backlog

What exactly has been discussed during triage? Shall we add this job to reach the Puppeteer compatibility or aren't we interested in headful tests right now and keep it as P3? That's not clear to me. Julian, maybe you can clarify? Thanks.

It's backlog P3 S3, so it's nice to have but not a priority.
The reasoning is that it's more important to work on actual features or bug fixes which enable Puppeteer to work with BiDi.

Flags: needinfo?(jdescottes)
Duplicate of this bug: 1854166

Joel, given that you work a lot with CI configuration do you have an idea how hard it would be to get a source test running in headful mode? Do we already have some other source tests jobs that do that? Or would our Puppeteer job be the first consumer of such a mode? Thanks.

Flags: needinfo?(jmaher)

the puppeteer jobs would be the first, but the docker image is the same for linux and the windows vm is the same on windows. source test is just a pool configured differently (disk, memory, etc.)

Flags: needinfo?(jmaher)
Depends on: 1917216
You need to log in before you can comment on or make changes to this bug.