Skip to content

Skip tests on Appveyor due to PSReadline issue - #8317

Merged
Aditya Patwardhan (adityapatwardhan) merged 4 commits into
PowerShell:masterfrom
adityapatwardhan:FixEnterPSHostProcessTests
Nov 20, 2018
Merged

Skip tests on Appveyor due to PSReadline issue#8317
Aditya Patwardhan (adityapatwardhan) merged 4 commits into
PowerShell:masterfrom
adityapatwardhan:FixEnterPSHostProcessTests

Conversation

@adityapatwardhan

@adityapatwardhan Aditya Patwardhan (adityapatwardhan) commented Nov 20, 2018

Copy link
Copy Markdown
Member

PR Summary

Enter-PSHostProcess tests fail on AppVeyor due to PSReadline issue. This change skips the tests on AppVeyor.

Expected 3948, but got @(<empty>, 'Oops, something went wrong.  Please report this bug with the details below.', 'Report on GitHub: https://github.com/lzybkr/PSReadLine/issues/new', '-----------------------------------------------------------------------', 'Last 0 Keys:', '```', <empty>, '```', <empty>, 'Exception:').

PR Checklist

@adityapatwardhan Aditya Patwardhan (adityapatwardhan) changed the title [Feature] Skip tests on Appveyor due to PSReadline issue Skip tests on Appveyor due to PSReadline issue Nov 20, 2018
@anmenaga

Copy link
Copy Markdown

Something is not right; CI's are failing for this change.

Comment thread test/powershell/Modules/Microsoft.PowerShell.Core/Enter-PSHostProcess.Tests.ps1 Outdated
Comment thread test/powershell/Modules/Microsoft.PowerShell.Core/Enter-PSHostProcess.Tests.ps1 Outdated
@adityapatwardhan
Aditya Patwardhan (adityapatwardhan) merged commit f27dfea into PowerShell:master Nov 20, 2018
@adityapatwardhan
Aditya Patwardhan (adityapatwardhan) deleted the FixEnterPSHostProcessTests branch November 20, 2018 18:17
@adityapatwardhan Aditya Patwardhan (adityapatwardhan) added the CL-Test Indicates that a PR should be marked as a test change in the Change Log label Nov 21, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CL-Test Indicates that a PR should be marked as a test change in the Change Log

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants