fixes #2 ITestResult.getThrowable() null for onTestSkipped

anshooarora · anshooarora · commit 2f9924239861 · 2018-12-11T23:42:45.000-05:00
diff --git a/src/main/java/com/aventstack/extentreports/testng/listener/ExtentITestListenerAdapter.java b/src/main/java/com/aventstack/extentreports/testng/listener/ExtentITestListenerAdapter.java
@@ -37,6 +37,10 @@ public synchronized void onTestFailure(ITestResult result) {
 
     @Override
     public synchronized void onTestSkipped(ITestResult result) {
+        if (result.getThrowable() == null) {
+            ExtentTestManager.getTest(result).skip("Test skipped");
+            return;
+        }
         ExtentTestManager.getTest(result).skip(result.getThrowable());
     }
 

Original file line number	Diff line number	Diff line change
`@@ -37,6 +37,10 @@ public synchronized void onTestFailure(ITestResult result) {`
`37`	`37`
`38`	`38`	`@Override`
`39`	`39`	`public synchronized void onTestSkipped(ITestResult result) {`
	`40`	`+ if (result.getThrowable() == null) {`
	`41`	`+ ExtentTestManager.getTest(result).skip("Test skipped");`
	`42`	`+ return;`
	`43`	`+ }`
`40`	`44`	`ExtentTestManager.getTest(result).skip(result.getThrowable());`
`41`	`45`	`}`
`42`	`46`