)]}'
{"tempest/tests/lib/base.py":[{"author":{"_account_id":11662,"name":"Paul Glass","email":"paul.glass@rackspace.com","username":"pglass"},"change_message_id":"0ea58a1ffa8701701e59ff082ddb7aedda510d35","unresolved":false,"context_lines":[{"line_number":36,"context_line":"        self.addCleanup(p.stop)"},{"line_number":37,"context_line":"        return m"},{"line_number":38,"context_line":""},{"line_number":39,"context_line":"    def patchobject(self, target, attribute, new\u003dmock.DEFAULT):"},{"line_number":40,"context_line":"        \"\"\"Convenient wrapper around `mock.patch.object`"},{"line_number":41,"context_line":""},{"line_number":42,"context_line":"        Returns a started mock that will be automatically stopped after the"}],"source_content_type":"text/x-python","patch_set":1,"id":"7a1b2928_317d301d","line":39,"updated":"2016-04-11 18:53:18.000000000","message":"This looks good, but as a note for other reviewers: I was curious about using mock.DEFAULT as the default value here. It is the same default value that patch.object() uses for the `new` argument: \"If new is omitted, then the target is replaced with a MagicMock.\"\n\nhttps://docs.python.org/3/library/unittest.mock.html#unittest.mock.patch","commit_id":"0e2e51596a4fad0b39c5502857f47c27a6f6a35a"},{"author":{"_account_id":7350,"name":"Jordan Pittier","email":"jordan.pittier@gmail.com","username":"JordanP"},"change_message_id":"e1c5c1a94106f636cb548acd7b5b103ec5477b56","unresolved":false,"context_lines":[{"line_number":36,"context_line":"        self.addCleanup(p.stop)"},{"line_number":37,"context_line":"        return m"},{"line_number":38,"context_line":""},{"line_number":39,"context_line":"    def patchobject(self, target, attribute, new\u003dmock.DEFAULT):"},{"line_number":40,"context_line":"        \"\"\"Convenient wrapper around `mock.patch.object`"},{"line_number":41,"context_line":""},{"line_number":42,"context_line":"        Returns a started mock that will be automatically stopped after the"}],"source_content_type":"text/x-python","patch_set":1,"id":"5a18252c_8df875e7","line":39,"in_reply_to":"7a1b2928_317d301d","updated":"2016-04-12 14:11:44.000000000","message":"Yes, this is correct. I wanted this `patchobject` method to have the same signature has `mock.patch.object`.","commit_id":"0e2e51596a4fad0b39c5502857f47c27a6f6a35a"}]}