search

microsoft / vstest

Visual Studio Test Platform is the runner and engine that powers test explorer and vstest.console.
MIT License
870 stars 316 forks source link

InProcessVsTestConsoleWrapper creates a VsTestConsoleProcessManager but doesn't Dispose() it #5117

Open omajid opened 1 week ago

omajid commented 1 week ago

https://github.com/microsoft/vstest/blob/5b69fd31169dd07ced917329bbb483f3b73ea98f/src/vstest.console/InProcessVsTestConsoleWrapper.cs#L129

The VsTestConsoleProcessManager is an IDisposable, but it's never disposed in this piece of code.

Does it need to be Dispose()d or is it safe to ignore?