)]}'
{"nova/tests/unit/conductor/tasks/test_migrate.py":[{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"9ed08cb17dcc9fa1e53129335020f0a71c27c880","unresolved":false,"context_lines":[{"line_number":224,"context_line":"    @mock.patch.object(scheduler_utils, \u0027fill_provider_mapping\u0027)"},{"line_number":225,"context_line":"    @mock.patch.object(scheduler_utils, \u0027claim_resources\u0027)"},{"line_number":226,"context_line":"    @mock.patch.object(context.RequestContext, \u0027elevated\u0027)"},{"line_number":227,"context_line":"    def test_execute_reschedule("},{"line_number":228,"context_line":"            self, mock_elevated, mock_claim, mock_fill_provider_mapping):"},{"line_number":229,"context_line":"        report_client \u003d report.SchedulerReportClient()"},{"line_number":230,"context_line":"        # setup task for re-schedule"}],"source_content_type":"text/x-python","patch_set":1,"id":"7faddb67_b1dc7045","line":227,"updated":"2019-09-03 14:02:23.000000000","message":"Technically this is redundant with the functional tests but it\u0027s probably good to have a unit test for this for later tweaking to this flow w/o having to adjust that complicated functional test again.","commit_id":"4d7b7cb07d6a767d75f5d728c7bf3c3084dcaab6"},{"author":{"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},"change_message_id":"388d3660abefdb3b24f0364aed33a97d5830aa72","unresolved":false,"context_lines":[{"line_number":224,"context_line":"    @mock.patch.object(scheduler_utils, \u0027fill_provider_mapping\u0027)"},{"line_number":225,"context_line":"    @mock.patch.object(scheduler_utils, \u0027claim_resources\u0027)"},{"line_number":226,"context_line":"    @mock.patch.object(context.RequestContext, \u0027elevated\u0027)"},{"line_number":227,"context_line":"    def test_execute_reschedule("},{"line_number":228,"context_line":"            self, mock_elevated, mock_claim, mock_fill_provider_mapping):"},{"line_number":229,"context_line":"        report_client \u003d report.SchedulerReportClient()"},{"line_number":230,"context_line":"        # setup task for re-schedule"}],"source_content_type":"text/x-python","patch_set":1,"id":"7faddb67_fc1b9fc7","line":227,"in_reply_to":"7faddb67_b1dc7045","updated":"2019-09-03 15:03:28.000000000","message":"It felt strange not adding a unit test for the positive case.","commit_id":"4d7b7cb07d6a767d75f5d728c7bf3c3084dcaab6"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"9ed08cb17dcc9fa1e53129335020f0a71c27c880","unresolved":false,"context_lines":[{"line_number":250,"context_line":""},{"line_number":251,"context_line":"        actual_selection \u003d task._reschedule()"},{"line_number":252,"context_line":""},{"line_number":253,"context_line":"        self.assertEqual(alternate_selection, actual_selection)"},{"line_number":254,"context_line":"        mock_claim.assert_called_once_with("},{"line_number":255,"context_line":"            mock_elevated.return_value, report_client, self.request_spec,"},{"line_number":256,"context_line":"            self.instance.uuid, alloc_req, \u00271.19\u0027)"}],"source_content_type":"text/x-python","patch_set":1,"id":"7faddb67_71e27803","line":253,"range":{"start_line":253,"start_character":13,"end_line":253,"end_character":24},"updated":"2019-09-03 14:02:23.000000000","message":"nit: this could probably just be assertIs","commit_id":"4d7b7cb07d6a767d75f5d728c7bf3c3084dcaab6"},{"author":{"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},"change_message_id":"388d3660abefdb3b24f0364aed33a97d5830aa72","unresolved":false,"context_lines":[{"line_number":250,"context_line":""},{"line_number":251,"context_line":"        actual_selection \u003d task._reschedule()"},{"line_number":252,"context_line":""},{"line_number":253,"context_line":"        self.assertEqual(alternate_selection, actual_selection)"},{"line_number":254,"context_line":"        mock_claim.assert_called_once_with("},{"line_number":255,"context_line":"            mock_elevated.return_value, report_client, self.request_spec,"},{"line_number":256,"context_line":"            self.instance.uuid, alloc_req, \u00271.19\u0027)"}],"source_content_type":"text/x-python","patch_set":1,"id":"7faddb67_dcca6318","line":253,"range":{"start_line":253,"start_character":13,"end_line":253,"end_character":24},"in_reply_to":"7faddb67_71e27803","updated":"2019-09-03 15:03:28.000000000","message":"Yepp. Done.","commit_id":"4d7b7cb07d6a767d75f5d728c7bf3c3084dcaab6"}]}