Skip to content

Enable skipped tests - #12894

Merged
Travis Plunk (TravisEz13) merged 4 commits into
PowerShell:masterfrom
iSazonov:tests-linux-enable
Jun 5, 2020
Merged

Enable skipped tests#12894
Travis Plunk (TravisEz13) merged 4 commits into
PowerShell:masterfrom
iSazonov:tests-linux-enable

Conversation

@iSazonov

@iSazonov Ilya (iSazonov) commented Jun 4, 2020

Copy link
Copy Markdown
Collaborator

PR Summary

PR Context

I discovered that CIs skip silently many tests which run after the Clipboard tests.

I have more general concern - I saw that tests was skipped on Linux but not on Windows and MacOS and I don't understand how this can be. My concern is that there is a bug. I hope MSFT can investigate this.

PR Checklist

Comment thread test/powershell/Modules/Microsoft.PowerShell.Management/Clipboard.Tests.ps1 Outdated
Comment thread test/powershell/Modules/Microsoft.PowerShell.Management/Clipboard.Tests.ps1 Outdated
Comment thread test/powershell/Modules/Microsoft.PowerShell.Management/Clipboard.Tests.ps1 Outdated
Comment thread test/powershell/Modules/Microsoft.PowerShell.Management/Clipboard.Tests.ps1 Outdated
@TravisEz13 Travis Plunk (TravisEz13) added the CL-BuildPackaging Indicates that a PR should be marked as a build or packaging change in the Change Log label Jun 4, 2020
@TravisEz13

Copy link
Copy Markdown
Member

Please investigate test failures

@iSazonov

Ilya (iSazonov) commented Jun 4, 2020

Copy link
Copy Markdown
Collaborator Author

Travis Plunk (@TravisEz13) two tests fail in Get-Process.Tests.ps1 - I did not find when we ruined it and how to fix it.

Comment thread test/powershell/Modules/Microsoft.PowerShell.Management/Clipboard.Tests.ps1 Outdated
@TravisEz13

Copy link
Copy Markdown
Member

PoshChan-Bot (@PoshChan) please rerun windows

@PoshChan

Copy link
Copy Markdown
Collaborator

Travis Plunk (@TravisEz13), successfully started rebuild of PowerShell-CI-Windows

@TravisEz13

Copy link
Copy Markdown
Member

/azp help

@azure-pipelines

This comment has been minimized.

@TravisEz13

Copy link
Copy Markdown
Member

/azp run powershell-ci-macos

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines successfully started running 1 pipeline(s).

@TravisEz13

Copy link
Copy Markdown
Member

/azp run powershell-ci-linux

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines successfully started running 1 pipeline(s).

@iSazonov

Copy link
Copy Markdown
Collaborator Author

Travis Plunk (@TravisEz13) Steve Lee (@SteveL-MSFT) I found why Linux tests fail. It is dotnet/runtime#33673.
I hope we get the fix in next .Net 5 Preview. While I put the tests in pending state.

@TravisEz13

Copy link
Copy Markdown
Member

Please file an issue to track re-enabling the tests.

@iSazonov
Ilya (iSazonov) deleted the tests-linux-enable branch June 6, 2020 02:42
@rajdeep-biswas

Copy link
Copy Markdown

I have found a solution to #12908. I would like to contribute.

@ghost

Copy link
Copy Markdown

🎉v7.1.0-preview.4 has been released which incorporates this pull request.:tada:

Handy links:

Thatgfsj (Thatgfsj) pushed a commit to Thatgfsj/PowerShell that referenced this pull request Aug 6, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CL-BuildPackaging Indicates that a PR should be marked as a build or packaging change in the Change Log

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants