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
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
,.ps1
and.psm1
files have the correct copyright headerWIP:
or[ WIP ]
to the beginning of the title (theWIP
bot will keep its status check atPending
while the prefix is present) and remove the prefix when the PR is ready.(which runs in a different PS Host).