Refactor ApprovalRules::GroupFinder interface
<!--IssueSummary start-->
<details>
<summary>
Everyone can contribute. [Help move this issue forward](https://handbook.gitlab.com/handbook/marketing/developer-relations/contributor-success/community-contributors-workflows/#contributor-links) while earning points, leveling up and collecting rewards.
</summary>
- [Close this issue](https://contributors.gitlab.com/manage-issue?action=close&projectId=278964&issueIid=455640)
</details>
<!--IssueSummary end-->
Finder classes typically only have a single public `execute` instance method.
`ApprovalRules::GroupFinder` has many public methods and none of them are `execute` :grimacing: https://gitlab.com/gitlab-org/gitlab/-/blob/38ad335189a1080c85c836716469d9774802ce41/ee/app/finders/approval_rules/group_finder.rb#L15-26)
We should refactor `ApprovalRules::GroupFinder` to have an interface consistent with other Finder classes.
issue