From 20243549e6a36bd59e5b12f829b691cf88eca8ce Mon Sep 17 00:00:00 2001
From: jonawals <jonawals@amazon.com>
Date: Fri, 30 Apr 2021 12:51:26 +0100
Subject: [PATCH] Address PR comments.

---
 .../Source/Artifact/Static/TestImpactDependencyGraphData.h | 7 ++++---
 .../Dependency/TestImpactTestSelectorAndPrioritizer.cpp    | 6 ++++--
 2 files changed, 8 insertions(+), 5 deletions(-)

diff --git a/Code/Tools/TestImpactFramework/Runtime/Code/Source/Artifact/Static/TestImpactDependencyGraphData.h b/Code/Tools/TestImpactFramework/Runtime/Code/Source/Artifact/Static/TestImpactDependencyGraphData.h
index 8dae41c579..3c9f455254 100644
--- a/Code/Tools/TestImpactFramework/Runtime/Code/Source/Artifact/Static/TestImpactDependencyGraphData.h
+++ b/Code/Tools/TestImpactFramework/Runtime/Code/Source/Artifact/Static/TestImpactDependencyGraphData.h
@@ -17,10 +17,11 @@
 
 namespace TestImpact
 {
+    //! Raw representation of the dependency graph for a given build target.
     struct DependencyGraphData
     {
-        AZStd::string m_root;
-        AZStd::vector<AZStd::string> m_vertexes;
-        AZStd::vector<AZStd::pair<AZStd::string, AZStd::string>> m_edges;
+        AZStd::string m_root; //!< The build target this dependency graph is for.
+        AZStd::vector<AZStd::string> m_vertices; //!< The depender/depending built targets in this graph.
+        AZStd::vector<AZStd::pair<AZStd::string, AZStd::string>> m_edges; //!< The dependency connectivity of the build targets in this graph.
     };
 } // namespace TestImpact
diff --git a/Code/Tools/TestImpactFramework/Runtime/Code/Source/Dependency/TestImpactTestSelectorAndPrioritizer.cpp b/Code/Tools/TestImpactFramework/Runtime/Code/Source/Dependency/TestImpactTestSelectorAndPrioritizer.cpp
index 2962e91381..549ef0c108 100644
--- a/Code/Tools/TestImpactFramework/Runtime/Code/Source/Dependency/TestImpactTestSelectorAndPrioritizer.cpp
+++ b/Code/Tools/TestImpactFramework/Runtime/Code/Source/Dependency/TestImpactTestSelectorAndPrioritizer.cpp
@@ -17,13 +17,15 @@
 
 namespace TestImpact
 {
-    TestSelectorAndPrioritizer::TestSelectorAndPrioritizer(const DynamicDependencyMap* dynamicDependencyMap, DependencyGraphDataMap&& dependencyGraphDataMap)
+    TestSelectorAndPrioritizer::TestSelectorAndPrioritizer(
+        const DynamicDependencyMap* dynamicDependencyMap, DependencyGraphDataMap&& dependencyGraphDataMap)
         : m_dynamicDependencyMap(dynamicDependencyMap)
         , m_dependencyGraphDataMap(AZStd::move(dependencyGraphDataMap))
     {
     }
 
-    AZStd::vector<const TestTarget*> TestSelectorAndPrioritizer::SelectTestTargets(const ChangeDependencyList& changeDependencyList, TestSelectionStrategy testSelectionStrategy)
+    AZStd::vector<const TestTarget*> TestSelectorAndPrioritizer::SelectTestTargets(
+        const ChangeDependencyList& changeDependencyList, TestSelectionStrategy testSelectionStrategy)
     {
         const auto selectedTestTargetAndDependerMap = SelectTestTargets(changeDependencyList);
         const auto prioritizedSelectedTests = PrioritizeSelectedTestTargets(selectedTestTargetAndDependerMap, testSelectionStrategy);