diff --git a/src/Runner.Worker/ExecutionContext.cs b/src/Runner.Worker/ExecutionContext.cs index a366dd5c1..24cc52444 100644 --- a/src/Runner.Worker/ExecutionContext.cs +++ b/src/Runner.Worker/ExecutionContext.cs @@ -477,28 +477,32 @@ namespace GitHub.Runner.Worker PublishStepTelemetry(); - var stepResult = new StepResult + if (_record.RecordType == "Task") { - ExternalID = _record.Id, - Conclusion = _record.Result ?? TaskResult.Succeeded, - Status = _record.State, - Number = _record.Order, - Name = _record.Name, - StartedAt = _record.StartTime, - CompletedAt = _record.FinishTime, - Annotations = new List() - }; - - _record.Issues?.ForEach(issue => - { - var annotation = issue.ToAnnotation(); - if (annotation != null) + var stepResult = new StepResult { - stepResult.Annotations.Add(annotation.Value); - } - }); + ExternalID = _record.Id, + Conclusion = _record.Result ?? TaskResult.Succeeded, + Status = _record.State, + Number = _record.Order, + Name = _record.Name, + StartedAt = _record.StartTime, + CompletedAt = _record.FinishTime, + Annotations = new List() + }; + + _record.Issues?.ForEach(issue => + { + var annotation = issue.ToAnnotation(); + if (annotation != null) + { + stepResult.Annotations.Add(annotation.Value); + } + }); + + Global.StepsResult.Add(stepResult); + } - Global.StepsResult.Add(stepResult); if (Root != this) { diff --git a/src/Test/L0/Worker/ExecutionContextL0.cs b/src/Test/L0/Worker/ExecutionContextL0.cs index 9d11508e1..91068d300 100644 --- a/src/Test/L0/Worker/ExecutionContextL0.cs +++ b/src/Test/L0/Worker/ExecutionContextL0.cs @@ -821,8 +821,7 @@ namespace GitHub.Runner.Common.Tests.Worker ec.Complete(); // Assert. - Assert.Equal(1, ec.Global.StepsResult.Count); - Assert.Equal(TaskResult.Succeeded, ec.Global.StepsResult.Single().Conclusion); + Assert.Equal(0, ec.Global.StepsResult.Count); } }