Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[BUG] Pressing the restart icon during a debugging session for a c# test fails with an error #904

Open
asilverman opened this issue Feb 8, 2024 · 9 comments
Labels
area-test Test discovery, execution, debugging enhancement New feature or request mac MacOS
Milestone

Comments

@asilverman
Copy link

Describe the Issue

In the screen recording below I am running a test in debug mode, I step through the test and then decide to restart the test, however the operation fails an VSCode says it couldn't find the process id
https://github.com/microsoft/vscode-dotnettools/assets/9611108/1f6ffbf2-ea4b-4bd9-932e-21817f4f0909

Steps To Reproduce

  1. Clone azure/bicep git repo
  2. run any test in debug from the test explorer
  3. press the restart debugger button

Expected Behavior

I would expect the test to restart and run again, breaking in the first breakpoint. It instead fails with an error message

Environment Information

  • OS: MacOS
  • ms-dotnettools.csdevkit version: v1.4.2
@asilverman asilverman added the bug Something isn't working label Feb 8, 2024
@microsoft-issue-labeler microsoft-issue-labeler bot added the area-dotnet-debugging Issues around .NET Debugging with console, ASP.NET, or Aspire label Feb 8, 2024
@WardenGnaw WardenGnaw added area-test Test discovery, execution, debugging and removed area-dotnet-debugging Issues around .NET Debugging with console, ASP.NET, or Aspire labels Feb 9, 2024
@WardenGnaw
Copy link
Member

The restart button will just relaunch with the same launch.json configuration provided. There needs to be a change in the test code to attach to the correct PID on restart.

@ocallesp ocallesp added the mac MacOS label Mar 7, 2024
Copy link

github-actions bot commented Apr 3, 2024

This issue has been marked as stale after 14 days of inactivity. @[ @ @], could you please take a look?

@asilverman
Copy link
Author

I would love to hear back on this

@peterwald peterwald added enhancement New feature or request and removed bug Something isn't working labels Apr 30, 2024
@peterwald
Copy link
Member

This is currently not supported in C# Dev Kit. We are evaluating whether to add restart functionality in a future release.

@peterwald peterwald added this to the Backlog milestone Apr 30, 2024
@asilverman
Copy link
Author

Ok, that is acceptable. I hope it's not too much trouble to add it

@asilverman
Copy link
Author

Hi @ocallesp, @peterwald any update on this bug?

@peterwald
Copy link
Member

It's on our radar, but unfortunately no updates to report at this time.

@asilverman
Copy link
Author

Hi @peterwald, it's been many months since I am experiencing this issue and would really appreciate if you can assist, it seems odd to me that this generally available feature for other languages simply doesn't work in C#

@AbhitejJohn
Copy link

@asilverman : We sadly have not been able to prioritize enhancements similar to this on Test Explorer given few other priorities(reliability in particular in the Test Explorer space). However, if we do hear back from more customers that this is required, we'd be happy to re-prioritize. At the moment, this is on our backlog and we'll pull it into one of the active milestones if this surfaces higher as a priority.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area-test Test discovery, execution, debugging enhancement New feature or request mac MacOS
Projects
None yet
Development

No branches or pull requests

5 participants