Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

EGRC-248: Test SSP System Characteristics #14

Merged
merged 7 commits into from
Mar 17, 2021
Merged

EGRC-248: Test SSP System Characteristics #14

merged 7 commits into from
Mar 17, 2021

Conversation

rgauss
Copy link
Contributor

@rgauss rgauss commented Mar 17, 2021

This adds tests for System Characteristics at both that component's level and within the SSP.

Also:

  • Fixes for null metadata, parties, and system implementation
  • Added key for info type categorization

]
}
],
"description": "Example information type",
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Other descriptions have periods, and this one doesn't—not sure whether that's something we want to keep consistent

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@hkps, sure, can you create commit suggestions for it/them?

rgauss and others added 2 commits March 17, 2021 13:27
Co-authored-by: Hannah Posey-Scholl <hkps.hannah@gmail.com>
Co-authored-by: Hannah Posey-Scholl <hkps.hannah@gmail.com>
src/OSCALSsp.test.js Outdated Show resolved Hide resolved
rgauss and others added 3 commits March 17, 2021 14:36
Co-authored-by: kkennedy26 <78984988+kkennedy26@users.noreply.github.com>
Co-authored-by: Hannah Posey-Scholl <hkps.hannah@gmail.com>
@rgauss rgauss merged commit 7848b80 into EGRC-130 Mar 17, 2021
@rgauss rgauss deleted the EGRC-248 branch March 17, 2021 19:08
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.

None yet

3 participants