feat(tests): add selenium based tests#999
Conversation
Note that this code is LLM generated, and is subject to change/be updated by LLMs. Co-authored-by: opencode
|
@filippomc : got opencode to write some selenium based tests, and ran them against v2dev. All tests pass, apart from those that need the workspaces API, as the API seems to be down. On the main page, the spinner keeps spinning, but no workspace cards are shown. Related bug/feature: if the api is down, perhaps we should show a warning/error to users (although we wouldnt expect this to happen on prod). Since these are only tests, no harm in them being mostly LLM generated---we'll keep them separate from the main code, and update them as necessary. For actual code, we'll make sure each line is properly reviewed if LLMs are used to assist. |
|
Thanks for adding these tests. I’ve been looking through the workspace frontend and noticed the delete-workspace test is failing because I’d be happy to investigate that failure and either propose a focused fix to the test or open a small follow-up PR. Would you prefer changes contributed directly to this branch, or handled separately after this PR is merged? |
Note that this code is LLM generated, and is subject to change/be updated by LLMs.
Co-authored-by: opencode