Cleanup PipeNameDescription

[Youssef1313]

This class is sealed and the current dispose implementation is unnecessarily complicated. The Dispose(bool) method is only relevant in non-sealed classes.

Corresponding testfx PR was merged as well: microsoft/testfx#6000

[Copilot]

Pull Request Overview

This PR simplifies the disposal pattern in the sealed PipeNameDescription class by removing the unused Dispose(bool) overload and streamlining the Dispose() method.

• Removed the redundant Dispose(bool disposing) overload and its call from Dispose().
• Simplified Dispose() to only check the disposal flag and handle the directory cleanup logic.

Comments suppressed due to low confidence (1)

src/Cli/dotnet/Commands/Test/IPC/PipeNameDescription.cs:15

• The Dispose method no longer sets the _disposed flag, so repeated calls will re-run the disposal logic. Add _disposed = true; after the cleanup to prevent multiple disposals.

    public void Dispose()

[Youssef1313]

/azp run

[azure-pipelines]

Azure Pipelines successfully started running 1 pipeline(s).