)]}'
{"barbican_tempest_plugin/tests/api/base.py":[{"author":{"_account_id":14676,"name":"Kairat Kushaev","email":"kkushaev@mirantis.com","username":"kairat_kushaev"},"change_message_id":"a494ca4109307de1a3799ee9263d3999e336ba86","unresolved":false,"context_lines":[{"line_number":67,"context_line":"    @classmethod"},{"line_number":68,"context_line":"    def skip_checks(cls):"},{"line_number":69,"context_line":"        super(BaseKeyManagerTest, cls).skip_checks()"},{"line_number":70,"context_line":"        if not CONF.service_available.barbican:"},{"line_number":71,"context_line":"            raise cls.skipException(\u0027Barbican is not enabled.\u0027)"},{"line_number":72,"context_line":""},{"line_number":73,"context_line":"    @classmethod"}],"source_content_type":"text/x-python","patch_set":2,"id":"bf659307_923442fd","line":70,"updated":"2018-03-27 13:17:21.000000000","message":"sorry, can we cal super after check for config. It allows us fail fast and not to do any not-needed check is config is incorrect.","commit_id":"8131e66bc895bcdcd4499f8089776ff3c8580ba3"},{"author":{"_account_id":10459,"name":"Luigi Toscano","email":"ltoscano@redhat.com","username":"ltoscano"},"change_message_id":"ea685cc9f1b9dd182a85bd0dc9c059afdc88caf1","unresolved":false,"context_lines":[{"line_number":67,"context_line":"    @classmethod"},{"line_number":68,"context_line":"    def skip_checks(cls):"},{"line_number":69,"context_line":"        super(BaseKeyManagerTest, cls).skip_checks()"},{"line_number":70,"context_line":"        if not CONF.service_available.barbican:"},{"line_number":71,"context_line":"            raise cls.skipException(\u0027Barbican is not enabled.\u0027)"},{"line_number":72,"context_line":""},{"line_number":73,"context_line":"    @classmethod"}],"source_content_type":"text/x-python","patch_set":2,"id":"16dfb038_678a6f8c","line":70,"in_reply_to":"bf659307_923442fd","updated":"2021-02-11 11:38:52.000000000","message":"the scenario tests has the same code, and it usually follows that pattern. It should be changed in both places, I guess, but it can leave it as it is.","commit_id":"8131e66bc895bcdcd4499f8089776ff3c8580ba3"}]}