Skip to content

Support multiple test names in /create-issue and fix underscore escaping in titles#16059

Draft
Copilot wants to merge 3 commits intomainfrom
copilot/support-multiple-test-names
Draft

Support multiple test names in /create-issue and fix underscore escaping in titles#16059
Copilot wants to merge 3 commits intomainfrom
copilot/support-multiple-test-names

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Apr 11, 2026

Description

The /create-issue command only supported a single test name per invocation, and issue titles contained unnecessary backslash-escaped underscores (e.g., Exec\_NginxContainer\_ListFiles) because EscapeMarkdownInline was applied to titles, which are plain text, not markdown.

Changes

  • JS parseCommand: Returns testQueries array. Accepts multiple --test flags and multiple positional args. Keeps testQuery (first element) for backward compat.
    /create-issue TestA TestB TestC
    /create-issue --test TestA --test TestB --test TestC
    /create-issue TestA TestB https://github.com/.../pull/123
    
  • Workflow YAML: Extract step outputs test_queries JSON array. Resolve step loops via jq, producing indexed result files. Create/update step processes each result independently and posts a combined summary comment.
  • C# title fix: Removed EscapeMarkdownInline() from issue title in CreateIssueArtifacts — GitHub issue titles are plain text.
  • Tests: 3 new tests for multi-test parsing; existing tests updated to assert testQueries array.

Mixing positional names with --test flags remains rejected as ambiguous.

Checklist

  • Is this feature complete?
    • Yes. Ready to ship.
    • No. Follow-up changes expected.
  • Are you including unit tests for the changes and scenario tests if relevant?
    • Yes
    • No
  • Did you add public API?
    • Yes
    • No
  • Does the change make any security assumptions or guarantees?
    • Yes
    • No
  • Does the change require an update in our Aspire docs?
    • Yes
    • No

Warning

Firewall rules blocked me from connecting to one or more addresses (expand for details)

I tried to connect to the following addresses, but was blocked by firewall rules:

  • dnceng.pkgs.visualstudio.com
    • Triggering command: /usr/share/dotnet/dotnet dotnet build tests/Infrastructure.Tests/Infrastructure.Tests.csproj /p:SkipNativeBuild=true (dns block)
    • Triggering command: /usr/share/dotnet/dotnet dotnet test tests/Infrastructure.Tests/Infrastructure.Tests.csproj /p:SkipNativeBuild=true -- --filter-class *.CreateFailingTestIssueWorkflowTests --filter-not-trait quarantined=true --filter-not-trait outerloop=true (dns block)
    • Triggering command: /usr/share/dotnet/dotnet dotnet build tools/CreateFailingTestIssue -v:q (dns block)

If you need me to access, download, or install something from one of these locations, you can either:

…ing in titles

- JS parseCommand: support multiple --test flags and multiple positional test names
- Workflow YAML: loop over test queries, process each result individually
- C# CreateIssueArtifacts: remove EscapeMarkdownInline from title (plain text, not markdown)
- Update help block to show multi-test usage patterns
- Add tests for multiple --test flags and multiple positional args

Agent-Logs-Url: https://github.com/microsoft/aspire/sessions/fe78d876-9255-42a5-b16f-5f9155469e80

Co-authored-by: radical <1472+radical@users.noreply.github.com>
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Apr 11, 2026

🚀 Dogfood this PR with:

⚠️ WARNING: Do not do this without first carefully reviewing the code of this PR to satisfy yourself it is safe.

curl -fsSL https://raw.githubusercontent.com/microsoft/aspire/main/eng/scripts/get-aspire-cli-pr.sh | bash -s -- 16059

Or

  • Run remotely in PowerShell:
iex "& { $(irm https://raw.githubusercontent.com/microsoft/aspire/main/eng/scripts/get-aspire-cli-pr.ps1) } 16059"

Copilot AI changed the title [WIP] Support multiple test names and fix underscore escaping in issue titles Support multiple test names in /create-issue and fix underscore escaping in titles Apr 11, 2026
Copilot AI requested a review from radical April 11, 2026 05:49
@radical
Copy link
Copy Markdown
Member

radical commented Apr 11, 2026

/create-issue Aspire.Cli.EndToEnd.Tests.PythonReactTemplateTests.CreateAndRunPythonReactProject

@github-actions
Copy link
Copy Markdown
Contributor

❌ The workflow run did not contain any failed test results in downloadable .trx artifacts.

@github-actions
Copy link
Copy Markdown
Contributor

❌ The /create-issue command failed. See the workflow run for details.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

/create-issue: support multiple test names and fix underscore escaping in issue titles

2 participants