commit 5f00cad0e257840c33117dc689e63984939f3649
author ittus <vuminhthang.cm@gmail.com> Sat Oct 15 10:32:40 2016 +0800
committer Nathaniel Manista <nathaniel@google.com> Mon Oct 31 09:57:15 2016 -0700
tree 3403797a48499269a1521a80bf479cbdea0a16ec
parent df61397973d36909a46fc4c3bf0e006dbb69f79a

Fix unprintable message with BatchError repr (#165)

googleapiclient/errors.py

diff --git a/googleapiclient/errors.py b/googleapiclient/errors.py
index e6f1f9d..6a741f8 100644
--- a/googleapiclient/errors.py
+++ b/googleapiclient/errors.py
@@ -126,6 +126,9 @@
     self.reason = reason
 
   def __repr__(self):
+    if getattr(self.resp, 'status', None) is None:
+      return '<BatchError "%s">' % (self.reason)
+    else:
       return '<BatchError %s "%s">' % (self.resp.status, self.reason)
 
   __str__ = __repr__

tests/test_http.py

diff --git a/tests/test_http.py b/tests/test_http.py
index 865d847..36b43bf 100644
--- a/tests/test_http.py
+++ b/tests/test_http.py
@@ -1053,7 +1053,9 @@
     upload = MediaFileUpload(
         datafile('small.png'), chunksize=500, resumable=True)
     self.request1.resumable = upload
-    self.assertRaises(BatchError, batch.add, self.request1, request_id='1')
+    with self.assertRaises(BatchError) as batch_error:
+      batch.add(self.request1, request_id='1')
+    str(batch_error.exception)
 
   def test_execute_empty_batch_no_http(self):
     batch = BatchHttpRequest()