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.

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

Add Expected Should Panic Message to Tests #397

Merged
merged 3 commits into from
Sep 5, 2023

Conversation

da2ce7
Copy link
Contributor

@da2ce7 da2ce7 commented Sep 5, 2023

closes #391

@da2ce7 da2ce7 changed the title 20230905 clippy work Add Expected Should Panic Message to Tests Sep 5, 2023
@codecov
Copy link

codecov bot commented Sep 5, 2023

Codecov Report

Merging #397 (efd1c99) into develop (8b48b0c) will decrease coverage by 0.10%.
The diff coverage is 100.00%.

@@             Coverage Diff             @@
##           develop     #397      +/-   ##
===========================================
- Coverage    84.07%   83.98%   -0.10%     
===========================================
  Files          102      102              
  Lines         7185     7182       -3     
===========================================
- Hits          6041     6032       -9     
- Misses        1144     1150       +6     
Files Changed Coverage Δ
src/servers/udp/connection_cookie.rs 100.00% <ø> (ø)
contrib/bencode/src/reference/decode.rs 96.20% <100.00%> (-0.02%) ⬇️
src/tracker/peer.rs 67.27% <100.00%> (+0.14%) ⬆️

... and 8 files with indirect coverage changes

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@da2ce7
Copy link
Contributor Author

da2ce7 commented Sep 5, 2023

ACK efd1c99

@da2ce7 da2ce7 merged commit 94aca98 into torrust:develop Sep 5, 2023
9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

Add Expected Message to #[should_panic] tests.
1 participant