-
-
Notifications
You must be signed in to change notification settings - Fork 22
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
add e2e of examples #697
Comments
Hello, |
@space-techy thanks! I experimented in this draft PR #600 but I have had several problems, the tests pass most of them fine, but there is a bug with open Bun processes. It's been a while since I've worked on this topic. So, it will be very welcome any contribution to it. It would be great if you could lend a hand. Feel free to use the draft PR for reference or start over. The idea would be to cover e2e of the examples with Playwright. The challenge here is that it's not a single project, it's several (each example is a different Brisa project). Maybe my mistake was to create a test file for everything, instead of each example having its own file. |
Hello @aralroca, Thank you for giving me this opportunity! I’m completely new to Brisa Repo and don’t have much knowledge about it yet. If possible, could you provide more information, along with any code pointers or documentation? I would truly appreciate it. I understand this might feel like a bit of an extra ask, but I assure you that I’ll give my best to contribute effectively. Additionally, I’d love to learn more details about the issue at hand. Thank you so much! |
@space-techy take a look at https://github.com/brisa-build/brisa/blob/canary/CONTRIBUTING.md to the deployment environment Then, all the examples are in: https://github.com/brisa-build/brisa/tree/canary/examples These examples are the same that appear here: https://brisa.build/examples In the |
Thank You @aralroca |
I tried it and it is as draft in a PR, but there are things to improve, such as Bun processes that remain open, and improve the GitHub Action...
I create the issue in case someone else wants to lend a hand with this and use the PR draft as a reference (if you want).
Currently, the examples folder does not have e2e tests, and it would be nice to have them to ensure that they do not break during releases.
The text was updated successfully, but these errors were encountered: