)]}'
{"cinder/tests/unit/brick/test_brick_lvm.py":[{"author":{"_account_id":4523,"name":"Eric Harney","email":"eharney@redhat.com","username":"eharney"},"change_message_id":"ff18709c56d67de31a5c95214081763f07cc7f15","unresolved":false,"context_lines":[{"line_number":350,"context_line":"        vg_name \u003d \"vg-name\""},{"line_number":351,"context_line":"        pool_name \u003d vg_name + \"-pool\""},{"line_number":352,"context_line":"        self.vg.create_thin_pool(pool_name, \"1G\")"},{"line_number":353,"context_line":"        self.vg._execute \u003d mock.Mock()"},{"line_number":354,"context_line":"        self.vg.create_volume(\"test\", \"1G\", lv_type\u003d\u0027thin\u0027)"},{"line_number":355,"context_line":"        if self.configuration.lvm_suppress_fd_warnings is False:"},{"line_number":356,"context_line":"            self.vg._execute.assert_called_once_with("}],"source_content_type":"text/x-python","patch_set":2,"id":"ff346bd7_7d75c031","line":353,"updated":"2017-07-26 18:19:45.000000000","message":"This should be done with a\n   with mock.patch()\n\ncall or a mock decorator rather than assigning directly to self.vg._execute.  Assigning this way can impact other tests.","commit_id":"9860e5fac60b0a8c6a3be790ff2d7568c3a6c0c5"}]}