feat!: Implement Enterprise SCIM - EnterpriseService.ListProvisionedSCIMGroups

[elminster-aom]

BREAKING CHANGE: SCIMService.ListSCIMProvisionedGroupsForEnterprise is now EnterpriseService.ListProvisionedSCIMEnterpriseGroups.

Fixes: #3813

Summary

From API Docs https://docs.github.com/en/enterprise-cloud@latest/rest/enterprise-admin/scim, implement:

• GET /scim/v2/enterprises/{enterprise}/Groups

Breaking Change

The SCIMService.ListSCIMProvisionedGroupsForEnterprise method has been moved to EnterpriseService.ListProvisionedSCIMGroups.

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 92.38%. Comparing base (d69610a) to head (7368118).
⚠️ Report is 1 commits behind head on master.

Additional details and impacted files

@@           Coverage Diff           @@
##           master    #3814   +/-   ##
=======================================
  Coverage   92.38%   92.38%           
=======================================
  Files         195      196    +1     
  Lines       14094    14094           
=======================================
  Hits        13021    13021           
  Misses        884      884           
  Partials      189      189           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

[gmlewis]

Thanks, @elminster-aom!
Did we agree in another PR not to have methods end with the suffix ...ForEnterprise?

[elminster-aom]

Did we agree in another PR not to have methods end with the suffix ...ForEnterprise?

@gmlewis, I couldn't find the PR you mention but I think the word "Enterprise" should appear somehow (Maybe with just an "E"? Suggestions are welcome) otherwise it may get confusing with the other SCIM resources in /github/scim.go.

[gmlewis]

Did we agree in another PR not to have methods end with the suffix ...ForEnterprise?

@gmlewis, I couldn't find the PR you mention but I think the word "Enterprise" should appear somehow (Maybe with just an "E"? Suggestions are welcome) otherwise it may get confusing with the other SCIM resources in /github/scim.go.

The idea is that for all our methods, the service must first be specified, like enterpriseService.MethodToCall so we shouldn't need to repeat the ForEnterprise part.

[gmlewis]

Thank you, @elminster-aom!
LGTM
Awaiting second LGTM+Approval before merging.

cc: @alexandear

[gmlewis]

Thank you, @elminster-aom and @alexandear!
Merging.