Address PR comments.

monroegm-disable-blank-issue-2
jonawals 5 years ago
parent eb5dd7ee47
commit 20243549e6

@ -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

@ -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);

Loading…
Cancel
Save