)]}'
{"patrole_tempest_plugin/tests/api/volume/rbac_base.py":[{"author":{"_account_id":23185,"name":"Samantha Blanco","email":"samantha.blanco@att.com","username":"sblanco"},"change_message_id":"355496daf83122d9a4e27c172ab7fa4b6c0a3cfd","unresolved":false,"context_lines":[{"line_number":61,"context_line":"    @classmethod"},{"line_number":62,"context_line":"    def create_volume_type(cls, name\u003dNone, **kwargs):"},{"line_number":63,"context_line":"        \"\"\"Create a test volume-type\"\"\""},{"line_number":64,"context_line":"        name \u003d name or data_utils.rand_name(cls.__name__ + \u0027-volume-type\u0027)"},{"line_number":65,"context_line":"        volume_type \u003d cls.volume_types_client.create_volume_type("},{"line_number":66,"context_line":"            name\u003dname, **kwargs)[\u0027volume_type\u0027]"},{"line_number":67,"context_line":"        cls.volume_types.append(volume_type[\u0027id\u0027])"}],"source_content_type":"text/x-python","patch_set":1,"id":"7ffa3b31_43c0f850","line":64,"updated":"2017-04-18 18:14:02.000000000","message":"This looks fine, but if we\u0027re going to add this function here, then we should refactor this test [0] (this can be a separate change)\n\n[0] https://github.com/openstack/patrole/blob/4d14c699fdc0bcf02ee82d06cb847a32e2773e39/patrole_tempest_plugin/tests/api/volume/test_volume_types_extra_specs_rbac.py","commit_id":"5c4b97de1cc8549eb9220853556b0c38f11b5ad7"}],"patrole_tempest_plugin/tests/api/volume/test_user_messages_rbac.py":[{"author":{"_account_id":23185,"name":"Samantha Blanco","email":"samantha.blanco@att.com","username":"sblanco"},"change_message_id":"355496daf83122d9a4e27c172ab7fa4b6c0a3cfd","unresolved":false,"context_lines":[{"line_number":63,"context_line":""},{"line_number":64,"context_line":"    @decorators.idempotent_id(\u0027bf7f31a1-509b-4a7d-a8a8-ad6ce68229c7\u0027)"},{"line_number":65,"context_line":"    @rbac_rule_validation.action("},{"line_number":66,"context_line":"        service\u003d\"cinder\","},{"line_number":67,"context_line":"        rule\u003d\"message:get_all\")"},{"line_number":68,"context_line":"    def test_list_messages(self):"},{"line_number":69,"context_line":"        self.rbac_utils.switch_role(self, toggle_rbac_role\u003dTrue)"},{"line_number":70,"context_line":"        self.client.list_messages()[\u0027messages\u0027]"}],"source_content_type":"text/x-python","patch_set":1,"id":"7ffa3b31_433b589b","line":67,"range":{"start_line":66,"start_character":8,"end_line":67,"end_character":30},"updated":"2017-04-18 18:14:02.000000000","message":"nit: I think these can fit on one line","commit_id":"5c4b97de1cc8549eb9220853556b0c38f11b5ad7"}]}