Skip to content

fix: prevent retry config accumulation#5606

Open
smalia2001 wants to merge 1 commit into
codeceptjs:4.xfrom
smalia2001:fix/retry-failed-step-accumulation
Open

fix: prevent retry config accumulation#5606
smalia2001 wants to merge 1 commit into
codeceptjs:4.xfrom
smalia2001:fix/retry-failed-step-accumulation

Conversation

@smalia2001
Copy link
Copy Markdown

@smalia2001 smalia2001 commented Jun 5, 2026

Motivation/Description of the PR

  • Description of this PR, which problem it solves
  • Resolves #issueId (if applicable).

Applicable helpers:

  • Playwright
  • Puppeteer
  • WebDriver
  • REST
  • FileHelper
  • Appium

Applicable plugins:

  • aiTrace
  • autoDelay
  • autoLogin
  • customLocator
  • pause
  • coverage
  • heal
  • retryFailedStep
  • screenshot
  • selenoid
  • stepTimeout
  • subtitles

Type of change

  • 🔥 Breaking changes
  • 🚀 New functionality
  • 🐛 Bug fix
  • 🧹 Chore
  • 📋 Documentation changes/updates
  • ♨️ Hot fix
  • 🔨 Markdown files fix - not related to source code
  • 💅 Polish code

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)

fixes #5598 (I hope it doesn't break something else 😄 )

…leanup their own config after each test

- Playwright & Puppeteer: store retry config in this._retryConfig and remove in _after()
- retryFailedStep plugin: avoid duplicate pushes and clean config on event.test.after
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Bug: retryOnFailedStep works only for the first test in the same run

1 participant