-
-
Notifications
You must be signed in to change notification settings - Fork 65
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
Playwright test timing out - flaky #1854
Comments
@tijlleenders the test shows passed because as mentioned in the line 39, one test is flaky means it failed initially but passed on retry. you can refer to https://playwright.dev/docs/test-retries. Tests sometimes fail because the elements related to sharing/collaboration feature doesn't always work as expected. For ex. when user shares a goal, the reciever should see it after refreshing the page, but this doesn't always happen. This unpredictable behavior is probably why the test passes when run again |
main branch is failing after soft delete PR. This is not a good situation. Please find a fix. |
@tijlleenders the test fails when it tries to accept the changes made to the goal during collaboration. I don't know why this is happening. This works fine when I do it myself (manually), but the test isn't passing. I need to look into this thoroughly, it might take some time. |
You mean running the same test in an automated way - so not manually - works on your machine? |
No. What I mean is that if I, as a user, perform the steps mentioned in the test manually, it works correctly. However, it does not work in the test environment. |
Ok, well then the first step is to get them working in an automated fashion on your local machine.
|
I tried everything I could to make the test run correctly (added refreshes, waits, page changes, playwright version updates, etc) but it doesn't seem to work, don't know why. |
Does it work automated locally? |
No it is not working locally also. It passes once in a while but mostly fails. |
Ok, well at least then you have a good setup to iterate possible solutions. What's the flop-rate? Does it succeed 10% or 90% of the time? Just retrying the test twice can be a workaround for now. |
10%. |
@vinaybadgujar102 Can we increase the time the test is allowed to run to two minutes to see if that fixes the flaky tests? |
@tijlleenders, Will give this a shot. |
Closed by #2049 |
Describe the bug
Playwright test is timing out. This is seen as a PASS in CI.
To Reproduce
See GH Action:
The text was updated successfully, but these errors were encountered: