Skip to content

test(desktop): fechar a janela principal escolhe a janela pela URL, não pelo título - #115

Merged
acacio90 merged 1 commit into
developfrom
fix/98-teste-fechar-janela-principal
Oct 27, 2026
Merged

acacio90 merged 1 commit into
developfrom
fix/98-teste-fechar-janela-principal

Conversation

@acacio90

Copy link
Copy Markdown
Owner

O que mudou

  • test-ui/detach.spec.ts › "fechar a janela principal encerra o app": a janela principal passa a ser escolhida pela URL (a destacada abre com ?detached=1), não pelo título, e o teste espera a aba chegar na janela destacada antes de fechar a principal.
  • Causa: ao carregar o index.html, a <title>MultTerm</title> sobrescreve o título da janela destacada até a aba ser adotada. Nesse intervalo as duas janelas se chamam "MultTerm" e a destacada vem primeiro em getAllWindows(); o teste fechava a destacada, a aba voltava à principal e o app não encerrava (o timeout de 30 s visto no CI). Num runner lento o intervalo é mais fácil de pegar.
  • Só teste; nenhum código do app mudou.

Testes

  • Sonda dos títulos logo após destacar (5 rodadas): PowerShell — MultTerm | MultTerm → MultTerm | MultTerm → PowerShell — MultTerm | MultTerm, sempre com a destacada primeiro
  • Fechando pelo seletor antigo exatamente nesse intervalo: o app continuou aberto em 3/3 (a falha do CI, reproduzida)
  • test-ui/detach.spec.ts com a correção, 10 repetições: 30/30
  • npm run typecheck: ok
  • npm test: 166/166 passaram
  • npm run test:ui: 90/90 passaram

Closes #98

…ão pelo título

Enquanto carrega o index.html, a janela destacada também se chama MultTerm e vinha primeiro na lista: o teste fechava a destacada, a aba voltava à principal e o app não encerrava.
@acacio90 acacio90 added this to the v0.10.0 milestone Oct 27, 2026
@acacio90 acacio90 added the bug Something isn't working label Oct 27, 2026
@acacio90
acacio90 merged commit eaf50ed into develop Oct 27, 2026
6 checks passed
@acacio90
acacio90 deleted the fix/98-teste-fechar-janela-principal branch October 27, 2026 22:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant