-
Notifications
You must be signed in to change notification settings - Fork 8.1k
Test fixes for stabilizing tests #19068
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
Test fixes for stabilizing tests #19068
Conversation
| <Reference Include="System.Management.Automation"> | ||
| <HintPath>.\Ref\System.Management.Automation.dll</HintPath> | ||
| </Reference> | ||
| <PackageReference Include="System.Management.Automation" Version="7.4.0-preview.1" /> |
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.
Should this be automated in some way?
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.
Good point. I will add this csproj to the dependabot monitored csproj files
d8e4c28 to
dee0a95
Compare
|
This PR is under test. Commenting so bot doesnt close the PR. |
dee0a95 to
be30e42
Compare
|
This pull request has been automatically marked as stale because it has been marked as requiring author feedback but has not had any activity for 15 days. It will be closed if no further activity occurs within 10 days of this comment. |
test/powershell/Modules/Microsoft.PowerShell.Security/CertificateProvider.Tests.ps1
Outdated
Show resolved
Hide resolved
test/powershell/Modules/Microsoft.PowerShell.Security/CertificateProvider.Tests.ps1
Outdated
Show resolved
Hide resolved
test/powershell/Modules/Microsoft.PowerShell.Security/CertificateProvider.Tests.ps1
Outdated
Show resolved
Hide resolved
test/powershell/Modules/Microsoft.PowerShell.Security/CertificateProvider.Tests.ps1
Outdated
Show resolved
Hide resolved
test/powershell/engine/ExperimentalFeature/Get-ExperimentalFeature.Tests.ps1
Outdated
Show resolved
Hide resolved
f76df1a to
9759db7
Compare
|
Removed the change adding the asset and also the change setting timeout for the pipeline task |
|
This PR has Quantification details
Why proper sizing of changes matters
Optimal pull request sizes drive a better predictable PR flow as they strike a
What can I do to optimize my changes
How to interpret the change counts in git diff output
Was this comment helpful? 👍 :ok_hand: :thumbsdown: (Email) |
|
🎉 Handy links: |
|
🎉 Handy links: |
|
🎉 Handy links: |
PR Summary
Make tests more robust when executing on various platforms.
PR Context
PR Checklist
.h,.cpp,.cs,.ps1and.psm1files have the correct copyright headerWIP:or[ WIP ]to the beginning of the title (theWIPbot will keep its status check atPendingwhile the prefix is present) and remove the prefix when the PR is ready.(which runs in a different PS Host).