-
Notifications
You must be signed in to change notification settings - Fork 115
Build Docker image on slim base #3204
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
Conversation
Can we track what tests we did with the slim-base to make sure its not breaking anything.
|
For Playwright end-to-end tests in CI, instead of running a development webserver with `yarn dev`, build and deploy a Docker container for Element Call and use that as the webserver to test against.
There weren't any existing tests that used Docker, so I added 073baca to run the Playwright tests against a freshly-built Docker container of EC. This should ensure that not only does the Docker image build properly, but that running containers can successfully serve EC. |
playwright/fixtures/widget-user.ts
Outdated
"Developer.elementCallUrl", | ||
null, | ||
"device", | ||
"https://localhost:3000/room", |
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.
For some reason, this wasn't able to use a variable, so I've defined two different versions of the function for each value of the URL.
Not sure why the end-to-end tests are hanging in CI, and they run properly for me in an Ubuntu VM... |
@AndrewFerr they are currently broken. |
I noticed "playwright" in some recent commit messages, so here's a merge to try tests again 🙂 EDIT: Still no luck. |
When using a containerized webserver, this stops the container once tests finish.
Now we're getting somewhere. Increasing the test job timeout revealed the errors that the tests are hitting, which do not reproduce when running them in a local Ubuntu VM. The tests are failing on expected page elements not being visible. Any idea what could be causing that? |
This reverts commit 298aced. The point of this was to figure out why the tests were being cancelled.
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.
This reduces the size of the runtime image by about 50%.
depends on #3294