Merge lp:~facundo/ols-store-tests/store-acceptance-tests--fix-new-format-2 into lp:~ubuntuone-pqm-team/ols-store-tests/store-acceptance-tests

Proposed by Facundo Batista
Status: Merged
Approved by: Ricardo Kirkner
Approved revision: 27
Merged at revision: 27
Proposed branch: lp:~facundo/ols-store-tests/store-acceptance-tests--fix-new-format-2
Merge into: lp:~ubuntuone-pqm-team/ols-store-tests/store-acceptance-tests
Diff against target: 26 lines (+10/-2)
1 file modified
tests/api/snap/test_snap_push.py (+10/-2)
To merge this branch: bzr merge lp:~facundo/ols-store-tests/store-acceptance-tests--fix-new-format-2
Reviewer Review Type Date Requested Status
Ricardo Kirkner (community) Approve
Review via email: mp+303603@code.launchpad.net

Commit message

Changed two other calls in Snap Push to support new style format error.

Description of the change

Changed two other calls in Snap Push to support new style format error.

To post a comment you must log in.
Revision history for this message
Ricardo Kirkner (ricardokirkner) :
review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'tests/api/snap/test_snap_push.py'
2--- tests/api/snap/test_snap_push.py 2016-06-24 20:15:28 +0000
3+++ tests/api/snap/test_snap_push.py 2016-08-22 17:04:54 +0000
4@@ -68,6 +68,14 @@
5 'name': ['This field is required.']
6 },
7 'success': False,
8+ 'error_list': [{
9+ 'code': 'missing-field',
10+ 'message': "The 'updown_id' field is required",
11+ }, {
12+ 'code': 'missing-field',
13+ 'message': "The 'name' field is required",
14+ }],
15+
16 }
17 self.assertEqual(body, expected)
18
19@@ -88,5 +96,5 @@
20 def test_push_snap_require_package_upload_permission(self):
21 auth = authenticate_with_macaroon(permissions=['package_access'])
22 response = push_snap(self.package_name, self.updown_id, auth=auth)
23- self.assert_error(response)
24- self.assert_forbidden(response)
25+ self.assert_error(response, content_type='application/json')
26+ self.assert_forbidden(response, new_style=True)

Subscribers

People subscribed via source and target branches