TestNg email reports are not current

Question

I have recently converted my project from JUnit5 to TestNG, solely for the purpose of getting decent reports.

I have added a listener that generates the report at the end of each run:

@Override
public void onFinish(ITestContext context) {
    System.out.println("FINISH. Sending email report.");
    utils.EmailHandler.sendEmail("Finished test", context.toString());
}

My problem is that the reports being sent by email are not from the current run, as desired, but the previous run.

Yet if I open the report in /test-output/custom-report.html using Eclipse IDE it is the correct one!

How do I ensure the emails sent out are current?

I have looked at a couple of similar questions here, but neither are appropriate to me:

Similar questions:

TestNg emailable-report is not updating?

ReportNG HTML report not updating

It finally worked when I moved the call to sendEmail to the end of the GenerateReport method of the listener. That removes all confusion and ensures that that output file is complete before attempting to send it out.

Please post some code to show how/when the file is created! – Xwris Stoixeia Feb 03 '18 at 03:14 — Xwris Stoixeia, Feb 03 '18 at 03:14

Xwris Stoixeia · Answer 1 · 2018-02-06T10:45:38.270

1

Unless, you 're somehow attaching the old version; from your description, I would say that most likely the file is created AFTER the email is being sent (hence previous version every-time). However, if this theory is correct, it must have emailed an empty file the first time :) Did it?

Idea: insert a couple of minutes delay in your code where the email is being sent. Go check the file as soon as the email leaves the ground, I think it will be the old version (as it hasn't been created yet!)

Have you tried using the @AfterTest annotation? Not sure, but onFinish(ITestContext context) could be lingering somewhere between @AfterMethod and AfterTestcausing your email to leave slightly earlier; before fully attached! Not sure though, why you send an email after each test and not after the whole suite has finished [so to use onFinish(ISuite suite)].

@AfterTest  
public void afterTest(ITestContext context) {

    //improving answer after initial comments
    if(bufferredWriter!=null){
    bufferredWriter.close();
    }       
    else{
    System.out.println("FINISH. Sending email report.");
    utils.EmailHandler.sendEmail("Finished test", context.toString());
    }   
}

Best of luck!

PS. Nevertheless, I would highly recommend to have a look at extentReports. Definitely better reporting than the built-in that comes with TestNG!

edited Feb 06 '18 at 10:45

answered Feb 03 '18 at 12:07

Xwris Stoixeia

1,831
21
22

1

I think you are right, @Xwris Stoixeia - I put some print statements in, and indeed, it does not close the writer until after sendEmail is run. Even when sendEmail is invoked from @AfterTest! – Steve Staple Feb 05 '18 at 10:59
Are you running things in parallel on different threads, perhaps? If not check if writer is closed before utils.EmailHandler.sendEmail https://stackoverflow.com/questions/36301347/how-should-i-check-if-bufferedwriter-is-already-closed Alternatively,also try @AfterSuite – Xwris Stoixeia Feb 06 '18 at 10:36
Steve also try @AfterClass, might be THE one – Xwris Stoixeia Feb 08 '18 at 20:35
@AfterClass also does not work. Is there a clean way to send the "emailable-report.html" which is automatically generated by testng ? – NKM May 22 '19 at 09:40
try @AfterSuite – Xwris Stoixeia May 23 '19 at 10:54

TestNg email reports are not current

1 Answers1