Added capability to execute test cases according to requirement from

RunSetup.csv.
Added new class, GloblaConstants for constants.

Author: AlfredStenwin

WebTestRunner.xml

@@ -1,26 +1,24 @@
 <?xml version="1.0" encoding="UTF-8"?>
 <!DOCTYPE suite SYSTEM "http://testng.org/testng-1.0.dtd">
-<suite verbose="1" name="Demo test suite" parallel="false" thread-count="1"  >
+<suite name="Demo test suite" >
 
 
 	<listeners>
-		<listener class-name="listeners.TestngListener" ></listener>
+		<listener class-name="listeners.TestListener" ></listener>
+		<listener class-name="listeners.MethodInterceptor" ></listener>
+
 	</listeners> 
 
 	<parameter name = "browser" value = "chrome"></parameter> 
-	<test name="Navigate to home page">
+	<test name="Navigate to home page" parallel="tests" thread-count="1">
 
 		<classes>
-			<class name="Tests.NewUserRegistration"> 
-				<methods>
-				  <include name="navigateToHomepage"></include>
-				</methods>
-			</class>
+			<class name="Tests.NewUserRegistration"/> 
 		</classes>
 	</test>
-
+<!--
 	<parameter name = "browser" value = "chrome"></parameter> 
-	<test name="Navigate to user signup page">
+	<test name="Navigate to user signup page" parallel="tests" thread-count="1">
 			<classes>
 				<class name="Tests.NewUserRegistration"> 
 					<methods>
@@ -31,7 +29,7 @@
 	</test>
 
 	<parameter name = "browser" value = "chrome"></parameter> 
-	<test name="Register new user">	
+	<test name="Register new user" parallel="tests" thread-count="1">	
 			<classes>
 				<class name="Tests.NewUserRegistration"> 
 					<methods>
@@ -40,5 +38,5 @@
 				</class>
 			</classes>
 	</test>
-	
+	-->
 </suite> 

src/main/java/globalconstants/GlobalConstants.java

@@ -0,0 +1,6 @@
+package globalconstants;
+
+public class GlobalConstants {
+	public static final String RUN_SETUP_CSV = System.getProperty("user.dir")+"/src/test/resources/RunSetup.csv";
+
+}

src/main/java/listeners/ExtentReportGenerator.java

@@ -0,0 +1,55 @@
+package listeners;
+
+import java.io.File;
+import java.io.IOException;
+import java.util.ArrayList;
+import java.util.List;
+import java.util.Map;
+
+import org.testng.IMethodInstance;
+import org.testng.IMethodInterceptor;
+import org.testng.ITestContext;
+
+import globalconstants.GlobalConstants;
+import utilities.filereaders.CsvReaderUtility;
+
+/* 
+ * TestNG listener to get the list of tests to run. 
+ */
+public class MethodInterceptor implements IMethodInterceptor{
+
+	List<IMethodInstance> testList = new ArrayList<>();
+
+	// Method will be executed only once for the entire test suite
+	// Returns the list of test cases to run according to the execute parameter in csv file	
+	@Override
+	public List<IMethodInstance> intercept(List<IMethodInstance> methods, ITestContext context) {
+		
+		List<Map<String, String>> mapList = new ArrayList<>();		
+		
+		try {
+			mapList.addAll(CsvReaderUtility.read(new File(GlobalConstants.RUN_SETUP_CSV)));
+			testList=getListOfTestCasesToRun(methods, mapList);
+			
+		} catch (IOException e) {
+			e.printStackTrace();	
+		}
+		
+		return testList;
+	}
+	
+	//Method to get the list of test cases to run
+	private List<IMethodInstance> getListOfTestCasesToRun(List<IMethodInstance> methods, List<Map<String, String>> mapList){
+		for(var method : methods){
+			for(var map : mapList){
+				if(method.getMethod().getMethodName().equalsIgnoreCase(map.get("name"))) {
+					if(map.get("execute").equalsIgnoreCase("true")) {
+						method.getMethod().setDescription(map.get("description"));
+						testList.add(method);
+					}
+				}
+			}
+		}
+		return testList;
+	}
+}

src/main/java/listeners/MethodInterceptor.java

@@ -1,7 +1,9 @@
 package listeners;
 
+import java.awt.Desktop;
 import java.io.File;
 import java.io.IOException;
+import java.util.Arrays;
 
 import org.openqa.selenium.io.FileHandler;
 import org.testng.ISuite;
@@ -16,22 +18,23 @@
 
 import decorators.Driver;
 import logsetup.Log;
+import reports.ExtentReport;
 
-public class TestngListener implements ITestListener, ISuiteListener{
+public class TestListener implements ITestListener, ISuiteListener{
 	ExtentReports extentReport;
 	ExtentTest extentTest ;
 
 	@Override
 	public void onStart(ISuite suite) {
 		//To generate extend report at the start of the suite execution
-		extentReport=ExtentReportGenerator.generateReport();	
+		extentReport=ExtentReport.initReport();	
 		Log.info("\""+suite.getName()+ "\" test suite execution started.");
 	}
 
 	@Override
 	public void onFinish(ISuite suite) {
 		//Flush is used to create the extend report
-		extentReport.flush();		
+		extentReport.flush();
 		Log.info("\""+suite.getName()+ "\" test suite execution ended.");				
 	}
 
@@ -42,31 +45,31 @@ public void onStart(ITestContext context) {
 
 	@Override
 	public void onTestStart(ITestResult result) {
-		String testName=result.getName();
-		extentTest = extentReport.createTest(testName);
-		extentTest.log(Status.INFO, testName+" started." );
-		Log.info("\""+testName+"\" execution started. EntentTest created");
+		String testDesc=result.getMethod().getDescription();
+		extentTest = extentReport.createTest(testDesc);
+		extentTest.log(Status.INFO, testDesc+" started." );
+		Log.info("\""+testDesc+"\" execution started. EntentTest created");
 
 	}
 
 	@Override
 	public void onTestSuccess(ITestResult result) {
-		String testName=result.getName();
-		extentTest.log(Status.PASS, testName+" test passed." );
-		Log.info("\""+testName+"\" passed.");		
+		String testDesc=result.getMethod().getDescription();
+		extentTest.log(Status.PASS, testDesc+" test passed." );
+		Log.info("\""+testDesc+"\" passed.");		
 
 	}
 
 	@Override
 	public void onTestFailure(ITestResult result) {
-		String testName=result.getName();
+		String testDesc=result.getMethod().getDescription();
 
 		//Getting the driver from result parameter for taking the screenshot on failure of test
 		try {
 			Driver driver = (Driver)result.getTestClass().getRealClass().getField("driver").get(result.getInstance());
 
 			//Take screenshot and copying to Screenshot folder in the project
-			String screenshotsFolderPath =System.getProperty("user.dir")+"/Screenshots/"+testName+".png";
+			String screenshotsFolderPath =System.getProperty("user.dir")+"/Screenshots/"+testDesc+".png";
 
 			//Call takescreenshot() method from DriverLogger class and copying the screenshot from source path to Screenshot folder 	
 			FileHandler.copy(driver.takescreenshot(), new File(screenshotsFolderPath));
@@ -78,14 +81,16 @@ public void onTestFailure(ITestResult result) {
 			e.printStackTrace();
 		}
 
-		extentTest.log(Status.FAIL, testName +" failed. \n"+result.getThrowable());
+		extentTest.log(Status.FAIL, testDesc +" failed.");
+		extentTest.fail(Arrays.toString(result.getThrowable().getStackTrace()));//beautify stacktrace		
 		Log.error("\""+result.getName()+"\" failed.", result.getThrowable());
 	}
 
 	@Override
 	public void onTestSkipped(ITestResult result) {
 		String testName=result.getName();
-		extentTest.log(Status.SKIP, testName +" skipped." + result.getThrowable());		
+		extentTest.log(Status.SKIP, testName +" skipped.");		
+		extentTest.skip(Arrays.toString(result.getThrowable().getStackTrace()));		
 		Log.error("\""+result.getName()+"\" skipped.", result.getThrowable());		
 	}
 

src/main/java/listeners/TestngListener.java renamed to src/main/java/listeners/TestListener.java

@@ -0,0 +1,25 @@
+package reports;
+
+import com.aventstack.extentreports.ExtentTest;
+
+public class ExtentManager {
+	
+	private ExtentManager() {} //Cannot invoke the constructor from other methods
+	
+	//To make the extent report thread safe, Thread local is used
+	private static ThreadLocal<ExtentTest> tlExtentTest = new ThreadLocal<>();
+	
+	public static ExtentTest GetExtentTest() {		
+		 return tlExtentTest.get();
+	}
+	
+	//Method to set the threadlocal version of extend report
+	public static void setExtentTest(ExtentTest test) {
+		tlExtentTest.set(test);
+	}
+	
+	public static void unload() {
+		tlExtentTest.remove();
+	}
+	
+}

src/main/java/reports/ExtentManager.java

@@ -0,0 +1,33 @@
+package utilities.filereaders;
+
+import java.io.File;
+import java.io.IOException;
+import java.util.LinkedList;
+import java.util.List;
+import java.util.Map;
+
+import com.fasterxml.jackson.core.JsonProcessingException;
+import com.fasterxml.jackson.databind.MappingIterator;
+import com.fasterxml.jackson.dataformat.csv.CsvMapper;
+import com.fasterxml.jackson.dataformat.csv.CsvSchema;
+
+public final class CsvReaderUtility  {
+	
+	private CsvReaderUtility() {}
+	
+	public static List<Map<String, String>> read(File file) throws JsonProcessingException, IOException {
+	    List<Map<String, String>> response = new LinkedList<Map<String, String>>();
+	    CsvMapper mapper = new CsvMapper();
+	    CsvSchema schema = CsvSchema.emptySchema().withHeader().withColumnSeparator(',').withComments();
+		MappingIterator<Map<String, String>> iterator = mapper.reader(Map.class)
+				.with(schema)
+	            .readValues(file);
+		
+	    while (iterator.hasNext()) {
+	        response.add(iterator.next());
+	    }
+	    return response;
+	}
+	
+
+}