TEST: Add edge case tests for option validation utilities#1256
Open
bhumigaddam wants to merge 2 commits intojonescompneurolab:masterfrom
Open
TEST: Add edge case tests for option validation utilities#1256bhumigaddam wants to merge 2 commits intojonescompneurolab:masterfrom
bhumigaddam wants to merge 2 commits intojonescompneurolab:masterfrom
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
While exploring the testing infrastructure in HNN-Core, related to improving the testing infrastructure discussed in issue #1118.I noticed that some utility functions responsible for validating input options did not have
dedicated tests covering edge cases.
This PR adds a small set of tests that verify both valid and invalid option
handling for the
_check_optionutility function. The tests ensure thatvalid inputs are accepted and that incorrect values raise the expected
exceptions.
Adding these edge-case tests helps strengthen the reliability of the
validation utilities and improves coverage for functions that are used
across different parts of the codebase.
I’m continuing to explore the testing infrastructure and look forward to
contributing further improvements.