From a05b1f511852d945fdb80f34b4f13ce17e10b20f Mon Sep 17 00:00:00 2001 From: "copilot-swe-agent[bot]" <198982749+Copilot@users.noreply.github.com> Date: Thu, 9 Apr 2026 04:04:39 +0000 Subject: [PATCH] fix: update step name check in step order validation test to be exact Agent-Logs-Url: https://github.com/github/gh-aw/sessions/05417776-28f4-49a8-bbbc-8f9fffc0d37b Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com> --- pkg/workflow/step_order_validation_integration_test.go | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/pkg/workflow/step_order_validation_integration_test.go b/pkg/workflow/step_order_validation_integration_test.go index 66ecffc0cf7..92b99479515 100644 --- a/pkg/workflow/step_order_validation_integration_test.go +++ b/pkg/workflow/step_order_validation_integration_test.go @@ -66,8 +66,9 @@ This workflow has a secret reference and safe-outputs. } // Upload Safe Outputs is now merged into the unified agent artifact; verify - // the old separate step no longer appears. - if strings.Contains(contentStr, "name: Upload Safe Outputs") { + // the old separate step no longer appears. Use exact matching (with leading "- " and + // trailing newline) to avoid false positives on steps like "Upload Safe Outputs Items". + if strings.Contains(contentStr, "- name: Upload Safe Outputs\n") { t.Error("Upload Safe Outputs should be removed (merged into unified agent artifact)") }