)]}'
{"neutron/tests/unit/services/vpn/device_drivers/test_cisco_ipsec.py":[{"author":{"_account_id":7787,"name":"Kevin Benton","email":"kevin@benton.pub","username":"blak111"},"change_message_id":"e28229cf54a19bc83063de361508c235c40b30f1","unresolved":false,"context_lines":[{"line_number":1534,"context_line":"                                       u\u0027updated_pending_status\u0027: True}}"},{"line_number":1535,"context_line":"                            }]"},{"line_number":1536,"context_line":"        self.assertEqual(expected_report,"},{"line_number":1537,"context_line":"                         sorted(report, key\u003doperator.itemgetter(\u0027id\u0027)))"},{"line_number":1538,"context_line":"        # Check that service and connection statuses are updated"},{"line_number":1539,"context_line":"        self.assertEqual(constants.ACTIVE, vpn_service1.last_status)"},{"line_number":1540,"context_line":"        self.assertEqual(constants.ACTIVE,"}],"source_content_type":"text/x-python","patch_set":4,"id":"fa98f980_87967f3d","line":1537,"updated":"2014-09-04 09:58:22.000000000","message":"I don\u0027t think importing operator is necessary here:\n\nsorted(report, key\u003dlambda x: x.get(\u0027id\u0027))","commit_id":"6cfb4948ccc0215d54976cbdffd9bdf795fbb41c"},{"author":{"_account_id":8124,"name":"cbrandily","email":"zzelle@gmail.com","username":"cbrandily"},"change_message_id":"ca54f01f2c7e3fedf1f150a843df4f4329a139da","unresolved":false,"context_lines":[{"line_number":1534,"context_line":"                                       u\u0027updated_pending_status\u0027: True}}"},{"line_number":1535,"context_line":"                            }]"},{"line_number":1536,"context_line":"        self.assertEqual(expected_report,"},{"line_number":1537,"context_line":"                         sorted(report, key\u003doperator.itemgetter(\u0027id\u0027)))"},{"line_number":1538,"context_line":"        # Check that service and connection statuses are updated"},{"line_number":1539,"context_line":"        self.assertEqual(constants.ACTIVE, vpn_service1.last_status)"},{"line_number":1540,"context_line":"        self.assertEqual(constants.ACTIVE,"}],"source_content_type":"text/x-python","patch_set":4,"id":"fa98f980_d8c5c88e","line":1537,"in_reply_to":"fa98f980_87967f3d","updated":"2014-09-04 11:20:42.000000000","message":"... but that\u0027s exactly where operator should be used","commit_id":"6cfb4948ccc0215d54976cbdffd9bdf795fbb41c"},{"author":{"_account_id":6635,"name":"John Davidge","email":"john.davidge@rackspace.com","username":"john-davidge"},"change_message_id":"acd409e2c919e20289baaa88a92195b5e5d011b2","unresolved":false,"context_lines":[{"line_number":1534,"context_line":"                                       u\u0027updated_pending_status\u0027: True}}"},{"line_number":1535,"context_line":"                            }]"},{"line_number":1536,"context_line":"        self.assertEqual(expected_report,"},{"line_number":1537,"context_line":"                         sorted(report, key\u003doperator.itemgetter(\u0027id\u0027)))"},{"line_number":1538,"context_line":"        # Check that service and connection statuses are updated"},{"line_number":1539,"context_line":"        self.assertEqual(constants.ACTIVE, vpn_service1.last_status)"},{"line_number":1540,"context_line":"        self.assertEqual(constants.ACTIVE,"}],"source_content_type":"text/x-python","patch_set":4,"id":"fa98f980_820cadee","line":1537,"in_reply_to":"fa98f980_87967f3d","updated":"2014-09-09 10:21:24.000000000","message":"Consensus seems to be to go with operator here, so I\u0027m putting it back to how it was in patch set 4.","commit_id":"6cfb4948ccc0215d54976cbdffd9bdf795fbb41c"},{"author":{"_account_id":2592,"name":"Mark McClain","email":"mark@mcclain.xyz","username":"markmcclain"},"change_message_id":"228e6e6c5bbd9e2f9856e5da1b1ffeebc6a75b9b","unresolved":false,"context_lines":[{"line_number":1534,"context_line":"                                       u\u0027updated_pending_status\u0027: True}}"},{"line_number":1535,"context_line":"                            }]"},{"line_number":1536,"context_line":"        self.assertEqual(expected_report,"},{"line_number":1537,"context_line":"                         sorted(report, key\u003doperator.itemgetter(\u0027id\u0027)))"},{"line_number":1538,"context_line":"        # Check that service and connection statuses are updated"},{"line_number":1539,"context_line":"        self.assertEqual(constants.ACTIVE, vpn_service1.last_status)"},{"line_number":1540,"context_line":"        self.assertEqual(constants.ACTIVE,"}],"source_content_type":"text/x-python","patch_set":4,"id":"fa98f980_24291c9b","line":1537,"in_reply_to":"fa98f980_d8c5c88e","updated":"2014-09-05 17:08:42.000000000","message":"+1 to Cedric.  importing operator is the proper usage here.","commit_id":"6cfb4948ccc0215d54976cbdffd9bdf795fbb41c"},{"author":{"_account_id":2592,"name":"Mark McClain","email":"mark@mcclain.xyz","username":"markmcclain"},"change_message_id":"f3aca535b0d9ef0eec91f462fc25ab86d53648a0","unresolved":false,"context_lines":[{"line_number":1533,"context_line":"                                       u\u0027updated_pending_status\u0027: True}}"},{"line_number":1534,"context_line":"                            }]"},{"line_number":1535,"context_line":"        self.assertEqual(expected_report,"},{"line_number":1536,"context_line":"                         sorted(report, key\u003dlambda x: x.get(\u0027id\u0027)))"},{"line_number":1537,"context_line":"        # Check that service and connection statuses are updated"},{"line_number":1538,"context_line":"        self.assertEqual(constants.ACTIVE, vpn_service1.last_status)"},{"line_number":1539,"context_line":"        self.assertEqual(constants.ACTIVE,"}],"source_content_type":"text/x-python","patch_set":6,"id":"fa98f980_c4605003","line":1536,"updated":"2014-09-05 17:07:27.000000000","message":"You should use:\n\nimport operator\n\nsorted(report, key\u003doperator.itemgetter(\u0027id\u0027))","commit_id":"a164326eb5fa4ef33661af39c9a3673a5c821509"},{"author":{"_account_id":6635,"name":"John Davidge","email":"john.davidge@rackspace.com","username":"john-davidge"},"change_message_id":"57d161a2d1cfdc6851bfa9b0bf4e8421287dd48e","unresolved":false,"context_lines":[{"line_number":1533,"context_line":"                                       u\u0027updated_pending_status\u0027: True}}"},{"line_number":1534,"context_line":"                            }]"},{"line_number":1535,"context_line":"        self.assertEqual(expected_report,"},{"line_number":1536,"context_line":"                         sorted(report, key\u003dlambda x: x.get(\u0027id\u0027)))"},{"line_number":1537,"context_line":"        # Check that service and connection statuses are updated"},{"line_number":1538,"context_line":"        self.assertEqual(constants.ACTIVE, vpn_service1.last_status)"},{"line_number":1539,"context_line":"        self.assertEqual(constants.ACTIVE,"}],"source_content_type":"text/x-python","patch_set":6,"id":"fa98f980_3044a8b6","line":1536,"in_reply_to":"fa98f980_c4605003","updated":"2014-09-09 10:21:31.000000000","message":"Consensus seems to be to go with operator here, so I\u0027m putting it back to how it was in patch set 4.","commit_id":"a164326eb5fa4ef33661af39c9a3673a5c821509"}]}