-
Notifications
You must be signed in to change notification settings - Fork 217
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Enable PsesInternalHostTests
(previously PowerShellContextTests
)
#1651
Conversation
Oops, gotta fix it for Windows 🤷 |
7c1a5cc
to
cceedab
Compare
All fixed! |
Hey @JustinGrote I covered #1629 with a regression test and cherry-picked the commit. Can you review this? We can close the other PR. |
c03154e
to
76a5791
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM! + one question
src/PowerShellEditorServices/Services/PowerShell/Host/PsesInternalHost.cs
Outdated
Show resolved
Hide resolved
76a5791
to
0903b6a
Compare
Co-authored-by: Andy Schwartzmeyer <[email protected]>
0903b6a
to
57bdb36
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
Part of #1677 |
I was fortunately able to re-enable the
PowerShellContextService
tests with the newPsesInternalHost
pretty easily.Also fixes PowerShell/vscode-powershell#3677.