fix(gitlab_provider): update member retrieval method to include all members

THEBAULT Julien · THEBAULT Julien · commit a41c2adb6ed2 · 2024-09-20T16:09:12.000+02:00
regression from 41342bc
diff --git a/gitlab2sentry/utils/gitlab_provider.py b/gitlab2sentry/utils/gitlab_provider.py
@@ -221,7 +221,7 @@ def _get_default_mentions(self, project: Project) -> str:
         return ", ".join(
             [
                 f"@{member.username}"
-                for member in project.members.list()
+                for member in project.members_all.list(get_all=True)
                 if (
                     member.access_level >= GITLAB_MENTIONS_ACCESS_LEVEL
                     and member.state != "blocked"
diff --git a/tests/conftest.py b/tests/conftest.py
@@ -63,7 +63,7 @@ def __init__(self, username, access_level, state):
 
 class TestGitlabProject:
     def __init__(self):
-        self.members = TestGitlabMemberManager()
+        self.members_all = TestGitlabMemberManager()
 
 
 class TestGitlabMemberManager:

Original file line number	Diff line number	Diff line change
`@@ -221,7 +221,7 @@ def _get_default_mentions(self, project: Project) -> str:`
`221`	`221`	`return ", ".join(`
`222`	`222`	`[`
`223`	`223`	`f"@{member.username}"`
`224`		`- for member in project.members.list()`
	`224`	`+ for member in project.members_all.list(get_all=True)`
`225`	`225`	`if (`
`226`	`226`	`member.access_level >= GITLAB_MENTIONS_ACCESS_LEVEL`
`227`	`227`	`and member.state != "blocked"`