Merge pull request #19 from ONS-Innovation/TotalDwarf03-patch-1

Bug: get_codeowner_users had explicit reference to ONSDigital. Replac…

Author: TotalDwarf03

github_api_toolkit/__init__.py

@@ -463,7 +463,7 @@ def get_team_maintainers(self, org: str, team_name: str) -> list | tuple:
         else:
             return self.get_error_message(response)
 
-    def get_codeowner_users(self, codeowners: list) -> list:
+    def get_codeowner_users(self, org: str, codeowners: list) -> list:
         """Gets a list of users from a list of users and teams. Will get the maintainers of any teams and add them as a user.
 
         Args:
@@ -477,7 +477,7 @@ def get_codeowner_users(self, codeowners: list) -> list:
 
         for codeowner in codeowners:
             if codeowner["type"] == "team":
-                team_maintainers = self.get_team_maintainers("ONSDigital", codeowner["name"])
+                team_maintainers = self.get_team_maintainers(org, codeowner["name"])
 
                 for maintainer in team_maintainers:
                     users.append(maintainer["login"])
@@ -531,8 +531,8 @@ def get_repository_email_list(self, org: str, repo: str, branch: str = "main") -
 
         codeowners = self.identify_teams_and_users(codeowners)
 
-        codeowners = self.get_codeowner_users(codeowners)
+        codeowners = self.get_codeowner_users(org, codeowners)
 
         emails = self.get_codeowner_emails(codeowners, org)
 
-        return emails
+        return emails