-
Notifications
You must be signed in to change notification settings - Fork 362
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
Dlaliberte playwright 06 22 #3184
Conversation
* Extend devcontainer docker compose file for playwright Create another docker compose file for playwright. That new one extends the existing docker compose Should be able to run: `npm run pwtests` * adjust gh action * adjust gh action * remove volumes * add timeout * fix volumes
* try running as user * again * try * change volume ownership * tidy up
* set script-shell to fix source not found and call npm run setup There has always been `source: not found` in the previous CI jobs The tests previously worked because npm run deps installs the requirements into the default environment and not the virtual environment. And since we actually did not call `source` correctly, it was okay because the default environment had the packages installed. The problem started to arise because we started to call `source` correctly in some places but not all places. This fixes by setting the shell consistently whenever we call `source` More info: https://morgan.cugerone.com/blog/troubleshooting-source-not-found-when-in-the-context-of-package-json-scripts/ * do each separately * Revert "do each separately" This reverts commit 4fc75f7. * do ci job * consistent to the right datastore mode * verbose * kill the emulator * remove verbose
One of the hacks/workarounds I did early on was to hide the google login, so it didn't get in the way of logging into the testing account, especially on Firefox where the login dialog always popped up when you were not logged in. But we probably want to allow developers to login to their own account as well, so I'll see if I can reenable the google login, now that we got everything else working. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I made a first pass through and have some questions.
Please give a specific PR summary line. |
Oh well. I did not see that you had merged it already. |
Add initial playwright visual tests.