)]}'
{"tempest/api/compute/servers/test_instance_actions.py":[{"author":{"_account_id":7350,"name":"Jordan Pittier","email":"jordan.pittier@gmail.com","username":"JordanP"},"change_message_id":"16d11b6ca97109884e641f8118216a913595f732","unresolved":false,"context_lines":[{"line_number":66,"context_line":"    @classmethod"},{"line_number":67,"context_line":"    def resource_setup(cls):"},{"line_number":68,"context_line":"        super(InstanceActionsV221TestJSON, cls).resource_setup()"},{"line_number":69,"context_line":"        server \u003d cls.create_test_server(wait_until\u003d\u0027ACTIVE\u0027)"},{"line_number":70,"context_line":"        cls.server_id \u003d server[\u0027id\u0027]"},{"line_number":71,"context_line":""},{"line_number":72,"context_line":"    @test.idempotent_id(\u00270a0f85d4-10fa-41f6-bf80-a54fb4aa2ae1\u0027)"}],"source_content_type":"text/x-python","patch_set":3,"id":"1a95cdbc_68832ecd","line":69,"updated":"2016-10-05 09:29:42.000000000","message":"Could you move this instance creation inside the test case it self. I.E, don\u0027t use the resource_setup here. We try to avoid using shared resources these days.","commit_id":"a0af40c51e383759e27a6fa294e282866ae148f2"},{"author":{"_account_id":7350,"name":"Jordan Pittier","email":"jordan.pittier@gmail.com","username":"JordanP"},"change_message_id":"16d11b6ca97109884e641f8118216a913595f732","unresolved":false,"context_lines":[{"line_number":76,"context_line":"        waiters.wait_for_server_termination(self.client, self.server_id)"},{"line_number":77,"context_line":"        body \u003d (self.client.list_instance_actions(self.server_id)"},{"line_number":78,"context_line":"                [\u0027instanceActions\u0027])"},{"line_number":79,"context_line":"        self.assertTrue(len(body) \u003d\u003d 2, str(body))"},{"line_number":80,"context_line":"        self.assertTrue(any([i for i in body if i[\u0027action\u0027] \u003d\u003d \u0027create\u0027]))"},{"line_number":81,"context_line":"        self.assertTrue(any([i for i in body if i[\u0027action\u0027] \u003d\u003d \u0027delete\u0027]))"}],"source_content_type":"text/x-python","patch_set":3,"id":"1a95cdbc_c83c4232","line":79,"updated":"2016-10-05 09:29:42.000000000","message":"use assertEqual here","commit_id":"a0af40c51e383759e27a6fa294e282866ae148f2"},{"author":{"_account_id":7350,"name":"Jordan Pittier","email":"jordan.pittier@gmail.com","username":"JordanP"},"change_message_id":"16d11b6ca97109884e641f8118216a913595f732","unresolved":false,"context_lines":[{"line_number":77,"context_line":"        body \u003d (self.client.list_instance_actions(self.server_id)"},{"line_number":78,"context_line":"                [\u0027instanceActions\u0027])"},{"line_number":79,"context_line":"        self.assertTrue(len(body) \u003d\u003d 2, str(body))"},{"line_number":80,"context_line":"        self.assertTrue(any([i for i in body if i[\u0027action\u0027] \u003d\u003d \u0027create\u0027]))"},{"line_number":81,"context_line":"        self.assertTrue(any([i for i in body if i[\u0027action\u0027] \u003d\u003d \u0027delete\u0027]))"}],"source_content_type":"text/x-python","patch_set":3,"id":"1a95cdbc_88407a58","line":80,"updated":"2016-10-05 09:29:42.000000000","message":"Use this, it\u0027s clearer\nassertEqual(sorted([i[\u0027action\u0027] for i in body]), [\u0027create\u0027, \u0027delete\u0027])","commit_id":"a0af40c51e383759e27a6fa294e282866ae148f2"}]}