retryTo leads to stale process on failure

[cjhille]

I'm sorry @kobenguyent, but we must have missed a regression with the last fix here #4110

It works fine as long as assertions are a success, but as soon as there is a failure – even outside the retryTo block – the test never terminates.

Simple test code:

Feature('foo');

Scenario('test issue', async ({ I }) => {
    I.amOnPage('http://example.org')
    I.waitForVisible('.nothing', 1); // should fail here but it won't terminate
    await retryTo( (tryNum) => {
        I.see(".doesNotMatter");
    }, 10);
});

EDIT: Adding an irregular await before the failing assertion await I.waitForVisible('.nothing', 1) will let the promise chain resolve.

[github-actions]

This issue is stale because it has been open for 90 days with no activity.

[cjhille]

The issue still persists (see test case). Unfortunately, retryTo can not be used currently. FYI @DavertMik