Fix python3 tests and the test error.

sigmavirus24 · sigmavirus24 · commit aea20f56c8c9 · 2012-09-18T13:38:10.000-04:00
Apparently author isn't always present. Interesting.
diff --git a/github3/decorators.py b/github3/decorators.py
@@ -14,7 +14,7 @@
     from StringIO import StringIO
 except ImportError:
     # python3
-    from io import StringIO
+    from io import BytesIO as StringIO
 
 
 def requires_auth(func):
@@ -42,6 +42,7 @@ def auth_wrapper(self, *args, **kwargs):
             # Mock a 401 response
             r = Response()
             r.status_code = 401
-            r.raw = StringIO('{"message": "Requires authentication"}')
+            r.encoding = 'utf-8'
+            r.raw = StringIO('{"message": "Requires authentication"}'.encode())
             raise GitHubError(r)
     return auth_wrapper
diff --git a/github3/git.py b/github3/git.py
@@ -60,13 +60,12 @@ def __init__(self, commit, session=None):
 
         #: dict containing at least the name, email and date the commit was
         #  created
-        self.author = commit.get('author')
+        self.author = commit.get('author', {})
         self._author_name = self.author.get('name', '')
 
         #: dict containing similar information to the author attribute
-        self.committer = commit.get('committer')
-        if commit.get('committer'):
-            self.committer = User(commit.get('committer'), None)
+        self.committer = commit.get('committer', {})
+        self._commit_name = self.committer.get('name', '')
 
         #: :class:`Tree <Tree>` the commit belongs to.
         self.tree = None
