mindersec / minder / 26918537574

61%

push

github

fix: gracefully skip invalid repositories in webhooks instead of aborting (#6360)

* fix: skip invalid repos in webhook batch instead of aborting entire payload

* refactor(webhook): apply review feedback on batch resilience tests

- Rename app_batch_test.go -> app_test.go per reviewer suggestion
- Use ptr.Ptr[T]() from internal/util/ptr instead of local strPtr/int64Ptr helpers
- Move validRepo/invalidRepo/zeroIDRepo to top-level package functions
- Change validRepo to accept int64 instead of int
- Fix processInstallationRepositoriesAppEvent to skip (not abort) on invalid repos
- Add zero-ID guard for removed repos in the same loop

Addresses review comments from @evankanderson

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

20410 of 33741 relevant lines covered (60.49%)

39.56 hits per line