fix: E2E tests timing out on CI due to uncached nix packages and missing Garnix wait - #87
Merged
Merged
Conversation
Co-authored-by: gfauredev <19304085+gfauredev@users.noreply.github.com>
gfauredev
marked this pull request as ready for review
March 15, 2026 19:12
📊 Coverage Report
|
🚀 Lighthouse Audit |
❌ Maestro Web E2E Failures📜 Maestro Console Output (Last 100 lines)📸 Screenshots |
…ro web tests Co-authored-by: gfauredev <19304085+gfauredev@users.noreply.github.com>
Copilot
AI
changed the title
[WIP] Add Maestro Web and Android E2E tests for user stories
Add Maestro Web & Android E2E tests per user story; fix Chrome/ChromeDriver version mismatch in CI
Mar 15, 2026
📊 Coverage Report
|
🚀 Lighthouse Audit |
❌ Maestro Web E2E Failures📜 Maestro Console Output (Last 100 lines)📸 Screenshots |
…house CI to pre-start server Co-authored-by: gfauredev <19304085+gfauredev@users.noreply.github.com>
Copilot
AI
changed the title
Add Maestro Web & Android E2E tests per user story; fix Chrome/ChromeDriver version mismatch in CI
Fix Maestro web E2E tests and Lighthouse CI audit
Mar 15, 2026
📊 Coverage Report
|
❌ Maestro Web E2E Failures📜 Maestro Console Output (Last 100 lines)📸 Screenshots |
🚀 Lighthouse Audit
Full reports: |
…mium (not system Chrome 145) Co-authored-by: gfauredev <19304085+gfauredev@users.noreply.github.com>
📊 Coverage Report
|
🚀 Lighthouse Audit |
❌ Maestro Web E2E Failures📜 Maestro Console Output (Last 100 lines)📸 Screenshots |
Co-authored-by: gfauredev <19304085+gfauredev@users.noreply.github.com>
…for temp dir cleanup Co-authored-by: gfauredev <19304085+gfauredev@users.noreply.github.com>
🚀 Lighthouse Audit
Full reports: |
Copilot
AI
changed the title
Add
fix: writeShellScriptBin for google-chrome wrapper so chromedriver finds it via PATH
Mar 16, 2026
pages and e2e-web Nix apps; fix CI Lighthouse and Maestro jobs
Copilot stopped work on behalf of
gfauredev due to an error
March 16, 2026 17:18
📊 Coverage Report
|
…r finds it via PATH Co-authored-by: gfauredev <19304085+gfauredev@users.noreply.github.com>
🚀 Lighthouse Audit
Full reports: |
gfauredev
force-pushed
the
copilot/add-maestro-e2e-tests-again
branch
from
March 16, 2026 17:22
b4e8e95 to
1f0f10b
Compare
Copilot stopped work on behalf of
gfauredev due to an error
March 16, 2026 17:24
📊 Coverage Report
|
📊 Coverage Report
|
🚀 Lighthouse Audit
Full reports: |
🚀 Lighthouse Audit
Full reports: |
…m reference in pagesE2eTester Co-authored-by: gfauredev <19304085+gfauredev@users.noreply.github.com>
📊 Coverage Report
|
🚀 Lighthouse Audit
Full reports: |
…t to 45min Co-authored-by: gfauredev <19304085+gfauredev@users.noreply.github.com>
Copilot
AI
changed the title
fix: writeShellScriptBin for google-chrome wrapper so chromedriver finds it via PATH
fix: E2E tests timing out on CI due to uncached nix packages and missing Garnix wait
Mar 16, 2026
Copilot stopped work on behalf of
gfauredev due to an error
March 16, 2026 17:56
Copilot stopped work on behalf of
gfauredev due to an error
March 16, 2026 18:25
📊 Coverage Report
|
🚀 Lighthouse Audit
Full reports: |
📊 Coverage Report
|
🚀 Lighthouse Audit
Full reports: |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
E2E CI jobs were hitting the 20-minute timeout because
nix run .#pagesE2eTesthad to download/build large packages (ungoogled-chromium~3.5GB,maestro,chromedriver) without any cache ready — Garnix hadn't finished building them yet when CI started. A secondary issue: thesleep 2server readiness check was a race condition.flake.nixpkgs.ungoogled-chromium→ungoogled-chromiuminsidewith env.pkgs; [...]—pkgs.prefix was incorrect in this scopesleep 2with acurlhealth check — pollshttp://localhost:8080/LogOut/until ready (consistent with lighthouse job); addcurltoruntimeInputssincewriteShellApplicationsets a strict PATH fromruntimeInputsonly.github/workflows/ci.ymle2e-webtimeout-minutes20 → 45 — accommodates downloading large nix packages on cache misslewagon/wait-on-check-action(Ruby, intermittently broken) with aghCLI polling loop — waits up to 20 min for the GarnixpagesE2eTestercheck to succeed before startingnix run, so packages are served fromcache.garnix.iorather than built on the runnercancelled()in addition tofailure()— previously a timeout produced no debug artifacts💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.