)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":33473,"name":"Atsushi Kawai","display_name":"akawai","email":"atsushi.kawai.bu@hitachi.com","username":"akawai"},"change_message_id":"8aa920a0043c2bda68280c1bf957cffe3a95a436","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":1,"id":"9f5d34bc_ee7e44f0","updated":"2023-12-13 08:08:24.000000000","message":"Does the patch fix same points in following patch?\n903503: Tests: Fix invalid assert calls | https://review.opendev.org/c/openstack/cinder/+/903503","commit_id":"cf88825a96a2c46bda74fc7b7a984b7010c19bb0"},{"author":{"_account_id":28621,"name":"Mauricio Faria de Oliveira","email":"mfo@canonical.com","username":"mfo"},"change_message_id":"a3775f5faf5de61e5b6d20da22c6314326125f92","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"34b3eb72_24014f49","in_reply_to":"9f5d34bc_ee7e44f0","updated":"2023-12-13 13:23:53.000000000","message":"\u003e Does the patch fix same points in following patch?\n\u003e 903503: Tests: Fix invalid assert calls | https://review.opendev.org/c/openstack/cinder/+/903503\n\nYes, with a slight difference on hunk 4.\nThis patch replaces called_with() with ASSERT_called_with();\nThe other patch replaces it with ASSERT_called_ONCE_with().","commit_id":"cf88825a96a2c46bda74fc7b7a984b7010c19bb0"}],"cinder/tests/unit/volume/drivers/solidfire/test_solidfire.py":[{"author":{"_account_id":4523,"name":"Eric Harney","email":"eharney@redhat.com","username":"eharney"},"change_message_id":"22dbb2bf8ffe6a661e88886fb6d083be6e939fb9","unresolved":true,"context_lines":[{"line_number":3226,"context_line":"            fake_context, fake_cinder_vols, secondary_id\u003d\u0027secondary\u0027,"},{"line_number":3227,"context_line":"            groups\u003dNone)"},{"line_number":3228,"context_line":""},{"line_number":3229,"context_line":"        mock_failover.assert_called_with(fake_context, fake_cinder_vols,"},{"line_number":3230,"context_line":"                                         \"secondary\", None)"},{"line_number":3231,"context_line":"        mock_failover_completed.assert_called_with(fake_context, \"secondary\")"},{"line_number":3232,"context_line":"        self.assertEqual(cluster_id, \"secondary\")"}],"source_content_type":"text/x-python","patch_set":1,"id":"721141bd_35137fb0","line":3229,"range":{"start_line":3229,"start_character":22,"end_line":3229,"end_character":40},"updated":"2024-01-19 16:22:39.000000000","message":"assert_called_once_with is a stronger and more specific assertion here.","commit_id":"cf88825a96a2c46bda74fc7b7a984b7010c19bb0"},{"author":{"_account_id":4523,"name":"Eric Harney","email":"eharney@redhat.com","username":"eharney"},"change_message_id":"22dbb2bf8ffe6a661e88886fb6d083be6e939fb9","unresolved":true,"context_lines":[{"line_number":3228,"context_line":""},{"line_number":3229,"context_line":"        mock_failover.assert_called_with(fake_context, fake_cinder_vols,"},{"line_number":3230,"context_line":"                                         \"secondary\", None)"},{"line_number":3231,"context_line":"        mock_failover_completed.assert_called_with(fake_context, \"secondary\")"},{"line_number":3232,"context_line":"        self.assertEqual(cluster_id, \"secondary\")"},{"line_number":3233,"context_line":"        self.assertEqual(fake_failover_updates, updates)"},{"line_number":3234,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"bc3a3bdb_5cef6cf6","line":3231,"range":{"start_line":3231,"start_character":32,"end_line":3231,"end_character":50},"updated":"2024-01-19 16:22:39.000000000","message":"assert_called_once_with is a stronger and more specific assertion here.","commit_id":"cf88825a96a2c46bda74fc7b7a984b7010c19bb0"}]}