)]}'
{"watcher/api/middleware/parsable_error.py":[{"author":{"_account_id":18971,"name":"Vincent Françoise","email":"Vincent.FRANCOISE@b-com.com","username":"v-francoise"},"change_message_id":"88906847090635fbc3fcf717451f5b869de486cc","unresolved":false,"context_lines":[{"line_number":73,"context_line":"                ):"},{"line_number":74,"context_line":"                try:"},{"line_number":75,"context_line":"                    # simple check xml is valid"},{"line_number":76,"context_line":"                    from_string \u003d \u0027{0}{1}{2}\u0027.format(\u0027\u003cerror_message\u003e\u0027,"},{"line_number":77,"context_line":"                                                     \u0027\\n\u0027.join(app_iter),"},{"line_number":78,"context_line":"                                                     \u0027\u003c/error_message\u003e\u0027)"},{"line_number":79,"context_line":"                    body \u003d [et.ElementTree.tostring("}],"source_content_type":"text/x-python","patch_set":3,"id":"5a710552_79ca65a3","line":76,"updated":"2015-12-15 14:17:54.000000000","message":"nit: you are trying to create an XML element here so IMHO, the following syntax would be more appropriate:\n\nbody \u003d [et.ElementTree.tostring(et.ElementTree.Element(\u0027error_message\u0027, text\u003d\u0027\\n\u0027.join(app_iter)))]","commit_id":"965381289330f4bbc1c26c8a958db967683ac042"},{"author":{"_account_id":18971,"name":"Vincent Françoise","email":"Vincent.FRANCOISE@b-com.com","username":"v-francoise"},"change_message_id":"88906847090635fbc3fcf717451f5b869de486cc","unresolved":false,"context_lines":[{"line_number":81,"context_line":"                except et.ElementTree.ParseError as err:"},{"line_number":82,"context_line":"                    LOG.error(_LE(\u0027Error parsing HTTP response: %s\u0027), err)"},{"line_number":83,"context_line":"                    status_code \u003d state[\u0027status_code\u0027]"},{"line_number":84,"context_line":"                    body \u003d [\u0027{0}{1}{2}\u0027.format(\u0027\u003cerror_message\u003e\u0027,"},{"line_number":85,"context_line":"                                               status_code,"},{"line_number":86,"context_line":"                                               \u0027\u003c/error_message\u003e\u0027)]"},{"line_number":87,"context_line":"                state[\u0027headers\u0027].append((\u0027Content-Type\u0027, \u0027application/xml\u0027))"}],"source_content_type":"text/x-python","patch_set":3,"id":"5a710552_346944bf","line":84,"updated":"2015-12-15 14:17:54.000000000","message":"ditto","commit_id":"965381289330f4bbc1c26c8a958db967683ac042"}],"watcher/applier/messaging/trigger.py":[{"author":{"_account_id":18971,"name":"Vincent Françoise","email":"Vincent.FRANCOISE@b-com.com","username":"v-francoise"},"change_message_id":"e56bdc8eea7316d6d726984c713cb22129148105","unresolved":false,"context_lines":[{"line_number":34,"context_line":"                                           action_plan_uuid)"},{"line_number":35,"context_line":"            cmd.execute()"},{"line_number":36,"context_line":"        except Exception as e:"},{"line_number":37,"context_line":"            LOG.error(e)"},{"line_number":38,"context_line":""},{"line_number":39,"context_line":"    def launch_action_plan(self, context, action_plan_uuid):"},{"line_number":40,"context_line":"        LOG.debug(\"Trigger ActionPlan %s\" % action_plan_uuid)"}],"source_content_type":"text/x-python","patch_set":7,"id":"3a7e1126_da1f3db5","line":37,"range":{"start_line":37,"start_character":22,"end_line":37,"end_character":23},"updated":"2015-12-21 10:02:34.000000000","message":"you should use LOG.exception() instead","commit_id":"17e69bea534ad47eb67523d4c2410bccdd50f881"}],"watcher/db/sqlalchemy/models.py":[{"author":{"_account_id":18971,"name":"Vincent Françoise","email":"Vincent.FRANCOISE@b-com.com","username":"v-francoise"},"change_message_id":"df3ddbe9951efb868822a53c842bbf9605ea78ea","unresolved":false,"context_lines":[{"line_number":40,"context_line":"               help\u003d\u0027MySQL engine to use.\u0027)"},{"line_number":41,"context_line":"]"},{"line_number":42,"context_line":""},{"line_number":43,"context_line":"sql_path \u003d paths.state_path_def(\u0027watcher.sqlite\u0027)"},{"line_number":44,"context_line":"_DEFAULT_SQL_CONNECTION \u003d \u0027sqlite:///{0}\u0027.format(sql_path)"},{"line_number":45,"context_line":""},{"line_number":46,"context_line":"cfg.CONF.register_opts(sql_opts, \u0027database\u0027)"}],"source_content_type":"text/x-python","patch_set":4,"id":"5a710552_68e36334","line":43,"updated":"2015-12-15 16:31:19.000000000","message":"This variable is in the global scope of the file and is not reused anywhere else. So maybe you can put the \"paths.state_path_def(\u0027watcher.sqlite\u0027)\" value directly within the .format() ?","commit_id":"551805b1c02c7c857a1b52be1514ad7fcf7c05f8"}],"watcher/tests/api/v1/test_audit_templates.py":[{"author":{"_account_id":18971,"name":"Vincent Françoise","email":"Vincent.FRANCOISE@b-com.com","username":"v-francoise"},"change_message_id":"df3ddbe9951efb868822a53c842bbf9605ea78ea","unresolved":false,"context_lines":[{"line_number":128,"context_line":"            audit_template \u003d obj_utils.create_test_audit_template("},{"line_number":129,"context_line":"                self.context, id\u003did_,"},{"line_number":130,"context_line":"                uuid\u003dutils.generate_uuid(),"},{"line_number":131,"context_line":"                name\u003d\u0027My Audit Template {0}\u0027.format(str(id_)))"},{"line_number":132,"context_line":"            audit_template_list.append(audit_template.uuid)"},{"line_number":133,"context_line":"        response \u003d self.get_json(\u0027/audit_templates\u0027)"},{"line_number":134,"context_line":"        self.assertEqual(len(audit_template_list),"}],"source_content_type":"text/x-python","patch_set":4,"id":"5a710552_884c8f43","line":131,"updated":"2015-12-15 16:31:19.000000000","message":"nit: you don\u0027t need to call the str() function anymore since the format() handles it","commit_id":"551805b1c02c7c857a1b52be1514ad7fcf7c05f8"},{"author":{"_account_id":18971,"name":"Vincent Françoise","email":"Vincent.FRANCOISE@b-com.com","username":"v-francoise"},"change_message_id":"df3ddbe9951efb868822a53c842bbf9605ea78ea","unresolved":false,"context_lines":[{"line_number":141,"context_line":"        for id_ in [1, 2, 3]:"},{"line_number":142,"context_line":"            audit_template \u003d obj_utils.create_test_audit_template("},{"line_number":143,"context_line":"                self.context, id\u003did_, uuid\u003dutils.generate_uuid(),"},{"line_number":144,"context_line":"                name\u003d\u0027My Audit Template {0}\u0027.format(str(id_)))"},{"line_number":145,"context_line":"            audit_template_list.append(audit_template.uuid)"},{"line_number":146,"context_line":"        for id_ in [4, 5]:"},{"line_number":147,"context_line":"            audit_template \u003d obj_utils.create_test_audit_template("}],"source_content_type":"text/x-python","patch_set":4,"id":"5a710552_685123d8","line":144,"updated":"2015-12-15 16:31:19.000000000","message":"ditto","commit_id":"551805b1c02c7c857a1b52be1514ad7fcf7c05f8"},{"author":{"_account_id":18971,"name":"Vincent Françoise","email":"Vincent.FRANCOISE@b-com.com","username":"v-francoise"},"change_message_id":"df3ddbe9951efb868822a53c842bbf9605ea78ea","unresolved":false,"context_lines":[{"line_number":146,"context_line":"        for id_ in [4, 5]:"},{"line_number":147,"context_line":"            audit_template \u003d obj_utils.create_test_audit_template("},{"line_number":148,"context_line":"                self.context, id\u003did_, uuid\u003dutils.generate_uuid(),"},{"line_number":149,"context_line":"                name\u003d\u0027My Audit Template {0}\u0027.format(str(id_)))"},{"line_number":150,"context_line":"            audit_template.soft_delete()"},{"line_number":151,"context_line":"        response \u003d self.get_json(\u0027/audit_templates\u0027)"},{"line_number":152,"context_line":"        self.assertEqual(3, len(response[\u0027audit_templates\u0027]))"}],"source_content_type":"text/x-python","patch_set":4,"id":"5a710552_c85857fd","line":149,"updated":"2015-12-15 16:31:19.000000000","message":"ditto","commit_id":"551805b1c02c7c857a1b52be1514ad7fcf7c05f8"},{"author":{"_account_id":18971,"name":"Vincent Françoise","email":"Vincent.FRANCOISE@b-com.com","username":"v-francoise"},"change_message_id":"df3ddbe9951efb868822a53c842bbf9605ea78ea","unresolved":false,"context_lines":[{"line_number":159,"context_line":"            audit_template \u003d obj_utils.create_test_audit_template("},{"line_number":160,"context_line":"                self.context, id\u003did_, uuid\u003dutils.generate_uuid(),"},{"line_number":161,"context_line":"                name\u003d\u0027My Audit Template {0}\u0027.format(str(id_)))"},{"line_number":162,"context_line":"            audit_template_list.append(audit_template.uuid)"},{"line_number":163,"context_line":"        for id_ in [4, 5]:"},{"line_number":164,"context_line":"            audit_template \u003d obj_utils.create_test_audit_template("},{"line_number":165,"context_line":"                self.context, id\u003did_, uuid\u003dutils.generate_uuid(),"}],"source_content_type":"text/x-python","patch_set":4,"id":"5a710552_88732f80","line":162,"updated":"2015-12-15 16:31:19.000000000","message":"ditto","commit_id":"551805b1c02c7c857a1b52be1514ad7fcf7c05f8"},{"author":{"_account_id":18971,"name":"Vincent Françoise","email":"Vincent.FRANCOISE@b-com.com","username":"v-francoise"},"change_message_id":"df3ddbe9951efb868822a53c842bbf9605ea78ea","unresolved":false,"context_lines":[{"line_number":163,"context_line":"        for id_ in [4, 5]:"},{"line_number":164,"context_line":"            audit_template \u003d obj_utils.create_test_audit_template("},{"line_number":165,"context_line":"                self.context, id\u003did_, uuid\u003dutils.generate_uuid(),"},{"line_number":166,"context_line":"                name\u003d\u0027My Audit Template {0}\u0027.format(str(id_)))"},{"line_number":167,"context_line":"            audit_template.soft_delete()"},{"line_number":168,"context_line":"            audit_template_list.append(audit_template.uuid)"},{"line_number":169,"context_line":"        response \u003d self.get_json(\u0027/audit_templates\u0027,"}],"source_content_type":"text/x-python","patch_set":4,"id":"5a710552_a86e2b94","line":166,"updated":"2015-12-15 16:31:19.000000000","message":"ditto","commit_id":"551805b1c02c7c857a1b52be1514ad7fcf7c05f8"},{"author":{"_account_id":18971,"name":"Vincent Françoise","email":"Vincent.FRANCOISE@b-com.com","username":"v-francoise"},"change_message_id":"df3ddbe9951efb868822a53c842bbf9605ea78ea","unresolved":false,"context_lines":[{"line_number":187,"context_line":"        for id_ in range(5):"},{"line_number":188,"context_line":"            obj_utils.create_test_audit_template("},{"line_number":189,"context_line":"                self.context, id\u003did_, uuid\u003dutils.generate_uuid(),"},{"line_number":190,"context_line":"                name\u003d\u0027My Audit Template {0}\u0027.format(str(id_)))"},{"line_number":191,"context_line":"        response \u003d self.get_json(\u0027/audit_templates/?limit\u003d3\u0027)"},{"line_number":192,"context_line":"        self.assertEqual(3, len(response[\u0027audit_templates\u0027]))"},{"line_number":193,"context_line":""}],"source_content_type":"text/x-python","patch_set":4,"id":"5a710552_08065fce","line":190,"updated":"2015-12-15 16:31:19.000000000","message":"ditto","commit_id":"551805b1c02c7c857a1b52be1514ad7fcf7c05f8"},{"author":{"_account_id":18971,"name":"Vincent Françoise","email":"Vincent.FRANCOISE@b-com.com","username":"v-francoise"},"change_message_id":"df3ddbe9951efb868822a53c842bbf9605ea78ea","unresolved":false,"context_lines":[{"line_number":199,"context_line":"        for id_ in range(5):"},{"line_number":200,"context_line":"            obj_utils.create_test_audit_template("},{"line_number":201,"context_line":"                self.context, id\u003did_, uuid\u003dutils.generate_uuid(),"},{"line_number":202,"context_line":"                name\u003d\u0027My Audit Template {0}\u0027.format(str(id_)))"},{"line_number":203,"context_line":"        response \u003d self.get_json(\u0027/audit_templates\u0027)"},{"line_number":204,"context_line":"        self.assertEqual(3, len(response[\u0027audit_templates\u0027]))"},{"line_number":205,"context_line":""}],"source_content_type":"text/x-python","patch_set":4,"id":"5a710552_e812f308","line":202,"updated":"2015-12-15 16:31:19.000000000","message":"ditto","commit_id":"551805b1c02c7c857a1b52be1514ad7fcf7c05f8"}],"watcher/tests/api/v1/test_audits.py":[{"author":{"_account_id":18971,"name":"Vincent Françoise","email":"Vincent.FRANCOISE@b-com.com","username":"v-francoise"},"change_message_id":"df3ddbe9951efb868822a53c842bbf9605ea78ea","unresolved":false,"context_lines":[{"line_number":169,"context_line":"        for id_ in range(5):"},{"line_number":170,"context_line":"            audit_template \u003d obj_utils.create_test_audit_template("},{"line_number":171,"context_line":"                self.context,"},{"line_number":172,"context_line":"                name\u003d\u0027at{0}\u0027.format(str(id_)),"},{"line_number":173,"context_line":"                uuid\u003dutils.generate_uuid())"},{"line_number":174,"context_line":"            obj_utils.create_test_audit("},{"line_number":175,"context_line":"                self.context, id\u003did_, uuid\u003dutils.generate_uuid(),"}],"source_content_type":"text/x-python","patch_set":4,"id":"5a710552_481b071c","line":172,"updated":"2015-12-15 16:31:19.000000000","message":"nit: no need for the str() here as well","commit_id":"551805b1c02c7c857a1b52be1514ad7fcf7c05f8"}],"watcher/tests/db/test_audit_template.py":[{"author":{"_account_id":18971,"name":"Vincent Françoise","email":"Vincent.FRANCOISE@b-com.com","username":"v-francoise"},"change_message_id":"df3ddbe9951efb868822a53c842bbf9605ea78ea","unresolved":false,"context_lines":[{"line_number":34,"context_line":"        for i in range(1, 6):"},{"line_number":35,"context_line":"            audit_template \u003d utils.create_test_audit_template("},{"line_number":36,"context_line":"                uuid\u003dw_utils.generate_uuid(),"},{"line_number":37,"context_line":"                name\u003d\u0027My Audit Template {0}\u0027.format(str(i)))"},{"line_number":38,"context_line":"            uuids.append(six.text_type(audit_template[\u0027uuid\u0027]))"},{"line_number":39,"context_line":"        res \u003d self.dbapi.get_audit_template_list(self.context)"},{"line_number":40,"context_line":"        res_uuids \u003d [r.uuid for r in res]"}],"source_content_type":"text/x-python","patch_set":4,"id":"5a710552_c8ddb743","line":37,"updated":"2015-12-15 16:31:19.000000000","message":"nit: here as well","commit_id":"551805b1c02c7c857a1b52be1514ad7fcf7c05f8"}]}