Merge pull request #27 from 21johnh21/HAWL/SWE-rename-UnitStatsDetail

Make _UnitStatsDetail a public class

Author: KyleKincer

…ect/Sources/Classes/_UnitStatsDetail.4dm …ject/Sources/Classes/UnitStatsDetail.4dmtesting/Project/Sources/Classes/_UnitStatsDetail.4dm renamed to testing/Project/Sources/Classes/UnitStatsDetail.4dm testing/Project/Sources/Classes/_UnitStatsDetail.4dm renamed to testing/Project/Sources/Classes/UnitStatsDetail.4dm

@@ -14,7 +14,7 @@ Function resetStatistics()
 	If (This:C1470._statsMap=Null:C1517)
 		return 
 	End if 
-	var $statistic : cs:C1710._UnitStatsDetail
+	var $statistic : cs:C1710.UnitStatsDetail
 	For each ($statistic; OB Values:C1718(This:C1470._statsMap))
 		If (OB Instance of:C1731($statistic.reset; 4D:C1709.Function))
 			$statistic.reset()
@@ -26,14 +26,14 @@ Function createStatistic($name : Text) : cs:C1710.UnitStatsTracker
 $name - text - the name of the statistic to create
 returns - cs.UnitStatsTracker - This object to chain call
 */
-	This:C1470._statsMap[$name]:=cs:C1710._UnitStatsDetail.new()
+	This:C1470._statsMap[$name]:=cs:C1710.UnitStatsDetail.new()
 	return This:C1470
 
-Function getStat($name : Text) : cs:C1710._UnitStatsDetail
+Function getStat($name : Text) : cs:C1710.UnitStatsDetail
 /* Gets a statistic. If one does not exist, a new statistic will
 be created an returned.
 $name - text - the name of the statistic to return
-returns - cs._UnitStatsDetail - a statistic
+returns - cs.UnitStatsDetail - a statistic
 */
 	If (Not:C34(This:C1470._doesStatisticExist($name)))
 		This:C1470.createStatistic($name)
@@ -47,7 +47,7 @@ $functionName - text - the name of the statistic to update
 $parameters - collection - the collection of parameters to push onto the related statistic
 $returnValue - variant - value this method will return
 */
-	var $statistic : cs:C1710._UnitStatsDetail
+	var $statistic : cs:C1710.UnitStatsDetail
 	$statistic:=This:C1470.getStat($functionName)
 	If ($statistic#Null:C1517)
 		$statistic.appendCalledParameters((($parameters#Null:C1517) ? $parameters : []))

testing/Project/Sources/Classes/UnitStatsTracker.4dm

@@ -9,7 +9,7 @@ Function test_mock_basic_functionality($t : cs:C1710.Testing)
 	$result:=$t.stats.mock("testFunction"; ["param1"; "param2"]; "mock_result")
 
 	// Verify the mock was called correctly
-	var $stat : cs:C1710._UnitStatsDetail
+	var $stat : cs:C1710.UnitStatsDetail
 	$stat:=$t.stats.getStat("testFunction")
 
 	$t.assert.areEqual($t; 1; $stat.getNumberOfCalls(); "Mock should be called once")
@@ -23,7 +23,7 @@ Function test_mock_multiple_calls($t : cs:C1710.Testing)
 	$t.stats.mock("validateUser"; ["guest"]; False:C215)
 
 	// Verify both calls were tracked
-	var $stat : cs:C1710._UnitStatsDetail
+	var $stat : cs:C1710.UnitStatsDetail
 	$stat:=$t.stats.getStat("validateUser")
 
 	$t.assert.areEqual($t; 2; $stat.getNumberOfCalls(); "Should be called twice")
@@ -38,7 +38,7 @@ Function test_mock_with_collections($t : cs:C1710.Testing)
 	$t.stats.mock("processCollection"; [$testCollection]; "processed")
 
 	// Verify collection parameter was captured
-	var $stat : cs:C1710._UnitStatsDetail
+	var $stat : cs:C1710.UnitStatsDetail
 	$stat:=$t.stats.getStat("processCollection")
 
 	$t.assert.areEqual($t; 1; $stat.getNumberOfCalls(); "Should be called once")
@@ -53,7 +53,7 @@ Function test_mock_statistics_reset($t : cs:C1710.Testing)
 	$t.stats.mock("testMethod"; ["param"]; "result")
 	$t.stats.mock("testMethod"; ["param2"]; "result2")
 
-	var $stat : cs:C1710._UnitStatsDetail
+	var $stat : cs:C1710.UnitStatsDetail
 	$stat:=$t.stats.getStat("testMethod")
 	$t.assert.areEqual($t; 2; $stat.getNumberOfCalls(); "Should have 2 calls before reset")
 
@@ -72,7 +72,7 @@ Function test_mock_parameter_details($t : cs:C1710.Testing)
 	$t.stats.mock("apiCall"; $params; New object:C1471("status"; 200; "data"; "success"))
 
 	// Verify parameter details
-	var $stat : cs:C1710._UnitStatsDetail
+	var $stat : cs:C1710.UnitStatsDetail
 	$stat:=$t.stats.getStat("apiCall")
 
 	$t.assert.areEqual($t; 1; $stat.getNumberOfCalls(); "Should be called once")

testing/Project/Sources/Classes/_MockingExampleTest.4dm

@@ -184,7 +184,7 @@ Function test_run_subtest_stats_isolated($t : cs:C1710.Testing)
         $testing.run("second"; Formula($1.assert.areEqual($1; 0; $1.stats.getStat("mocked").getNumberOfCalls(); "Stats should not carry over between subtests")))
 
         // Parent stats should remain unaffected
-        var $parentStat : cs:C1710._UnitStatsDetail
+        var $parentStat : cs:C1710.UnitStatsDetail
         $parentStat:=$testing.stats.getStat("mocked")
         $t.assert.areEqual($t; 0; $parentStat.getNumberOfCalls(); "Parent stats should remain unaffected")
 

testing/Project/Sources/Classes/_TestingTest.4dm

@@ -81,7 +81,7 @@
 				"Summary": ""
 			}
 		},
-		"_UnitStatsDetail": {
+		"UnitStatsDetail": {
 			"new()": {
 				"Syntax": "**.new**()",
 				"Params": [],
@@ -931,7 +931,7 @@
 			"Summary": ""
 		}
 	},
-	"_UnitStatsDetail": {
+	"UnitStatsDetail": {
 		"getXCallParamLength()": {
 			"Syntax": "**.getXCallParamLength**( *xCall* : Integer ) : Integer",
 			"Params": [
@@ -2396,7 +2396,7 @@
 			"Summary": ""
 		},
 		"getStat()": {
-			"Syntax": "**.getStat**( *name* : Text ) : cs.Testing._UnitStatsDetail",
+			"Syntax": "**.getStat**( *name* : Text ) : cs.Testing.UnitStatsDetail",
 			"Params": [
 				[
 					"name",
@@ -2405,7 +2405,7 @@
 				],
 				[
 					"",
-					"cs.Testing._UnitStatsDetail",
+					"cs.Testing.UnitStatsDetail",
 					"<-"
 				]
 			],