)]}'
{"nova/tests/unit/compute/test_compute_api.py":[{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"946388dc0aacaa2a41a3eda678f57c7fac258c6c","unresolved":false,"context_lines":[{"line_number":1621,"context_line":"        mock_elevated.return_value \u003d self.context"},{"line_number":1622,"context_line":"        mock_get_migration.return_value \u003d fake_mig"},{"line_number":1623,"context_line":"        mock_reverse_upsize_quota_delta.return_value \u003d \u0027deltas\u0027"},{"line_number":1624,"context_line":"        resvs \u003d [\u0027resvs\u0027]"},{"line_number":1625,"context_line":"        fake_quotas \u003d objects.Quotas.from_reservations(self.context, resvs)"},{"line_number":1626,"context_line":"        mock_reserve_quota_delta.return_value \u003d fake_quotas"},{"line_number":1627,"context_line":""},{"line_number":1628,"context_line":"        def _check_state(expected_task_state\u003dNone):"}],"source_content_type":"text/x-python","patch_set":4,"id":"1a6eadb0_d90b537c","line":1625,"range":{"start_line":1624,"start_character":0,"end_line":1625,"end_character":75},"updated":"2016-12-12 15:46:01.000000000","message":"nit: if you have to rework, could you move this above the mocks. Makes this a little more readable","commit_id":"46920cb044c4551bf5238b331582ca43ab13ab6d"},{"author":{"_account_id":22771,"name":"Christopher Martin","email":"cm876n@att.com","username":"cm876n"},"change_message_id":"9385d3d9428d0cc9398d06d626585b54c6db37c6","unresolved":false,"context_lines":[{"line_number":1621,"context_line":"        mock_elevated.return_value \u003d self.context"},{"line_number":1622,"context_line":"        mock_get_migration.return_value \u003d fake_mig"},{"line_number":1623,"context_line":"        mock_reverse_upsize_quota_delta.return_value \u003d \u0027deltas\u0027"},{"line_number":1624,"context_line":"        resvs \u003d [\u0027resvs\u0027]"},{"line_number":1625,"context_line":"        fake_quotas \u003d objects.Quotas.from_reservations(self.context, resvs)"},{"line_number":1626,"context_line":"        mock_reserve_quota_delta.return_value \u003d fake_quotas"},{"line_number":1627,"context_line":""},{"line_number":1628,"context_line":"        def _check_state(expected_task_state\u003dNone):"}],"source_content_type":"text/x-python","patch_set":4,"id":"1a6eadb0_dcb5017f","line":1625,"range":{"start_line":1624,"start_character":0,"end_line":1625,"end_character":75},"in_reply_to":"1a6eadb0_d90b537c","updated":"2016-12-12 16:00:26.000000000","message":"Yea I can do that","commit_id":"46920cb044c4551bf5238b331582ca43ab13ab6d"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"946388dc0aacaa2a41a3eda678f57c7fac258c6c","unresolved":false,"context_lines":[{"line_number":1635,"context_line":"        with test.nested("},{"line_number":1636,"context_line":"            mock.patch.object(fake_inst, \u0027save\u0027, side_effect\u003d_check_state),"},{"line_number":1637,"context_line":"            mock.patch.object(fake_mig, \u0027save\u0027, side_effect\u003d_check_mig),"},{"line_number":1638,"context_line":"            mock.patch.object(self.compute_api, \u0027_record_action_start\u0027),"},{"line_number":1639,"context_line":"            mock.patch.object(self.compute_api.compute_rpcapi, \u0027revert_resize\u0027)"},{"line_number":1640,"context_line":"        ) as (mock_inst_save, mock_mig_save, mock_record_action,"},{"line_number":1641,"context_line":"              mock_revert_resize):"},{"line_number":1642,"context_line":"            self.compute_api.revert_resize(self.context, fake_inst)"}],"source_content_type":"text/x-python","patch_set":4,"id":"1a6eadb0_39ffe735","line":1639,"range":{"start_line":1638,"start_character":0,"end_line":1639,"end_character":79},"updated":"2016-12-12 15:46:01.000000000","message":"nit: Could we move these to decorators?","commit_id":"46920cb044c4551bf5238b331582ca43ab13ab6d"},{"author":{"_account_id":22771,"name":"Christopher Martin","email":"cm876n@att.com","username":"cm876n"},"change_message_id":"9385d3d9428d0cc9398d06d626585b54c6db37c6","unresolved":false,"context_lines":[{"line_number":1635,"context_line":"        with test.nested("},{"line_number":1636,"context_line":"            mock.patch.object(fake_inst, \u0027save\u0027, side_effect\u003d_check_state),"},{"line_number":1637,"context_line":"            mock.patch.object(fake_mig, \u0027save\u0027, side_effect\u003d_check_mig),"},{"line_number":1638,"context_line":"            mock.patch.object(self.compute_api, \u0027_record_action_start\u0027),"},{"line_number":1639,"context_line":"            mock.patch.object(self.compute_api.compute_rpcapi, \u0027revert_resize\u0027)"},{"line_number":1640,"context_line":"        ) as (mock_inst_save, mock_mig_save, mock_record_action,"},{"line_number":1641,"context_line":"              mock_revert_resize):"},{"line_number":1642,"context_line":"            self.compute_api.revert_resize(self.context, fake_inst)"}],"source_content_type":"text/x-python","patch_set":4,"id":"1a6eadb0_bc676524","line":1639,"range":{"start_line":1638,"start_character":0,"end_line":1639,"end_character":79},"in_reply_to":"1a6eadb0_39ffe735","updated":"2016-12-12 16:00:26.000000000","message":"Sure","commit_id":"46920cb044c4551bf5238b331582ca43ab13ab6d"},{"author":{"_account_id":7634,"name":"Takashi Natsume","email":"takanattie@gmail.com","username":"natsumet"},"change_message_id":"c40881860790a735c80ea6e4f5496efbc6275007","unresolved":false,"context_lines":[{"line_number":1641,"context_line":"        ) as (mock_inst_save, mock_mig_save, mock_record_action,"},{"line_number":1642,"context_line":"              mock_revert_resize):"},{"line_number":1643,"context_line":"            self.compute_api.revert_resize(self.context, fake_inst)"},{"line_number":1644,"context_line":""},{"line_number":1645,"context_line":"            mock_get_migration.assert_called_once_with("},{"line_number":1646,"context_line":"                self.context, fake_inst[\u0027uuid\u0027], \u0027finished\u0027)"},{"line_number":1647,"context_line":"            mock_reverse_upsize_quota_delta.assert_called_once_with("}],"source_content_type":"text/x-python","patch_set":5,"id":"da4df55a_1586cbda","line":1644,"updated":"2016-12-28 04:56:28.000000000","message":"The following assert statement should be added.\n\nmock_elevated.assert_called_once_with()","commit_id":"1bf3b08eb51b347616e37312c66fa0af416acc98"},{"author":{"_account_id":7634,"name":"Takashi Natsume","email":"takanattie@gmail.com","username":"natsumet"},"change_message_id":"c40881860790a735c80ea6e4f5496efbc6275007","unresolved":false,"context_lines":[{"line_number":1697,"context_line":"                          self.compute_api.revert_resize,"},{"line_number":1698,"context_line":"                          self.context,"},{"line_number":1699,"context_line":"                          fake_inst)"},{"line_number":1700,"context_line":""},{"line_number":1701,"context_line":"            mock_get_migration.assert_called_once_with("},{"line_number":1702,"context_line":"                self.context, fake_inst[\u0027uuid\u0027], \u0027finished\u0027)"},{"line_number":1703,"context_line":"            mock_reverse_upsize_quota_delta.assert_called_once_with("}],"source_content_type":"text/x-python","patch_set":5,"id":"da4df55a_b5617f3b","line":1700,"updated":"2016-12-28 04:56:28.000000000","message":"ditto","commit_id":"1bf3b08eb51b347616e37312c66fa0af416acc98"},{"author":{"_account_id":782,"name":"John Garbutt","email":"john@johngarbutt.com","username":"johngarbutt"},"change_message_id":"552f3baaccf89c205eedd597eaf0eeaaa4c11dd6","unresolved":false,"context_lines":[{"line_number":1640,"context_line":"            mock.patch.object(self.compute_api.compute_rpcapi, \u0027revert_resize\u0027)"},{"line_number":1641,"context_line":"        ) as (mock_inst_save, mock_mig_save, mock_record_action,"},{"line_number":1642,"context_line":"              mock_revert_resize):"},{"line_number":1643,"context_line":"            self.compute_api.revert_resize(self.context, fake_inst)"},{"line_number":1644,"context_line":""},{"line_number":1645,"context_line":"            mock_elevated.assert_called_once_with()"},{"line_number":1646,"context_line":"            mock_get_migration.assert_called_once_with("}],"source_content_type":"text/x-python","patch_set":6,"id":"1a430d35_7d05f111","line":1643,"updated":"2017-02-09 10:08:55.000000000","message":"Nit: a new line before the method under test would make it a little clearer.","commit_id":"f3d3c79a5d793b731fe7f62e7f29b5a327ec6b8a"}]}