Skip to content

Fix missing attachments from TestNG configuration listeners - #1423

Merged
baev merged 1 commit into
mainfrom
fix-776
Sep 7, 2026
Merged

Fix missing attachments from TestNG configuration listeners#1423
baev merged 1 commit into
mainfrom
fix-776

Conversation

@baev

@baev baev commented Sep 7, 2026

Copy link
Copy Markdown
Member

Context

Preserve attachments and steps added by TestNG configuration success, failure, and skip listeners on the corresponding fixture. Screenshots captured in onConfigurationFailure after a failed @BeforeMethod now appear in the report.

On TestNG 7.10 or newer, this works regardless of listener registration order, including listeners registered through annotations or suite XML. Older versions retain their default configuration-listener order.

fixes #776

Checklist

@baev baev added the pr:improvement Change that improves some user experience but can't be considered as a new feature label Sep 7, 2026
@github-actions

github-actions Bot commented Sep 7, 2026

Copy link
Copy Markdown

Allure Report Summary

Name Duration Stats New Flaky Retry Report
Allure Java – Pull request #1423 32m 42s Passed tests 2289 0 0 0 View
Allure Java GitHub actions run (2026-09-07T13:25:31.093Z) 32m 42s Passed tests 2289 0 0 0 View

@baev
baev merged commit 59c1722 into main Sep 7, 2026
9 of 10 checks passed
@baev
baev deleted the fix-776 branch September 7, 2026 14:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

pr:improvement Change that improves some user experience but can't be considered as a new feature

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[allure-testng] Attachment on failed config is not added to the failed configuration method

2 participants