)]}'
{"nova/tests/unit/virt/libvirt/test_utils.py":[{"author":{"_account_id":679,"name":"Kevin L. Mitchell","email":"klmitch@mit.edu","username":"klmitch"},"change_message_id":"4e4de84045d2f5393df546eb9f2f82c87332f264","unresolved":false,"context_lines":[{"line_number":729,"context_line":"        with mock.patch.object(libvirt_utils, \u0027execute\u0027,"},{"line_number":730,"context_line":"                               side_effect\u003dprocessutils.ProcessExecutionError):"},{"line_number":731,"context_line":"            with mock.patch.object(libvirt_utils.LOG, \u0027warning\u0027) as mock_log:"},{"line_number":732,"context_line":"                libvirt_utils.update_mtime(mock.sentinel.path)"},{"line_number":733,"context_line":"        self.assertTrue(mock_log.called)"}],"source_content_type":"text/x-python","patch_set":1,"id":"fa6399be_ee9da70b","line":732,"updated":"2016-10-17 15:22:37.000000000","message":"Could switch to using the decorator form of mock.patch.object() to avoid nested with.  For that matter, \u0027with\u0027 allows multiple clauses in 2.7+, which would also allow you to skip the nesting.  (suggestion)","commit_id":"53da313a86e81bf1df75119ca0e8f857e7b2909c"}]}
