)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":6486,"name":"Brant Knudson","email":"blk@acm.org","username":"blk-u"},"change_message_id":"e4fdb25721f3cf98baec9e6ce92e73a19afb6e45","unresolved":false,"context_lines":[{"line_number":7,"context_line":"Create new user plugin tests"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Create a base test setup for user auth plugin tests and implement some"},{"line_number":10,"context_line":"basic user plugin tests."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Change-Id: Ia31b0ee5be204a678369e4f2f8cd8e3401087cfd"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"3acd31a7_477bc08c","line":10,"updated":"2015-05-04 20:19:29.000000000","message":"why? Does this improve coverage or something?","commit_id":"22afdedce8fcd6f92faafa422c8099a5710d9407"}],"keystonemiddleware/tests/unit/auth_token/test_user_auth_plugin.py":[{"author":{"_account_id":6486,"name":"Brant Knudson","email":"blk@acm.org","username":"blk-u"},"change_message_id":"84596954264fe2b49af9a22cffc6196bc22b8380","unresolved":false,"context_lines":[{"line_number":9,"context_line":"# WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. See the"},{"line_number":10,"context_line":"# License for the specific language governing permissions and limitations"},{"line_number":11,"context_line":"# under the License."},{"line_number":12,"context_line":"#"},{"line_number":13,"context_line":"import uuid"},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"from keystoneclient import auth"}],"source_content_type":"text/x-python","patch_set":2,"id":"7a016987_51cd6a2f","line":12,"updated":"2015-05-22 22:34:52.000000000","message":"(nit) add a blank line","commit_id":"22afdedce8fcd6f92faafa422c8099a5710d9407"},{"author":{"_account_id":6486,"name":"Brant Knudson","email":"blk@acm.org","username":"blk-u"},"change_message_id":"84596954264fe2b49af9a22cffc6196bc22b8380","unresolved":false,"context_lines":[{"line_number":17,"context_line":""},{"line_number":18,"context_line":"from keystonemiddleware.tests.unit.auth_token import base"},{"line_number":19,"context_line":""},{"line_number":20,"context_line":"BASE_URI \u003d \u0027https://keystone.example.com:1234\u0027"},{"line_number":21,"context_line":"AUTH_URL \u003d \u0027https://keystone.auth.com:1234\u0027"},{"line_number":22,"context_line":""},{"line_number":23,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"7a016987_71f1866a","line":20,"updated":"2015-05-22 22:34:52.000000000","message":"why is keystone.example.com:1234 used here instead of something else? add a comment.","commit_id":"22afdedce8fcd6f92faafa422c8099a5710d9407"},{"author":{"_account_id":6486,"name":"Brant Knudson","email":"blk@acm.org","username":"blk-u"},"change_message_id":"e42ea0d830bcf9d59ad095fc35acba5c144932f0","unresolved":false,"context_lines":[{"line_number":39,"context_line":"        self.assertEqual(token.user_id, token_data.user_id)"},{"line_number":40,"context_line":"        try:"},{"line_number":41,"context_line":"            trust_id \u003d token.trust_id"},{"line_number":42,"context_line":"        except KeyError:"},{"line_number":43,"context_line":"            trust_id \u003d None"},{"line_number":44,"context_line":"        self.assertEqual(trust_id, token_data.trust_id)"},{"line_number":45,"context_line":"        self.assertEqual(self.get_role_names(token), token_data.role_names)"}],"source_content_type":"text/x-python","patch_set":2,"id":"7a016987_d766eaeb","line":42,"updated":"2015-05-22 22:49:33.000000000","message":"accessing an attribute can raise KeyError?","commit_id":"22afdedce8fcd6f92faafa422c8099a5710d9407"}]}