)]}'
{"keystone/tests/unit/test_v3_federation.py":[{"author":{"_account_id":5046,"name":"Lance Bragstad","email":"lbragstad@redhat.com","username":"ldbragst"},"change_message_id":"7958a9f31300de7c3e9debe9098eb819e02575e0","unresolved":false,"context_lines":[{"line_number":2153,"context_line":"        # delete the idp, the related federated user should be deleted as well."},{"line_number":2154,"context_line":"        PROVIDERS.federation_api.delete_idp(idp_id)"},{"line_number":2155,"context_line":""},{"line_number":2156,"context_line":"        hints \u003d driver_hints.Hints()"},{"line_number":2157,"context_line":"        hints.add_filter(\u0027idp_id\u0027, idp_id)"},{"line_number":2158,"context_line":"        fed_users \u003d PROVIDERS.identity_api.list_users(idp_id, hints)"},{"line_number":2159,"context_line":"        self.assertEqual([], fed_users)"},{"line_number":2160,"context_line":""},{"line_number":2161,"context_line":"    def test_scope_to_bad_project(self):"},{"line_number":2162,"context_line":"        \"\"\"Scope unscoped token with a project we don\u0027t have access to.\"\"\""}],"source_content_type":"text/x-python","patch_set":1,"id":"3f79a3b5_b9ac939f","line":2159,"range":{"start_line":2156,"start_character":8,"end_line":2159,"end_character":39},"updated":"2018-08-22 16:10:28.000000000","message":"Can we check to make sure this returns users prior to deleting the identity provider?","commit_id":"d33a7348b4ff0a2ad825bbfe81872463d0b9d9e0"},{"author":{"_account_id":15054,"name":"wangxiyuan","email":"wangxiyuan1007@gmail.com","username":"wangxiyuan"},"change_message_id":"2fb9de01fd6f95cda4ecf31b793d8e79c5ee306a","unresolved":false,"context_lines":[{"line_number":2153,"context_line":"        # delete the idp, the related federated user should be deleted as well."},{"line_number":2154,"context_line":"        PROVIDERS.federation_api.delete_idp(idp_id)"},{"line_number":2155,"context_line":""},{"line_number":2156,"context_line":"        hints \u003d driver_hints.Hints()"},{"line_number":2157,"context_line":"        hints.add_filter(\u0027idp_id\u0027, idp_id)"},{"line_number":2158,"context_line":"        fed_users \u003d PROVIDERS.identity_api.list_users(idp_id, hints)"},{"line_number":2159,"context_line":"        self.assertEqual([], fed_users)"},{"line_number":2160,"context_line":""},{"line_number":2161,"context_line":"    def test_scope_to_bad_project(self):"},{"line_number":2162,"context_line":"        \"\"\"Scope unscoped token with a project we don\u0027t have access to.\"\"\""}],"source_content_type":"text/x-python","patch_set":1,"id":"3f79a3b5_000bb97c","line":2159,"range":{"start_line":2156,"start_character":8,"end_line":2159,"end_character":39},"in_reply_to":"3f79a3b5_b9ac939f","updated":"2018-08-23 03:53:07.000000000","message":"Done, I add a pre-check to ensure there are some fed users with the specified idp first.","commit_id":"d33a7348b4ff0a2ad825bbfe81872463d0b9d9e0"},{"author":{"_account_id":5046,"name":"Lance Bragstad","email":"lbragstad@redhat.com","username":"ldbragst"},"change_message_id":"5127ae7b5d23b6347ab8465545cbceedb6d2079f","unresolved":false,"context_lines":[{"line_number":2153,"context_line":"        # There are som fed users with the specified idp."},{"line_number":2154,"context_line":"        hints \u003d driver_hints.Hints()"},{"line_number":2155,"context_line":"        hints.add_filter(\u0027idp_id\u0027, idp_id)"},{"line_number":2156,"context_line":"        fed_users \u003d PROVIDERS.identity_api.list_users(hints\u003dhints)"},{"line_number":2157,"context_line":"        idp_domain_id \u003d PROVIDERS.federation_api.get_idp(idp_id)[\u0027domain_id\u0027]"},{"line_number":2158,"context_line":"        for fed_user in fed_users:"},{"line_number":2159,"context_line":"            self.assertEqual(idp_domain_id, fed_user[\u0027domain_id\u0027])"}],"source_content_type":"text/x-python","patch_set":4,"id":"3f79a3b5_51365a53","line":2156,"updated":"2018-08-23 12:30:09.000000000","message":"We could assert that this has a specific length, otherwise if it is an empty list we won\u0027t hit the assertion at line 2159.","commit_id":"14722c6f5fd160a509bfb91c808a3859dd71832b"}]}