Skip to content

Improve test coverage for test manifest parsing (new_test) #1267

@ZainabTravadi

Description

@ZainabTravadi

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:

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions