Commit 62f3c917 authored by Romain Courteaud's avatar Romain Courteaud 🐙

erp5_json_rpc_api: return a default success content

parent 4a5509ed
......@@ -353,7 +353,11 @@ class JsonRpcAPIService(OpenAPIService):
raise JsonRpcAPIInvalidJsonDictContent(str(e))
response = request.RESPONSE
# XXX Hardcoded JSONForm behaviour
if result == "Nothing to do":
result = {}
if (result == "Nothing to do") or (not result):
result = {
'status': 200,
'type': 'success-type',
'title': 'query completed'
}
response.setHeader("Content-Type", "application/json")
return json.dumps(result).encode()
......@@ -773,7 +773,14 @@ class TestJsonRpcAPIJsonFormHandling(JsonRpcAPITestCase):
env={'CONTENT_TYPE': 'application/json'})
self.assertEqual(response.getStatus(), 200)
self.assertEqual(response.getHeader('content-type'), 'application/json')
self.assertEqual(json.loads(response.getBody()), {})
self.assertEqual(
response.getBody(),
json.dumps(
{
'type': 'success-type',
'title': "query completed",
'status': 200
}).encode())
def test_jsonFormHandling_noInputSchemaAndBodyContent(self):
self.addJSONForm(
......@@ -790,8 +797,13 @@ class TestJsonRpcAPIJsonFormHandling(JsonRpcAPITestCase):
self.assertEqual(response.getStatus(), 200)
self.assertEqual(response.getHeader('content-type'), 'application/json')
self.assertEqual(
json.loads(response.getBody()), {
})
response.getBody(),
json.dumps(
{
'type': 'success-type',
'title': "query completed",
'status': 200
}).encode())
def test_jsonFormHandling_invalidBodyContent(self):
self.addJSONForm(
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment