Skip to content

Conversation

@CXWorks
Copy link

@CXWorks CXWorks commented Jul 14, 2023

Hi,

Thanks for your time & patience to review this PR.

We are researchers focusing on Rust unit tests. By examine the existing code, we found a unit test can be added to improve the repo's overall unit test coverage(this project is already been well tested).
For the first 4 cases, you can easily find their error locations and for the last case, the untested area is:
https://github.com/tailhook/humantime/blob/12ce6f50894a56a410b390e5608ac9db8afe2407/src/duration.rs#L181-L187
including both the error cases and correct path.

Thanks again for reviewing.

@djc
Copy link
Member

djc commented Mar 12, 2025

Thanks!

@djc djc added this pull request to the merge queue Mar 12, 2025
Merged via the queue into chronotope:main with commit 171a43d Mar 12, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants