Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Tests with #[should_panic] should not be able to return Result #4376

Open
daxpedda opened this issue Dec 22, 2024 · 0 comments
Open

Tests with #[should_panic] should not be able to return Result #4376

daxpedda opened this issue Dec 22, 2024 · 0 comments
Labels
breaking-change Tracking breaking changes for the next major version bump (if ever) enhancement

Comments

@daxpedda
Copy link
Collaborator

To align with cargo test, we should not allow returning Result from a test that is marked with #[should_panic].
@daxpedda daxpedda added breaking-change Tracking breaking changes for the next major version bump (if ever) enhancement labels Dec 22, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
breaking-change Tracking breaking changes for the next major version bump (if ever) enhancement
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@daxpedda