)]}'
{"test/functional/test_access_control.py":[{"author":{"_account_id":15343,"name":"Tim Burke","email":"tburke@nvidia.com","username":"tburke"},"change_message_id":"d371b6ad62f2a3ac815dfa185e1e1b94683c0dc6","unresolved":false,"context_lines":[{"line_number":3042,"context_line":"            self.client.auth(test_case[\u0027auth_user_name\u0027])"},{"line_number":3043,"context_line":"        resp \u003d self.client.send_request(test_case[\u0027http_method\u0027],"},{"line_number":3044,"context_line":"                                        self.info_url, token)"},{"line_number":3045,"context_line":"        return resp.status"},{"line_number":3046,"context_line":""},{"line_number":3047,"context_line":"    def _cleanup(self):"},{"line_number":3048,"context_line":"        pass"}],"source_content_type":"text/x-python","patch_set":7,"id":"dab17558_2ac2e133","line":3045,"updated":"2016-05-06 22:16:02.000000000","message":"I kinda feel like we ought to make some sort of assertion about the content of the response, rather than *just* the response code. Even just adding\n\n swift_info \u003d json.loads(resp.body)\n self.assertIn(\u0027swift\u0027, swift_info)\n\nbefore the return would be significant, I think.","commit_id":"b42a519e6bcb36163f71c38f22fd90fcdff4f55c"},{"author":{"_account_id":15343,"name":"Tim Burke","email":"tburke@nvidia.com","username":"tburke"},"change_message_id":"d371b6ad62f2a3ac815dfa185e1e1b94683c0dc6","unresolved":false,"context_lines":[{"line_number":3055,"context_line":"        if any((tf.skip, tf.skip2, tf.skip3, tf.skip_if_not_v3,"},{"line_number":3056,"context_line":"                tf.skip_if_no_reseller_admin)):"},{"line_number":3057,"context_line":"            raise SkipTest"},{"line_number":3058,"context_line":"        self.info_url \u003d self._get_info_url()"},{"line_number":3059,"context_line":"        scenario_rbac_info \u003d RBAC_INFO_GET + RBAC_INFO_HEAD + RBAC_INFO_OPTIONS"},{"line_number":3060,"context_line":"        shuffle(scenario_rbac_info)"},{"line_number":3061,"context_line":"        self._run_scenario(scenario_rbac_info)"}],"source_content_type":"text/x-python","patch_set":7,"id":"dab17558_4abf65fb","line":3058,"updated":"2016-05-06 22:16:02.000000000","message":"nit: I wasn\u0027t really expecting self.info_url to be set here. __init__ or setUp or even _prepare, but not inline with the test...","commit_id":"b42a519e6bcb36163f71c38f22fd90fcdff4f55c"}]}