Skip to content

Commit b08b8b0

Browse files
committed
Update JSON response format for consistency with other endpoints
1 parent eaa5612 commit b08b8b0

2 files changed

Lines changed: 4 additions & 3 deletions

File tree

tests/publisher/cve/test_has_cve_api.py

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -46,6 +46,7 @@ def test_has_cves_for_canonical_user(self, mock_get_snap_info, mock_get):
4646

4747
self.assertEqual(response.status_code, 200)
4848
self.assertEqual(data["success"], True)
49+
self.assertEqual(data["data"]["revisions"], [123, 321])
4950

5051
@patch(
5152
"webapp.publisher.cve.cve_helper.CveHelper.get_revisions_with_cves",

webapp/publisher/cve/cve_views.py

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -48,21 +48,21 @@ def get_revisions_with_cves(snap_name):
4848
)
4949
if not has_access:
5050
return (
51-
flask.jsonify({"success": False, "error": error_message}),
51+
flask.jsonify({"success": False, "message": error_message}),
5252
status_code,
5353
)
5454

5555
revisions_with_cves = CveHelper.get_revisions_with_cves(snap_name)
5656
if len(revisions_with_cves) > 0:
5757
return flask.jsonify(
58-
{"success": True, "revisions": revisions_with_cves}
58+
{"success": True, "data": {"revisions": revisions_with_cves}}
5959
)
6060
else:
6161
return (
6262
flask.jsonify(
6363
{
6464
"success": False,
65-
"error": f"CVEs data for '{snap_name}' snap not found.",
65+
"message": f"CVEs data for '{snap_name}' snap not found.",
6666
}
6767
),
6868
404,

0 commit comments

Comments
 (0)