dotnet / upgrade-assistant

A tool to assist developers in upgrading .NET Framework applications to .NET 6 and beyond
MIT License
1.11k stars 163 forks source link

Controller with a method changed to Async makes it appear not migrated in the endpoint explorer #1592

Open sfabriece opened 3 months ago

sfabriece commented 3 months ago

Describe the bug

I would expect changing a method to async in a migrated controller to not cause it to be treated as not migrated in the upgrade assistant endpoints dashboard

To Reproduce

Migrate a controller from a .net framework 4.8.1 to a NET 8 mvc app.

Change the index method from public ActionResult Index() to public async Task<IActionResult> Index().

Go to the upgrade assistant dashboard and look at the endpoint explorer. The method is shown as not migrated.

Exceptions (if any)

Further technical details

6.0.424 [C:\Program Files\dotnet\sdk]
7.0.317 [C:\Program Files\dotnet\sdk]
8.0.303 [C:\Program Files\dotnet\sdk]