fix: global retry

[kobenguyent]

Motivation/Description of the PR

• Resolves retry inside codecept.conf.js not working #3663
• Resolves The global retries config option is not applied #3607

Type of change

• 🐛 Bug fix

Checklist:

• Tests have been added
• Documentation has been added (Run npm run docs)
• Lint checking (Run npm run lint)
• Local tests are passed (Run npm test)

[DavertMik]

Thank you!

[jhalborg]

Thank you!

@DavertMik - when can we expect a patch release?

Just tested this change. For the simple config with just a number of retries, i.e. retry: 2,, it works 🎉 However, the grep functionality does not seem to work still. For example, for a config like this:

retry: [
    {
      Scenario: 1,
      Feature: 1,
    },
    {
      grep: '@flaky',
      Scenario: 4,
      Feature: 3,
    },
  ]

I would expect 1 retry for tests, unless marked with @flaky, in which case more retries would be done. However, this is not the case, and as far as I could see, no tests have been added for the grep functionality in this PR

[ua1wcz]

@kobenguyent Is it support Gherkin?

cc @DavertMik

Cause what I receiving when trying to use it with Gherkin (Added Before to cover Background)

Error processing suite.before event:
TypeError: Cannot read properties of undefined (reading 'retryBefore')