Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

E2E tests instability: electronApplication.browserWindow: Execution context was destroyed, most likely because of a navigation #4983

Closed
Tracked by #3936
odockal opened this issue Nov 24, 2023 · 5 comments · Fixed by #5524

Comments

@odockal
Copy link
Contributor

odockal commented Nov 24, 2023

Bug description

The patch for #4968 was merged and it introduced probably a regression. E2E main tests started to fail. It is very interesting that Smoke E2E suite is still passing without any issues.

Error differs a bit when run locally, run the nature could be the same.

Playwright test log (this one differs)

Error: electronApplication.browserWindow: Execution context was destroyed, most likely because of a navigation.

Console output:

main ↪️ System ready. Loading extensions...
STDERR: Error creating window Error: ERR_FAILED (-2) loading 'file:///home/odockal/git/podman-desktop/packages/renderer/dist/index.html'
    at rejectAndCleanup (node:electron/js2c/browser_init:2:84559)
    at WebContents.stopLoadingListener (node:electron/js2c/browser_init:2:84946)
    at WebContents.emit (node:events:514:28) {
  errno: -2,
  code: 'ERR_FAILED',
  url: 'file:///home/odockal/git/podman-desktop/packages/renderer/dist/index.html'
}

Operating system

Linux

Installation Method

Other

Version

next (development version)

Steps to reproduce

No response

Relevant log output

main ↪️ System ready. Loading extensions...
STDERR: Error creating window Error: ERR_FAILED (-2) loading 'file:///home/odockal/git/podman-desktop/packages/renderer/dist/index.html'
    at rejectAndCleanup (node:electron/js2c/browser_init:2:84559)
    at WebContents.stopLoadingListener (node:electron/js2c/browser_init:2:84946)
    at WebContents.emit (node:events:514:28) {
  errno: -2,
  code: 'ERR_FAILED',
  url: 'file:///home/odockal/git/podman-desktop/packages/renderer/dist/index.html'
}

Additional context

No response

@odockal
Copy link
Contributor Author

odockal commented Nov 27, 2023

The problem seems to be introduced with remove playwright dep
and eventually partially fixed for e2e smoke tests with add playwright install step in gha file. Now @ScrewTSW is working on the fix for all e2e tests.

@odockal odockal changed the title E2E main test suite: electronApplication.browserWindow: Execution context was destroyed, most likely because of a navigation E2E tests instability: electronApplication.browserWindow: Execution context was destroyed, most likely because of a navigation Nov 28, 2023
@odockal odockal assigned odockal and unassigned ScrewTSW Nov 29, 2023
@odockal
Copy link
Contributor Author

odockal commented Nov 29, 2023

It might be that the fix mentioned above is not directly solving this issue.
Instead we might adjust opening - closing developer tools window in application startup and preferences, decribed in #5031.

@odockal
Copy link
Contributor Author

odockal commented Nov 29, 2023

Outstanding PR could also fix an issue, but it seems that it is not a way how to properly fix the issue.

@odockal
Copy link
Contributor Author

odockal commented Dec 19, 2023

Solved by: #5274

@cbr7
Copy link
Contributor

cbr7 commented Jan 12, 2024

Reopened because issue is still occurring.

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

Successfully merging a pull request may close this issue.

3 participants