Skip to content

fix: update Chrome installation for Puppeteer across platforms#11642

Open
KauanAfonso wants to merge 2 commits into
loopbackio:masterfrom
KauanAfonso:fix/puppeteer-chrome-installation
Open

fix: update Chrome installation for Puppeteer across platforms#11642
KauanAfonso wants to merge 2 commits into
loopbackio:masterfrom
KauanAfonso:fix/puppeteer-chrome-installation

Conversation

@KauanAfonso

@KauanAfonso KauanAfonso commented Jun 22, 2026

Copy link
Copy Markdown

Fix Chrome Installation for Puppeteer Across Platforms

Description

This PR updates the CI workflow to properly install and configure Google Chrome for Puppeteer across all supported operating systems (Linux, macOS, and Windows).

Previously, Chrome was only installed and configured for Linux, causing Puppeteer-based tests to fail on other platforms.

Changes

  • Added Chrome installation steps for:
    • macOS using brew install --cask google-chrome
    • Windows using choco install googlechrome
  • Ensured Chrome is installed on Linux via apt-get
  • Configured PUPPETEER_EXECUTABLE_PATH environment variable for:
    • Linux (/usr/bin/google-chrome)
    • macOS (/Applications/Google Chrome.app/Contents/MacOS/Google Chrome)
    • Windows (C:/Program Files/Google/Chrome/Application/chrome.exe)

Motivation

Puppeteer requires a Chromium/Chrome executable to run browser-based tests. Without proper installation and configuration, tests fail in CI environments.

This change ensures consistent test execution across all supported CI platforms.

Checklist

  • DCO (Developer Certificate of Origin) signed in all commits
  • npm test passes on your machine
  • New tests added or existing tests modified (N/A)
  • Code conforms with the style guide
  • Documentation updated (N/A)

@KauanAfonso KauanAfonso requested a review from raymondfeng as a code owner June 22, 2026 18:48
@dhmlau

dhmlau commented Jun 23, 2026

Copy link
Copy Markdown
Member

@KauanAfonso, thanks for looking into the CI issue and creating a separate PR for the fix.
When the PR is ready to merge, could you please:

  • squash the commit
  • update the prefix of commit message to be fix (or fix(ci)) instead of feat

Thanks.

@KauanAfonso

KauanAfonso commented Jun 23, 2026

Copy link
Copy Markdown
Author

Ok, thanks for the review!
I wil do.

@KauanAfonso KauanAfonso force-pushed the fix/puppeteer-chrome-installation branch from e6e9ea4 to 0ef3fa4 Compare June 23, 2026 13:36
Signed-off-by: kauanAfonso <kauan.afonso@ibm.com>
Signed-off-by: Kauan Afonso <kauanafon3@gmail.com>
@KauanAfonso KauanAfonso force-pushed the fix/puppeteer-chrome-installation branch from f661d5d to e670208 Compare June 23, 2026 14:23
@KauanAfonso KauanAfonso changed the title feat: update Chrome installation for Puppeteer on Linux fix: update Chrome installation for Puppeteer on Linux Jun 23, 2026
@KauanAfonso KauanAfonso changed the title fix: update Chrome installation for Puppeteer on Linux fix: update Chrome installation for Puppeteer across platforms Jun 23, 2026

@dhmlau dhmlau left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@KauanAfonso, thanks for the fix—LGTM from my side. Let’s wait for additional reviews before merging.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants