Cap the validators a Native Staking Strategy can hold

[naddison36]

Changes

• Added a MAX_VALIDATORS immutable to ValidatorRegistrator that caps the number of validators in the strategy

Code Change Checklist

To be completed before internal review begins:

• [ ] The contract code is complete
• [ ] Executable deployment file
• [ ] Fork tests that test after the deployment file runs
• [ ] Unit tests *if needed
• [ ] The owner has done a full checklist review of the code + tests

Internal review:

• [ ] Two approvals by internal reviewers

[github-actions[bot]]

	Warnings
:warning:	:eyes: This PR needs at least 2 reviewers

Generated by :no_entry_sign: dangerJS against aed1661a5c8bf4551faf56477b2016d481ab2bf5

[codecov[bot]]

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 63.06%. Comparing base (49439ec) to head (aed1661).

Additional details and impacted files

```diff @@ Coverage Diff @@ ## sparrowDom/nativeStaking #2087 +/- ## ============================================================ + Coverage 62.55% 63.06% +0.51% ============================================================ Files 65 65 Lines 3247 3249 +2 Branches 632 841 +209 ============================================================ + Hits 2031 2049 +18 + Misses 1213 1197 -16 Partials 3 3 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

[openzeppelin-code[bot]]

Cap the validators a Native Staking Strategy can hold

Generated at commit: aed1661a5c8bf4551faf56477b2016d481ab2bf5

🚨 Report Summary

	Severity Level	Results
Contracts	Critical High Medium Low Note Total	3 3 0 18 42 66
Dependencies	Critical High Medium Low Note Total	0 0 0 0 0 0

---

For more details view the full report in OpenZeppelin Code Inspector