)]}'
{"compute_hyperv/tests/unit/test_vmops.py":[{"author":{"_account_id":8213,"name":"Claudiu Belu","email":"cbelu@cloudbasesolutions.com","username":"claudiub"},"change_message_id":"81515731177322558dcdac5448615ecc1579af84","unresolved":false,"context_lines":[{"line_number":1257,"context_line":"            mock_instance.name,"},{"line_number":1258,"context_line":"            self._pathutils.get_instance_dir.return_value)"},{"line_number":1259,"context_line":""},{"line_number":1260,"context_line":"    def test_destroy_inexistent_instance(self):"},{"line_number":1261,"context_line":"        mock_instance \u003d fake_instance.fake_instance_obj(self.context)"},{"line_number":1262,"context_line":"        self._vmops._vmutils.vm_exists.return_value \u003d False"},{"line_number":1263,"context_line":"        self._vmops._vmutils.planned_vm_exists.return_value \u003d False"}],"source_content_type":"text/x-python","patch_set":1,"id":"7f287b81_f40f1334","side":"PARENT","line":1260,"updated":"2017-08-31 15:49:17.000000000","message":"i don\u0027t think this has to be removed.","commit_id":"fedb492a1466ab0fdb9a055b4399f1876e19f291"},{"author":{"_account_id":8213,"name":"Claudiu Belu","email":"cbelu@cloudbasesolutions.com","username":"claudiub"},"change_message_id":"d84b098fb89619ad8b567f27c4668aa50c30939c","unresolved":false,"context_lines":[{"line_number":1257,"context_line":"            mock_instance.name,"},{"line_number":1258,"context_line":"            self._pathutils.get_instance_dir.return_value)"},{"line_number":1259,"context_line":""},{"line_number":1260,"context_line":"    def test_destroy_inexistent_instance(self):"},{"line_number":1261,"context_line":"        mock_instance \u003d fake_instance.fake_instance_obj(self.context)"},{"line_number":1262,"context_line":"        self._vmops._vmutils.vm_exists.return_value \u003d False"},{"line_number":1263,"context_line":"        self._vmops._vmutils.planned_vm_exists.return_value \u003d False"}],"source_content_type":"text/x-python","patch_set":1,"id":"7f287b81_ce9da358","side":"PARENT","line":1260,"in_reply_to":"7f287b81_aeb6d73d","updated":"2017-08-31 16:07:29.000000000","message":"this test covers the case in which there is no VM and no Planned VM, the last else in the method. It\u0027s only a LOG.debug though, but still.. it\u0027s not hurting anyone and it\u0027s a bit outside the scope of the patch. :)","commit_id":"fedb492a1466ab0fdb9a055b4399f1876e19f291"},{"author":{"_account_id":8543,"name":"Lucian Petrut","email":"lpetrut@cloudbasesolutions.com","username":"plucian"},"change_message_id":"e95ebcaea2d3cd5ed9922cd0efc097677f8a4908","unresolved":false,"context_lines":[{"line_number":1257,"context_line":"            mock_instance.name,"},{"line_number":1258,"context_line":"            self._pathutils.get_instance_dir.return_value)"},{"line_number":1259,"context_line":""},{"line_number":1260,"context_line":"    def test_destroy_inexistent_instance(self):"},{"line_number":1261,"context_line":"        mock_instance \u003d fake_instance.fake_instance_obj(self.context)"},{"line_number":1262,"context_line":"        self._vmops._vmutils.vm_exists.return_value \u003d False"},{"line_number":1263,"context_line":"        self._vmops._vmutils.planned_vm_exists.return_value \u003d False"}],"source_content_type":"text/x-python","patch_set":1,"id":"7f287b81_0ea28b4c","side":"PARENT","line":1260,"in_reply_to":"7f287b81_ce9da358","updated":"2017-08-31 16:18:12.000000000","message":"Yeah, I was trying to avoid duplicating some stuff from the other test.","commit_id":"fedb492a1466ab0fdb9a055b4399f1876e19f291"},{"author":{"_account_id":8543,"name":"Lucian Petrut","email":"lpetrut@cloudbasesolutions.com","username":"plucian"},"change_message_id":"42b08a5ac5bc1eb385841f7549839615b12592ba","unresolved":false,"context_lines":[{"line_number":1257,"context_line":"            mock_instance.name,"},{"line_number":1258,"context_line":"            self._pathutils.get_instance_dir.return_value)"},{"line_number":1259,"context_line":""},{"line_number":1260,"context_line":"    def test_destroy_inexistent_instance(self):"},{"line_number":1261,"context_line":"        mock_instance \u003d fake_instance.fake_instance_obj(self.context)"},{"line_number":1262,"context_line":"        self._vmops._vmutils.vm_exists.return_value \u003d False"},{"line_number":1263,"context_line":"        self._vmops._vmutils.planned_vm_exists.return_value \u003d False"}],"source_content_type":"text/x-python","patch_set":1,"id":"7f287b81_aeb6d73d","side":"PARENT","line":1260,"in_reply_to":"7f287b81_f40f1334","updated":"2017-08-31 15:54:28.000000000","message":"Seems redundant, it\u0027s already covered by test_destroy. Actually, in that test, a planned vm always exists.","commit_id":"fedb492a1466ab0fdb9a055b4399f1876e19f291"}]}