Skip to content

test(validate): enable component-definition port validation coverage#2195

Open
Acuspeedster wants to merge 1 commit into
oscal-compass:developfrom
Acuspeedster:fix/validate-component-definition-ports-tests
Open

test(validate): enable component-definition port validation coverage#2195
Acuspeedster wants to merge 1 commit into
oscal-compass:developfrom
Acuspeedster:fix/validate-component-definition-ports-tests

Conversation

@Acuspeedster

Copy link
Copy Markdown
Contributor

Types of changes

  • Hot fix (emergency fix and release)
  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Documentation (change which affects the documentation site)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Release (develop -> main)

Quality assurance (all should be covered).

  • My code follows the code style of this project.
  • Documentation for my change is up to date?
  • My PR meets testing requirements.
  • All new and existing tests passed.
  • All commits are signed-off.

Summary

Enable the previously skipped component-definition port validation coverage. The validation command already handles these fixtures correctly, so the PR turns the skipped cases into active regression tests:

  • valid component-definition with port ranges
  • invalid component-definition with a negative port range

Key links:

Before you merge

  • Ensure it is a 'squash commit' if not a release.
  • Ensure CI is currently passing
  • Check sonar. If you are working for a fork a maintainer will reach out, if required.

Signed-off-by: Acuspeedster <arnavrajsingh@gmail.com>
@Acuspeedster Acuspeedster requested a review from a team as a code owner April 8, 2026 21:55
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