Skip to content

Mark parent step as failed if child step failed with soft check (fix #827)#854

Closed
sharovd wants to merge 3 commits into
allure-framework:masterfrom
sharovd:fix-827-soft-check-nested-step-fail
Closed

Mark parent step as failed if child step failed with soft check (fix #827)#854
sharovd wants to merge 3 commits into
allure-framework:masterfrom
sharovd:fix-827-soft-check-nested-step-fail

Conversation

@sharovd

@sharovd sharovd commented Apr 10, 2025

Copy link
Copy Markdown
Contributor

Context

  • Implementation of feature request Mark high level step as failed if low level step with soft check failed #827
  • Added sync_steps_statuses() method to synchronise the status for the parent step based on the status of the child steps. This method returns True if at least one nested step has failed — allowing the parent step or the test itself to be marked as failed
  • Added test test_pytest_fail_in_nested_step_with_soft_check() to validate the described test scenario
  • Here is an example of allure report after the fix:

Checklist

@sharovd sharovd force-pushed the fix-827-soft-check-nested-step-fail branch from d0a51d7 to 557d037 Compare May 16, 2025 08:47
@sharovd

sharovd commented Jun 15, 2025

Copy link
Copy Markdown
Contributor Author

@delatrie, hi!
Could you please review this PR?
Please let me know if any changes are needed.

@vochicuong18

Copy link
Copy Markdown

Could you please help merge this branch into master? This issue still occurs in allure-pytest version 2.14.3.

@Solo-TT

Solo-TT commented Oct 29, 2025

Copy link
Copy Markdown
微信图片_20251029172320_4149_10 用了大佬的方法,我纯搬运工,解决了我的难题,膜拜! @sharovd 谢谢

@ExplorerOL

ExplorerOL commented Nov 20, 2025

Copy link
Copy Markdown

I really need this functionality too! Maintainers, respond, please!

@Solo-TT

Solo-TT commented Nov 20, 2025

Copy link
Copy Markdown

I really need this functionality too! Maintainers, respond, please!

The author has already submitted it. I wonder where you're from?

@delatrie

Copy link
Copy Markdown
Contributor

Hello, everyone! Sorry, I didn't respond more quickly. Unfortunately, I have to close this PR. Please, see my comment in the associated issue for the motivation and the workaround.

@delatrie delatrie closed this Dec 29, 2025
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.

5 participants