)]}'
{"neutron/tests/unit/services/loadbalancer/drivers/haproxy/test_namespace_driver.py":[{"author":{"_account_id":2592,"name":"Mark McClain","email":"mark@mcclain.xyz","username":"markmcclain"},"change_message_id":"099c0d00ea77b6b1d16cd15cd588ae3489a7ca48","unresolved":false,"context_lines":[{"line_number":287,"context_line":"            mock.patch(\u0027neutron.agent.linux.utils.execute\u0027),"},{"line_number":288,"context_line":"            mock.patch.object(namespace_driver.LOG, \u0027exception\u0027),"},{"line_number":289,"context_line":"        ) as (path_exists, mock_open, mock_execute,"},{"line_number":290,"context_line":"              namespace_driver.LOG.exception):"},{"line_number":291,"context_line":"            file_mock \u003d mock.MagicMock()"},{"line_number":292,"context_line":"            mock_open.return_value \u003d file_mock"},{"line_number":293,"context_line":"            file_mock.__enter__.return_value \u003d file_mock"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAS3%2F%2FYJQ%3D","line":290,"updated":"2013-09-27 20:27:03.000000000","message":"You should not be assigning a symbol in another file to the result of patch object.","commit_id":"ad18b59b63b9adb3ad4d69bd36510707a4415e1e"},{"author":{"_account_id":2592,"name":"Mark McClain","email":"mark@mcclain.xyz","username":"markmcclain"},"change_message_id":"dfdf1c3d42bfe5753799a64df848c7911b35257a","unresolved":false,"context_lines":[{"line_number":291,"context_line":"            mock_open.return_value \u003d file_mock"},{"line_number":292,"context_line":"            file_mock.__enter__.return_value \u003d file_mock"},{"line_number":293,"context_line":"            file_mock.__iter__.return_value \u003d iter([\u0027123\u0027])"},{"line_number":294,"context_line":"            mock_log.return_value \u003d mock.MagicMock()"},{"line_number":295,"context_line":""},{"line_number":296,"context_line":"            path_exists.return_value \u003d False"},{"line_number":297,"context_line":"            namespace_driver.kill_pids_in_file(\u0027sudo\u0027, \u0027test_path\u0027)"}],"source_content_type":"text/x-python","patch_set":4,"id":"AAAAS3%2F%2FVRY%3D","line":294,"updated":"2013-09-30 02:29:24.000000000","message":"The return value of a mock is a mock. This is unnecessary","commit_id":"811c0e9c1d6c5c4c68ad75272ede1ecf2eff0776"},{"author":{"_account_id":2592,"name":"Mark McClain","email":"mark@mcclain.xyz","username":"markmcclain"},"change_message_id":"3bbd67da40cbd4f6791f04b000d4b8db8ca5ea7a","unresolved":false,"context_lines":[{"line_number":301,"context_line":"            path_exists.return_value \u003d True"},{"line_number":302,"context_line":"            mock_execute.side_effect \u003d RuntimeError"},{"line_number":303,"context_line":"            namespace_driver.kill_pids_in_file(\u0027sudo\u0027, \u0027test_path\u0027)"},{"line_number":304,"context_line":"            mock_execute.assert_called_once_with("},{"line_number":305,"context_line":"                [\u0027kill\u0027, \u0027-9\u0027, \u0027123\u0027], \u0027sudo\u0027)"},{"line_number":306,"context_line":""},{"line_number":307,"context_line":"    def test_get_state_file_path(self):"}],"source_content_type":"text/x-python","patch_set":5,"id":"AAAATH%2F%2FdCw%3D","line":304,"updated":"2013-10-15 23:27:59.000000000","message":"There should be a check here to make sure that the mock log was called here.","commit_id":"1aedcf89061353bf69f76033e8dcc6a309e3a466"}]}