Update RemoveSecurityManagerTeam to use org roles APIs

Author: just-joshing

github/orgs_security_managers.go

@@ -69,15 +69,14 @@ func (s *OrganizationsService) AddSecurityManagerTeam(ctx context.Context, org,
 
 // RemoveSecurityManagerTeam removes a team from the list of security managers for an organization.
 //
-// GitHub API docs: https://docs.github.com/rest/orgs/security-managers#remove-a-security-manager-team
+// GitHub API docs: https://docs.github.com/en/rest/orgs/organization-roles#remove-an-organization-role-from-a-team
 //
-//meta:operation DELETE /orgs/{org}/security-managers/teams/{team_slug}
+//meta:operation DELETE /orgs/{org}/organization-roles/teams/{team_slug}/{security_manager_role_id}
 func (s *OrganizationsService) RemoveSecurityManagerTeam(ctx context.Context, org, team string) (*Response, error) {
-	u := fmt.Sprintf("orgs/%v/security-managers/teams/%v", org, team)
-	req, err := s.client.NewRequest("DELETE", u, nil)
+	securityManagerRole, resp, err := s.GetSecurityManagerRole(ctx, org)
 	if err != nil {
-		return nil, err
+		return resp, err
 	}
 
-	return s.client.Do(ctx, req, nil)
+	return s.RemoveOrgRoleFromTeam(ctx, org, team, securityManagerRole.GetID())
 }

github/orgs_security_managers_test.go

@@ -109,7 +109,8 @@ func TestOrganizationsService_RemoveSecurityManagerTeam(t *testing.T) {
 	t.Parallel()
 	client, mux, _ := setup(t)
 
-	mux.HandleFunc("/orgs/o/security-managers/teams/t", func(w http.ResponseWriter, r *http.Request) {
+	handleGetSecurityManagerRole(t, mux, "o")
+	mux.HandleFunc("/orgs/o/organization-roles/teams/t/138", func(w http.ResponseWriter, r *http.Request) {
 		testMethod(t, r, "DELETE")
 	})
 
@@ -141,10 +142,12 @@ func TestOrganizationsService_RemoveSecurityManagerTeam_invalidOrg(t *testing.T)
 
 func TestOrganizationsService_RemoveSecurityManagerTeam_invalidTeam(t *testing.T) {
 	t.Parallel()
-	client, _, _ := setup(t)
+	client, mux, _ := setup(t)
+
+	handleGetSecurityManagerRole(t, mux, "o")
 
 	ctx := context.Background()
-	_, err := client.Organizations.RemoveSecurityManagerTeam(ctx, "%", "t")
+	_, err := client.Organizations.RemoveSecurityManagerTeam(ctx, "o", "%")
 	testURLParseError(t, err)
 }