Skip to content

Commit d721eef

Browse files
authored
Merge pull request #103 from oslokommune/298-get-team-members
T#298 Add method for retrieving team members
2 parents a890274 + 9ac8a1f commit d721eef

File tree

3 files changed

+22
-0
lines changed

3 files changed

+22
-0
lines changed

CHANGELOG.md

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,7 @@
11
## ?.?.?
22

33
* New parameter `include` added to `TeamClient.get_teams`.
4+
* New method `TeamClient.get_team_members` for getting the members of a team.
45

56
## 2.1.0
67

okdata/sdk/team/client.py

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -44,3 +44,9 @@ def get_team_by_name(self, team_name):
4444
url = "{}/teams/name/{}".format(self.api_url, quote(team_name))
4545
log.info(f"SDK:Getting team from: {url}")
4646
return self.get(url).json()
47+
48+
def get_team_members(self, team_id):
49+
"""Return the members of a team."""
50+
url = "{}/teams/{}/members".format(self.api_url, quote(team_id))
51+
log.info(f"SDK:Getting team members from: {url}")
52+
return self.get(url).json()

tests/team/test_client.py

Lines changed: 15 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -37,3 +37,18 @@ def test_get_team_by_name(requests_mock):
3737
status_code=200,
3838
)
3939
assert TeamClient().get_team_by_name(team_name) == team
40+
41+
42+
def test_get_team_members(requests_mock):
43+
team_id = "abc"
44+
members = [
45+
{"name": "Foo", "username": "foo"},
46+
{"name": "Bar", "username": "bar"},
47+
]
48+
requests_mock.register_uri(
49+
"GET",
50+
re.compile(f"teams/{team_id}/members"),
51+
text=json.dumps(members),
52+
status_code=200,
53+
)
54+
assert TeamClient().get_team_members(team_id) == members

0 commit comments

Comments
 (0)