Upgrade to Java 21 (Selenium/TestNG test framework)

pom.xml

@@ -13,8 +13,8 @@
 
 	<properties>
 		<project.build.sourceEncoding>UTF-8</project.build.sourceEncoding>
-		<maven.compiler.source>1.8</maven.compiler.source>
-		<maven.compiler.target>1.8</maven.compiler.target>
+		<maven.compiler.source>21</maven.compiler.source>
+		<maven.compiler.target>21</maven.compiler.target>
 		<p>x</p>
 	</properties>
 
@@ -29,7 +29,7 @@
 		<dependency>
 			<groupId>org.testng</groupId>
 			<artifactId>testng</artifactId>
-			<version>6.14.3</version>
+			<version>7.10.2</version>
 			<scope>test</scope>
 		</dependency>
 		<!-- https://mvnrepository.com/artifact/com.relevantcodes/extentreports -->
@@ -62,7 +62,7 @@
 			<plugin>
 				<groupId>org.apache.maven.plugins</groupId>
 				<artifactId>maven-surefire-plugin</artifactId>
-				<version>2.19.1</version>
+				<version>3.5.2</version>
 				<configuration>
 					<suiteXmlFiles>
 						<suiteXmlFile>./src/test/resources/suites/testng.xml</suiteXmlFile>

src/test/java/example/example/listeners/SuiteSummaryListener.java

@@ -0,0 +1,51 @@
+package example.example.listeners;
+
+import org.testng.ISuite;
+import org.testng.ISuiteListener;
+import org.testng.ISuiteResult;
+import org.testng.ITestContext;
+
+import example.example.util.LoggerUtil;
+import example.example.util.MailUtil;
+import example.example.util.TestProperties;
+
+/**
+ * Suite-level listener that loads global properties before the suite runs and,
+ * once the whole suite has finished, logs an execution summary and emails the
+ * report. This replaces the former {@code @BeforeSuite}/{@code @AfterSuite}
+ * hooks in {@code BaseTest}: TestNG 7 no longer supports native parameter
+ * injection into {@code @AfterSuite} methods, and a suite listener guarantees
+ * the summary runs exactly once per suite while aggregating results across every
+ * {@code <test>} tag.
+ */
+public class SuiteSummaryListener implements ISuiteListener {
+
+	@Override
+	public void onStart(ISuite suite) {
+		LoggerUtil.log("************************** Test Execution Started ************************************");
+		TestProperties.loadAllPropertie();
+	}
+
+	@Override
+	public void onFinish(ISuite suite) {
+		int total = 0;
+		int passed = 0;
+		int failed = 0;
+		int skipped = 0;
+		for (ISuiteResult result : suite.getResults().values()) {
+			ITestContext context = result.getTestContext();
+			total += context.getAllTestMethods().length;
+			passed += context.getPassedTests().size();
+			failed += context.getFailedTests().size();
+			skipped += context.getSkippedTests().size();
+		}
+		LoggerUtil.log("Total number of testcases : " + total);
+		LoggerUtil.log("Number of testcases Passed : " + passed);
+		LoggerUtil.log("Number of testcases Failed : " + failed);
+		LoggerUtil.log("Number of testcases Skipped  : " + skipped);
+		boolean mailSent = MailUtil.sendMail(total, passed, failed, skipped);
+		LoggerUtil.log("Mail sent : " + mailSent);
+		LoggerUtil.log("************************** Test Execution Finished ************************************");
+	}
+
+}

src/test/java/example/example/tests/BaseTest.java

@@ -1,23 +1,17 @@
 package example.example.tests;
 
-import java.util.concurrent.TimeUnit;
+import java.time.Duration;
 
 import org.openqa.selenium.WebDriver;
 import org.openqa.selenium.chrome.ChromeDriver;
 import org.openqa.selenium.chrome.ChromeOptions;
-import org.testng.ITestContext;
 import org.testng.annotations.AfterClass;
-import org.testng.annotations.AfterSuite;
 import org.testng.annotations.BeforeClass;
-import org.testng.annotations.BeforeSuite;
 import org.testng.annotations.Listeners;
 
 import example.example.context.WebDriverContext;
 import example.example.listeners.LogListener;
 import example.example.listeners.ReportListener;
-import example.example.util.LoggerUtil;
-import example.example.util.MailUtil;
-import example.example.util.TestProperties;
 import io.github.bonigarcia.wdm.WebDriverManager;
 
 /**
@@ -31,35 +25,6 @@ public class BaseTest {
 	/** The driver. */
 	protected WebDriver driver;
 
-	/**
-	 * Global setup.
-	 */
-	@BeforeSuite(alwaysRun = true)
-	public void globalSetup() {
-		LoggerUtil.log("************************** Test Execution Started ************************************");
-		TestProperties.loadAllPropertie();
-	}
-
-	/**
-	 * Wrap all up.
-	 *
-	 * @param context the context
-	 */
-	@AfterSuite(alwaysRun = true)
-	public void wrapAllUp(ITestContext context) {
-		int total = context.getAllTestMethods().length;
-		int passed = context.getPassedTests().size();
-		int failed = context.getFailedTests().size();
-		int skipped = context.getSkippedTests().size();
-		LoggerUtil.log("Total number of testcases : " + total);
-		LoggerUtil.log("Number of testcases Passed : " + passed);
-		LoggerUtil.log("Number of testcases Failed : " + failed);
-		LoggerUtil.log("Number of testcases Skipped  : " + skipped);
-		boolean mailSent = MailUtil.sendMail(total, passed, failed, skipped);
-		LoggerUtil.log("Mail sent : " + mailSent);
-		LoggerUtil.log("************************** Test Execution Finished ************************************");
-	}
-
 	/**
 	 * Setup.
 	 */
@@ -74,7 +39,7 @@ protected void setup() {
 		ops.addArguments("--disable-dev-shm-usage");
 		driver = new ChromeDriver(ops);
 		driver.manage().window().maximize();
-		driver.manage().timeouts().implicitlyWait(10, TimeUnit.SECONDS);
+		driver.manage().timeouts().implicitlyWait(Duration.ofSeconds(10));
 		WebDriverContext.setDriver(driver);
 	}
 

src/test/resources/suites/testng.xml

@@ -1,7 +1,9 @@
 <?xml version="1.0" encoding="UTF-8"?>
 <!DOCTYPE suite SYSTEM "http://testng.org/testng-1.0.dtd">
 <suite name="Suite">
-	<listeners></listeners>
+	<listeners>
+		<listener class-name="example.example.listeners.SuiteSummaryListener" />
+	</listeners>
 	<test thread-count="5" name="Test" parallel="classes">
 		<classes>
 			<class name="example.example.tests.GoogleSearchTest" />