Metadata
- Source
- FLUID-6383
- Type
- Task
- Priority
- Major
- Status
- Closed
- Resolution
- Fixed
- Assignee
- Alan Harnum
- Reporter
- Alan Harnum
- Created
2019-07-19T10:34:03.951-0400 - Updated
2020-07-29T09:24:55.454-0400 - Versions
- N/A
- Fixed Versions
- N/A
- Component
-
- Testing Infrastructure
Description
"Chrome" and "Headless Chrome" are registered as separate launchers by Testem, and many systems (including the current CI infrastructure) will have both.
Running Headless Chrome in addition to Chrome results in:
- redundant testing
- harder to read test results
- longer CI jobs
- possible intermittent failures due to the lack of a graphical interface (reflected most immediately in the second instance of Chrome tests consistently failing in https://github.com/fluid-project/infusion/pull/956 until Headless Chrome was added to the skip list)
Currently we exclude the following browser launchers in Infusion's tests/testem.js testemOptions block:
{
...
skip: "PhantomJS,Opera,Safari",
...
{{}}}
I would propose we add Headless Chrome and skip it if available. I don't believe we've ever had a discussion about a need to have both available, but I may be wrong.
Comments
-
Alan Harnum commented
2019-07-19T10:43:55.885-0400 I've opened a PR to make this change at https://github.com/fluid-project/infusion/pull/957
-
Tony Atkins [RtF] commented
2019-07-22T07:36:53.230-0400 In previous similar discussions in other projects, @@Antranig Basman and I went the other way, i.e. disabling "headed" Chrome:
I think there is a speed boost to running headless.
-
Tony Atkins [RtF] commented
2019-07-22T07:44:08.539-0400 @@Justin Obara asked whether there were any issues with focus with Headless Chrome, all tests passed for me using a command like:
node node_modules/testem/testem.js ci --file tests/testem.js --launch "Headless Chrome"
-
Justin Obara commented
2019-07-22T08:14:52.551-0400 If we switch to Headless Chrome, maybe we can also run Firefox headless. I was talking to @@Tony Atkins [RtF] about this in the fluid-work channel today and it seemed to work for him.
-
Alan Harnum commented
2019-07-22T12:29:15.140-0400 Thanks @@Tony Atkins [RtF] and @@Justin Obara - I will see if I can get the PR running both in Headless mode. Agree using both (if it doesn't impact the focus of the tests) is ideal for speed purposes.
-
Giovanni Tirloni commented
2020-04-17T15:18:28.608-0400 +1 for running headless Chrome/Firefox if we can make it work. It would simplify CI enormously.
-
Justin Obara commented
2020-04-27T08:08:02.962-0400 @@Giovanni Tirloni I vaguely remember having issues with using headless browsers in GitHub Actions. I think it's related to https://issues.gpii.net/browse/GPII-4064
-
Justin Obara commented
2020-07-29T09:24:51.143-0400 Work on FLUID-6368 switched to only support the headless version of chrome for testem configuration.