)]}'
{"tempest/api/object_storage/test_object_temp_url.py":[{"author":{"_account_id":6968,"name":"Christian Schwede","email":"cschwede@nvidia.com","username":"cschwede"},"change_message_id":"7a1b32d06e7b728097cd481eddda83291241c80f","unresolved":false,"context_lines":[{"line_number":127,"context_line":"        self.account_client_metadata, _ \u003d \\"},{"line_number":128,"context_line":"            self.account_client.list_account_metadata()"},{"line_number":129,"context_line":""},{"line_number":130,"context_line":"        EXPIRATION_TIME \u003d 10000  # high to ensure the test finishes."},{"line_number":131,"context_line":"        expires \u003d int(time.time() + EXPIRATION_TIME)"},{"line_number":132,"context_line":"        url \u003d self.get_temp_url(self.container_name,"},{"line_number":133,"context_line":"                                self.object_name, \"GET\","}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAATn%2F%2BfzY%3D","line":130,"updated":"2013-11-25 13:50:39.000000000","message":"Maybe set expiration_time once in setUp for all tests instead of defining this in every test?","commit_id":"77afd496a02d04646bf944323bc9e83be169dd09"},{"author":{"_account_id":7872,"name":"Marc Koderer","email":"marc@koderer.com","username":"mkoderer"},"change_message_id":"e0e802b855bb83c01ad70530c2b0cb5b2aa795bf","unresolved":false,"context_lines":[{"line_number":128,"context_line":"            self.account_client.list_account_metadata()"},{"line_number":129,"context_line":""},{"line_number":130,"context_line":"        EXPIRATION_TIME \u003d 10000  # high to ensure the test finishes."},{"line_number":131,"context_line":"        expires \u003d int(time.time() + EXPIRATION_TIME)"},{"line_number":132,"context_line":"        url \u003d self.get_temp_url(self.container_name,"},{"line_number":133,"context_line":"                                self.object_name, \"GET\","},{"line_number":134,"context_line":"                                expires, key2)"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAATn%2F%2BdCk%3D","line":131,"updated":"2013-11-25 21:42:46.000000000","message":"I agree to Christian. Maybe we could put these lines into a function? something like:\n\n_get_expiration_time(expiration_time \u003d 10000){\n    return int(time.time() + expiration_time)\n}","commit_id":"77afd496a02d04646bf944323bc9e83be169dd09"},{"author":{"_account_id":6167,"name":"Ken\u0027ichi Ohmichi","email":"ken1ohmichi@gmail.com","username":"oomichi"},"change_message_id":"f708cb3e83f50fed2f9906967556b1fd5ae682d8","unresolved":false,"context_lines":[{"line_number":132,"context_line":"        url \u003d self._get_temp_url(self.container_name,"},{"line_number":133,"context_line":"                                 self.object_name, \"GET\","},{"line_number":134,"context_line":"                                 expires, key2)"},{"line_number":135,"context_line":"        _, body \u003d self.object_client.get_object_using_temp_url(url)"},{"line_number":136,"context_line":""},{"line_number":137,"context_line":"    @attr(type\u003d\u0027gate\u0027)"},{"line_number":138,"context_line":"    def test_put_object_using_temp_url(self):"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAATn%2F%2BJFk%3D","line":135,"updated":"2013-12-02 02:17:57.000000000","message":"better to add a resp[\u0027status\u0027] check because of api behavior test.","commit_id":"694d8f448f78af15b0d83c663e7182b14980a630"}]}