REname JobMetaContainer to JobMetaWrapper

Code/Tools/TestImpactFramework/Runtime/Code/Source/Process/JobRunner/TestImpactProcessJob.h

@@ -22,7 +22,7 @@ namespace TestImpact
     //! @tparam JobPayloadT The resulting output of the processed artifact produced by this job.
     template<typename JobInfoT, typename JobPayloadT>
     class Job
-        : public JobMetaContainer
+        : public JobMetaWrapper
     {
     public:
         using Info = JobInfoT;
@@ -47,7 +47,7 @@ namespace TestImpact
 
     template<typename JobInfoT, typename JobPayloadT>
     Job<JobInfoT, JobPayloadT>::Job(const Info& jobInfo, JobMeta&& jobMeta, AZStd::optional<Payload>&& payload)
-        : JobMetaContainer(AZStd::move(jobMeta))
+        : JobMetaWrapper(AZStd::move(jobMeta))
         , m_jobInfo(jobInfo)
         , m_payload(AZStd::move(payload))
     {

Code/Tools/TestImpactFramework/Runtime/Code/Source/Process/JobRunner/TestImpactProcessJobMeta.cpp

@@ -16,32 +16,32 @@
 
 namespace TestImpact
 {
-    JobMetaContainer::JobMetaContainer(const JobMeta& jobMeta)
+    JobMetaWrapper::JobMetaWrapper(const JobMeta& jobMeta)
         : m_meta(jobMeta)
     {
     }
 
-    JobMetaContainer::JobMetaContainer(JobMeta&& jobMeta)
+    JobMetaWrapper::JobMetaWrapper(JobMeta&& jobMeta)
         : m_meta(AZStd::move(jobMeta))
     {
     }
 
-    JobResult JobMetaContainer::GetJobResult() const
+    JobResult JobMetaWrapper::GetJobResult() const
     {
         return m_meta.m_result;
     }
 
-    AZStd::optional<ReturnCode> JobMetaContainer::GetReturnCode() const
+    AZStd::optional<ReturnCode> JobMetaWrapper::GetReturnCode() const
     {
         return m_meta.m_returnCode;
     }
 
-    AZStd::chrono::high_resolution_clock::time_point JobMetaContainer::GetStartTime() const
+    AZStd::chrono::high_resolution_clock::time_point JobMetaWrapper::GetStartTime() const
     {
         return m_meta.m_startTime.value_or(AZStd::chrono::high_resolution_clock::time_point());
     }
 
-    AZStd::chrono::high_resolution_clock::time_point JobMetaContainer::GetEndTime() const
+    AZStd::chrono::high_resolution_clock::time_point JobMetaWrapper::GetEndTime() const
     {
         if (m_meta.m_startTime.has_value() && m_meta.m_duration.has_value())
         {
@@ -53,7 +53,7 @@ namespace TestImpact
         }
     }
 
-    AZStd::chrono::milliseconds JobMetaContainer::GetDuration() const
+    AZStd::chrono::milliseconds JobMetaWrapper::GetDuration() const
     {
         return m_meta.m_duration.value_or(AZStd::chrono::milliseconds{ 0 });
     }

Code/Tools/TestImpactFramework/Runtime/Code/Source/Process/JobRunner/TestImpactProcessJobMeta.h

@@ -39,11 +39,12 @@ namespace TestImpact
         AZStd::optional<ReturnCode> m_returnCode; //!< The return code of the underlying processes of this job.
     };
 
-    class JobMetaContainer
+    //! Wrapper for job meta structure to inheritance/aggregation without being coupled to the JobInfo or Job classes.
+    class JobMetaWrapper
     {
     public:
-        JobMetaContainer(const JobMeta& jobMeta);
-        JobMetaContainer(JobMeta&& jobMeta);
+        JobMetaWrapper(const JobMeta& jobMeta);
+        JobMetaWrapper(JobMeta&& jobMeta);
 
         //! Returns the result of this job.
         JobResult GetJobResult() const;