)]}'
{"nova/tests/unit/virt/vmwareapi/test_driver_api.py":[{"author":{"_account_id":11428,"name":"Ajaya Agrawal","email":"ajku.agr@gmail.com","username":"ajayaa"},"change_message_id":"033a66cad1d75466538567cbe991d2befb2111e3","unresolved":false,"context_lines":[{"line_number":783,"context_line":"            info \u003d self._get_info()"},{"line_number":784,"context_line":"            self._check_vm_info(info, power_state.RUNNING)"},{"line_number":785,"context_line":"            vmwareapi_fake.assertPathExists(self, str(root))"},{"line_number":786,"context_line":"            self.assertEqual(1, fake_extend_virtual_disk[0].call_count)"},{"line_number":787,"context_line":""},{"line_number":788,"context_line":"    @mock.patch.object(nova.virt.vmwareapi.images.VMwareImage,"},{"line_number":789,"context_line":"                       \u0027from_image\u0027)"}],"source_content_type":"text/x-python","patch_set":2,"id":"9a68dd71_d4d074f8","line":786,"range":{"start_line":786,"start_character":17,"end_line":786,"end_character":28},"updated":"2016-01-23 21:26:40.000000000","message":"I think you can use assert_any_call here. Also the test itself could be decorated with mock.patch rather than creating a context with test.nested. As far as I understand test.nested is used when you have multiple objects to patch.","commit_id":"692b88006ebc4a31c91762d26a9ec58a0456b646"},{"author":{"_account_id":7400,"name":"Tracy Jones","email":"tjones@vmware.com","username":"tracyajones"},"change_message_id":"4e3701807e3dedaac3eb6808c17f9d36af06ac8a","unresolved":false,"context_lines":[{"line_number":783,"context_line":"            info \u003d self._get_info()"},{"line_number":784,"context_line":"            self._check_vm_info(info, power_state.RUNNING)"},{"line_number":785,"context_line":"            vmwareapi_fake.assertPathExists(self, str(root))"},{"line_number":786,"context_line":"            self.assertEqual(1, fake_extend_virtual_disk[0].call_count)"},{"line_number":787,"context_line":""},{"line_number":788,"context_line":"    @mock.patch.object(nova.virt.vmwareapi.images.VMwareImage,"},{"line_number":789,"context_line":"                       \u0027from_image\u0027)"}],"source_content_type":"text/x-python","patch_set":2,"id":"9a68dd71_bfd35903","line":786,"range":{"start_line":786,"start_character":17,"end_line":786,"end_character":28},"in_reply_to":"9a68dd71_d4d074f8","updated":"2016-01-24 01:07:19.000000000","message":"I don\u0027t think i can do the decoration because at that point self is not defined.  Agree i could have used called_once.  If i have to push a new patch i will do that.  Is that ok?","commit_id":"692b88006ebc4a31c91762d26a9ec58a0456b646"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"f11368a41b69a58d0f124c178affce6eed003d2c","unresolved":false,"context_lines":[{"line_number":561,"context_line":"            self._create_vm()"},{"line_number":562,"context_line":"            self.assertEqual(2, self.iso_index)"},{"line_number":563,"context_line":"            self.assertEqual(fake_attach_cdrom_to_vm.call_count, 2)"},{"line_number":564,"context_line":"            self.assertEqual(fake_create_config_drive.call_count, 1)"},{"line_number":565,"context_line":""},{"line_number":566,"context_line":"    def test_ephemeral_disk_attach(self):"},{"line_number":567,"context_line":"        self._create_vm(ephemeral\u003d50)"}],"source_content_type":"text/x-python","patch_set":3,"id":"7a5de9d1_8cc86baa","line":564,"range":{"start_line":564,"start_character":67,"end_line":564,"end_character":68},"updated":"2016-02-02 09:39:12.000000000","message":"More precisely, we can check expected mock calls with expected arg also. But that can be improved later if needed.","commit_id":"5c9228c95985e90a22f6894c1578f95d7134c88e"}]}