)]}'
{"patrole_tempest_plugin/tests/api/identity/v3/test_regions_rbac.py":[{"author":{"_account_id":23185,"name":"Samantha Blanco","email":"samantha.blanco@att.com","username":"sblanco"},"change_message_id":"f55678772cfd8018f7bc4849a1ef279bd1c44c9d","unresolved":false,"context_lines":[{"line_number":52,"context_line":"    @decorators.idempotent_id(\u00276bdaecd4-0843-4ed6-ab64-3a57ab0cd120\u0027)"},{"line_number":53,"context_line":"    def test_update_region(self):"},{"line_number":54,"context_line":"        region \u003d self._create_region()"},{"line_number":55,"context_line":"        new_description \u003d data_utils.rand_name(\u0027test_update_region\u0027)"},{"line_number":56,"context_line":""},{"line_number":57,"context_line":"        self.rbac_utils.switch_role(self, switchToRbacRole\u003dTrue)"},{"line_number":58,"context_line":"        self.regions_client.update_region(region[\u0027id\u0027],"}],"source_content_type":"text/x-python","patch_set":2,"id":"9a30ddce_a51f8259","line":55,"updated":"2017-03-10 16:27:50.000000000","message":"nit: I don\u0027t think it\u0027s necessary to use rand_name for this. There\u0027s no uniqueness check for description as far as I\u0027m aware.","commit_id":"2c50f368b14e26c642368dfa67b856b1f08751ea"},{"author":{"_account_id":23186,"name":"Felipe Monteiro","email":"felipe.carneiro.monteiro@gmail.com","username":"felipe.monteiro"},"change_message_id":"a2e6581ba80ff819da70bdeaba0e08f1d3552ce7","unresolved":false,"context_lines":[{"line_number":52,"context_line":"    @decorators.idempotent_id(\u00276bdaecd4-0843-4ed6-ab64-3a57ab0cd120\u0027)"},{"line_number":53,"context_line":"    def test_update_region(self):"},{"line_number":54,"context_line":"        region \u003d self._create_region()"},{"line_number":55,"context_line":"        new_description \u003d data_utils.rand_name(\u0027test_update_region\u0027)"},{"line_number":56,"context_line":""},{"line_number":57,"context_line":"        self.rbac_utils.switch_role(self, switchToRbacRole\u003dTrue)"},{"line_number":58,"context_line":"        self.regions_client.update_region(region[\u0027id\u0027],"}],"source_content_type":"text/x-python","patch_set":2,"id":"9a30ddce_48e09b96","line":55,"in_reply_to":"9a30ddce_a51f8259","updated":"2017-03-10 17:00:07.000000000","message":"Other than having to import data_utils, it doesn\u0027t detract from the test in any way. I\u0027m indifferent regarding keeping it or removing it.","commit_id":"2c50f368b14e26c642368dfa67b856b1f08751ea"}]}