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
chore(muted): handle new Muted status #3570
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## prowler-4.0-dev #3570 +/- ##
===================================================
- Coverage 86.21% 86.20% -0.01%
===================================================
Files 703 703
Lines 21633 21631 -2
===================================================
- Hits 18650 18648 -2
Misses 2983 2983 ☔ View full report in Codecov by Sentry. |
@@ -11,16 +11,17 @@ | |||
from prowler.lib.utils.utils import hash_sha512 | |||
|
|||
|
|||
def generate_json_asff_status(status: str) -> str: | |||
def generate_json_asff_status(status: str, muted: bool = False) -> str: |
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.
Where are you calling this function with the new muted field?
Please add a test to check the new behaviour?
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.
I added where it is needed, good catch. There is already a test for this.
Description
Handle new Muted status in check with
mute_non_default_regions
variable.License
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.