mindersec / minder / 31330237371

62%

push

github

fix(ruletest): initialize params map to prevent panic on template rendering (#6688)

When a Starlark test omits the params= argument, the paramsMap was nil.
This caused the rule type engine to skip ValidateParamsAgainstSchema,
so param_schema defaults (e.g. branch: '') were never applied. When
the ingestion template later called index .Params "branch", it panicked
on the nil map, resulting in an 'internal minder error'.

Fix: initialize paramsMap as an empty map so the engine always runs
schema validation and applies defaults.

Fixes 4 failing tests in minder-rules-and-profiles:
- branch_protection_allow_deletions_test.star/test_force_push_not_allowed
- branch_protection_allow_force_pushes_test.star/test_force_push_not_allowed
- branch_protection_require_status_checks_test.star/test_status_checks_required
- branch_protection_require_status_checks_test.star/test_status_checks_required_with_strict_mode

1 of 1 new or added line in 1 file covered. (100.0%)

21405 of 34713 relevant lines covered (61.66%)

39.69 hits per line