You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
Repository navigation
[allure-testng] Attachment on failed config is not added to the failed configuration method #776
Describe the bug
There is a test with a configuration method (BeforeMethod) that fails with an exception.
There is an IConfigurationListener implementation provided, that creates an attachment on failed configuration.
An attachment is expected to be present in the report inside the failed configuration method container, but it is not.
An attachment can be found in the allure-results directory but seems not to be attached to any test container.
I'm not sure we will ever support such behaviour since Allure uses listeners itself to collect data from TestNG and we can't control the listeners order
Ok, I will have a look.
But my idea was that MAYBE all the logic of collecting the results of a before configuration methods should be handled inside onConfigurationSuccess/onConfigurationFailure methods of IConfigurationListener.
Activity
baflQA commentedon Jun 20, 2022
Hi @baev! Would You be able to have a look?
baev commentedon Jun 21, 2022
I'm not sure we will ever support such behaviour since Allure uses listeners itself to collect data from TestNG and we can't control the listeners order
baev commentedon Jun 21, 2022
in most cases as workaround you can use AllureLifecycle listeners instead of TestNG listeners
baflQA commentedon Jun 21, 2022
Ok, I will have a look.
But my idea was that MAYBE all the logic of collecting the results of a before configuration methods should be handled inside onConfigurationSuccess/onConfigurationFailure methods of IConfigurationListener.
fix missing attachments from TestNG configuration listeners (fixes al…