)]}'
{"keystone/contrib/oauth1/controllers.py":[{"author":{"_account_id":6486,"name":"Brant Knudson","email":"blk@acm.org","username":"blk-u"},"change_message_id":"a11f945785e10044b82f946d96d4dc4f6f6b984f","unresolved":false,"context_lines":[{"line_number":39,"context_line":"    def base_url(cls, path\u003dNone):"},{"line_number":40,"context_line":"        \"\"\"Construct a path and pass it to V3Controller.base_url method.\"\"\""},{"line_number":41,"context_line":""},{"line_number":42,"context_line":"        path \u003d \u0027/OS-OAUTH1/\u0027 + cls.collection_name"},{"line_number":43,"context_line":"        return controller.V3Controller.base_url(path\u003dpath)"},{"line_number":44,"context_line":""},{"line_number":45,"context_line":"    @controller.protected()"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAUX%2F%2FIos%3D","line":42,"updated":"2014-02-16 14:16:58.000000000","message":"this function takes path as an argument and then ignores it. Add a comment to explain why this is correct.","commit_id":"bdd381c81043f47a8aa96c74cdf782a080cb0fa0"}],"keystone/tests/test_v3_oauth1.py":[{"author":{"_account_id":6486,"name":"Brant Knudson","email":"blk@acm.org","username":"blk-u"},"change_message_id":"33db382e0907cb9b742fcae4a783a90b10bf810f","unresolved":false,"context_lines":[{"line_number":159,"context_line":"        self.assertEqual(resp.result[\u0027consumer\u0027][\u0027id\u0027], consumer_id)"},{"line_number":160,"context_line":""},{"line_number":161,"context_line":"    def test_consumer_list(self):"},{"line_number":162,"context_line":"        self._consumer_create()"},{"line_number":163,"context_line":"        resp \u003d self.get(self.CONSUMER_URL)"},{"line_number":164,"context_line":"        entities \u003d resp.result[\u0027consumers\u0027]"},{"line_number":165,"context_line":"        self.assertIsNotNone(entities)"}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAAVn%2F%2FG4M%3D","line":162,"updated":"2014-03-09 15:45:55.000000000","message":"why does this test need to do self._consumer_create() now when it didn\u0027t before?","commit_id":"96c978e4f99f900521143707574c4c28f2e5b00e"},{"author":{"_account_id":6482,"name":"Steve Martinelli","email":"s.martinelli@gmail.com","username":"stevemar"},"change_message_id":"b6cb31f5c48aa2d47743d81b017700c71b855df4","unresolved":false,"context_lines":[{"line_number":159,"context_line":"        self.assertEqual(resp.result[\u0027consumer\u0027][\u0027id\u0027], consumer_id)"},{"line_number":160,"context_line":""},{"line_number":161,"context_line":"    def test_consumer_list(self):"},{"line_number":162,"context_line":"        self._consumer_create()"},{"line_number":163,"context_line":"        resp \u003d self.get(self.CONSUMER_URL)"},{"line_number":164,"context_line":"        entities \u003d resp.result[\u0027consumers\u0027]"},{"line_number":165,"context_line":"        self.assertIsNotNone(entities)"}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAAVn%2F%2FGUQ%3D","line":162,"in_reply_to":"AAAAVn%2F%2FG4M%3D","updated":"2014-03-09 20:46:43.000000000","message":"i suppose it doesn\u0027t necessarily need it. if you really want it removed, i can do so. It just adds a consumer, instead of expecting an empty list.","commit_id":"96c978e4f99f900521143707574c4c28f2e5b00e"},{"author":{"_account_id":6486,"name":"Brant Knudson","email":"blk@acm.org","username":"blk-u"},"change_message_id":"b35bfebea7b89221c5a3c33361d22549e8eb148d","unresolved":false,"context_lines":[{"line_number":159,"context_line":"        self.assertEqual(resp.result[\u0027consumer\u0027][\u0027id\u0027], consumer_id)"},{"line_number":160,"context_line":""},{"line_number":161,"context_line":"    def test_consumer_list(self):"},{"line_number":162,"context_line":"        self._consumer_create()"},{"line_number":163,"context_line":"        resp \u003d self.get(self.CONSUMER_URL)"},{"line_number":164,"context_line":"        entities \u003d resp.result[\u0027consumers\u0027]"},{"line_number":165,"context_line":"        self.assertIsNotNone(entities)"}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAAVn%2F%2FGSs%3D","line":162,"in_reply_to":"AAAAVn%2F%2FGUQ%3D","updated":"2014-03-09 20:49:57.000000000","message":"ok, just wondering why the change. Sounds like it\u0027s better for testing.","commit_id":"96c978e4f99f900521143707574c4c28f2e5b00e"}]}