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

feature: playwright e2e tests form electron application #1644

Draft
wants to merge 15 commits into
base: master
Choose a base branch
from

Conversation

vojtech-cerveny
Copy link
Contributor

Same as #1643 but for in-browser-merged branch.

  • basic flow how we can use playwright for basic tests of UI

  • Adding new way how to handle incoming urls (call it from test)
    tidepooluploader://localhost/keycloak-redirect#state....

        await window.evaluate((url) => {
          window.electron.handleIncomingUrl(url);
        }, href);

How to test:
Add these variables (qa2 personal account) into .env

E2E_USER_EMAIL=
E2E_USER_PASSWORD=
  • yarn test:e2e --headed for visible Chrome
  • yarn test:e2e

It is draft, please let me know what do you think about my changes.

@vojtech-cerveny
Copy link
Contributor Author

Currently I am trying to setup circleCI and I am not sure where can be a problem - it seems, that Electron doesn't have access to internet. But I tried to curl to google before running tests and it returns status code 200.

Check video from test.
It stucks on this screen:
image

1f81c21831e5194fb3eadd406437e929.webm

CircleCI build

Base automatically changed from in-browser-merged to master July 29, 2024 13:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants