Description
The current test suite for new_test in the manifest module primarily validates
basic success and failure cases, but does not comprehensively verify parsed
field values or default behavior.
This limits confidence when modifying or extending test configuration logic.
Observed Gaps
-
Parsed values for fields such as:
source-dir
main
link
dependencies
are not explicitly asserted
-
Default values for optional fields are not tested
-
Error handling (e.g., invalid test names) is not fully validated
Proposed Improvement
Add unit tests that:
- Verify full parsing of all supported fields
- Validate default values when optional fields are omitted
- Ensure invalid configurations produce meaningful errors
Related Work
A pull request has been opened to address these gaps:
Description
The current test suite for
new_testin the manifest module primarily validatesbasic success and failure cases, but does not comprehensively verify parsed
field values or default behavior.
This limits confidence when modifying or extending test configuration logic.
Observed Gaps
Parsed values for fields such as:
source-dirmainlinkdependenciesare not explicitly asserted
Default values for optional fields are not tested
Error handling (e.g., invalid test names) is not fully validated
Proposed Improvement
Add unit tests that:
Related Work
A pull request has been opened to address these gaps: