)]}'
{"nova/tests/unit/compute/test_resource_tracker.py":[{"author":{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},"change_message_id":"8f22d565b5e311b83741e36b279d0814052af4dc","unresolved":false,"context_lines":[{"line_number":565,"context_line":"                       pinned_cpus\u003dset([]))])"},{"line_number":566,"context_line":""},{"line_number":567,"context_line":"    @mock.patch(\u0027nova.objects.MigrationList.get_in_progress_by_host_and_node\u0027)"},{"line_number":568,"context_line":"    def test_deleted_instances_with_migrations(self, mock_migration_list):"},{"line_number":569,"context_line":"        migration \u003d objects.Migration(context\u003dself.context,"},{"line_number":570,"context_line":"                                      migration_type\u003d\u0027resize\u0027,"},{"line_number":571,"context_line":"                                      instance_uuid\u003d\u0027invalid\u0027)"}],"source_content_type":"text/x-python","patch_set":1,"id":"7aa08908_184d543e","side":"PARENT","line":568,"updated":"2016-06-13 14:24:33.000000000","message":"Can you at least explain in the commit message why you don\u0027t think that this is useful? I haven\u0027t looked deeply at the code it\u0027s testing, but the commit message says it\u0027s not testing anything at all, and I\u0027m not sure I agree with that assessment. It might not be testing anything useful, in which case maybe we should remove it, but...","commit_id":"c0cbea952866d7d26338edfd28b0c063216f34c6"},{"author":{"_account_id":7,"name":"Jay Pipes","email":"jaypipes@gmail.com","username":"jaypipes"},"change_message_id":"06ed81fd39c89b1e13513e217fbdca48f1f1b9d4","unresolved":false,"context_lines":[{"line_number":565,"context_line":"                       pinned_cpus\u003dset([]))])"},{"line_number":566,"context_line":""},{"line_number":567,"context_line":"    @mock.patch(\u0027nova.objects.MigrationList.get_in_progress_by_host_and_node\u0027)"},{"line_number":568,"context_line":"    def test_deleted_instances_with_migrations(self, mock_migration_list):"},{"line_number":569,"context_line":"        migration \u003d objects.Migration(context\u003dself.context,"},{"line_number":570,"context_line":"                                      migration_type\u003d\u0027resize\u0027,"},{"line_number":571,"context_line":"                                      instance_uuid\u003d\u0027invalid\u0027)"}],"source_content_type":"text/x-python","patch_set":1,"id":"7aa08908_08597f35","side":"PARENT","line":568,"updated":"2016-06-13 15:15:23.000000000","message":"Sure, I will update the commit message, no problem. This test is ostensibly testing \"deleted instances with migrations\" but it isn\u0027t passing any deleted instances -- involved or not involved with migrations -- and the code paths being \"validated\" here (that MigrationList.get_in_progress_by_host_and_node() is called with no arguments) is fully tested in test_tracker.TestUpdateAvailableResource.test_no_instance_dest_migration()","commit_id":"c0cbea952866d7d26338edfd28b0c063216f34c6"}]}