Skip to content

Add a test case for decode_slice_unchecked panic#284

Open
AbeZbm wants to merge 1 commit intomarshallpierce:masterfrom
AbeZbm:add-tests
Open

Add a test case for decode_slice_unchecked panic#284
AbeZbm wants to merge 1 commit intomarshallpierce:masterfrom
AbeZbm:add-tests

Conversation

@AbeZbm
Copy link

@AbeZbm AbeZbm commented May 28, 2025

Hi,

Thanks for your time to review this PR.

By examining the existing code, we found that one test can be added to improve the repo's overall test coverage. We've added
one test for

DecodeSliceError::OutputSliceTooSmall => {
panic!("Output slice is too small")

Thanks again for reviewing.

@AbeZbm
Copy link
Author

AbeZbm commented May 30, 2025

@marshallpierce Hello! I was wondering if there's any feedback on this PR. I'm happy to address any concerns or make changes as needed. Thank you!

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.

1 participant