test: Onda 7 — cover app components (nodes, Canvas, App); gate to 92/83/91/94 - #32
Open
RVerdiF wants to merge 11 commits into
Open
test: Onda 7 — cover app components (nodes, Canvas, App); gate to 92/83/91/94#32RVerdiF wants to merge 11 commits into
RVerdiF wants to merge 11 commits into
Conversation
Boots the packaged Electron app under Xvfb (ubuntu runner) and runs e2e/app.spec.ts, e2e/canvas.spec.ts and e2e/portal.spec.ts. The specs launch .vite/build/main.js, so the job builds with electron-forge package first; playwright install-deps provides the system libs (no browser download — Electron ships via npm).
GitHub Actions runners run as root in a container, where Chromium's SUID sandbox helper is unavailable and the app never opens a window. Pass --no-sandbox only when process.env.CI is set; local runs are unchanged.
Coverage (suite 553 tests / 42 files, all green): - icons.tsx 37% -> 100% (39 tests) - Panel.tsx 45% -> 93%, Composer.tsx 63% -> 99%, TerminalPalette.tsx 70% -> 100% - presetStore.ts 64% -> 100%, skillInstall.ts 55% -> 100% - fsService.ts 80% -> 100%, gitService.ts 65% -> 99% (lines/funcs 100%) e2e CI: shared launchApp() helper adds --no-sandbox --disable-gpu --disable-dev-shm-usage on CI runners (root container); ELECTRON_ENABLE_LOGGING surfaces app stderr for diagnosable failures. Local launches unchanged.
The runner log only showed a silent firstWindow timeout; the Electron child's own output was swallowed. Attach listeners on launch so the next run reveals the actual boot failure.
…holds to 92/83/91/94
RVerdiF
added
the
tests
This was referenced Aug 14, 2026
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
1 participant
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.
Onda 7 — cobertura do app renderer (alvo 60%+ estourado)
Resultado
O que entrou (só testes + ratchet, produção intocada)
Validação
Findings (sem fix — registrados para PR futuro)