Skip to content

Commit 5a02134

Browse files
author
THEBAULT Julien
committed
fix(gitlab_provider): update member retrieval method to include all members
regression from 41342bc
1 parent 41342bc commit 5a02134

File tree

3 files changed

+4
-4
lines changed

3 files changed

+4
-4
lines changed

gitlab2sentry/utils/gitlab_provider.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -221,7 +221,7 @@ def _get_default_mentions(self, project: Project) -> str:
221221
return ", ".join(
222222
[
223223
f"@{member.username}"
224-
for member in project.members.list()
224+
for member in project.members_all.list(get_all=True)
225225
if (
226226
member.access_level >= GITLAB_MENTIONS_ACCESS_LEVEL
227227
and member.state != "blocked"

tests/conftest.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -63,11 +63,11 @@ def __init__(self, username, access_level, state):
6363

6464
class TestGitlabProject:
6565
def __init__(self):
66-
self.members = TestGitlabMemberManager()
66+
self.members_all = TestGitlabMemberManager()
6767

6868

6969
class TestGitlabMemberManager:
70-
def list(self):
70+
def list(self, *args, **kwargs):
7171
return TEST_GITLAB_PROJECT_MEMBERS
7272

7373

tests/test_gitlab_provider.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -92,7 +92,7 @@ def test_get_default_mentions(gitlab_provider_fixture, gitlab_project_fixture):
9292
)
9393
_project_non_blocked_members = [
9494
member
95-
for member in gitlab_project_fixture.members.list()
95+
for member in gitlab_project_fixture.members_all.list()
9696
if member.state != "blocked"
9797
]
9898
assert len(_mentioned_members) == len(_project_non_blocked_members)

0 commit comments

Comments
 (0)