-
-
Notifications
You must be signed in to change notification settings - Fork 8
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
Add test for trivy fs
#35
Conversation
MAINT: bump create-pull-request to v6
Sorry for the late response. Can you resolve the conflicts? |
Thanks. However, the test on Windows failed accidentally. So, I'll check it this weekend. |
I guess we can ignore the windows error for now. #51 (comment) Is this review ready? |
Yes, it's ready. |
DIRS_RESULT="${DIRS_RESULT}${dir}," | ||
fi | ||
done | ||
MATRIX_JSON=$(echo "${DIRS_RESULT}" | sed 's/,$//' | sed 's/\\n//g' | jq -R -s -c 'split(",") | [.[] | {dir: ., command: split("/")[2], type: split("/")[4], label: [split("/")[2],split("/")[3]] | join("/")}]') |
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.
Can you add some comments?
I don't quite understand what this script is doing.
@haya14busa how do you think about this? |
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.
Thanks!
Add test for
trivy fs
to check #17, the test was added.Changes:
./testdata
directory with./testdata/[trivy command]/[platform label]/[with_detections|without_detections]
./testdata/config/terraform/with_detections
and./testdata/config/terraform/without_detections