)]}'
{"keystone/tests/unit/test_v3_federation.py":[{"author":{"_account_id":8978,"name":"Marek Denis","email":"marek.denis+openstack@gmail.com","username":"marek-denis"},"change_message_id":"3bff55a59421076ade85bb45ebe87a9cb8760b0d","unresolved":false,"context_lines":[{"line_number":1310,"context_line":"            self.assertEqual(project_ids, expected_project_ids,"},{"line_number":1311,"context_line":"                             \u0027Projects match failed for url %s\u0027 % url)"},{"line_number":1312,"context_line":""},{"line_number":1313,"context_line":"        # Replace the project inherited assignment by the previous domain one"},{"line_number":1314,"context_line":"        self.assignment_api.delete_grant("},{"line_number":1315,"context_line":"            role_id\u003dself.role_customer[\u0027id\u0027],"},{"line_number":1316,"context_line":"            group_id\u003dself.group_customers[\u0027id\u0027],"}],"source_content_type":"text/x-python","patch_set":1,"id":"ba7be1f8_7bc4125f","line":1313,"updated":"2015-02-24 09:44:33.000000000","message":"what\u0027s the purpose of the following changes? To revert the initial role/domain/groups/project configuration?","commit_id":"524b35e87f6177916bd333def880c87a4e1bf2ce"},{"author":{"_account_id":9142,"name":"Samuel de Medeiros Queiroz","email":"samueldmq@lsd.ufcg.edu.br","username":"samuel-z","inactive":true},"change_message_id":"b5be1cca4f5cd121f8cd275f58d614b13de56330","unresolved":false,"context_lines":[{"line_number":1310,"context_line":"            self.assertEqual(project_ids, expected_project_ids,"},{"line_number":1311,"context_line":"                             \u0027Projects match failed for url %s\u0027 % url)"},{"line_number":1312,"context_line":""},{"line_number":1313,"context_line":"        # Replace the project inherited assignment by the previous domain one"},{"line_number":1314,"context_line":"        self.assignment_api.delete_grant("},{"line_number":1315,"context_line":"            role_id\u003dself.role_customer[\u0027id\u0027],"},{"line_number":1316,"context_line":"            group_id\u003dself.group_customers[\u0027id\u0027],"}],"source_content_type":"text/x-python","patch_set":1,"id":"ba7be1f8_9ec9ad28","line":1313,"in_reply_to":"ba7be1f8_7bc4125f","updated":"2015-02-24 13:13:05.000000000","message":"Yes... I don\u0027t think it loads sample data for each test, does it?\n\nWe then need to keep the previous state in order to not affect next tests.","commit_id":"524b35e87f6177916bd333def880c87a4e1bf2ce"},{"author":{"_account_id":5707,"name":"Henry Nash","email":"henryn@linux.vnet.ibm.com","username":"henry-nash"},"change_message_id":"a6e26d09a1716bf84119a52724b3bc369a193d84","unresolved":false,"context_lines":[{"line_number":2304,"context_line":"        # Enable OS-INHERIT config switch"},{"line_number":2305,"context_line":"        self.config_fixture.config(group\u003d\u0027os_inherit\u0027, enabled\u003dTrue)"},{"line_number":2306,"context_line":""},{"line_number":2307,"context_line":"        # Replace the existing domain inherited assignment by a project one"},{"line_number":2308,"context_line":"        self.assignment_api.delete_grant("},{"line_number":2309,"context_line":"            role_id\u003dself.role_customer[\u0027id\u0027],"},{"line_number":2310,"context_line":"            group_id\u003dself.group_customers[\u0027id\u0027],"}],"source_content_type":"text/x-python","patch_set":4,"id":"da9b358b_ae3623e5","line":2307,"updated":"2015-03-31 05:56:31.000000000","message":"I don\u0027t like that we are monkeying with the (extensive) setup of domain/projects/assignments here. I have to believe that we\u0027ll need more than one test over time to do with project inheritances here.  I\u0027d prefer to change the setup so that we have either another domain that has only project inherited roles, a token create already etc.","commit_id":"f2634755efc2bcc0737979b88a71cd71127e6ed1"},{"author":{"_account_id":9142,"name":"Samuel de Medeiros Queiroz","email":"samueldmq@lsd.ufcg.edu.br","username":"samuel-z","inactive":true},"change_message_id":"528ef4e1c859154c8e16cc2b0f0cc313a647f71c","unresolved":false,"context_lines":[{"line_number":2304,"context_line":"        # Enable OS-INHERIT config switch"},{"line_number":2305,"context_line":"        self.config_fixture.config(group\u003d\u0027os_inherit\u0027, enabled\u003dTrue)"},{"line_number":2306,"context_line":""},{"line_number":2307,"context_line":"        # Replace the existing domain inherited assignment by a project one"},{"line_number":2308,"context_line":"        self.assignment_api.delete_grant("},{"line_number":2309,"context_line":"            role_id\u003dself.role_customer[\u0027id\u0027],"},{"line_number":2310,"context_line":"            group_id\u003dself.group_customers[\u0027id\u0027],"}],"source_content_type":"text/x-python","patch_set":4,"id":"da9b358b_4913e70d","line":2307,"in_reply_to":"da9b358b_ae3623e5","updated":"2015-04-01 00:38:02.000000000","message":"Ok, I will provide a better setup inside this method\u0027s scope.\n\nWe could, in a follow-on patch, add a class for tokens obtained from inherited role assignments.\n\nI am not going to do this as part of this patch, since there are some other tests with role inheritance (on domains) here.\n\nWhat do you think?","commit_id":"f2634755efc2bcc0737979b88a71cd71127e6ed1"},{"author":{"_account_id":6486,"name":"Brant Knudson","email":"blk@acm.org","username":"blk-u"},"change_message_id":"90452266d810d424ff8f9174cffa6afd5c7c1212","unresolved":false,"context_lines":[{"line_number":2291,"context_line":"                self.assertEqual(projects_ref, projects,"},{"line_number":2292,"context_line":"                                 \u0027match failed for url %s\u0027 % url)"},{"line_number":2293,"context_line":""},{"line_number":2294,"context_line":"    # TODO(samueldmq): Create another test class for role inheritance tests."},{"line_number":2295,"context_line":"    # The advantage would be to reduce the complexity of this test class and"},{"line_number":2296,"context_line":"    # have tests specific to this fuctionality grouped, easing readability and"},{"line_number":2297,"context_line":"    # maintenability."}],"source_content_type":"text/x-python","patch_set":6,"id":"baa041b7_07f4d8ef","line":2294,"updated":"2015-04-03 01:12:22.000000000","message":"why not just do it?","commit_id":"51317c80836ea63e4a6ca39c92c97927e8a9733e"},{"author":{"_account_id":6486,"name":"Brant Knudson","email":"blk@acm.org","username":"blk-u"},"change_message_id":"90452266d810d424ff8f9174cffa6afd5c7c1212","unresolved":false,"context_lines":[{"line_number":2293,"context_line":""},{"line_number":2294,"context_line":"    # TODO(samueldmq): Create another test class for role inheritance tests."},{"line_number":2295,"context_line":"    # The advantage would be to reduce the complexity of this test class and"},{"line_number":2296,"context_line":"    # have tests specific to this fuctionality grouped, easing readability and"},{"line_number":2297,"context_line":"    # maintenability."},{"line_number":2298,"context_line":"    @utils.wip(\u0027waiting on bug #1424500\u0027)"},{"line_number":2299,"context_line":"    def test_list_projects_for_inherited_project_assignment(self):"}],"source_content_type":"text/x-python","patch_set":6,"id":"baa041b7_678cc47f","line":2296,"updated":"2015-04-03 01:12:22.000000000","message":"change \"fuctionality\" to \"functionality\"","commit_id":"51317c80836ea63e4a6ca39c92c97927e8a9733e"},{"author":{"_account_id":5707,"name":"Henry Nash","email":"henryn@linux.vnet.ibm.com","username":"henry-nash"},"change_message_id":"6fd4f14977b3b2f6b32a75ff5613e493a948be00","unresolved":false,"context_lines":[{"line_number":2294,"context_line":"    # TODO(samueldmq): Create another test class for role inheritance tests."},{"line_number":2295,"context_line":"    # The advantage would be to reduce the complexity of this test class and"},{"line_number":2296,"context_line":"    # have tests specific to this fuctionality grouped, easing readability and"},{"line_number":2297,"context_line":"    # maintenability."},{"line_number":2298,"context_line":"    @utils.wip(\u0027waiting on bug #1424500\u0027)"},{"line_number":2299,"context_line":"    def test_list_projects_for_inherited_project_assignment(self):"},{"line_number":2300,"context_line":"        # Enable os_inherit extension"}],"source_content_type":"text/x-python","patch_set":6,"id":"da9b358b_25eb167a","line":2297,"updated":"2015-04-01 12:49:18.000000000","message":"++","commit_id":"51317c80836ea63e4a6ca39c92c97927e8a9733e"},{"author":{"_account_id":6486,"name":"Brant Knudson","email":"blk@acm.org","username":"blk-u"},"change_message_id":"90452266d810d424ff8f9174cffa6afd5c7c1212","unresolved":false,"context_lines":[{"line_number":2294,"context_line":"    # TODO(samueldmq): Create another test class for role inheritance tests."},{"line_number":2295,"context_line":"    # The advantage would be to reduce the complexity of this test class and"},{"line_number":2296,"context_line":"    # have tests specific to this fuctionality grouped, easing readability and"},{"line_number":2297,"context_line":"    # maintenability."},{"line_number":2298,"context_line":"    @utils.wip(\u0027waiting on bug #1424500\u0027)"},{"line_number":2299,"context_line":"    def test_list_projects_for_inherited_project_assignment(self):"},{"line_number":2300,"context_line":"        # Enable os_inherit extension"}],"source_content_type":"text/x-python","patch_set":6,"id":"baa041b7_8791089c","line":2297,"updated":"2015-04-03 01:12:22.000000000","message":"change \"maintenability\" to \"maintainability\"","commit_id":"51317c80836ea63e4a6ca39c92c97927e8a9733e"},{"author":{"_account_id":6486,"name":"Brant Knudson","email":"blk@acm.org","username":"blk-u"},"change_message_id":"90452266d810d424ff8f9174cffa6afd5c7c1212","unresolved":false,"context_lines":[{"line_number":2325,"context_line":"            r \u003d self.get(url, token\u003dself.tokens[\u0027EMPLOYEE_ASSERTION\u0027])"},{"line_number":2326,"context_line":"            project_ids \u003d [project[\u0027id\u0027] for project in r.result[\u0027projects\u0027]]"},{"line_number":2327,"context_line":""},{"line_number":2328,"context_line":"            self.assertEqual(len(expected_project_ids), len(project_ids))"},{"line_number":2329,"context_line":"            for expected_project_id in expected_project_ids:"},{"line_number":2330,"context_line":"                self.assertIn(expected_project_id, project_ids,"},{"line_number":2331,"context_line":"                              \u0027Projects match failed for url %s\u0027 % url)"}],"source_content_type":"text/x-python","patch_set":6,"id":"baa041b7_475f80e5","line":2328,"updated":"2015-04-03 01:12:22.000000000","message":"use HasLength matcher.","commit_id":"51317c80836ea63e4a6ca39c92c97927e8a9733e"}]}
