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
TestNG optional reporting for @After @Before failures #356
On Before/After(Class/Test) failures allure create reports as it was 'Test' and we have some inconvenience to displaying TeamCity allure reports. For example, next test class will report about two tests(one failed and one skipped) public class Mytest { @BeforeMethod public void before() { Assert.fail(); } @Test public void test() { } }
This behavior was implemented with #135
What is the expected behavior?
Reporting of After/Before method to be optional. For example, using configuration flags.
What is the motivation / use case for changing the behavior?
Disabling the After/Before reporting was requested a sew times ago and those requests was rejected because of #135. My call is to make it optional to satisfy both behaviors with actual by default.
I'd also love for this to get worked on/merged in. We recently upgraded to 2.13.6 of allure-testng, and didn't expect to see our config methods increasing the total count of tests.
If this were an added feature where you could decide to either include them or to not, that'd be nice. It throws our reporting numbers off now if anything breaks in config methods for TestNG.
Is there any more work to be done for this PR? If need be I can pickup where @Gzyablik started/left off.
Activity
Proryanator commentedon Mar 29, 2021
I'd also love for this to get worked on/merged in. We recently upgraded to 2.13.6 of allure-testng, and didn't expect to see our config methods increasing the total count of tests.
If this were an added feature where you could decide to either include them or to not, that'd be nice. It throws our reporting numbers off now if anything breaks in config methods for TestNG.
Is there any more work to be done for this PR? If need be I can pickup where @Gzyablik started/left off.
PavelChachotkinKudo commentedon Jun 1, 2022
I vote for this addition flag ))
allure-testng: add property to disable configuration failures logging (…