)]}'
{"tempest/api/compute/admin/test_hosts_negative.py":[{"author":{"_account_id":7350,"name":"Jordan Pittier","email":"jordan.pittier@gmail.com","username":"JordanP"},"change_message_id":"0ed0b1420cc932c494334bc4ff516dad08fd273d","unresolved":false,"context_lines":[{"line_number":28,"context_line":"        cls.non_admin_client \u003d cls.os.hosts_client"},{"line_number":29,"context_line":""},{"line_number":30,"context_line":"    @classmethod"},{"line_number":31,"context_line":"    def resource_setup(cls):"},{"line_number":32,"context_line":"        super(HostsAdminNegativeTestJSON, cls).resource_setup()"},{"line_number":33,"context_line":"        hosts \u003d cls.client.list_hosts()[\u0027hosts\u0027]"},{"line_number":34,"context_line":"        if not hosts:"}],"source_content_type":"text/x-python","patch_set":1,"id":"7a77a97e_16b543ed","line":31,"updated":"2016-11-16 11:32:41.000000000","message":"I am not sure about this.\n\nIn theory, resource_setup is here to actually create resources. Not to define variables or aliases to variables.\n\nAlso, I don\u0027t think we should introduce new class variables (cls.hostname, cls.nonexistent_hostname) lightly because then in the tests we have to scroll up to know and see where these variables come from.","commit_id":"34c1b79f8ef1dbc1d5ac35b539b5c5f3cf936a4b"}]}