Skip to content

fix: support for pytest-retries#836

Merged
Pierre-Sassoulas merged 3 commits into
pytest-dev:masterfrom
acolombier:fix/support-for-pytest-retries
Dec 2, 2025
Merged

fix: support for pytest-retries#836
Pierre-Sassoulas merged 3 commits into
pytest-dev:masterfrom
acolombier:fix/support-for-pytest-retries

Conversation

@acolombier
Copy link
Copy Markdown
Contributor

Fixes #815

@BeyondEvil
Copy link
Copy Markdown
Contributor

Thanks for the PR.

Please rebase with master.

@acolombier acolombier force-pushed the fix/support-for-pytest-retries branch from 5a8bedd to d9c4736 Compare January 31, 2025 14:11
@disha-harish-prabhu
Copy link
Copy Markdown

We are facing this issue in our test suites as well. Any ETA on merge?

Copy link
Copy Markdown
Member

@Pierre-Sassoulas Pierre-Sassoulas left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I can merge in main but the release will need some coordination. In the meantime please use a pip install from a commit directly using git + ssh.

@Pierre-Sassoulas Pierre-Sassoulas merged commit e243582 into pytest-dev:master Dec 2, 2025
34 of 35 checks passed
iisenfrank added a commit to iisenfrank/pytest-html that referenced this pull request Apr 17, 2026
…for comma logic

The comma condition in the outcomes loop was hardcoded to check
`result != "rerun"`, which breaks since pytest-dev#836 added "retried" after
"rerun" in the outcomes dict. Using Jinja2's `loop.last` is the
correct, order-independent approach.

Ref: pytest-dev#836
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.

Crashes with KeyError: 'retried' when test is retried with pytest-retry

4 participants