fix #2 - npe when onTestStart was not called for ITestListener

Author: anshooarora

src/main/java/com/aventstack/extentreports/service/ExtentTestManager.java

@@ -96,7 +96,7 @@ private static synchronized ExtentTest createTest(ITestResult result, ExtentTest
         return test;
     }
 
-    public static synchronized void log(ITestResult result) {
+    public static synchronized void log(ITestResult result, Boolean createTestAsChild) {
         String msg = "Test ";
         Status status = Status.PASS;
         switch (result.getStatus()) {
@@ -112,11 +112,18 @@ public static synchronized void log(ITestResult result) {
                 msg += "passed";
                 break;
         }
+        if (ExtentTestManager.getTest(result) == null) {
+            ExtentTestManager.createMethod(result, createTestAsChild);
+        }
         if (result.getThrowable() != null) {
             ExtentTestManager.getTest(result).log(status, result.getThrowable());
             return;
         }
         ExtentTestManager.getTest(result).log(status, msg);
     }
+    
+    public static synchronized void log(ITestResult result) {
+        log(result, false);
+    }
 
 }

src/main/java/com/aventstack/extentreports/testng/listener/ExtentITestListenerClassAdapter.java

@@ -27,17 +27,17 @@ public synchronized void onTestStart(ITestResult result) {
 
     @Override
     public synchronized void onTestSuccess(ITestResult result) {
-        ExtentTestManager.log(result);
+        ExtentTestManager.log(result, true);
     }
 
     @Override
     public synchronized void onTestFailure(ITestResult result) {
-        ExtentTestManager.log(result);
+        ExtentTestManager.log(result, true);
     }
 
     @Override
     public synchronized void onTestSkipped(ITestResult result) {
-        ExtentTestManager.log(result);
+        ExtentTestManager.log(result, true);
     }
 
     @Override

src/test/java/com/aventstack/extentreports/adapter/testlistener/Class1Tests.java

@@ -1,13 +1,21 @@
 package com.aventstack.extentreports.adapter.testlistener;
 
 import org.testng.Assert;
+import org.testng.SkipException;
+import org.testng.annotations.BeforeSuite;
 import org.testng.annotations.Listeners;
 import org.testng.annotations.Test;
 
 import com.aventstack.extentreports.testng.listener.ExtentITestListenerAdapter;
 
 @Listeners({ExtentITestListenerAdapter.class})
 public class Class1Tests extends BaseDataProvider {
+	
+	@BeforeSuite
+	public void BeforeSuite() {
+		System.out.println("Before Suite");
+		throw new SkipException("Skip Suite");
+	}
 
 	@Test(dataProvider = "Authentication", groups = { "functest", "checkintest" })
 	public void passClass1(String user, String password) {
@@ -19,4 +27,10 @@ public void failClass1(String user, String password) {
 		Assert.assertTrue(false);
 	}
 
+	@Test
+	public void SkipClass1() {
+		System.out.println("Inside Skip Test");
+		throw new SkipException("Skip Class1 Test skipped");
+	}
+	
 }

src/test/java/com/aventstack/extentreports/adapter/testlistener/Class3Tests.java

@@ -0,0 +1,29 @@
+package com.aventstack.extentreports.adapter.testlistener;
+
+import org.testng.Assert;
+import org.testng.SkipException;
+import org.testng.annotations.BeforeSuite;
+import org.testng.annotations.Listeners;
+import org.testng.annotations.Test;
+
+import com.aventstack.extentreports.testng.listener.ExtentITestListenerAdapter;
+
+@Listeners({ExtentITestListenerAdapter.class})
+public class Class3Tests extends BaseDataProvider {
+
+    @BeforeSuite
+    public void beforeSuite() {
+        throw new SkipException("intentional skip");
+    }
+    
+	@Test
+	public void passClass2() {
+		Assert.assertTrue(true);
+	}
+	
+	@Test(groups = { "functest", "checkintest" })
+	public void failClass2() {
+		Assert.assertTrue(false);
+	}
+	
+}