fix summaries for actions results (actions#3174)

SrRyan · web-flow · commit d7777fd6320d · 2024-02-27T15:22:26.000-05:00
* fix summaries for actions results

* remove negative
diff --git a/src/Runner.Worker/FileCommandManager.cs b/src/Runner.Worker/FileCommandManager.cs
@@ -244,7 +244,7 @@ public void ProcessCommand(IExecutionContext context, string filePath, Container
                 if (resultsReceiverEndpoint != null)
                 {
                     Trace.Info($"Queueing results file ({filePath}) for attachment upload ({attachmentName})");
-                    var stepId = context.Id;
+                    var stepId = context.IsEmbedded ? context.EmbeddedId : context.Id;
                     // Attachments must be added to the parent context (job), not the current context (step)
                     context.Root.QueueSummaryFile(attachmentName, scrubbedFilePath, stepId);
                 }

Original file line number	Diff line number	Diff line change
`@@ -244,7 +244,7 @@ public void ProcessCommand(IExecutionContext context, string filePath, Container`
`244`	`244`	`if (resultsReceiverEndpoint != null)`
`245`	`245`	`{`
`246`	`246`	`Trace.Info($"Queueing results file ({filePath}) for attachment upload ({attachmentName})");`
`247`		`- var stepId = context.Id;`
	`247`	`+ var stepId = context.IsEmbedded ? context.EmbeddedId : context.Id;`
`248`	`248`	`// Attachments must be added to the parent context (job), not the current context (step)`
`249`	`249`	`context.Root.QueueSummaryFile(attachmentName, scrubbedFilePath, stepId);`
`250`	`250`	`}`