-
Notifications
You must be signed in to change notification settings - Fork 55
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
feat: adds control origination to ssp-filter #1375
feat: adds control origination to ssp-filter #1375
Conversation
Adds test to test one and multiple control origination value inputs Adds test to test bad control origintation value input Adds filtering logic to ssp.py Closes oscal-compass#1361 Signed-off-by: Jennifer Power <barnabei.jennifer@gmail.com>
… origination Signed-off-by: Jennifer Power <barnabei.jennifer@gmail.com>
Signed-off-by: Jennifer Power <barnabei.jennifer@gmail.com>
When filtering for control origination, the property could be specified more than one time. This change adds a break and changes to the test component defintion to ensure this case is covered. Signed-off-by: Jennifer Power <barnabei.jennifer@gmail.com>
Hey @jpower432 . Glad to see this PR in place. As soon as you get ready for review please put me and Vikas as reviewers and we will be more than glad to help you with it. If you have any questions or any assistance I can provide, please do not hesitate to reach me out :) |
Thanks @AleJo2995! This is ready, but I am not able to add reviewers. Would you be able to re-run the failed jobs on this PR? I believe the original failure was due to a GitHub Actions outage. |
@AleJo2995 @vikas-agarwal76 Just wanted to reach out about this PR. I am not able to re-run the failed job for windows, but it looks like it was a pytest internal error. Can you please take a look when you have a chance? Appreciate it! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Thanks @vikas-agarwal76. Wanted to double check with you and @AleJo2995 about requirements around merging. I see that I need to squash the commit. Any other requirements since I am working from a fork? Thanks :) |
@jpower432 Yes, whenever any branch is merged into develop branch it is "Squash and Merge". |
Types of changes
develop
->main
)Quality assurance (all should be covered).
Summary
This PR adds an optional control origination flag to the trestle author ssp-filter command. This will filter implemented requirements by the control origination property.
This PR also adds corresponding docs changes to update ssp-filter functionality.
Closes #1361
Key links:
Before you merge