Add author attribute to Release objects

omgjlk · omgjlk · commit ecf7ea97e1cc · 2018-01-04T11:58:49.000-08:00
The Author data comes from the API and fits nicely into the ShortUser
object, so make it happen. A test was added to cover it too.

Signed-off-by: Jesse Keating &lt;jkeating@j2solutions.net&gt;
diff --git a/github3/repos/release.py b/github3/repos/release.py
@@ -7,6 +7,7 @@
 
 from .. import utils
 from .. import models
+from .. import users
 from ..decorators import requires_auth
 from ..exceptions import error_for
 
@@ -62,6 +63,10 @@ def _update_attributes(self, release):
         #: URITemplate to upload an asset with
         self.upload_urlt = URITemplate(release['upload_url'])
 
+        #: :class:`User <github3.users.ShortUser>` object representing the
+        #:  creator of the release
+        self.author = users.ShortUser(release['author'])
+
         #: URLs to various attributes
         for urltype in ['assets_url', 'html_url', 'tarball_url',
                         'zipball_url']:
diff --git a/tests/unit/test_repos_release.py b/tests/unit/test_repos_release.py
@@ -1,4 +1,5 @@
 from github3.repos.release import Release, Asset
+from github3.users import ShortUser
 
 from .helper import (UnitHelper, UnitIteratorHelper, create_url_helper, mock,
                      create_example_data_helper)
@@ -29,6 +30,10 @@ def test_original_assets(self):
     def test_has_upload_urlt(self):
         assert self.instance.upload_urlt is not None
 
+    def test_has_author(self):
+        assert self.instance.author is not None
+        assert isinstance(self.instance.author, ShortUser)
+
     # Method tests
     def test_tarball_archive(self):
         """Verify that we generate the correct URL for a tarball archive."""
