)]}'
{"troveclient/tests/test_capabilities.py":[{"author":{"_account_id":8415,"name":"Denis M.","email":"lildee1991@gmail.com","username":"dmakogon"},"change_message_id":"45e337a57ccbe7f2117a2d4112638e11c97bc9fa","unresolved":false,"context_lines":[{"line_number":31,"context_line":"        super(CapabilitiesTest, self).tearDown()"},{"line_number":32,"context_line":""},{"line_number":33,"context_line":"    def test_list(self):"},{"line_number":34,"context_line":"        list_mock \u003d mock.Mock()"},{"line_number":35,"context_line":"        self.capabilities._list \u003d list_mock"},{"line_number":36,"context_line":"        self.capabilities.list()"},{"line_number":37,"context_line":"        list_mock.assert_called_with(\"/capabilities\", \"capabilities\")"}],"source_content_type":"text/x-python","patch_set":1,"id":"1ae5cdf2_e4349e4a","line":34,"updated":"2014-06-27 22:14:50.000000000","message":"Line #34-35  can be merged into one.\nAlso, seems like one test presented here. That is why all mocking should be moved into setUp method.","commit_id":"25fb5373de788c27056f15a4b2191a2c40de07af"},{"author":{"_account_id":10266,"name":"Simon Chang","email":"schang@tesora.com","username":"changsimon"},"change_message_id":"99125169578dfeff71e19343f65b754717f6f21d","unresolved":false,"context_lines":[{"line_number":31,"context_line":"        super(CapabilitiesTest, self).tearDown()"},{"line_number":32,"context_line":""},{"line_number":33,"context_line":"    def test_list(self):"},{"line_number":34,"context_line":"        list_mock \u003d mock.Mock()"},{"line_number":35,"context_line":"        self.capabilities._list \u003d list_mock"},{"line_number":36,"context_line":"        self.capabilities.list()"},{"line_number":37,"context_line":"        list_mock.assert_called_with(\"/capabilities\", \"capabilities\")"}],"source_content_type":"text/x-python","patch_set":1,"id":"1ae5cdf2_d7502d0c","line":34,"in_reply_to":"1ae5cdf2_e4349e4a","updated":"2014-06-30 14:57:57.000000000","message":"The reason behind the mock being here, but not in setUp is because there will be more test methods being added to this module. LIST is just the first one being implemented.  I agree merging #34-35 to one line.","commit_id":"25fb5373de788c27056f15a4b2191a2c40de07af"}]}