Selenium extent reports framework not generating failed status

Question

Currently, I am using Selenium TestNG and Extent Reports framework for reporting. I am facing a problem where all are my test cases are showing Pass and not showing Fail.

@BeforeTest
public void setUp()
{
    //where we need to generate the report
    htmlReporter = new ExtentHtmlReporter(System.getProperty("user.dir")+"/test-output/MyReport.html");
    extent = new ExtentReports();
    extent.attachReporter(htmlReporter);

    // Set our document title, theme etc..
    htmlReporter.config().setDocumentTitle("My Test Report");
    htmlReporter.config().setReportName("Test Report");
    htmlReporter.config().setTestViewChartLocation(ChartLocation.TOP);
    htmlReporter.config().setTheme(Theme.DARK);
}

@Test
public void On_Page_IM() throws InterruptedException {
    test = extent.createTest("On_Page_IM");
    wd.manage().window().maximize();        
    Thread.sleep(10000);       
    test.log(Status.INFO,"On page intent media: Show");
}

@AfterSuite
public void tearDown()
{
    extent.flush();
    wd.quit();
}

Answer 1

After Added this code its solved

   @AfterMethod
public void getResult(ITestResult result)
{
    if(result.getStatus()==ITestResult.FAILURE)
    {
        test.log(Status.FAIL, result.getThrowable());

    }
   // extent.endTest(test);
}

Answer 2

Selenium WebDriver 4 + JUnit 5 + Extent Reports 5 (JDK 17)

I used this and this to write the modern JUnit 5 solution, since most examples are for the older TestNG. ExtentHtmlReporter has also been deprecated.

abstract class BaseTest {
    private static ExtentReports extent; // Prevent overwriting reports per test
    protected WebDriver driver; // WebDriver should never be static

    @RegisterExtension
    protected AfterEachExtension afterEachExtension = new AfterEachExtension();

    @BeforeAll
    public static void beforeAll() {
        extent = new ExtentReports();
        spark = new ExtentSparkReporter("target/spark-reports/index.html");
        extent.attachReporter(spark);
        spark.config(
            ExtentSparkReporterConfig.builder()
                .theme(Theme.DARK)
                .documentTitle("Extent Reports")
                .build()
        );
    }

    @BeforeEach
    public void beforeEach() {
        driver = // Initialise driver
    }

    @AfterEach
    public void afterEach() {
        AfterEachExtension.setExtent(extent);
        afterEachExtension.setDriver(driver);
    }

    @AfterAll
    public static void afterAll() {
        extent.flush();
    }
}


import com.aventstack.extentreports.Status;
// Other imports

public class AfterEachExtension implements AfterEachCallback {
    private static ExtentReports extent;
    private WebDriver driver;

    public static void setExtent(ExtentReports extent) {
        AfterEachExtension.extent = extent;
    }

    public void setDriver(WebDriver driver) {
        this.driver = driver;
    }

    @Override
    public void afterEach(ExtensionContext context) throws Exception {
        var test = extent.createTest(context.getDisplayName());
        context.getExecutionException().ifPresent(value -> test.log(Status.FAIL, value));
        driver.quit();
    }
}

Answer 3

Just add below code at end of ur tests

@AfterMethod
public void AfterMethod(ITestResult result) {

    if (result.getStatus() == ITestResult.FAILURE) {
        test.log(Status.FAIL,
                MarkupHelper.createLabel(result.getName()
                        + " Test case FAILED due to below issues:",
                        ExtentColor.RED));
        test.fail(result.getThrowable());
    } else if (result.getStatus() == ITestResult.SUCCESS) {
        test.log(
                Status.PASS,
                MarkupHelper.createLabel(result.getName()
                        + " Test Case PASSED", ExtentColor.GREEN));
    } else {
        test.log(
                Status.SKIP,
                MarkupHelper.createLabel(result.getName()
                        + " Test Case SKIPPED", ExtentColor.ORANGE));
        test.skip(result.getThrowable());
    }
}

@AfterTest
public void AfterTest() {

    extent.flush();

}