From 9c36b41d6e65319788d475e0560864ea8b73c83d Mon Sep 17 00:00:00 2001
From: Eamonn Mansour <47121388+eamansour@users.noreply.github.com>
Date: Wed, 21 Aug 2024 16:06:41 +0100
Subject: [PATCH] Update comments in unit tests

Signed-off-by: Eamonn Mansour <47121388+eamansour@users.noreply.github.com>
---
 .../api/ras/internal/routes/TestRunQuery.java | 56 -------------------
 1 file changed, 56 deletions(-)

diff --git a/galasa-parent/dev.galasa.framework.api.ras/src/test/java/dev/galasa/framework/api/ras/internal/routes/TestRunQuery.java b/galasa-parent/dev.galasa.framework.api.ras/src/test/java/dev/galasa/framework/api/ras/internal/routes/TestRunQuery.java
index d5e61af5e..8b3e41cf9 100644
--- a/galasa-parent/dev.galasa.framework.api.ras/src/test/java/dev/galasa/framework/api/ras/internal/routes/TestRunQuery.java
+++ b/galasa-parent/dev.galasa.framework.api.ras/src/test/java/dev/galasa/framework/api/ras/internal/routes/TestRunQuery.java
@@ -490,9 +490,7 @@ public void testQueryWithRequestorNotSortedWithEmptyDBServiceReturnsOK() throws
 		//Then...
 		// Expecting:
 		//  {
-		//   "pageNum": 1,
 		//   "pageSize": 100,
-		//   "numPages": 1,
 		//   "amountOfRuns": 0,
 		//   "runs": [
 		// 	 ]
@@ -526,9 +524,7 @@ public void testQueryWithRequestorNotSortedWithDBServiceWithOneRecordReturnsOK()
 		//Then...
 		// Expecting:
 		//  {
-		//   "pageNum": 1,
 		//   "pageSize": 100,
-		//   "numPages": 1,
 		//   "amountOfRuns": 1,
 		//   "runs": [
 		//     {
@@ -568,9 +564,7 @@ public void testQueryWithRequestorNotSortedWithDBServiceTenRecordsReturnsOK() th
 		//Then...
 		// Expecting:
 		//  {
-		//   "pageNum": 1,
 		//   "pageSize": 100,
-		//   "numPages": 1,
 		//   "amountOfRuns": 10,
 		//   "runs": [
 		//     {
@@ -623,9 +617,7 @@ public void testQueryWithFromDateNotSortedWithDBServiceWithOneRecordReturnsOK()
 		//Then...
 		// Expecting:
 		//  {
-		//   "pageNum": 1,
 		//   "pageSize": 100,
-		//   "numPages": 1,
 		//   "amountOfRuns": 1,
 		//   "runs": [
 		//     {
@@ -662,9 +654,6 @@ public void testQueryWithoutFromDateOrRunNameNotSortedWithDBServiceTenRecordsRet
 		servlet.init();
 		servlet.doGet(req,resp);
 
-		//Then...
-		// Expecting:
-
 		//Then...
 		assertThat(resp.getStatus()).isEqualTo(400);
 		assertThat(outStream.toString()).contains("GAL5010E: Error parsing the query parameters. from time is a mandatory field if no runname is supplied.");
@@ -690,9 +679,6 @@ public void testNoQueryWithDBServiceTenRecordsReturnsOK() throws Exception {
 		servlet.init();
 		servlet.doGet(req,resp);
 
-		//Then...
-		// Expecting:
-
 		//Then...
 		List<String> expectedRunNames = generateExpectedRunNames(mockInputRunResults);
         String actualOutput = outStream.toString();
@@ -732,9 +718,7 @@ public void testQueryWithoutFromDateWithRunNameNotSortedWithDBServiceTenRecordsR
 		//Then...
 		// Expecting:
 		//  {
-		//   "pageNum": 1,
 		//   "pageSize": 100,
-		//   "numPages": 1,
 		//   "amountOfRuns": 0,
 		//   "runs": []
 		// }
@@ -771,9 +755,7 @@ public void testQueryWithFromDateAndRunnameNotSortedWithDBServiceTenRecordsRetur
 		//Then...
 		// Expecting:
 		//  {
-		//   "pageNum": 1,
 		//   "pageSize": 100,
-		//   "numPages": 1,
 		//   "amountOfRuns": 0,
 		//   "runs": []
 		// }
@@ -809,9 +791,7 @@ public void testQueryWithFromDateAndRunnameOutsideNotSortedWithDBServiceTenRecor
 		//Then...
 		// Expecting:
 		//  {
-		//   "pageNum": 1,
 		//   "pageSize": 100,
-		//   "numPages": 1,
 		//   "amountOfRuns": 0,
 		//   "runs": []
 		// }
@@ -844,9 +824,7 @@ public void testQueryWithRequestorNotSortedWithDBServiceTenRecordsPageSizeFiveRe
 		//Then...
 		// Expecting:
 		//  {
-		//   "pageNum": 1,
 		//   "pageSize": 5,
-		//   "numPages": 2,
 		//   "amountOfRuns": 10,
 		//   "runs": [
 		//     {
@@ -902,9 +880,7 @@ public void testQueryWithRequestorNotSortedWithDBServiceTenRecordsPageSizeFivePa
 		//Then...
 		// Expecting:
 		//  {
-		//   "pageNum": 2,
 		//   "pageSize": 5,
-		//   "numPages": 2,
 		//   "amountOfRuns": 10,
 		//   "runs": [
 		//     {
@@ -963,9 +939,7 @@ public void testQueryWithRequestorNotSortedWithDBServiceTwentyRecordsPageSizeFiv
 		//Then...
 		// Expecting:
 		//  {
-		//   "pageNum": 3,
 		//   "pageSize": 5,
-		//   "numPages": 4,
 		//   "amountOfRuns": 20,
 		//   "runs": [
 		//     {
@@ -1299,9 +1273,7 @@ public void testQueryWithFromDateAndToDateNotSortedWithDBServiceTenRecordsReturn
 		//Then...
 		// Expecting:
 		//  {
-		//   "pageNum": 1,
 		//   "pageSize": 100,
-		//   "numPages": 1,
 		//   "amountOfRuns": 10,
 		//   "runs": [...]
 		// }
@@ -1342,9 +1314,7 @@ public void testQueryWithFromDateAndToDatetSortedToDescendingWithDBServiceTenRec
 		//Then...
 		// Expecting:
 		//  {
-		//   "pageNum": 1,
 		//   "pageSize": 100,
-		//   "numPages": 1,
 		//   "amountOfRuns": 10,
 		//   "runs": [...]
 		// }
@@ -1387,9 +1357,7 @@ public void testQueryWithFromDateAndToDatetSortedToAscendingWithDBServiceTenReco
 		//Then...
 		// Expecting:
 		//  {
-		//   "pageNum": 1,
 		//   "pageSize": 100,
-		//   "numPages": 1,
 		//   "amountOfRuns": 10,
 		//   "runs": [...]
 		// }
@@ -1432,9 +1400,7 @@ public void testQueryWithFromDateAndToDatetSortedResultDescendingWithDBServiceTe
 		//Then...
 		// Expecting:
 		//  {
-		//   "pageNum": 1,
 		//   "pageSize": 100,
-		//   "numPages": 1,
 		//   "amountOfRuns": 10,
 		//   "runs": [...]
 		// }
@@ -1477,9 +1443,7 @@ public void testQueryWithFromDateAndToDateResultSortedResultAscendingWithDBServi
 		//Then...
 		// Expecting:
 		//  {
-		//   "pageNum": 1,
 		//   "pageSize": 100,
-		//   "numPages": 1,
 		//   "amountOfRuns": 10,
 		//   "runs": [...]
 		// }
@@ -1518,9 +1482,7 @@ public void testQueryWithFromDateSortedTestclassDescendingWithDBServiceTenRecord
 		//Then...
 		// Expecting:
 		//  {
-		//   "pageNum": 1,
 		//   "pageSize": 100,
-		//   "numPages": 1,
 		//   "amountOfRuns": 10,
 		//   "runs": [...]
 		// }
@@ -1562,9 +1524,7 @@ public void testQueryWithFromDateAndToDateResultSortedTestclassAscendingWithDBSe
 		//Then...
 		// Expecting:
 		//  {
-		//   "pageNum": 1,
 		//   "pageSize": 100,
-		//   "numPages": 1,
 		//   "amountOfRuns": 10,
 		//   "runs": [...]
 		// }
@@ -1602,9 +1562,7 @@ public void testQueryWithFromDateAndToDateBadSortWithDBServiceTenRecordsReturnsE
 		//Then...
 		// Expecting:
 		//  {
-		//   "pageNum": 1,
 		//   "pageSize": 100,
-		//   "numPages": 1,
 		//   "amountOfRuns": 10,
 		//   "runs": [...]
 		// }
@@ -1662,9 +1620,7 @@ public void testQueryWithFromDateBadSortKeyWithDBServiceTenRecordsReturnsOkUnsor
 		//Then...
 		// Expecting:
 		//  {
-		//   "pageNum": 1,
 		//   "pageSize": 100,
-		//   "numPages": 1,
 		//   "amountOfRuns": 10,
 		//   "runs": [...]
 		// }
@@ -1706,9 +1662,7 @@ public void testQueryWithFromResultNotSortedWithDBServiceTenRecordsReturnsOK() t
 		//Then...
 		// Expecting:
 		//  {
-		//   "pageNum": 1,
 		//   "pageSize": 100,
-		//   "numPages": 1,
 		//   "amountOfRuns": 10,
 		//   "runs": [...]
 		// }
@@ -1750,9 +1704,7 @@ public void testQueryWithFromtestNameNotSortedWithDBServiceTenRecordsReturnsOK()
 		//Then...
 		// Expecting:
 		//  {
-		//   "pageNum": 1,
 		//   "pageSize": 100,
-		//   "numPages": 1,
 		//   "amountOfRuns": 10,
 		//   "runs": [...]
 		// }
@@ -1794,9 +1746,7 @@ public void testQueryWithFromBundleNotSortedWithDBServiceTenRecordsReturnsOK() t
 		//Then...
 		// Expecting:
 		//  {
-		//   "pageNum": 1,
 		//   "pageSize": 100,
-		//   "numPages": 1,
 		//   "amountOfRuns": 10,
 		//   "runs": [...]
 		// }
@@ -1838,9 +1788,7 @@ public void testQueryWithFromRunIdNotSortedWithDBServiceTenRecordsReturnsOK() th
 		//Then...
 		// Expecting:
 		//  {
-		//   "pageNum": 1,
 		//   "pageSize": 100,
-		//   "numPages": 1,
 		//   "amountOfRuns": 10,
 		//   "runs": [...]
 		// }
@@ -1881,9 +1829,7 @@ public void testQueryWithFromRunIdsNotSortedWithDBServiceTenRecordsReturnsOK() t
 		//Then...
 		// Expecting:
 		//  {
-		//   "pageNum": 1,
 		//   "pageSize": 100,
-		//   "numPages": 1,
 		//   "amountOfRuns": 10,
 		//   "runs": [...]
 		// }
@@ -1924,9 +1870,7 @@ public void testQueryWithRunIdsNotSortedWithDBServiceTenRecordsReturnsOK() throw
 		//Then...
 		// Expecting:
 		//  {
-		//   "pageNum": 1,
 		//   "pageSize": 100,
-		//   "numPages": 1,
 		//   "amountOfRuns": 10,
 		//   "runs": [...]
 		// }