Skip to content

fix: Real support for .Net xUint test#67

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

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

Conversation

@Yumitoya8569
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.

@Yumitoya8569
Copy link
Copy Markdown
Author

@microsoft-github-policy-service agree

@Yumitoya8569
Copy link
Copy Markdown
Author

Yumitoya8569 commented May 8, 2026

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.

@JohnGalt1717
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?

@Yumitoya8569
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.

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.

2 participants