)]}'
{"openstackclient/image/v2/image.py":[{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"e711602ea429857090be81aaa16096251ed81786","unresolved":false,"context_lines":[{"line_number":395,"context_line":"                parsed_args.project_domain,"},{"line_number":396,"context_line":"            ).id"},{"line_number":397,"context_line":""},{"line_number":398,"context_line":"        if parsed_args.import:"},{"line_number":399,"context_line":"            kwargs[\u0027use_import\u0027] \u003d True"},{"line_number":400,"context_line":""},{"line_number":401,"context_line":"        # open the file first to ensure any failures are handled before the"}],"source_content_type":"text/x-python","patch_set":1,"id":"bf51134e_19b3f999","line":398,"updated":"2020-06-24 00:37:36.000000000","message":"pep8: E999 SyntaxError: invalid syntax","commit_id":"6ce5e819119cfaf00a389bde7b0fbb9adfed49b4"},{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"8c9a2ce7a99c2a207cbb80109b9cff2b6027a6e1","unresolved":false,"context_lines":[{"line_number":395,"context_line":"                parsed_args.project_domain,"},{"line_number":396,"context_line":"            ).id"},{"line_number":397,"context_line":""},{"line_number":398,"context_line":"        if parsed_args.import:"},{"line_number":399,"context_line":"            kwargs[\u0027use_import\u0027] \u003d True"},{"line_number":400,"context_line":""},{"line_number":401,"context_line":"        # open the file first to ensure any failures are handled before the"}],"source_content_type":"text/x-python","patch_set":3,"id":"bf51134e_543ccbdf","line":398,"updated":"2020-06-29 19:13:30.000000000","message":"pep8: E999 SyntaxError: invalid syntax","commit_id":"5a9df2232a9692a8db65d797706ceeb96937322c"}],"openstackclient/tests/unit/image/v2/test_image.py":[{"author":{"_account_id":28522,"name":"Hervé Beraud","email":"herveberaud.pro@gmail.com","username":"hberaud"},"change_message_id":"54052f848e9482cf29a07d5add27caae2b2ef704","unresolved":false,"context_lines":[{"line_number":272,"context_line":"            self.cmd.take_action, parsed_args)"},{"line_number":273,"context_line":""},{"line_number":274,"context_line":"    @mock.patch(\u0027sys.stdin\u0027, side_effect\u003d[None])"},{"line_number":275,"context_line":"    def test_image_create_import(self, raw_input):"},{"line_number":276,"context_line":""},{"line_number":277,"context_line":"        arglist \u003d ["},{"line_number":278,"context_line":"            \u0027--import\u0027,"}],"source_content_type":"text/x-python","patch_set":2,"id":"bf51134e_6b2e9792","line":275,"range":{"start_line":275,"start_character":39,"end_line":275,"end_character":48},"updated":"2020-06-24 09:03:44.000000000","message":"Do we need to patch `sys.stdin` here?\n`raw_input` is never used and if I correctly understood this code arguments will be passed by using `arglist`.","commit_id":"553ab3207645d6baadceb6c53171f6ee298cbb72"},{"author":{"_account_id":2,"name":"Monty Taylor","email":"mordred@inaugust.com","username":"mordred"},"change_message_id":"9e41795af93a9e5cb4203508d516f849c6fa936e","unresolved":false,"context_lines":[{"line_number":272,"context_line":"            self.cmd.take_action, parsed_args)"},{"line_number":273,"context_line":""},{"line_number":274,"context_line":"    @mock.patch(\u0027sys.stdin\u0027, side_effect\u003d[None])"},{"line_number":275,"context_line":"    def test_image_create_import(self, raw_input):"},{"line_number":276,"context_line":""},{"line_number":277,"context_line":"        arglist \u003d ["},{"line_number":278,"context_line":"            \u0027--import\u0027,"}],"source_content_type":"text/x-python","patch_set":2,"id":"bf51134e_f64ba727","line":275,"range":{"start_line":275,"start_character":39,"end_line":275,"end_character":48},"in_reply_to":"bf51134e_2b1c9fbf","updated":"2020-06-24 13:38:19.000000000","message":"Yeah, I think we could move the mock.patch to be done in the method instead of as a decorator since we don\u0027t actually need raw_input. But I\u0027m not *super* worried about it if it\u0027s match ing existing pattern in the file (seems like something we could more generally refactor into a setUp method or similar.","commit_id":"553ab3207645d6baadceb6c53171f6ee298cbb72"},{"author":{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"change_message_id":"59894797d802ce97f3053394d27daf3a87d66051","unresolved":false,"context_lines":[{"line_number":272,"context_line":"            self.cmd.take_action, parsed_args)"},{"line_number":273,"context_line":""},{"line_number":274,"context_line":"    @mock.patch(\u0027sys.stdin\u0027, side_effect\u003d[None])"},{"line_number":275,"context_line":"    def test_image_create_import(self, raw_input):"},{"line_number":276,"context_line":""},{"line_number":277,"context_line":"        arglist \u003d ["},{"line_number":278,"context_line":"            \u0027--import\u0027,"}],"source_content_type":"text/x-python","patch_set":2,"id":"bf51134e_2b1c9fbf","line":275,"range":{"start_line":275,"start_character":39,"end_line":275,"end_character":48},"in_reply_to":"bf51134e_6b2e9792","updated":"2020-06-24 09:59:32.000000000","message":"I copied that from the other tests above after things failed and it resolves the failure I was getting without this mock. Not sure how exactly this works, but since it works for others, I think this must be fine.\n\nMaybe I should move this test right after test_image_reserve_options to make the inheritance more apparent?","commit_id":"553ab3207645d6baadceb6c53171f6ee298cbb72"},{"author":{"_account_id":28522,"name":"Hervé Beraud","email":"herveberaud.pro@gmail.com","username":"hberaud"},"change_message_id":"ac216f513a638a172fb1271ffc7971c524460dcc","unresolved":false,"context_lines":[{"line_number":272,"context_line":"            self.cmd.take_action, parsed_args)"},{"line_number":273,"context_line":""},{"line_number":274,"context_line":"    @mock.patch(\u0027sys.stdin\u0027, side_effect\u003d[None])"},{"line_number":275,"context_line":"    def test_image_create_import(self, raw_input):"},{"line_number":276,"context_line":""},{"line_number":277,"context_line":"        arglist \u003d ["},{"line_number":278,"context_line":"            \u0027--import\u0027,"}],"source_content_type":"text/x-python","patch_set":2,"id":"bf51134e_f3709c42","line":275,"range":{"start_line":275,"start_character":39,"end_line":275,"end_character":48},"in_reply_to":"bf51134e_f64ba727","updated":"2020-06-29 12:57:59.000000000","message":"Yep seems good","commit_id":"553ab3207645d6baadceb6c53171f6ee298cbb72"}]}