Skip to content

Commit

Permalink
Correct typos and URLs for GSA#833 constraint re @Gabeblis feedback
Browse files Browse the repository at this point in the history
Co-authored-by: Gabeblis <[email protected]>
  • Loading branch information
aj-stein-gsa and Gabeblis committed Nov 19, 2024
1 parent d9e9f6f commit f13dd5c
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 5 deletions.
5 changes: 2 additions & 3 deletions src/validations/constraints/fedramp-external-constraints.xml
Original file line number Diff line number Diff line change
Expand Up @@ -62,9 +62,8 @@
</let>
<let var="minor-version-valid" expression="$doc-oscal-version-parts[2] >= $required-doc-oscal-version-parts[2]"/>
<let var="patch-version-valid" expression="$doc-oscal-version-parts[3] >= $required-doc-oscal-version-parts[3]"/>
<expect id="oscal-version-matches-fedramp-version" target="oscal-version" level="ERROR"
test="$major-version-valid and $minor-version-valid and $patch-version-valid">
<prop namespace="https://docs.oasis-open.org/sarif/sarif/v2.1.0" name="help-url" value="https://docs.oasis-open.org/sarif/sarif/v2.1.0"/>
<expect id="oscal-version-matches-fedramp-version" target="oscal-version" test="$major-version-valid and $minor-version-valid and $patch-version-valid" level="ERROR">
<prop namespace="https://docs.oasis-open.org/sarif/sarif/v2.1.0" name="help-url" value="https://automate.fedramp.gov/documentation/general-concepts/oscal-file-concepts/#oscal-syntax-version"/>
<message>A FedRAMP document SHOULD have an OSCAL version that matches the minimally required version for FedRAMP packages ({$fedramp-required-minimum-version} not {.}).</message>
</expect>
</constraints>
Expand Down
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
test-case:
name: Positive Test for import-profile-has-available-document
name: Positive Test for oscal-version-matches-fedramp-version
description: >-
This test case validates the behavior of constraint import-profile-has-available-document.
This test case validates the behavior of constraint oscal-version-matches-fedramp-version.
Scenario 1 tests: an invalid match below the minimum required version threshold.
Scenario 2 tests: an invalid match above the implied maximum required version threshold, a major version greater than required.
Scenario 3 tests: an invalid match because the fedramp-version prop is missing, so a fallthrough default is required.
Expand Down

0 comments on commit f13dd5c

Please sign in to comment.