-
Notifications
You must be signed in to change notification settings - Fork 13.4k
Allow checking of run-pass execution output in compiletest #63825
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
Conversation
Moved into ```fn load_compare_outputs(&self, proc_res: &ProcRes, explicit_format: bool) -> usize```
(rust_highfive has picked a reviewer for you, use r? to override) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks! A few comments, but this looks good overall.
457bfce
to
3dc3cf3
Compare
The job Click to expand the log.
I'm a bot! I can only do what humans tell me to, so if this was not helpful or you have suggestions for improvements, please ping or otherwise contact |
3dc3cf3
to
1c6b986
Compare
The job Click to expand the log.
I'm a bot! I can only do what humans tell me to, so if this was not helpful or you have suggestions for improvements, please ping or otherwise contact |
1c6b986
to
29cc305
Compare
The job Click to expand the log.
I'm a bot! I can only do what humans tell me to, so if this was not helpful or you have suggestions for improvements, please ping or otherwise contact |
29cc305
to
cdf3beb
Compare
📌 Commit cdf3beb178dd6ae8eabb7519b6b7007e8d30e19a has been approved by |
⌛ Testing commit cdf3beb178dd6ae8eabb7519b6b7007e8d30e19a with merge f970d3d042fcb718c2ab908190bf0cbbb47a8271... |
💔 Test failed - checks-azure |
The job Click to expand the log.
I'm a bot! I can only do what humans tell me to, so if this was not helpful or you have suggestions for improvements, please ping or otherwise contact |
The job Click to expand the log.
I'm a bot! I can only do what humans tell me to, so if this was not helpful or you have suggestions for improvements, please ping or otherwise contact |
💔 Test failed - checks-azure |
Hm, this makes them "pass" on that platform though, instead of being marked as "ignored" as it probably should. |
Yeah, I'd rather we didn't mark them as having passed. |
Okay, I'll go ahead and make that change. |
Ignores run-pass tests with the `check-run-results` flag enabled for the wasm32-bare ("wasm32-unknown-unknown") target, as it does not support printing to stdout/stderr.
e86d112
to
bf44f12
Compare
Yeah, I think it's a bit ambiguous as to what the correct behavior there is but overall I'd prefer to ignore the test entirely I think. We can of course change that at a later point. @bors r+ |
📌 Commit bf44f12 has been approved by |
Allow checking of run-pass execution output in compiletest Closes #63751 Adds a `check-run-results` flag to compiletest headers, which if enabled checks the output of the execution of a run-pass test's binary against expected output.
The job Click to expand the log.
I'm a bot! I can only do what humans tell me to, so if this was not helpful or you have suggestions for improvements, please ping or otherwise contact |
💔 Test failed - checks-azure |
The remote-test-client outputs a message of the form "uploaded "<build_dir>/<executable_path>", waiting for result" onto stdout when executing a test, which is then captured in the process result. This needs to be removed when comparing the results of the run-pass test execution.
The most recent commit should take care of the failing test, and allow for the PR to merge. |
@bors r+ |
📌 Commit 12adc39 has been approved by |
Allow checking of run-pass execution output in compiletest Closes #63751 Adds a `check-run-results` flag to compiletest headers, which if enabled checks the output of the execution of a run-pass test's binary against expected output.
☀️ Test successful - checks-azure |
Closes #63751
Adds a
check-run-results
flag to compiletest headers, which if enabled checks the output of the execution of a run-pass test's binary against expected output.