)]}'
{"test/unit/cli/test_ringbuilder.py":[{"author":{"_account_id":18838,"name":"Paul Dardeau","email":"paul.dardeau@intel.com","username":"pdardeau"},"change_message_id":"7bf2fcf0de024d53e03cdc81103e3c30af53350e","unresolved":false,"context_lines":[{"line_number":285,"context_line":"        except SystemExit as e:"},{"line_number":286,"context_line":"            err \u003d e"},{"line_number":287,"context_line":"        self.assertEqual(err.code, 2)"},{"line_number":288,"context_line":""},{"line_number":289,"context_line":"    def test_parse_remove_values_number_of_arguments(self):"},{"line_number":290,"context_line":"        # Test Number of arguments abnormal"},{"line_number":291,"context_line":"        argv \u003d [\"--region\", \"2\", \"test\"]"}],"source_content_type":"text/x-python","patch_set":7,"id":"da6ed579_cae888b5","line":288,"updated":"2016-01-15 18:07:04.000000000","message":"This test works without your updates in ringbuilder.py, but fails here with your patch. I believe your test should be modified here to just run the _parse_set_info_values without the try block and without the assertEqual since err will still be None due to your patch on ringbuilder.py.","commit_id":"e94cc81d99e5b1f8f44b2818788f877a3142fc55"},{"author":{"_account_id":16226,"name":"tobe","email":"chendihao@unitedstack.com","username":"tobe"},"change_message_id":"b7fb3f734d3d7cf47a2e792444786109f6b48d3d","unresolved":false,"context_lines":[{"line_number":285,"context_line":"        except SystemExit as e:"},{"line_number":286,"context_line":"            err \u003d e"},{"line_number":287,"context_line":"        self.assertEqual(err.code, 2)"},{"line_number":288,"context_line":""},{"line_number":289,"context_line":"    def test_parse_remove_values_number_of_arguments(self):"},{"line_number":290,"context_line":"        # Test Number of arguments abnormal"},{"line_number":291,"context_line":"        argv \u003d [\"--region\", \"2\", \"test\"]"}],"source_content_type":"text/x-python","patch_set":7,"id":"da6ed579_484a52ae","line":288,"in_reply_to":"da6ed579_cae888b5","updated":"2016-01-16 00:27:29.000000000","message":"Agree. We should not get error after applying this patch.","commit_id":"e94cc81d99e5b1f8f44b2818788f877a3142fc55"}]}