Added Kics Error count properties TEST_KEYWORDS=repository_kics_test

[TommyE123]

Link to issue: #3392

Fixes

The Kics linter is incorrectly reporting the number of errors. This discrepancy becomes apparent when running the linter on files with multiple warnings or errors, as MegaLinter incorrectly displays "1" error instead of the actual count in the summary.

The linter descriptor must contain properties cli_lint_errors_count and cli_lint_errors_regex to be able to count the errors by getting info from the linter output log.

Proposed Changes

1. Added to: KICS descriptor

cli_lint_errors_count: regex_number
cli_lint_errors_regex: "TOTAL: ([0-9]+)"

Readiness Checklist

Author/Contributor

• Add entry to the CHANGELOG listing the change and linking to the corresponding issue (if appropriate)

Reviewing Maintainer

• Label as breaking if this is a large fundamental change
• Label as either automation, bug, documentation, enhancement, infrastructure, or performance

[TommyE123]

HI @nvuillam,

I believe I have implemented the change you suggested, but I'm not certain how to verify it is working correctly. Do you simply need to approve the test workflow?

I also considered adding ERROR and SUCCESS reports to the automation tests for additional testing when KICS runs. However, since KICS produces quite a bit of output compared to other linters, I wasn't sure if that would work well? I also wasnt sure if this was even needed.
Please advise if you think adding error/success reports would be beneficial or if the current test workflow is sufficient.

Any guidance you can provide on testing the change would be greatly appreciated. Please let me know if you need any clarification or have additional suggestions.

regards
Tom

[nvuillam]

@TommyE123 error and success report files are from a previous version of MegaLinter and i think i broke it somehow ^^

You can add the example fils if you want, it's still good for documentation :)

[nvuillam]

Works for me, thanks for your contribution :)