)]}'
{"keystone/assignment/controllers.py":[{"author":{"_account_id":17860,"name":"Samuel de Medeiros Queiroz","email":"samueldmq@gmail.com","username":"samueldmq"},"change_message_id":"5f111ecc9c30e80b9fcc46d38b254a059d000f4e","unresolved":false,"context_lines":[{"line_number":475,"context_line":"                endpoint, implied_role)"},{"line_number":476,"context_line":"            response[\"role_inference\"][\u0027implies\u0027].append(implied_response)"},{"line_number":477,"context_line":"        response[\"links\"] \u003d {"},{"line_number":478,"context_line":"            \"self\": endpoint + \"/v3/prior_role/\" + prior_id"},{"line_number":479,"context_line":"        }"},{"line_number":480,"context_line":"        return response"},{"line_number":481,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"7aa08908_0d789b29","line":478,"range":{"start_line":478,"start_character":31,"end_line":478,"end_character":59},"updated":"2016-06-07 13:13:27.000000000","message":"Given this represents a link to all the implied roles of a given prior role, the link should be:\n\n    /v3/roles/--prior-role-id--/implies\"","commit_id":"2ebb486ab4e23bfd37c6a30296e46190c9889dd0"},{"author":{"_account_id":7725,"name":"David Stanek","email":"dstanek@dstanek.com","username":"dstanek"},"change_message_id":"bc85b0820efdd69222981072e647cc44d19d5441","unresolved":false,"context_lines":[{"line_number":475,"context_line":"                endpoint, implied_role)"},{"line_number":476,"context_line":"            response[\"role_inference\"][\u0027implies\u0027].append(implied_response)"},{"line_number":477,"context_line":"        response[\"links\"] \u003d {"},{"line_number":478,"context_line":"            \"self\": endpoint + \"/v3/roles/--prior-role-id--/implies\""},{"line_number":479,"context_line":"        }"},{"line_number":480,"context_line":"        return response"},{"line_number":481,"context_line":""}],"source_content_type":"text/x-python","patch_set":4,"id":"7aa08908_7f69d48b","line":478,"updated":"2016-06-13 18:18:19.000000000","message":"Do we literally want --prior-role-id-- or the actual role id?","commit_id":"3870f611160f50e366044562a46f18e4e01e3e73"},{"author":{"_account_id":5046,"name":"Lance Bragstad","email":"lbragstad@redhat.com","username":"ldbragst"},"change_message_id":"b3449700343d29f010b4f37983ad7bc8a3e5fb5a","unresolved":false,"context_lines":[{"line_number":475,"context_line":"                endpoint, implied_role)"},{"line_number":476,"context_line":"            response[\"role_inference\"][\u0027implies\u0027].append(implied_response)"},{"line_number":477,"context_line":"        response[\"links\"] \u003d {"},{"line_number":478,"context_line":"            \"self\": endpoint + \"/v3/roles/--prior-role-id--/implies\""},{"line_number":479,"context_line":"        }"},{"line_number":480,"context_line":"        return response"},{"line_number":481,"context_line":""}],"source_content_type":"text/x-python","patch_set":4,"id":"7aa08908_50596234","line":478,"in_reply_to":"7aa08908_7f69d48b","updated":"2016-06-13 20:36:47.000000000","message":"++","commit_id":"3870f611160f50e366044562a46f18e4e01e3e73"}],"keystone/tests/unit/test_v3_assignment.py":[{"author":{"_account_id":17860,"name":"Samuel de Medeiros Queiroz","email":"samueldmq@gmail.com","username":"samueldmq"},"change_message_id":"715625fb96c930eede20bd75f8f126770d400868","unresolved":false,"context_lines":[{"line_number":2447,"context_line":"        actual_implied_ids \u003d [implied[\u0027id\u0027]"},{"line_number":2448,"context_line":"                              for implied in role_inference[\u0027implies\u0027]]"},{"line_number":2449,"context_line":""},{"line_number":2450,"context_line":"        for expected_id in expected_implied_ids:"},{"line_number":2451,"context_line":"            self.assertIn(expected_id, actual_implied_ids)"},{"line_number":2452,"context_line":"        self.assertEqual(len(expected_implied_ids),"},{"line_number":2453,"context_line":"                         len(role_inference[\u0027implies\u0027]))"},{"line_number":2454,"context_line":""},{"line_number":2455,"context_line":"        self.assertIsNotNone(role_inference[\u0027prior_role\u0027][\u0027links\u0027][\u0027self\u0027])"},{"line_number":2456,"context_line":"        for implied in role_inference[\u0027implies\u0027]:"}],"source_content_type":"text/x-python","patch_set":5,"id":"7aa08908_13d6c44e","line":2453,"range":{"start_line":2450,"start_character":8,"end_line":2453,"end_character":56},"updated":"2016-06-13 21:02:28.000000000","message":"This could be replaced with assertItemsEqual(..)","commit_id":"ff748037748aacefc098bf4a879617c0cb658a8d"},{"author":{"_account_id":17860,"name":"Samuel de Medeiros Queiroz","email":"samueldmq@gmail.com","username":"samueldmq"},"change_message_id":"67f9e9d866737889a09ec0fa171676509aaad537","unresolved":false,"context_lines":[{"line_number":2447,"context_line":"        actual_implied_ids \u003d [implied[\u0027id\u0027]"},{"line_number":2448,"context_line":"                              for implied in role_inference[\u0027implies\u0027]]"},{"line_number":2449,"context_line":""},{"line_number":2450,"context_line":"        self.assertItemsEqual(expected_implied_ids, role_inference[\u0027implies\u0027])"},{"line_number":2451,"context_line":""},{"line_number":2452,"context_line":"        self.assertIsNotNone(role_inference[\u0027prior_role\u0027][\u0027links\u0027][\u0027self\u0027])"},{"line_number":2453,"context_line":"        for implied in role_inference[\u0027implies\u0027]:"}],"source_content_type":"text/x-python","patch_set":6,"id":"7aa08908_3683ee57","line":2450,"range":{"start_line":2450,"start_character":52,"end_line":2450,"end_character":77},"updated":"2016-06-13 21:25:42.000000000","message":"this needs to be actual_implied_ids","commit_id":"555a9b4ee387ee7f7a5c60f150caf49586d92d23"}]}
