Skip to content

./x.py test --pass check fails #66929

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Closed
RalfJung opened this issue Dec 1, 2019 · 4 comments · Fixed by #66932
Closed

./x.py test --pass check fails #66929

RalfJung opened this issue Dec 1, 2019 · 4 comments · Fixed by #66932
Assignees
Labels
A-testsuite Area: The testsuite used to check the correctness of rustc C-bug Category: This is a bug. P-high High priority T-compiler Relevant to the compiler team, which will review and decide on the PR/issue.

Comments

@RalfJung
Copy link
Member

RalfJung commented Dec 1, 2019

The following command fails on master:

./x.py -j4 test --stage 1 src/test/ui --test-args issue-65419 --pass check

says

[...]
---- [ui] ui/async-await/issues/issue-65419/issue-65419-async-fn-resume-after-panic.rs stdout ----

error: error pattern ' thread 'main' panicked at '`async fn` resumed after panicking'' not found!
status: exit code: 0
command: "/home/r/src/rust/rustc.2/build/x86_64-unknown-linux-gnu/stage1/bin/rustc" "/home/r/src/rust/rustc.2/src/test/ui/async-await/issues/issue-65419/issue-65419-async-fn-resume-after-panic.rs" "-Zthreads=1" "--target=x86_64-unknown-linux-gnu" "--error-format" "json" "-Zui-testing" "--emit" "metadata" "-C" "prefer-dynamic" "--out-dir" "/home/r/src/rust/rustc.2/build/x86_64-unknown-linux-gnu/test/ui/async-await/issues/issue-65419/issue-65419-async-fn-resume-after-panic" "-Crpath" "-O" "-Cdebuginfo=0" "-Zunstable-options" "-Lnative=/home/r/src/rust/rustc.2/build/x86_64-unknown-linux-gnu/native/rust-test-helpers" "--edition=2018" "-L" "/home/r/src/rust/rustc.2/build/x86_64-unknown-linux-gnu/test/ui/async-await/issues/issue-65419/issue-65419-async-fn-resume-after-panic/auxiliary" "-A" "unused"
stdout:
------------------------------------------

------------------------------------------
stderr:
------------------------------------------

------------------------------------------




failures:
    [ui] ui/async-await/issues/issue-65419/issue-65419-async-fn-resume-after-completion.rs
    [ui] ui/async-await/issues/issue-65419/issue-65419-async-fn-resume-after-panic.rs
    [ui] ui/async-await/issues/issue-65419/issue-65419-generator-resume-after-completion.rs

Omitting the --pass check fixes that. The offending tests are all run-fail tests with an error-pattern, looks like those don't work properly with --pass check?

Cc @Mark-Simulacrum

@petrochenkov
Copy link
Contributor

cc @ninjasource #66321
cc @nvlbg #65506
Please, don't move run-fail tests to UI until this is addressed.

cc #65865

@jonas-schievink jonas-schievink added A-testsuite Area: The testsuite used to check the correctness of rustc C-bug Category: This is a bug. labels Dec 1, 2019
@Centril Centril added P-high High priority T-compiler Relevant to the compiler team, which will review and decide on the PR/issue. labels Dec 1, 2019
@ninjasource
Copy link

ninjasource commented Dec 1, 2019

Hi @petrochenkov, Should I create a PR to move the issue-65419 tests back to src/test/run-fail? Happy to do so.

@Centril
Copy link
Contributor

Centril commented Dec 1, 2019

I have a possible fix in #66932.

@Centril Centril self-assigned this Dec 1, 2019
@ninjasource
Copy link

@Centril, great. Let me know if it doesn't work out and I'll submit my change as an alternative fix.

@bors bors closed this as completed in 6e9172f Dec 22, 2019
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
A-testsuite Area: The testsuite used to check the correctness of rustc C-bug Category: This is a bug. P-high High priority T-compiler Relevant to the compiler team, which will review and decide on the PR/issue.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants