TestNG: No need of listener by class

BeforeScreenshotTest.java

package test;

import org.testng.Assert;
import org.testng.annotations.BeforeMethod;
import org.testng.annotations.Listeners;
import org.testng.annotations.Test;

@Listeners(ScreenshotMaker.class)
public class BeforeScreenshotTest implements Screen {

  public String getDriver() {
    return "driver of BeforeScreenshotTest";
  }

  @BeforeMethod
  public void beforeMethod() {
    Assert.fail("BOUM");
  }

  @Test
  public void test() { }
}

Screen.java

package test;

public interface Screen {

  String getDriver();
}

ScreenshotMaker.java

package test;

import org.testng.IInvokedMethod;
import org.testng.IInvokedMethodListener;
import org.testng.ITestResult;

public class ScreenshotMaker implements IInvokedMethodListener {

  public void beforeInvocation(IInvokedMethod method, ITestResult testResult) {

  }

  public void afterInvocation(IInvokedMethod method, ITestResult testResult) {
    if (testResult.getStatus() == ITestResult.FAILURE) {
      Object testInstance = testResult.getInstance();
      if (testInstance instanceof Screen) {
        Screen screen = (Screen) testInstance;
        System.out.println("after invocation of " +testResult.getName());
        System.out.println("\t" + screen.getDriver());
      }
    }
  }
}

ScreenshotTest.java

package test;

import org.testng.Assert;
import org.testng.annotations.Listeners;
import org.testng.annotations.Test;

@Listeners(ScreenshotMaker.class)
public class ScreenshotTest implements Screen {

  public String getDriver() {
    return "driver of ScreenshotTest";
  }

  @Test
  public void test() {
    Assert.fail("BOUM");
  }
}

Interesting. I didn't know applying listeners with annotation applies the listener once for whole suite. This is actually the cleanest way to add a screenshot listener I've seen so far! GJ and Thanks

Thanks mate your awesome