)]}'
{"nova/tests/unit/api/openstack/compute/test_server_start_stop.py":[{"author":{"_account_id":12175,"name":"Eli Qiao","email":"qiaoliyong@gmail.com","username":"Eli"},"change_message_id":"4b6b8067c6620c1703976bff9b1bfd81083d23e7","unresolved":false,"context_lines":[{"line_number":31,"context_line":"from nova.tests.unit.api.openstack import fakes"},{"line_number":32,"context_line":""},{"line_number":33,"context_line":""},{"line_number":34,"context_line":"def fake_instance_get(context, instance_id,"},{"line_number":35,"context_line":"                      columns_to_join\u003dNone, use_slave\u003dFalse):"},{"line_number":36,"context_line":"    result \u003d fakes.stub_instance(id\u003d1, uuid\u003dinstance_id)"},{"line_number":37,"context_line":"    result[\u0027created_at\u0027] \u003d None"},{"line_number":38,"context_line":"    result[\u0027deleted_at\u0027] \u003d None"},{"line_number":39,"context_line":"    result[\u0027updated_at\u0027] \u003d None"},{"line_number":40,"context_line":"    result[\u0027deleted\u0027] \u003d 0"},{"line_number":41,"context_line":"    result[\u0027info_cache\u0027] \u003d {\u0027network_info\u0027: \u0027[]\u0027,"},{"line_number":42,"context_line":"                            \u0027instance_uuid\u0027: result[\u0027uuid\u0027]}"},{"line_number":43,"context_line":"    return result"},{"line_number":44,"context_line":""},{"line_number":45,"context_line":""},{"line_number":46,"context_line":"def fake_start_stop_not_ready(self, context, instance):"},{"line_number":47,"context_line":"    raise exception.InstanceNotReady(instance_id\u003dinstance[\"uuid\"])"},{"line_number":48,"context_line":""},{"line_number":49,"context_line":""},{"line_number":50,"context_line":"def fake_start_stop_locked_server(self, context, instance):"},{"line_number":51,"context_line":"    raise exception.InstanceIsLocked(instance_uuid\u003dinstance[\u0027uuid\u0027])"},{"line_number":52,"context_line":""},{"line_number":53,"context_line":""},{"line_number":54,"context_line":"def fake_start_stop_invalid_state(self, context, instance):"},{"line_number":55,"context_line":"    raise exception.InstanceIsLocked(instance_uuid\u003dinstance[\u0027uuid\u0027])"},{"line_number":56,"context_line":""},{"line_number":57,"context_line":""},{"line_number":58,"context_line":"class ServerStartStopTestV21(test.TestCase):"}],"source_content_type":"text/x-python","patch_set":2,"id":"9a68dd71_2154af52","line":55,"range":{"start_line":34,"start_character":0,"end_line":55,"end_character":68},"updated":"2016-01-27 03:22:37.000000000","message":"please remove these unused fake function too.","commit_id":"62e64a5343d5f92fc11b6e9c2c3c1b62c813ee40"},{"author":{"_account_id":15888,"name":"Zhenyu Zheng","email":"zheng.zhenyu@outlook.com","username":"Kevin_Zheng"},"change_message_id":"48830458a19b0758881786b946e433a3deeb19ea","unresolved":false,"context_lines":[{"line_number":31,"context_line":"from nova.tests.unit.api.openstack import fakes"},{"line_number":32,"context_line":""},{"line_number":33,"context_line":""},{"line_number":34,"context_line":"def fake_instance_get(context, instance_id,"},{"line_number":35,"context_line":"                      columns_to_join\u003dNone, use_slave\u003dFalse):"},{"line_number":36,"context_line":"    result \u003d fakes.stub_instance(id\u003d1, uuid\u003dinstance_id)"},{"line_number":37,"context_line":"    result[\u0027created_at\u0027] \u003d None"},{"line_number":38,"context_line":"    result[\u0027deleted_at\u0027] \u003d None"},{"line_number":39,"context_line":"    result[\u0027updated_at\u0027] \u003d None"},{"line_number":40,"context_line":"    result[\u0027deleted\u0027] \u003d 0"},{"line_number":41,"context_line":"    result[\u0027info_cache\u0027] \u003d {\u0027network_info\u0027: \u0027[]\u0027,"},{"line_number":42,"context_line":"                            \u0027instance_uuid\u0027: result[\u0027uuid\u0027]}"},{"line_number":43,"context_line":"    return result"},{"line_number":44,"context_line":""},{"line_number":45,"context_line":""},{"line_number":46,"context_line":"def fake_start_stop_not_ready(self, context, instance):"},{"line_number":47,"context_line":"    raise exception.InstanceNotReady(instance_id\u003dinstance[\"uuid\"])"},{"line_number":48,"context_line":""},{"line_number":49,"context_line":""},{"line_number":50,"context_line":"def fake_start_stop_locked_server(self, context, instance):"},{"line_number":51,"context_line":"    raise exception.InstanceIsLocked(instance_uuid\u003dinstance[\u0027uuid\u0027])"},{"line_number":52,"context_line":""},{"line_number":53,"context_line":""},{"line_number":54,"context_line":"def fake_start_stop_invalid_state(self, context, instance):"},{"line_number":55,"context_line":"    raise exception.InstanceIsLocked(instance_uuid\u003dinstance[\u0027uuid\u0027])"},{"line_number":56,"context_line":""},{"line_number":57,"context_line":""},{"line_number":58,"context_line":"class ServerStartStopTestV21(test.TestCase):"}],"source_content_type":"text/x-python","patch_set":2,"id":"9a68dd71_c1f2736f","line":55,"range":{"start_line":34,"start_character":0,"end_line":55,"end_character":68},"in_reply_to":"9a68dd71_2154af52","updated":"2016-01-27 03:24:57.000000000","message":"Hi, I used these functions in the second patch set, check out  ln 78","commit_id":"62e64a5343d5f92fc11b6e9c2c3c1b62c813ee40"},{"author":{"_account_id":12175,"name":"Eli Qiao","email":"qiaoliyong@gmail.com","username":"Eli"},"change_message_id":"9be5a5cec651e6cedc34f839295796afc27cfbbf","unresolved":false,"context_lines":[{"line_number":31,"context_line":"from nova.tests.unit.api.openstack import fakes"},{"line_number":32,"context_line":""},{"line_number":33,"context_line":""},{"line_number":34,"context_line":"def fake_instance_get(context, instance_id,"},{"line_number":35,"context_line":"                      columns_to_join\u003dNone, use_slave\u003dFalse):"},{"line_number":36,"context_line":"    result \u003d fakes.stub_instance(id\u003d1, uuid\u003dinstance_id)"},{"line_number":37,"context_line":"    result[\u0027created_at\u0027] \u003d None"},{"line_number":38,"context_line":"    result[\u0027deleted_at\u0027] \u003d None"},{"line_number":39,"context_line":"    result[\u0027updated_at\u0027] \u003d None"},{"line_number":40,"context_line":"    result[\u0027deleted\u0027] \u003d 0"},{"line_number":41,"context_line":"    result[\u0027info_cache\u0027] \u003d {\u0027network_info\u0027: \u0027[]\u0027,"},{"line_number":42,"context_line":"                            \u0027instance_uuid\u0027: result[\u0027uuid\u0027]}"},{"line_number":43,"context_line":"    return result"},{"line_number":44,"context_line":""},{"line_number":45,"context_line":""},{"line_number":46,"context_line":"def fake_start_stop_not_ready(self, context, instance):"},{"line_number":47,"context_line":"    raise exception.InstanceNotReady(instance_id\u003dinstance[\"uuid\"])"},{"line_number":48,"context_line":""},{"line_number":49,"context_line":""},{"line_number":50,"context_line":"def fake_start_stop_locked_server(self, context, instance):"},{"line_number":51,"context_line":"    raise exception.InstanceIsLocked(instance_uuid\u003dinstance[\u0027uuid\u0027])"},{"line_number":52,"context_line":""},{"line_number":53,"context_line":""},{"line_number":54,"context_line":"def fake_start_stop_invalid_state(self, context, instance):"},{"line_number":55,"context_line":"    raise exception.InstanceIsLocked(instance_uuid\u003dinstance[\u0027uuid\u0027])"},{"line_number":56,"context_line":""},{"line_number":57,"context_line":""},{"line_number":58,"context_line":"class ServerStartStopTestV21(test.TestCase):"}],"source_content_type":"text/x-python","patch_set":2,"id":"9a68dd71_a1bbbf74","line":55,"range":{"start_line":34,"start_character":0,"end_line":55,"end_character":68},"in_reply_to":"9a68dd71_c1f2736f","updated":"2016-01-27 03:27:27.000000000","message":"sorry, I missed it due to gerrit.\nbut actually, you don\u0027t need to use it at all.\nyou can try:\n\n@mock.patch.object(compute_api.API, \u0027start\u0027, side_effect\u003dexception.InstanceIsLocked(instance_uuid\u003dinstance[\u0027uuid\u0027]))\ndef test_start(self, start_mock):","commit_id":"62e64a5343d5f92fc11b6e9c2c3c1b62c813ee40"},{"author":{"_account_id":7634,"name":"Takashi Natsume","email":"takanattie@gmail.com","username":"natsumet"},"change_message_id":"955478bec1f3fc9cc5b699fb4b6139f8d8af0efd","unresolved":false,"context_lines":[{"line_number":31,"context_line":"from nova.tests.unit.api.openstack import fakes"},{"line_number":32,"context_line":""},{"line_number":33,"context_line":""},{"line_number":34,"context_line":"def fake_instance_get(instance_id, columns_to_join\u003dNone, use_slave\u003dFalse):"},{"line_number":35,"context_line":"    result \u003d fakes.stub_instance(id\u003d1, uuid\u003dinstance_id)"},{"line_number":36,"context_line":"    result[\u0027created_at\u0027] \u003d None"},{"line_number":37,"context_line":"    result[\u0027deleted_at\u0027] \u003d None"}],"source_content_type":"text/x-python","patch_set":4,"id":"7a5de9d1_6a439daf","line":34,"range":{"start_line":34,"start_character":35,"end_line":34,"end_character":72},"updated":"2016-02-02 03:48:13.000000000","message":"Now \u0027columns_to_join\u0027 and \u0027use_slave\u0027 are unnecessary.\nSo it would be better to remove them.","commit_id":"93f8d405e0d4b562926cf5545eb416536bebd5c5"},{"author":{"_account_id":12299,"name":"Paweł Koniszewski","email":"pawel.koniszewski@gmail.com","username":"pawel.koniszewski"},"change_message_id":"97ef17b50cb1506f160f120331e4222cf4852502","unresolved":false,"context_lines":[{"line_number":71,"context_line":"    def test_start(self):"},{"line_number":72,"context_line":"        self.stub_out(\u0027nova.db.instance_get_by_uuid\u0027, fake_instance_get)"},{"line_number":73,"context_line":"        self.mox.StubOutWithMock(compute_api.API, \u0027start\u0027)"},{"line_number":74,"context_line":"        compute_api.API.start(mox.IgnoreArg(), mox.IgnoreArg())"},{"line_number":75,"context_line":"        self.mox.ReplayAll()"},{"line_number":76,"context_line":""},{"line_number":77,"context_line":"        body \u003d dict(start\u003d\"\")"}],"source_content_type":"text/x-python","patch_set":5,"id":"dae33548_e3145c45","side":"PARENT","line":74,"range":{"start_line":74,"start_character":8,"end_line":74,"end_character":63},"updated":"2016-02-16 18:49:57.000000000","message":"This assert is missing","commit_id":"2fc930a84e0e934c3f3d2c24679c698aeb98f8df"},{"author":{"_account_id":12299,"name":"Paweł Koniszewski","email":"pawel.koniszewski@gmail.com","username":"pawel.koniszewski"},"change_message_id":"97ef17b50cb1506f160f120331e4222cf4852502","unresolved":false,"context_lines":[{"line_number":114,"context_line":"    def test_stop(self):"},{"line_number":115,"context_line":"        self.stub_out(\u0027nova.db.instance_get_by_uuid\u0027, fake_instance_get)"},{"line_number":116,"context_line":"        self.mox.StubOutWithMock(compute_api.API, \u0027stop\u0027)"},{"line_number":117,"context_line":"        compute_api.API.stop(mox.IgnoreArg(), mox.IgnoreArg())"},{"line_number":118,"context_line":"        self.mox.ReplayAll()"},{"line_number":119,"context_line":""},{"line_number":120,"context_line":"        body \u003d dict(stop\u003d\"\")"}],"source_content_type":"text/x-python","patch_set":5,"id":"dae33548_432a687f","side":"PARENT","line":117,"range":{"start_line":117,"start_character":8,"end_line":117,"end_character":62},"updated":"2016-02-16 18:49:57.000000000","message":"This assert is missing","commit_id":"2fc930a84e0e934c3f3d2c24679c698aeb98f8df"}]}