)]}'
{"designate_tempest_plugin/tests/api/v2/test_blacklists.py":[{"author":{"_account_id":11662,"name":"Paul Glass","email":"paul.glass@rackspace.com","username":"pglass"},"change_message_id":"a5a189812c418bade9f56e84eaacbb4899e8d32b","unresolved":false,"context_lines":[{"line_number":30,"context_line":"    def setup_clients(cls):"},{"line_number":31,"context_line":"        super(BaseBlacklistsTest, cls).setup_clients()"},{"line_number":32,"context_line":""},{"line_number":33,"context_line":"        cls.admin_client \u003d cls.os_adm.blacklists_client"},{"line_number":34,"context_line":""},{"line_number":35,"context_line":""},{"line_number":36,"context_line":"class BlacklistsAdminTest(BaseBlacklistsTest):"}],"source_content_type":"text/x-python","patch_set":1,"id":"1aa78d24_21f6bf8e","line":33,"updated":"2016-07-07 19:50:20.000000000","message":"If this is going to instantiate an admin client, then I think we should put \"Admin\" in the class name and move `credentials \u003d [\"admin\"]` up to the base class as well.","commit_id":"b87fa7abc9d2569b4084f40e24052030ae3ec86d"},{"author":{"_account_id":741,"name":"Kiall Mac Innes","email":"kiall@macinnes.ie","username":"kiall"},"change_message_id":"8bb6839cbddf4defd7c79ac526845805404bc9e2","unresolved":false,"context_lines":[{"line_number":30,"context_line":"    def setup_clients(cls):"},{"line_number":31,"context_line":"        super(BaseBlacklistsTest, cls).setup_clients()"},{"line_number":32,"context_line":""},{"line_number":33,"context_line":"        cls.admin_client \u003d cls.os_adm.blacklists_client"},{"line_number":34,"context_line":""},{"line_number":35,"context_line":""},{"line_number":36,"context_line":"class BlacklistsAdminTest(BaseBlacklistsTest):"}],"source_content_type":"text/x-python","patch_set":1,"id":"dada55a8_37218db5","line":33,"in_reply_to":"1aa78d24_21f6bf8e","updated":"2016-07-19 13:52:52.000000000","message":"Good catch! Will fix..","commit_id":"b87fa7abc9d2569b4084f40e24052030ae3ec86d"}]}