Skip to content

Add coverage for parameterized Generate Method#13035

Merged
davidwengier merged 2 commits intodotnet:mainfrom
davidwengier:davidwengier/enable-generate-parameterized-method
Apr 15, 2026
Merged

Add coverage for parameterized Generate Method#13035
davidwengier merged 2 commits intodotnet:mainfrom
davidwengier:davidwengier/enable-generate-parameterized-method

Conversation

@davidwengier
Copy link
Copy Markdown
Member

Summary

  • add GenerateMethodTests coverage for method invocations with parameters inside @code
  • add matching implicit-expression coverage to confirm the parameterized Roslyn path is exercised

Testing

  • dotnet test src\Razor\test\Microsoft.VisualStudio.LanguageServices.Razor.Test\Microsoft.VisualStudio.LanguageServices.Razor.Test.csproj --filter "FullyQualifiedName~GenerateMethod"
  • dotnet test src\Razor\test\Microsoft.VisualStudioCode.RazorExtension.Test\Microsoft.VisualStudioCode.RazorExtension.Test.csproj --filter "FullyQualifiedName~GenerateMethod"

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@davidwengier davidwengier requested a review from a team as a code owner April 10, 2026 04:46
@davidwengier
Copy link
Copy Markdown
Member Author

When I looked at Roslyn I thought "Generate Parameterized Method" was a separate code action that would have to be allow-listed, but its reported as just a plain Generate Method action, so it was already working. Just needed some test coverage.

@davidwengier davidwengier enabled auto-merge April 15, 2026 03:37
@davidwengier
Copy link
Copy Markdown
Member Author

Without the Roslyn insertion that preceeded this PR, these tests won't run, but it's still worth adding, so have updated them to be skipped. We can unskip all of the generate method tests at once post repo merge.

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.

3 participants