-
Notifications
You must be signed in to change notification settings - Fork 8.3k
Do we need Enable-RunspaceDebug.Tests.ps1 file? #13470
Copy link
Copy link
Closed
Labels
Issue-Questionideally support can be provided via other mechanisms, but sometimes folks do open an issue to get aideally support can be provided via other mechanisms, but sometimes folks do open an issue to get aResolution-FixedThe issue is fixed.The issue is fixed.Up-for-GrabsUp-for-grabs issues are not high priorities, and may be opportunities for external contributorsUp-for-grabs issues are not high priorities, and may be opportunities for external contributorsWG-Quality-Testissues in a test or in test infrastructureissues in a test or in test infrastructure
Metadata
Metadata
Assignees
Labels
Issue-Questionideally support can be provided via other mechanisms, but sometimes folks do open an issue to get aideally support can be provided via other mechanisms, but sometimes folks do open an issue to get aResolution-FixedThe issue is fixed.The issue is fixed.Up-for-GrabsUp-for-grabs issues are not high priorities, and may be opportunities for external contributorsUp-for-grabs issues are not high priorities, and may be opportunities for external contributorsWG-Quality-Testissues in a test or in test infrastructureissues in a test or in test infrastructure
Do we need
Enable-RunspaceDebug.Tests.ps1file?https://github.com/PowerShell/PowerShell/blob/master/test/powershell/Modules/Microsoft.PowerShell.Utility/Enable-RunspaceDebug.Tests.ps1 was added as part of #8923, then in #10338 the functionality it tests was removed.
When run, its tests are skipped because
Microsoft.PowerShell.Utility.PSDebugRunspaceWithBreakpointsexperimental feature doesn't exist, or else the tests would fail.PowerShell/test/powershell/Modules/Microsoft.PowerShell.Utility/Enable-RunspaceDebug.Tests.ps1
Lines 9 to 14 in 904e551
Should we remove the file?
Steps to reproduce
Expected behavior
Actual behavior
Environment data