From: Javier M. Mellid <jmunhoz@igalia.com>
Date: Wed, 9 Sep 2015 19:28:32 +0000 (+0200)
Subject: s3tests: Add test_object_requestid_on_error
X-Git-Url: http://git-server-git.apps.pok.os.sepia.ceph.com/?a=commitdiff_plain;h=63048b7bdfc742f8cff7d689315d8fe6437b48a2;p=s3-tests.git

s3tests: Add test_object_requestid_on_error

Error responses must contains a RequestId tag. This is the ID of the
request associated with the error.

Signed-off-by: Javier M. Mellid <jmunhoz@igalia.com>
---

diff --git a/s3tests/functional/test_s3.py b/s3tests/functional/test_s3.py
index 41bc72cc..a59a76c8 100644
--- a/s3tests/functional/test_s3.py
+++ b/s3tests/functional/test_s3.py
@@ -23,6 +23,7 @@ import threading
 import itertools
 import string
 import random
+import re
 
 import xml.etree.ElementTree as ET
 
@@ -832,6 +833,16 @@ def test_object_read_notexist():
     eq(e.reason, 'Not Found')
     eq(e.error_code, 'NoSuchKey')
 
+@attr(resource='object')
+@attr(method='get')
+@attr(operation='read contents that were never written to raise one error response')
+@attr(assertion='RequestId appears in the error response')
+def test_object_requestid_on_error():
+    bucket = get_new_bucket()
+    key = bucket.new_key('foobar')
+    e = assert_raises(boto.exception.S3ResponseError, key.get_contents_as_string)
+    request_id = re.search(r'<RequestId>.*</RequestId>', e.body.encode('utf-8')).group(0)
+    assert request_id is not None
 
 @attr(resource='object')
 @attr(method='put')