Skip to content

Miri tests: skip parts of test_char_range #72812

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

Merged
merged 1 commit into from
Jun 1, 2020

Conversation

RalfJung
Copy link
Member

The new test_char_range test takes forever in Miri as it loops over all values of char. This makes it skip most of them when being run in Miri.

@rust-highfive
Copy link
Contributor

r? @LukasKalbertodt

(rust_highfive has picked a reviewer for you, use r? to override)

@rust-highfive rust-highfive added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label May 31, 2020
@jonas-schievink
Copy link
Contributor

@bors r+ rollup

@bors
Copy link
Collaborator

bors commented May 31, 2020

📌 Commit 532dabd has been approved by jonas-schievink

@bors bors added S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels May 31, 2020
@CAD97
Copy link
Contributor

CAD97 commented May 31, 2020

Sorry miri 😅

bors added a commit to rust-lang-ci/rust that referenced this pull request May 31, 2020
Rollup of 5 pull requests

Successful merges:

 - rust-lang#72691 (Fix escape key handling)
 - rust-lang#72807 (Avoid setting wrong obligation cause span of associated type mismatch)
 - rust-lang#72812 (Miri tests: skip parts of test_char_range)
 - rust-lang#72829 (Clarify terms in doc comments)
 - rust-lang#72830 (Fix release notes for niche initialization change)

Failed merges:

r? @ghost
@bors bors merged commit 5480120 into rust-lang:master Jun 1, 2020
@RalfJung RalfJung deleted the miri-char-test branch June 1, 2020 22:47
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants