Skip to content

fix(e2e): move playwright-go to mxschmitt path @v0.6100.0 (dead CDN) - #623

Closed
CybotTM wants to merge 1 commit into
mainfrom
fix/playwright-go-cdn-bump
Closed

fix(e2e): move playwright-go to mxschmitt path @v0.6100.0 (dead CDN)#623
CybotTM wants to merge 1 commit into
mainfrom
fix/playwright-go-cdn-bump

Conversation

@CybotTM

@CybotTM CybotTM commented Jul 19, 2026

Copy link
Copy Markdown
Member

Problem

The CI / E2E Tests job has failed on main for 3 weeks on the unchanged commit. playwright-go v0.5700.1 downloads its driver from the retired Azure/Edgio mirrors, which now 404:

install playwright: could not install driver: got non 200 status code: 404
  https://playwright.azureedge.net/builds/driver/playwright-1.57.0-linux.zip
  https://playwright-akamai.azureedge.net/...
  https://playwright-verizon.azureedge.net/...
FAIL github.com/netresearch/ldap-manager/internal/e2e

Fix

Upstream reverted the canonical module path back to github.com/mxschmitt/playwright-go (both mxschmitt/* and playwright-community/* repos declare module github.com/mxschmitt/playwright-go at the latest release). v0.6100.0 fetches the driver from registry.npmjs.org + nodejs.org/dist instead of the dead CDN.

  • v0.6000.0 (community path) still hardcodes azureedge → does not fix it.
  • The playwright-community v0.6100.0 tag ships a mismatched go.mod module path → not go get-able under that path.

So this switches the import path (11 files, all in internal/e2e/) and pins v0.6100.0.

Verification

  • go build -tags e2e ./... → passes (no breaking API changes).
  • go mod tidy → stable, no stale entries.
  • playwright install chromium on a clean driver cache → exit 0 (driver now downloads; the old version exits 1 with the 404 above).

The E2E job failed on main: playwright-go v0.5700.1 downloads its driver
from the retired Azure/Edgio mirrors (playwright*.azureedge.net), which
now 404 for playwright-1.57.0-linux.zip, aborting 'install playwright:
could not install driver'.

Upstream reverted the canonical module path back to
github.com/mxschmitt/playwright-go (both repos' go.mod declare it at the
latest release), and v0.6100.0 fetches the driver from registry.npmjs.org
+ nodejs.org/dist instead of the dead CDN. v0.6000.0 under the
playwright-community path still hardcodes azureedge, and the
playwright-community v0.6100.0 tag ships a mismatched module path, so the
fix is to switch the import path and pin v0.6100.0.

Verified: go build -tags e2e ./... passes and
'playwright install chromium' succeeds on a clean cache (exit 0).

Signed-off-by: Sebastian Mendel <info@sebastianmendel.de>
Copilot AI review requested due to automatic review settings July 19, 2026 05:33
@gemini-code-assist

Copy link
Copy Markdown

Warning

You have reached your daily quota limit. Please wait up to 24 hours and I will start processing your requests again!

@github-actions github-actions Bot added dependencies Pull requests that update a dependency file tests labels Jul 19, 2026
@github-actions

Copy link
Copy Markdown
Contributor

Dependency Review

✅ No vulnerabilities or license issues or OpenSSF Scorecard issues found.

OpenSSF Scorecard

PackageVersionScoreDetails
gomod/github.com/mxschmitt/playwright-go 0.6100.0 🟢 5.2
Details
CheckScoreReason
Maintained🟢 1030 commit(s) and 23 issue activity found in the last 90 days -- score normalized to 10
Code-Review⚠️ 1Found 5/30 approved changesets -- score normalized to 1
Packaging⚠️ -1packaging workflow not detected
Dangerous-Workflow🟢 10no dangerous workflow patterns detected
Token-Permissions⚠️ 0detected GitHub workflow tokens with excessive permissions
Binary-Artifacts🟢 9binaries present in source code
CII-Best-Practices⚠️ 0no effort to earn an OpenSSF best practices badge detected
Fuzzing⚠️ 0project is not fuzzed
Branch-Protection⚠️ -1internal error: error during branchesHandler.setup: internal error: some github tokens can't read classic branch protection rules: https://github.com/ossf/scorecard-action/blob/main/docs/authentication/fine-grained-auth-token.md
Signed-Releases⚠️ -1no releases found
License🟢 10license file detected
Pinned-Dependencies⚠️ 2dependency not pinned by hash detected -- score normalized to 2
Security-Policy🟢 10security policy file detected
SAST⚠️ 0SAST tool is not run on all commits -- score normalized to 0

Scanned Files

  • go.mod

@github-actions github-actions Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Automated approval for maintainer PR

All automated quality gates passed. See SECURITY_CONTROLS.md for compensating controls.

@sonarqubecloud

Copy link
Copy Markdown

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@github-actions

Copy link
Copy Markdown
Contributor

⚠️ Mutation Testing Results

Mutation Score: 0% (threshold: 60%)

⚠️ Score is below threshold. Consider improving test coverage or test quality.

What is mutation testing?

Mutation testing measures test quality by introducing small changes (mutations) to the code and checking if tests detect them. A higher score means better test effectiveness.

  • Killed mutants: Tests caught the mutation (good!)
  • Survived mutants: Tests missed the mutation (needs improvement)

@codecov

codecov Bot commented Jul 19, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 69.94%. Comparing base (e1d0e09) to head (344b9d7).

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #623   +/-   ##
=======================================
  Coverage   69.94%   69.94%           
=======================================
  Files          34       34           
  Lines        3437     3437           
=======================================
  Hits         2404     2404           
  Misses        873      873           
  Partials      160      160           
Flag Coverage Δ
e2e 58.99% <ø> (ø)
unittests 70.40% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@CybotTM
CybotTM enabled auto-merge July 19, 2026 06:41
@CybotTM

CybotTM commented Jul 19, 2026

Copy link
Copy Markdown
Member Author

Duplicate of #621 (same playwright-go module-rename fix, opened 2026-07-15). I opened this without checking for the existing PR — closing in favor of #621. Both are blocked on the repo's merge-queue + code_scanning ruleset deadlock (a required CodeQL result + a merge_group-only container check), which needs a repo-config fix before either can merge.

@CybotTM CybotTM closed this Jul 19, 2026
auto-merge was automatically disabled July 19, 2026 10:27

Pull request was closed

@CybotTM
CybotTM deleted the fix/playwright-go-cdn-bump branch August 26, 2026 06:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dependencies Pull requests that update a dependency file tests

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants