)]}'
{"nova/tests/unit/compute/test_compute_mgr.py":[{"author":{"_account_id":10135,"name":"Lee Yarwood","display_name":"Lee Yarwood","email":"lyarwood@redhat.com","username":"lyarwood"},"change_message_id":"6d512ab0c8b971f35cd0bde842996b4ee2771de9","unresolved":false,"context_lines":[{"line_number":6344,"context_line":"                                     mock_start, mock_finish):"},{"line_number":6345,"context_line":"        self._do_build_instance_update(mock_save)"},{"line_number":6346,"context_line":""},{"line_number":6347,"context_line":"        orig_do_build_and_run \u003d self.compute._do_build_and_run_instance"},{"line_number":6348,"context_line":""},{"line_number":6349,"context_line":"        def _wrapped_do_build_and_run_instance(*args, **kwargs):"},{"line_number":6350,"context_line":"            ret \u003d orig_do_build_and_run(*args, **kwargs)"},{"line_number":6351,"context_line":"            self.assertEqual(build_results.ACTIVE, ret)"},{"line_number":6352,"context_line":"            return ret"},{"line_number":6353,"context_line":""},{"line_number":6354,"context_line":"        with mock.patch.object("},{"line_number":6355,"context_line":"            self.compute, \u0027_do_build_and_run_instance\u0027,"},{"line_number":6356,"context_line":"            side_effect\u003d_wrapped_do_build_and_run_instance,"},{"line_number":6357,"context_line":"        ):"},{"line_number":6358,"context_line":"            self.compute.build_and_run_instance("},{"line_number":6359,"context_line":"                self.context, self.instance,"},{"line_number":6360,"context_line":"                self.image, request_spec\u003d{},"}],"source_content_type":"text/x-python","patch_set":1,"id":"ff570b3c_797b0027","line":6357,"range":{"start_line":6347,"start_character":0,"end_line":6357,"end_character":10},"updated":"2020-05-19 18:53:32.000000000","message":"This did initially confuse me until I went and looked at build_and_run_instance:\n\nhttps://github.com/openstack/nova/blob/382d9b23362a69c86a9e050ecad789e4b57cd258/nova/compute/manager.py#L2125-L2133\n\nThe use of utils.spawn_n and the lack of any return value means this is the only way to assert the return within _do_build_and_run_instance as _assert_build_instance_hook_called was doing previously.","commit_id":"039eb709ccf5b10a04f02a4cd8bc15a8bab53553"}]}