)]}'
{"neutron/tests/unit/bigswitch/test_servermanager.py":[{"author":{"_account_id":2592,"name":"Mark McClain","email":"mark@mcclain.xyz","username":"markmcclain"},"change_message_id":"cb7b613b812ad5b5809cc0295b8988696c25fdfa","unresolved":false,"context_lines":[{"line_number":129,"context_line":"    def test_no_reconnect_recurse_to_infinity(self):"},{"line_number":130,"context_line":"        # retry uses recursion when a reconnect is necessary"},{"line_number":131,"context_line":"        # this test makes sure it stops after 1 recursive call"},{"line_number":132,"context_line":"        reqmock \u003d mock.MagicMock()"},{"line_number":133,"context_line":"        respmock \u003d mock.MagicMock().return_value"},{"line_number":134,"context_line":"        respmock.getheader.return_value \u003d \u0027HASHHEADER\u0027"},{"line_number":135,"context_line":"        sp \u003d servermanager.ServerPool()"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAVn%2F%2B%2BkA%3D","line":132,"updated":"2014-03-10 22:22:10.000000000","message":"I\u0027d suggest removing this line and moving line 149 inside of the with context.","commit_id":"57fc13f963e498d084eb4180a4759970199eca2c"},{"author":{"_account_id":7787,"name":"Kevin Benton","email":"kevin@benton.pub","username":"blak111"},"change_message_id":"31d515fc6b039f60b76de61978cca78acf7986d9","unresolved":false,"context_lines":[{"line_number":129,"context_line":"    def test_no_reconnect_recurse_to_infinity(self):"},{"line_number":130,"context_line":"        # retry uses recursion when a reconnect is necessary"},{"line_number":131,"context_line":"        # this test makes sure it stops after 1 recursive call"},{"line_number":132,"context_line":"        reqmock \u003d mock.MagicMock()"},{"line_number":133,"context_line":"        respmock \u003d mock.MagicMock().return_value"},{"line_number":134,"context_line":"        respmock.getheader.return_value \u003d \u0027HASHHEADER\u0027"},{"line_number":135,"context_line":"        sp \u003d servermanager.ServerPool()"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAVn%2F%2B%2BYk%3D","line":132,"in_reply_to":"AAAAVn%2F%2B%2BkA%3D","updated":"2014-03-10 22:59:17.000000000","message":"Done","commit_id":"57fc13f963e498d084eb4180a4759970199eca2c"},{"author":{"_account_id":2592,"name":"Mark McClain","email":"mark@mcclain.xyz","username":"markmcclain"},"change_message_id":"cb7b613b812ad5b5809cc0295b8988696c25fdfa","unresolved":false,"context_lines":[{"line_number":130,"context_line":"        # retry uses recursion when a reconnect is necessary"},{"line_number":131,"context_line":"        # this test makes sure it stops after 1 recursive call"},{"line_number":132,"context_line":"        reqmock \u003d mock.MagicMock()"},{"line_number":133,"context_line":"        respmock \u003d mock.MagicMock().return_value"},{"line_number":134,"context_line":"        respmock.getheader.return_value \u003d \u0027HASHHEADER\u0027"},{"line_number":135,"context_line":"        sp \u003d servermanager.ServerPool()"},{"line_number":136,"context_line":"        with mock.patch(HTTPCON) as conmock:"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAVn%2F%2B%2Bk0%3D","line":133,"updated":"2014-03-10 22:22:10.000000000","message":"Consider removing line 133 and merging line 134 into 138.","commit_id":"57fc13f963e498d084eb4180a4759970199eca2c"},{"author":{"_account_id":7787,"name":"Kevin Benton","email":"kevin@benton.pub","username":"blak111"},"change_message_id":"31d515fc6b039f60b76de61978cca78acf7986d9","unresolved":false,"context_lines":[{"line_number":130,"context_line":"        # retry uses recursion when a reconnect is necessary"},{"line_number":131,"context_line":"        # this test makes sure it stops after 1 recursive call"},{"line_number":132,"context_line":"        reqmock \u003d mock.MagicMock()"},{"line_number":133,"context_line":"        respmock \u003d mock.MagicMock().return_value"},{"line_number":134,"context_line":"        respmock.getheader.return_value \u003d \u0027HASHHEADER\u0027"},{"line_number":135,"context_line":"        sp \u003d servermanager.ServerPool()"},{"line_number":136,"context_line":"        with mock.patch(HTTPCON) as conmock:"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAVn%2F%2B%2BWo%3D","line":133,"in_reply_to":"AAAAVn%2F%2B%2Bk0%3D","updated":"2014-03-10 22:59:17.000000000","message":"Done","commit_id":"57fc13f963e498d084eb4180a4759970199eca2c"},{"author":{"_account_id":2592,"name":"Mark McClain","email":"mark@mcclain.xyz","username":"markmcclain"},"change_message_id":"cb7b613b812ad5b5809cc0295b8988696c25fdfa","unresolved":false,"context_lines":[{"line_number":156,"context_line":"            instance \u003d conmock.return_value"},{"line_number":157,"context_line":"            instance.request \u003d reqmock"},{"line_number":158,"context_line":"            resp \u003d sp.servers[0].rest_call(\u0027GET\u0027, \u0027/\u0027)"},{"line_number":159,"context_line":"        self.assertEqual(resp, (0, None, None, None))"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAVn%2F%2B%2Bjs%3D","line":159,"updated":"2014-03-10 22:22:10.000000000","message":"this should be indented","commit_id":"57fc13f963e498d084eb4180a4759970199eca2c"},{"author":{"_account_id":7787,"name":"Kevin Benton","email":"kevin@benton.pub","username":"blak111"},"change_message_id":"31d515fc6b039f60b76de61978cca78acf7986d9","unresolved":false,"context_lines":[{"line_number":156,"context_line":"            instance \u003d conmock.return_value"},{"line_number":157,"context_line":"            instance.request \u003d reqmock"},{"line_number":158,"context_line":"            resp \u003d sp.servers[0].rest_call(\u0027GET\u0027, \u0027/\u0027)"},{"line_number":159,"context_line":"        self.assertEqual(resp, (0, None, None, None))"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAVn%2F%2B%2BaQ%3D","line":159,"in_reply_to":"AAAAVn%2F%2B%2Bjs%3D","updated":"2014-03-10 22:59:17.000000000","message":"Done","commit_id":"57fc13f963e498d084eb4180a4759970199eca2c"}]}