Decode received mboxes

Signed-off-by: Stephen Finucane <[email protected]>
Closes: #47

Author: stephenfin

git_pw/bundle.py

@@ -72,7 +72,9 @@ def download_cmd(bundle_id, output):
     bundle = _get_bundle(bundle_id)
 
     if output:
-        output.write(api.get(bundle['mbox']).text)
+        content = api.get(bundle['mbox']).content
+
+        output.write(content)
 
         if output != sys.stdout:
             path = output.name

git_pw/patch.py

@@ -72,7 +72,7 @@ def download_cmd(patch_id, output, fmt):
         if fmt == 'diff':
             content = patch['diff']
         else:
-            content = api.get(patch['mbox']).text
+            content = api.get(patch['mbox']).content
 
         output.write(content)
 

git_pw/series.py

@@ -52,7 +52,9 @@ def download_cmd(series_id, output):
     series = api.detail('series', series_id)
 
     if output:
-        output.write(api.get(series['mbox']).text)
+        content = api.get(series['mbox']).content
+
+        output.write(content)
 
         if output != sys.stdout:
             path = output.name

releasenotes/notes/issue-47-a9ac87642050d289.yaml

@@ -0,0 +1,5 @@
+---
+fixes:
+  - |
+    Resolved an issue that prevented viewing patch diffs/mboxes in stdout on
+    Python 3.

tests/test_bundle.py

@@ -117,7 +117,7 @@ def test_download_to_file(self, mock_get, mock_download, mock_get_bundle):
 
         class MockResponse(object):
             @property
-            def text(self):
+            def content(self):
                 return b'alpha-beta'
 
         rsp = {'mbox': 'http://example.com/api/patches/123/mbox/'}

tests/test_patch.py

@@ -115,7 +115,7 @@ def test_download_to_file(self, mock_get, mock_log, mock_download,
 
         class MockResponse(object):
             @property
-            def text(self):
+            def content(self):
                 return b'alpha-beta'
 
         rsp = {'mbox': 'hello, world', 'diff': 'test'}

tests/test_series.py

@@ -68,7 +68,7 @@ def test_download_to_file(self, mock_get, mock_download, mock_detail):
 
         class MockResponse(object):
             @property
-            def text(self):
+            def content(self):
                 return b'alpha-beta'
 
         rsp = {'mbox': 'http://example.com/api/patches/123/mbox/'}