Skip to content

Enable skipped tests#12894

Merged
TravisEz13 merged 4 commits intoPowerShell:masterfrom
iSazonov:tests-linux-enable
Jun 5, 2020
Merged

Enable skipped tests#12894
TravisEz13 merged 4 commits intoPowerShell:masterfrom
iSazonov:tests-linux-enable

Conversation

@iSazonov
Copy link
Copy Markdown
Collaborator

@iSazonov iSazonov commented Jun 4, 2020

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

@TravisEz13 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
Copy link
Copy Markdown
Collaborator Author

iSazonov commented Jun 4, 2020

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

@iSazonov iSazonov force-pushed the tests-linux-enable branch from baed45e to 663cc5c Compare June 4, 2020 17:20
@TravisEz13
Copy link
Copy Markdown
Member

@PoshChan please rerun windows

@PoshChan
Copy link
Copy Markdown
Collaborator

PoshChan commented Jun 4, 2020

@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 iSazonov closed this Jun 5, 2020
@iSazonov iSazonov reopened this Jun 5, 2020
@iSazonov iSazonov force-pushed the tests-linux-enable branch from 62f7a5d to 17ecd02 Compare June 5, 2020 10:25
@iSazonov
Copy link
Copy Markdown
Collaborator Author

iSazonov commented Jun 5, 2020

@TravisEz13 @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.

@iSazonov iSazonov force-pushed the tests-linux-enable branch from 17ecd02 to 5347689 Compare June 5, 2020 10:48
@iSazonov iSazonov force-pushed the tests-linux-enable branch from 5347689 to 1da45b3 Compare June 5, 2020 11:23
@TravisEz13
Copy link
Copy Markdown
Member

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

@TravisEz13 TravisEz13 added this to the 7.1.0-preview.4 milestone Jun 5, 2020
@iSazonov 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

ghost commented Jun 25, 2020

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

Handy links:

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