)]}'
{"tempest/api/orchestration/stacks/test_swift_resources.py":[{"author":{"_account_id":7385,"name":"Thomas Herve","email":"therve@redhat.com","username":"therve"},"change_message_id":"f05872e2730501e33ba2255a1c56d8d0612820d9","unresolved":false,"context_lines":[{"line_number":23,"context_line":"from tempest.test import attr"},{"line_number":24,"context_line":""},{"line_number":25,"context_line":"LOG \u003d logging.getLogger(__name__)"},{"line_number":26,"context_line":""},{"line_number":27,"context_line":""},{"line_number":28,"context_line":"class SwiftResourcesTestJSON(base.BaseOrchestrationTest):"},{"line_number":29,"context_line":"    _interface \u003d \u0027json\u0027"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAUH%2F%2B8Ho%3D","line":26,"updated":"2014-01-20 15:45:33.000000000","message":"This seems unused, and the logging import can be removed as well.","commit_id":"2a55d6a0fbe6fb3cf784a357202273cea7205da6"},{"author":{"_account_id":7385,"name":"Thomas Herve","email":"therve@redhat.com","username":"therve"},"change_message_id":"f05872e2730501e33ba2255a1c56d8d0612820d9","unresolved":false,"context_lines":[{"line_number":61,"context_line":"        cls.stack_name \u003d data_utils.rand_name(\u0027heat\u0027)"},{"line_number":62,"context_line":"        os \u003d clients.Manager()"},{"line_number":63,"context_line":"        if not os.config.service_available.swift:"},{"line_number":64,"context_line":"            raise cls.skipException(\"Neutron support is required\")"},{"line_number":65,"context_line":"        cls.account_client \u003d os.account_client"},{"line_number":66,"context_line":"        cls.container_client \u003d os.container_client"},{"line_number":67,"context_line":"        # create the stack"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAUH%2F%2B8Fg%3D","line":64,"updated":"2014-01-20 15:45:33.000000000","message":"The message doesn\u0027t map the check.","commit_id":"2a55d6a0fbe6fb3cf784a357202273cea7205da6"},{"author":{"_account_id":7385,"name":"Thomas Herve","email":"therve@redhat.com","username":"therve"},"change_message_id":"f05872e2730501e33ba2255a1c56d8d0612820d9","unresolved":false,"context_lines":[{"line_number":81,"context_line":"        resources \u003d [(\u0027SwiftContainer\u0027, \u0027OS::Swift::Container\u0027),"},{"line_number":82,"context_line":"                     (\u0027SwiftContainerWebsite\u0027, \u0027OS::Swift::Container\u0027)]"},{"line_number":83,"context_line":"        for resource_name, resource_type in resources:"},{"line_number":84,"context_line":"            resource \u003d self.test_resources.get(resource_name, None)"},{"line_number":85,"context_line":"            self.assertIsInstance(resource, dict)"},{"line_number":86,"context_line":"            self.assertEqual(resource_type, resource[\u0027resource_type\u0027])"},{"line_number":87,"context_line":"            self.assertEqual(resource_name, resource[\u0027logical_resource_id\u0027])"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAUH%2F%2B8Es%3D","line":84,"updated":"2014-01-20 15:45:33.000000000","message":"Is the get(None) necessary? I would just access test_resources directly.","commit_id":"2a55d6a0fbe6fb3cf784a357202273cea7205da6"},{"author":{"_account_id":7385,"name":"Thomas Herve","email":"therve@redhat.com","username":"therve"},"change_message_id":"f05872e2730501e33ba2255a1c56d8d0612820d9","unresolved":false,"context_lines":[{"line_number":92,"context_line":"        resp, container_list \u003d \\"},{"line_number":93,"context_line":"            self.account_client.list_account_containers(params\u003dparams)"},{"line_number":94,"context_line":"        self.assertEqual(\u0027200\u0027, resp[\u0027status\u0027])"},{"line_number":95,"context_line":"        self.assertEqual(len(container_list), 2)"},{"line_number":96,"context_line":"        for cont in container_list:"},{"line_number":97,"context_line":"            self.assertTrue(cont[\u0027name\u0027].startswith(self.stack_name))"},{"line_number":98,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAUH%2F%2B8EU%3D","line":95,"updated":"2014-01-20 15:45:33.000000000","message":"The arguments should be inverted.","commit_id":"2a55d6a0fbe6fb3cf784a357202273cea7205da6"},{"author":{"_account_id":1192,"name":"David Kranz","email":"dkranz@redhat.com","username":"david-kranz"},"change_message_id":"5b49cf46f4fe19693c66adc1ed13033ec2aeb619","unresolved":false,"context_lines":[{"line_number":70,"context_line":"        for resource in resources:"},{"line_number":71,"context_line":"            cls.test_resources[resource[\u0027logical_resource_id\u0027]] \u003d resource"},{"line_number":72,"context_line":""},{"line_number":73,"context_line":"    @attr(type\u003d\u0027slow\u0027)"},{"line_number":74,"context_line":"    def test_created_resources(self):"},{"line_number":75,"context_line":"        \"\"\"Created stack should be on the list of existing stacks.\"\"\""},{"line_number":76,"context_line":"        resources \u003d [(\u0027SwiftContainer\u0027, \u0027OS::Swift::Container\u0027),"}],"source_content_type":"text/x-python","patch_set":4,"id":"AAAAUH%2F%2BHtg%3D","line":73,"updated":"2014-01-29 23:21:45.000000000","message":"Why is just this test marked as slow?","commit_id":"eb30c387dbbf72b8d6e9dcc2acd29ddb3c0c8c40"},{"author":{"_account_id":866,"name":"Chmouel Boudjnah","email":"chmouel@chmouel.com","username":"chmouel"},"change_message_id":"b6bac5d14832e83d041c260efec7df598b107d3b","unresolved":false,"context_lines":[{"line_number":70,"context_line":"        for resource in resources:"},{"line_number":71,"context_line":"            cls.test_resources[resource[\u0027logical_resource_id\u0027]] \u003d resource"},{"line_number":72,"context_line":""},{"line_number":73,"context_line":"    @attr(type\u003d\u0027slow\u0027)"},{"line_number":74,"context_line":"    def test_created_resources(self):"},{"line_number":75,"context_line":"        \"\"\"Created stack should be on the list of existing stacks.\"\"\""},{"line_number":76,"context_line":"        resources \u003d [(\u0027SwiftContainer\u0027, \u0027OS::Swift::Container\u0027),"}],"source_content_type":"text/x-python","patch_set":4,"id":"AAAAUH%2F%2BGH0%3D","line":73,"in_reply_to":"AAAAUH%2F%2BHtg%3D","updated":"2014-01-30 07:05:22.000000000","message":"humm I kind of assume that create resource just are slow but since it\u0027s only present in the neutron test (from where i started) which I guess make sense that test can be slow in this case.","commit_id":"eb30c387dbbf72b8d6e9dcc2acd29ddb3c0c8c40"}]}