You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
@bahmutov, do you think cypress-pipe should retry all funtions that contain Cypress chainers? I thought about it and decided not to for 2 reasons off the top of my head - retrying side effect things like click which have their own timeouts and the command log getting spammed with cy logs.
There are probably other edge cases as well. I could always try it to see what happens. You could probably experiment by removing the retry restriction.
I definitely see value in retrying everything even things that have side-effects - especially if the user really wants to. Real world problem solved here: cypress-io/cypress#3135 (comment)
Retrying is handled by verifyUpcomingAssertions. I dug around there a bit to see why assertions don't retry from functions that contain Cypress commands. It is quite complex and hard to follow. Do you have any insights. It looks like that code was last touched by Brian over a year ago.
but it only evaluates functions that return cypress chainers once.
I have done my own little code experiment in https://github.com/bahmutov/retry-task-demo (which found a problem right away cypress-io/cypress#3144) but it is working.
The text was updated successfully, but these errors were encountered: