)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":1179,"name":"Clay Gerrard","email":"clay.gerrard@gmail.com","username":"clay-gerrard"},"change_message_id":"c3217c99ae8b8b6be02d5c5ea29673c72c3664d0","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"327b0792_1b695b35","updated":"2023-03-01 16:04:56.000000000","message":"seems reasonable, i assume some tests didn\u0027t create their app from our pipeline wrappers\n\nwould be nice to revert this small diff in controllers.base and see some test that asserts 2 requests fails when there\u0027s 3 or 4 ","commit_id":"beeb4c009b7c6537a530d5deca089bbef1362ea5"},{"author":{"_account_id":1179,"name":"Clay Gerrard","email":"clay.gerrard@gmail.com","username":"clay-gerrard"},"change_message_id":"dd4eb3741b1474ca722773880c308d32f83c0a47","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"591efe26_e91b5d0e","updated":"2023-03-02 18:00:06.000000000","message":"nice test","commit_id":"b68cc893f74bf61a03439549d1ca5412cfc8d0ff"}],"swift/proxy/controllers/base.py":[{"author":{"_account_id":1179,"name":"Clay Gerrard","email":"clay.gerrard@gmail.com","username":"clay-gerrard"},"change_message_id":"c3217c99ae8b8b6be02d5c5ea29673c72c3664d0","unresolved":true,"context_lines":[{"line_number":440,"context_line":""},{"line_number":441,"context_line":"    # Try to cut through all the layers to the proxy app"},{"line_number":442,"context_line":"    try:"},{"line_number":443,"context_line":"        app \u003d app._pipeline_final_app"},{"line_number":444,"context_line":"    except AttributeError:"},{"line_number":445,"context_line":"        pass"},{"line_number":446,"context_line":"    # Check in environment cache and in memcache (in that order)"}],"source_content_type":"text/x-python","patch_set":1,"id":"f6b5d405_f10cc9b0","line":443,"updated":"2023-03-01 16:04:56.000000000","message":"oh i love pipeline final app!","commit_id":"beeb4c009b7c6537a530d5deca089bbef1362ea5"}],"test/unit/proxy/controllers/test_base.py":[{"author":{"_account_id":1179,"name":"Clay Gerrard","email":"clay.gerrard@gmail.com","username":"clay-gerrard"},"change_message_id":"dd4eb3741b1474ca722773880c308d32f83c0a47","unresolved":true,"context_lines":[{"line_number":376,"context_line":"        req \u003d Request.blank(\"/v1/a/c/o\", environ\u003d{\u0027swift.cache\u0027: FakeCache()})"},{"line_number":377,"context_line":"        req.get_response(filtered_app)"},{"line_number":378,"context_line":"        self.assertEqual([e[\u0027PATH_INFO\u0027] for e in final_app.captured_envs],"},{"line_number":379,"context_line":"                         [\u0027/v1/a\u0027, \u0027/v1/a/c\u0027, \u0027/v1/a/c/o\u0027])"},{"line_number":380,"context_line":""},{"line_number":381,"context_line":"    def test_get_object_info_swift_source(self):"},{"line_number":382,"context_line":"        app \u003d FakeApp()"}],"source_content_type":"text/x-python","patch_set":3,"id":"a44f6fa2_ef23e44c","line":379,"updated":"2023-03-02 18:00:06.000000000","message":"oh hell yes, AssertionError: Lists differ: [\u0027/v1/a\u0027, \u0027/v1/a\u0027, \u0027/v1/a\u0027, \u0027/v1/a/c\u0027, \u0027/v1/a/c\u0027, \u0027/v1/a/c\u0027, \u0027/v1/a/c/o\u0027] !\u003d [\u0027/v1/a\u0027, \u0027/v1/a/c\u0027, \u0027/v1/a/c/o\u0027]","commit_id":"b68cc893f74bf61a03439549d1ca5412cfc8d0ff"}]}
