)]}'
{"patrole_tempest_plugin/tests/api/identity/v3/test_groups_rbac.py":[{"author":{"_account_id":23659,"name":"Jaewoo Park","email":"jaewoopark91@gmail.com","username":"aselius"},"change_message_id":"03323f0e5504a465ccac6b4fed323dc71cc9d014","unresolved":false,"context_lines":[{"line_number":27,"context_line":""},{"line_number":28,"context_line":"class IdentityGroupsV3AdminRbacTest(rbac_base.BaseIdentityV3RbacAdminTest):"},{"line_number":29,"context_line":""},{"line_number":30,"context_line":"    def _create_group(self):"},{"line_number":31,"context_line":"        \"\"\"Creates a group for test.\"\"\""},{"line_number":32,"context_line":"        name \u003d data_utils.rand_name(\u0027Group\u0027)"},{"line_number":33,"context_line":"        group \u003d self.groups_client \\"}],"source_content_type":"text/x-python","patch_set":4,"id":"fa31d9ce_cd1ea062","line":30,"range":{"start_line":30,"start_character":8,"end_line":30,"end_character":21},"updated":"2017-02-13 18:45:47.000000000","message":"looks good but I think the order for the methods before the test cases are typically: teardown -\u003e classmethods -\u003e other methods.","commit_id":"56b509d32955505b9db4ebd04394f476882c69c1"},{"author":{"_account_id":23185,"name":"Samantha Blanco","email":"samantha.blanco@att.com","username":"sblanco"},"change_message_id":"208647f5f45357b40d48967450b304f9142ada7f","unresolved":false,"context_lines":[{"line_number":27,"context_line":""},{"line_number":28,"context_line":"class IdentityGroupsV3AdminRbacTest(rbac_base.BaseIdentityV3RbacAdminTest):"},{"line_number":29,"context_line":""},{"line_number":30,"context_line":"    def _create_group(self):"},{"line_number":31,"context_line":"        \"\"\"Creates a group for test.\"\"\""},{"line_number":32,"context_line":"        name \u003d data_utils.rand_name(\u0027Group\u0027)"},{"line_number":33,"context_line":"        group \u003d self.groups_client \\"}],"source_content_type":"text/x-python","patch_set":4,"id":"fa31d9ce_28356af6","line":30,"range":{"start_line":30,"start_character":8,"end_line":30,"end_character":21},"in_reply_to":"fa31d9ce_cd1ea062","updated":"2017-02-13 19:36:20.000000000","message":"Jaewoo\u0027s right; helpers methods should be below tearDown and class methods","commit_id":"56b509d32955505b9db4ebd04394f476882c69c1"},{"author":{"_account_id":23185,"name":"Samantha Blanco","email":"samantha.blanco@att.com","username":"sblanco"},"change_message_id":"208647f5f45357b40d48967450b304f9142ada7f","unresolved":false,"context_lines":[{"line_number":119,"context_line":"    @decorators.idempotent_id(\u0027b3e394a7-079e-4a0d-a4ff-9b266293d1ee\u0027)"},{"line_number":120,"context_line":"    def test_list_user_group(self):"},{"line_number":121,"context_line":"        group \u003d self._create_group()"},{"line_number":122,"context_line":"        self._add_user_to_group(group[\u0027id\u0027])"},{"line_number":123,"context_line":""},{"line_number":124,"context_line":"        rbac_utils.switch_role(self, switchToRbacRole\u003dTrue)"},{"line_number":125,"context_line":"        self.groups_client.list_group_users(group[\u0027id\u0027])"}],"source_content_type":"text/x-python","patch_set":4,"id":"fa31d9ce_fa5dd915","line":122,"updated":"2017-02-13 19:36:20.000000000","message":"Not technically necessary to add a user to the group since we\u0027re not testing functionality","commit_id":"56b509d32955505b9db4ebd04394f476882c69c1"}]}