diff --git a/briar_wrapper/models/contacts.py b/briar_wrapper/models/contacts.py index 63005d55c5f1025b47290a4edc271fc77c9f65eb..0438275947254156ade69b635fdba8eb7177bf5a 100644 --- a/briar_wrapper/models/contacts.py +++ b/briar_wrapper/models/contacts.py @@ -38,13 +38,14 @@ class Contacts(Model): _post(url, headers=self._headers, json={"link": link, "alias": alias}) def set_alias(self, contact_id: int, alias: str) -> None: + # pylint: disable=line-too-long """ Sets the alias of a given user [Upstream documentation](https://code.briarproject.org/briar/briar/-/blob/main/briar-headless/README.md#changing-alias-of-a-contact) """ - url = urljoin(BASE_HTTP_URL, self._API_ENDPOINT + str(contact_id) + alias) + url = urljoin(BASE_HTTP_URL, self._API_ENDPOINT + f"{str(contact_id)}/alias") _post(url, headers=self._headers, json={"alias": alias}) def delete(self, contact_id: int) -> None: diff --git a/tests/briar_wrapper/models/test_contacts.py b/tests/briar_wrapper/models/test_contacts.py index 07945135f4b5ffdbf48f200f640f39be506a72d3..f7531a2cb0bc67f5c03f721a6a1e970f715789f1 100644 --- a/tests/briar_wrapper/models/test_contacts.py +++ b/tests/briar_wrapper/models/test_contacts.py @@ -55,10 +55,10 @@ def test_set_alias(api, request_headers, requests_mock): contacts = Contacts(api) contact_id = 137 requests_mock.register_uri("POST", - BASE_HTTP_URL + f"/{contact_id}/alias", + BASE_HTTP_URL + f"{contact_id}/alias", request_headers=request_headers, - additional_matcher=match_request_add_pending) - contacts.add_pending(contact_id, TEST_ALIAS) + additional_matcher=match_request_set_alias) + contacts.set_alias(contact_id, TEST_ALIAS) def match_request_set_alias(request):