diff --git a/easy_my_coop_api/services/subscription_request_service.py b/easy_my_coop_api/services/subscription_request_service.py index 7017cbf..5331643 100644 --- a/easy_my_coop_api/services/subscription_request_service.py +++ b/easy_my_coop_api/services/subscription_request_service.py @@ -51,7 +51,8 @@ class SubscriptionRequestService(Component): } return response - def create(self, **params): + def create(self, **params): # pylint: disable=method-required-super + params = self._prepare_create(params) sr = self.env["subscription.request"].create(params) return self._to_dict(sr) @@ -69,7 +70,7 @@ class SubscriptionRequestService(Component): def _to_dict(self, sr): sr.ensure_one() return { - "id": sr.id, + "id": sr.id, "name": sr.name, "email": sr.email, "state": sr.state, diff --git a/easy_my_coop_api/tests/common.py b/easy_my_coop_api/tests/common.py index 39e7a6f..7a7222d 100644 --- a/easy_my_coop_api/tests/common.py +++ b/easy_my_coop_api/tests/common.py @@ -73,8 +73,8 @@ class BaseEMCRestCase(BaseRestCase): def http_get_content(self, route, headers=None): response = self.http_get(route, headers=headers) self.assertEquals(response.status_code, 200) - - return json.loads(response.content) + content = response.content.decode("utf-8") + return json.loads(content) def http_post(self, url, data, headers=None): headers = _add_api_key(headers) diff --git a/easy_my_coop_api/tests/test_ping.py b/easy_my_coop_api/tests/test_ping.py index 36c9433..dcfbe2a 100644 --- a/easy_my_coop_api/tests/test_ping.py +++ b/easy_my_coop_api/tests/test_ping.py @@ -30,12 +30,12 @@ class TestPing(BaseEMCRestCase): url = "http://%s:%s%s" % (HOST, PORT, path) response = requests.get(url) self.assertEquals(response.status_code, 200) - content = json.loads(response.content) + content = json.loads(response.content.decode("utf-8")) self.assertTrue("message" in content) def test_search_route(self): response = self.http_get("/api/ping") self.assertEquals(response.status_code, 200) - content = json.loads(response.content) + content = json.loads(response.content.decode("utf-8")) self.assertTrue("message" in content) diff --git a/easy_my_coop_api/tests/test_subscription_requests.py b/easy_my_coop_api/tests/test_subscription_requests.py index df97d3e..71643b8 100644 --- a/easy_my_coop_api/tests/test_subscription_requests.py +++ b/easy_my_coop_api/tests/test_subscription_requests.py @@ -113,7 +113,7 @@ class TestSRController(BaseEMCRestCase): response = self.http_post(url, data=data) self.assertEquals(response.status_code, 200) - content = json.loads(response.content) + content = json.loads(response.content.decode("utf-8")) content.pop("id") # can't know id in advance expected = { @@ -135,7 +135,7 @@ class TestSRController(BaseEMCRestCase): response = self.http_post(url, data=data) self.assertEquals(response.status_code, 200) - content = json.loads(response.content) + content = json.loads(response.content.decode("utf-8")) expected = self.demo_request_1_dict expected["state"] = "done"