)]}'
{"nova/tests/test_libvirt.py":[{"author":{"_account_id":679,"name":"Kevin L. Mitchell","email":"klmitch@mit.edu","username":"klmitch"},"change_message_id":"54fcafdbdbda64e43d2a341bcc378d397256fc2b","unresolved":false,"context_lines":[{"line_number":1162,"context_line":"        libvirt_driver.LibvirtDriver._conn.lookupByName \u003d self.fake_lookup"},{"line_number":1163,"context_line":"        self.mox.StubOutWithMock(libvirt_driver.utils, \u0027execute\u0027)"},{"line_number":1164,"context_line":"        libvirt_driver.utils.execute \u003d self.fake_execute"},{"line_number":1165,"context_line":"        libvirt_driver.libvirt_utils.disk_type \u003d \"raw\""},{"line_number":1166,"context_line":""},{"line_number":1167,"context_line":"        self.mox.ReplayAll()"},{"line_number":1168,"context_line":""}],"source_content_type":"text/x-python","patch_set":8,"id":"AAAALH%2F%2FNTo%3D","line":1165,"updated":"2012-10-04 20:29:43.000000000","message":"These last two lines bother me; how do we know these values get changed back correctly?  It seems to me we should be using self.stubs.Set() here rather than direct assignment to attributes.","commit_id":"1a808ae91f6faa200e716b166e5a1c6a7808d3e1"},{"author":{"_account_id":1904,"name":"Boris Filippov","email":"bfilippov@griddynamics.com","username":"bfilippov"},"change_message_id":"1e34c310023dab389fdd2b54554cd8d3b2fea29a","unresolved":false,"context_lines":[{"line_number":1162,"context_line":"        libvirt_driver.LibvirtDriver._conn.lookupByName \u003d self.fake_lookup"},{"line_number":1163,"context_line":"        self.mox.StubOutWithMock(libvirt_driver.utils, \u0027execute\u0027)"},{"line_number":1164,"context_line":"        libvirt_driver.utils.execute \u003d self.fake_execute"},{"line_number":1165,"context_line":"        libvirt_driver.libvirt_utils.disk_type \u003d \"raw\""},{"line_number":1166,"context_line":""},{"line_number":1167,"context_line":"        self.mox.ReplayAll()"},{"line_number":1168,"context_line":""}],"source_content_type":"text/x-python","patch_set":8,"id":"AAAALH%2F%2FKo4%3D","line":1165,"in_reply_to":"AAAALH%2F%2FNTo%3D","updated":"2012-10-09 21:32:47.000000000","message":"Actually - partially done. I use self.stubs.Set for disk_type, but this fake_execute trick is used anywhere in these tests and seems to be stubbed/unstubbed in setUps and tearDowns along with other \"magic\".","commit_id":"1a808ae91f6faa200e716b166e5a1c6a7808d3e1"}]}