)]}'
{"keystone/tests/unit/test_auth_plugin.py":[{"author":{"_account_id":1916,"name":"Guang Yee","email":"gyee@suse.com","username":"guang-yee"},"change_message_id":"ecc8edb94fc1dd9bd87f1b3a37193f3045f16f8d","unresolved":false,"context_lines":[{"line_number":137,"context_line":"        config_files.append(tests.dirs.tests_conf(\u0027test_auth_plugin.conf\u0027))"},{"line_number":138,"context_line":"        return config_files"},{"line_number":139,"context_line":""},{"line_number":140,"context_line":"    def auth_plugin_config_override(self, methods\u003dNone, **method_classes):"},{"line_number":141,"context_line":"        # Do not apply the auth plugin overrides so that the config file is"},{"line_number":142,"context_line":"        # tested"},{"line_number":143,"context_line":"        pass"}],"source_content_type":"text/x-python","patch_set":2,"id":"da20952f_b6f58184","line":140,"updated":"2015-08-26 19:55:31.000000000","message":"I don\u0027t understand this change. If I am reading the code correctly, config_overrides() for these tests was intended to be a no-op. Not it will use the config_overrides() from the base class? Seems like a change in behavior, and Jenkins still go green?","commit_id":"a0ad88b333301da2cba27e7f7bc352209874b80f"},{"author":{"_account_id":2903,"name":"Morgan Fainberg","email":"morgan.fainberg@gmail.com","username":"mdrnstm"},"change_message_id":"120b38b9ff9eb692618b396a5abd2122ad837860","unresolved":false,"context_lines":[{"line_number":137,"context_line":"        config_files.append(tests.dirs.tests_conf(\u0027test_auth_plugin.conf\u0027))"},{"line_number":138,"context_line":"        return config_files"},{"line_number":139,"context_line":""},{"line_number":140,"context_line":"    def auth_plugin_config_override(self, methods\u003dNone, **method_classes):"},{"line_number":141,"context_line":"        # Do not apply the auth plugin overrides so that the config file is"},{"line_number":142,"context_line":"        # tested"},{"line_number":143,"context_line":"        pass"}],"source_content_type":"text/x-python","patch_set":2,"id":"da20952f_3b0ab1d3","line":140,"in_reply_to":"da20952f_b6f58184","updated":"2015-08-26 23:11:55.000000000","message":"The whole reason to make it a no op was so it wouldn\u0027t configure the auth plugins. Override the correct method instead.","commit_id":"a0ad88b333301da2cba27e7f7bc352209874b80f"}]}