-
Notifications
You must be signed in to change notification settings - Fork 288
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
[pipeline/e2e] end-to-end-tests.ps1 should fail with failure test #1518
[pipeline/e2e] end-to-end-tests.ps1 should fail with failure test #1518
Conversation
86eeba9
to
7755480
Compare
adf8f06
to
29bce0b
Compare
TestExample of un-expected pass before fixed Example of failed as expected after fixed
Example of error message
|
35a4312
to
4ecab71
Compare
As far as I can tell it already does this. Do you have a repro for it not doing so? (EDIT: I see you have an example in a comment here but it's not clear how you got that to happen) |
It looks like the repro might only happen with something like this "Pester" module ( as in #1512 ) because all other existing e2e tests fail by throwing an exception, thus stopping further test runs. |
Sorry I forgot to attach the test commit 7755480 Run this script with BeforeAll {
Import-Module $PSScriptRoot/not-exist/posh-vcpkg
}
Describe 'Module posh-vcpkg tests' {
It 'Expect fail' {}
} This issue has been fixed by:
|
I'm now trying to fix the error message Update: Fixed by adding P.S. The command |
4ecab71
to
8c9ed70
Compare
Unable change runner machine It hangs with PR check:
Update: It might be waiting the branch protection As a transitional, add arm64 https://github.com/actions/runner-images?tab=readme-ov-file#available-images Update 2: |
d9d0cba
to
50b6d36
Compare
Note
https://devblogs.microsoft.com/devops/upcoming-deprecation-of-macos-12-hosted-pipeline-image/ |
50b6d36
to
d214a44
Compare
ae8ffd4
to
927019e
Compare
927019e
to
5acf856
Compare
TestExpected fail with failed test case https://github.com/microsoft/vcpkg-tool/actions/runs/11925979891 |
Thanks! |
Fix
end-to-end-tests.ps1
, which should fail and stop with failure test case.Change
$LASTEXITCODE
$LASTEXITCODE
forThrow-IfNotFailed
autocomplete-command.ps1
Throw-IfNotFailed
for non-windowsRemove-Problem-Matchers
for malformed portaction/checkout@4
,actions/setup-node@v4
,lukka/[email protected]