ci: use chrome browser for cypress tests

[mrrajamanickam-coveo]

• Also
  • update to cypress-io/github-action@v5 (which supports Node 18 along with 16)
  • add workaround for hardcoded node.js version in cypress github action made redundant by retry changes
  • update couple of pkgs from cypress 9 to cypress 10
  • unify version of cypress across pkgs in the mono-repo
• Merged changes from ci: add retry in CI for failed cypress test suites #2911 to test them together
  • Adds retry to failed cypress actions and a couple of recommended vscode extensions (KIT-2481)
• Refactor bunch of flaky test suites, primarily by restructuring before setup to before each (KIT-2448)
• No changes to test exec time observed (averaging around ~5min per suite as before with electron)
  • Unless a test section is retried on failure (which could add 2-3x depending on times failed to the test sections that were retried)

TODO

• Fix error in 3 pkgs: Cypress version 10.0.0 no longer supports cypress.json. - possibly surfaced due to switching to system node (v18)

https://coveord.atlassian.net/browse/KIT-2488

[github-actions]

Dependency Review

✅ No vulnerabilities or license issues found.

Scanned Manifest Files

package-lock.json

• node-getopt@0.3.2
• retry-cli@0.7.0

package.json

• retry-cli@0.7.0

packages/atomic-hosted-page/package.json

• cypress@10.8.0
• cypress@9.7.0

packages/samples/headless-react/package.json

• cypress@10.8.0

packages/samples/stencil/package.json

• cypress@10.8.0
• cypress@9.7.0

packages/samples/vuejs/package.json

• cypress@10.8.0
• cypress@9.7.0

[github-actions]

Pull Request Report

PR Title

✅ Title follows the conventional commit spec.

Bundle Size

File	Old (kb)	New (kb)	Change (%)
case-assist	176.7	176.7	0
search	320.9	320.9	0
insight	281.4	281.4	0
product-listing	219.6	219.6	0
product-recommendation	151.7	151.7	0
recommendation	185.5	185.5	0

[mrrajamanickam-coveo]

@olamothe Can you please review the changes since you last reviewed.
The only failure now is from Review / Dependency Review - probably from a change in master

The following dependencies have incompatible licenses:
package-lock.json » axe-core@4.7.0 – License: MPL-2.0
Error: Dependency review detected incompatible licenses.

Looks like the new version is getting pulled as a transitive dep. Should we pin to an older version?

├─┬ @coveo/atomic@2.29.0 -> ./packages/atomic
│ ├── axe-core@4.4.3
│ └─┬ cypress-axe@1.0.0
│ └── axe-core@4.4.3 deduped
└─┬ @coveo/headless-react-samples@0.0.0 -> ./packages/samples/headless-react
└─┬ react-scripts@5.0.1
└─┬ eslint-config-react-app@7.0.1
└─┬ eslint-plugin-jsx-a11y@6.7.1
└── axe-core@4.7.0