)]}'
{"cinder/tests/unit/volume/flows/test_manage_snapshot_flow.py":[{"author":{"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},"change_message_id":"21bcfc8da67219a07a42c02da22c3bcd167863a9","unresolved":false,"context_lines":[{"line_number":226,"context_line":"        mock_host \u003d mock.MagicMock()"},{"line_number":227,"context_line":""},{"line_number":228,"context_line":"        fake_size \u003d 3"},{"line_number":229,"context_line":"        fake_snap \u003d fake_snapshot.fake_snapshot_obj(self.ctxt, volume_size\u003dfake_size)"},{"line_number":230,"context_line":"        mock_get_by_id \u003d fake_snap"},{"line_number":231,"context_line":""},{"line_number":232,"context_line":"        FAKE_SNAPSHOT_ID \u003d 1"}],"source_content_type":"text/x-python","patch_set":2,"id":"3fa7e38b_5a469352","line":229,"range":{"start_line":229,"start_character":51,"end_line":229,"end_character":52},"updated":"2019-10-28 10:29:29.000000000","message":"Add a line break after the opening ( and indent the next line to keep it under 80 characters.","commit_id":"a4daf6e2847373c5fb9f42eb51f47e67a5e27b92"},{"author":{"_account_id":31055,"name":"Anastasiya Zhyrkevich","email":"Anastasiya.Zhyrkevich@gmail.com","username":"anastzhyr"},"change_message_id":"d99bc74dda448f3de9f680f99186eda8bbcd184a","unresolved":false,"context_lines":[{"line_number":226,"context_line":"        mock_host \u003d mock.MagicMock()"},{"line_number":227,"context_line":""},{"line_number":228,"context_line":"        fake_size \u003d 3"},{"line_number":229,"context_line":"        fake_snap \u003d fake_snapshot.fake_snapshot_obj(self.ctxt, volume_size\u003dfake_size)"},{"line_number":230,"context_line":"        mock_get_by_id \u003d fake_snap"},{"line_number":231,"context_line":""},{"line_number":232,"context_line":"        FAKE_SNAPSHOT_ID \u003d 1"}],"source_content_type":"text/x-python","patch_set":2,"id":"3fa7e38b_44d0d8f7","line":229,"range":{"start_line":229,"start_character":51,"end_line":229,"end_character":52},"in_reply_to":"3fa7e38b_5a469352","updated":"2020-01-08 16:18:48.000000000","message":"Done","commit_id":"a4daf6e2847373c5fb9f42eb51f47e67a5e27b92"},{"author":{"_account_id":20813,"name":"Sofia Enriquez","email":"lsofia.enriquez@gmail.com","username":"enriquetaso"},"change_message_id":"b0e1e6058390c247d7121b70accbd44677b578e4","unresolved":false,"context_lines":[{"line_number":215,"context_line":""},{"line_number":216,"context_line":"    @mock.patch(\u0027cinder.objects.snapshot.Snapshot.get_by_id\u0027)"},{"line_number":217,"context_line":"    @mock.patch(\u0027cinder.volume.volume_utils.notify_about_snapshot_usage\u0027)"},{"line_number":218,"context_line":"    def test_create_snap_on_finish_task_notify("},{"line_number":219,"context_line":"            self,"},{"line_number":220,"context_line":"            mock_notify_about_usage,"},{"line_number":221,"context_line":"            _mock_get_by_id):"},{"line_number":222,"context_line":"        \"\"\"Test to create snapshot on finish.\"\"\""},{"line_number":223,"context_line":"        mock_status \u003d mock.MagicMock()"},{"line_number":224,"context_line":"        mock_db \u003d mock.MagicMock()"}],"source_content_type":"text/x-python","patch_set":5,"id":"3fa7e38b_51864667","line":221,"range":{"start_line":218,"start_character":47,"end_line":221,"end_character":28},"updated":"2019-12-10 18:24:46.000000000","message":"In my opinion, we should refactor this to look like L162.","commit_id":"809e0cd829efe4ce001d7266a3299e2f4a3b0761"},{"author":{"_account_id":1736,"name":"Ivan Kolodyazhny","email":"e0ne@e0ne.info","username":"e0ne"},"change_message_id":"9d6be75edf7b6a0028134b01ebef8957afdf7807","unresolved":false,"context_lines":[{"line_number":219,"context_line":"            self,"},{"line_number":220,"context_line":"            mock_notify_about_usage,"},{"line_number":221,"context_line":"            _mock_get_by_id):"},{"line_number":222,"context_line":"        \"\"\"Test to create snapshot on finish.\"\"\""},{"line_number":223,"context_line":"        mock_status \u003d mock.MagicMock()"},{"line_number":224,"context_line":"        mock_db \u003d mock.MagicMock()"},{"line_number":225,"context_line":"        mock_event_suffix \u003d mock.MagicMock()"}],"source_content_type":"text/x-python","patch_set":5,"id":"3fa7e38b_34df9167","line":222,"range":{"start_line":222,"start_character":8,"end_line":222,"end_character":48},"updated":"2019-11-28 19:10:33.000000000","message":"IMO, we don\u0027t need docstring here","commit_id":"809e0cd829efe4ce001d7266a3299e2f4a3b0761"},{"author":{"_account_id":20813,"name":"Sofia Enriquez","email":"lsofia.enriquez@gmail.com","username":"enriquetaso"},"change_message_id":"b0e1e6058390c247d7121b70accbd44677b578e4","unresolved":false,"context_lines":[{"line_number":219,"context_line":"            self,"},{"line_number":220,"context_line":"            mock_notify_about_usage,"},{"line_number":221,"context_line":"            _mock_get_by_id):"},{"line_number":222,"context_line":"        \"\"\"Test to create snapshot on finish.\"\"\""},{"line_number":223,"context_line":"        mock_status \u003d mock.MagicMock()"},{"line_number":224,"context_line":"        mock_db \u003d mock.MagicMock()"},{"line_number":225,"context_line":"        mock_event_suffix \u003d mock.MagicMock()"}],"source_content_type":"text/x-python","patch_set":5,"id":"3fa7e38b_f178927e","line":222,"range":{"start_line":222,"start_character":8,"end_line":222,"end_character":48},"in_reply_to":"3fa7e38b_34df9167","updated":"2019-12-10 18:24:46.000000000","message":"Please remote the docstring.","commit_id":"809e0cd829efe4ce001d7266a3299e2f4a3b0761"},{"author":{"_account_id":20813,"name":"Sofia Enriquez","email":"lsofia.enriquez@gmail.com","username":"enriquetaso"},"change_message_id":"b0bafe20b1f138676911694289d4dde811c972ae","unresolved":false,"context_lines":[{"line_number":230,"context_line":""},{"line_number":231,"context_line":"        task \u003d manager.CreateSnapshotOnFinishTask(mock_db, mock_event_suffix,"},{"line_number":232,"context_line":"                                                  mock_host)"},{"line_number":233,"context_line":"        task.execute(self.ctxt, fake_snapshot_descr, mock_status)"},{"line_number":234,"context_line":"        mock_notify_about_usage.assert_called_once_with("},{"line_number":235,"context_line":"            self.ctxt, fake_snapshot_descr, mock_event_suffix, host\u003dmock_host)"},{"line_number":236,"context_line":""}],"source_content_type":"text/x-python","patch_set":8,"id":"3fa7e38b_0f496db6","line":233,"range":{"start_line":233,"start_character":32,"end_line":233,"end_character":51},"updated":"2020-01-09 14:45:31.000000000","message":"-1: The execute method expects a ful snapshot.\n \nhttps://opendev.org/openstack/cinder/src/branch/master/cinder/volume/flows/manager/manage_existing_snapshot.py#L286","commit_id":"932cf7f23090a65f89fe16bc21dda5fb9115a520"}]}