Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

fix: Real support for .Net xUint test#67

Open
Yumitoya8569 wants to merge 5 commits into
microsoft:main from
Yumitoya8569:main
Open

fix: Real support for .Net xUint test #67
Yumitoya8569 wants to merge 5 commits into
microsoft:main from
Yumitoya8569:main

Conversation

@Yumitoya8569

@Yumitoya8569 Yumitoya8569 commented May 7, 2026

Copy link
Copy Markdown

Current progress

The current version (1.1.3) does not work when running C# xUnit tests at all.
The reason is that the test process is not attached to the debugger.
This PR uses a workaround to attach it.

Follow-up

Currently, it uses a presumed name (testhost) to scan processes and obtain the PID.
There should be a better approach in the future.

Copy link
Copy Markdown
Author

@microsoft-github-policy-service agree

Yumitoya8569 commented May 8, 2026
edited
Loading

Copy link
Copy Markdown
Author

Update

  1. Fixed an issue where the server would fail to start if the .csproj and .cs files were not in the same directory.
  2. Fixed an issue where continuous requests from the agent would cause the server to crash.

Follow-up

Continuous requests cause the server to crash due to the transport implementation.
Additionally, the singleton stateless mode described in the StreamableHTTPServerTransport doc does not actually work.
So for now, just fixed it with a workaround.
There should be a better approach in the future.

Copy link
Copy Markdown

It would be nice if it just worked for all tests in the vs code test harness to be able to debug a given tests. It shouldn't care about which test framework is used.

Does this work?

Copy link
Copy Markdown
Author

It would be nice if it just worked for all tests in the vs code test harness to be able to debug a given tests. It shouldn't care about which test framework is used.

Does this work?

Basically, it runs based on dotnet test, so it shouldn't be bound to any test framework, but I haven't tested it on other test frameworks yet.

JohnGalt1717 reacted with thumbs up emoji

Copy link
Copy Markdown

Great! Is there a way to get the AI to know to use the actual test harness so the tests show in realtime instead of kicking it off out of band? (or does it already do that with what you're doing?)

Copy link
Copy Markdown
Author

Great! Is there a way to get the AI to know to use the actual test harness so the tests show in realtime instead of kicking it off out of band? (or does it already do that with what you're doing?)

This is unrelated to whether the AI knows the test framework or not.
It works just like the other parts of the project.
This PR only fixes the issue where dotnet test was not working.
If your test framework is supported by dotnet test and the C# Dev Kit extension, it should work.

JohnGalt1717 reacted with thumbs up emoji

Copy link
Copy Markdown
Author

@ozzafar

Has this issue already been fixed by #78?
Should I close this?

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

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

AltStyle によって変換されたページ (->オリジナル) /