From 7248e66831d4ffe09ef1f30a1df59ec0a9331ece Mon Sep 17 00:00:00 2001 From: Vincent Jacques Date: Wed, 30 May 2012 18:58:18 +0200 Subject: [PATCH] Check HTTP status on all requests --- .../GithubObject.MethodBody.UseResult.py | 35 +++--- src/github/AuthenticatedUser.py | 58 +++++++++- src/github/Authorization.py | 4 + src/github/Commit.py | 4 + src/github/CommitComment.py | 4 + src/github/Download.py | 2 + src/github/Gist.py | 16 +++ src/github/GistComment.py | 4 + src/github/GitRef.py | 4 + src/github/Hook.py | 6 + src/github/Issue.py | 20 ++++ src/github/IssueComment.py | 4 + src/github/Label.py | 4 + src/github/Milestone.py | 6 + src/github/NamedUser.py | 24 ++++ src/github/Organization.py | 28 +++++ src/github/PullRequest.py | 14 +++ src/github/PullRequestComment.py | 4 + src/github/Repository.py | 104 ++++++++++++++++++ src/github/RepositoryKey.py | 4 + src/github/Team.py | 16 +++ src/github/UserKey.py | 4 + 22 files changed, 352 insertions(+), 17 deletions(-) diff --git a/codegen/templates/GithubObject.MethodBody.UseResult.py b/codegen/templates/GithubObject.MethodBody.UseResult.py index 1e9446d2..3ef9a0df 100644 --- a/codegen/templates/GithubObject.MethodBody.UseResult.py +++ b/codegen/templates/GithubObject.MethodBody.UseResult.py @@ -1,38 +1,43 @@ +{% if method.type.name != "bool" %} + if self.__requester.isFailureStatus( status ): # pragma no branch + raise GithubException( status, data ) # pragma no cover +{% endif %} + {% if method.isMutation %} self.__useAttributes( data ) {% endif %} {% if method.type.simple %} -{% if method.type.cardinality == "scalar" %} + {% if method.type.cardinality == "scalar" %} -{% if method.type.name == "@todo" %} - return data -{% endif %} - -{% if method.type.name == "bool" %} + {% if method.type.name == "bool" %} return status == 204 -{% endif %} + {% endif %} -{% endif %} - -{% if method.type.cardinality == "list" %} + {% if method.type.name == "@todo" %} return data -{% endif %} + {% endif %} + + {% endif %} + + {% if method.type.cardinality == "list" %} + return data + {% endif %} {% else %} -{% if method.type.cardinality == "scalar" %} + {% if method.type.cardinality == "scalar" %} return {% if method.type.name != class.name %}{{ method.type.name }}.{% endif %}{{ method.type.name }}( self.__requester, data, completed = True ) -{% endif %} + {% endif %} -{% if method.type.cardinality == "list" %} + {% if method.type.cardinality == "list" %} return PaginatedList.PaginatedList( {% if method.type.name != class.name %}{{ method.type.name }}.{% endif %}{{ method.type.name }}, self.__requester, headers, data ) -{% endif %} + {% endif %} {% endif %} diff --git a/src/github/AuthenticatedUser.py b/src/github/AuthenticatedUser.py index 0514c9b4..6f6e179b 100644 --- a/src/github/AuthenticatedUser.py +++ b/src/github/AuthenticatedUser.py @@ -154,6 +154,8 @@ class AuthenticatedUser( object ): None, post_parameters ) + if self.__requester.isFailureStatus( status ): # pragma no branch + raise GithubException( status, data ) # pragma no cover def add_to_following( self, following ): assert isinstance( following, NamedUser.NamedUser ), following @@ -163,6 +165,8 @@ class AuthenticatedUser( object ): None, None ) + if self.__requester.isFailureStatus( status ): # pragma no branch + raise GithubException( status, data ) # pragma no cover def add_to_watched( self, watched ): assert isinstance( watched, Repository.Repository ), watched @@ -172,6 +176,8 @@ class AuthenticatedUser( object ): None, None ) + if self.__requester.isFailureStatus( status ): # pragma no branch + raise GithubException( status, data ) # pragma no cover def create_authorization( self, scopes = DefaultValueForOptionalParameters, note = DefaultValueForOptionalParameters, note_url = DefaultValueForOptionalParameters ): if scopes is not DefaultValueForOptionalParameters: @@ -193,6 +199,8 @@ class AuthenticatedUser( object ): None, post_parameters ) + if self.__requester.isFailureStatus( status ): # pragma no branch + raise GithubException( status, data ) # pragma no cover return Authorization.Authorization( self.__requester, data, completed = True ) def create_fork( self, repo ): @@ -203,6 +211,8 @@ class AuthenticatedUser( object ): None, None ) + if self.__requester.isFailureStatus( status ): # pragma no branch + raise GithubException( status, data ) # pragma no cover return Repository.Repository( self.__requester, data, completed = True ) def create_gist( self, public, files, description = DefaultValueForOptionalParameters ): @@ -221,6 +231,8 @@ class AuthenticatedUser( object ): None, post_parameters ) + if self.__requester.isFailureStatus( status ): # pragma no branch + raise GithubException( status, data ) # pragma no cover return Gist.Gist( self.__requester, data, completed = True ) def create_key( self, title, key ): @@ -236,8 +248,8 @@ class AuthenticatedUser( object ): None, post_parameters ) - if self.__requester.isFailureStatus( status ): - raise GithubException( status, data ) + if self.__requester.isFailureStatus( status ): # pragma no branch + raise GithubException( status, data ) # pragma no cover return UserKey.UserKey( self.__requester, data, completed = True ) def create_repo( self, name, description = DefaultValueForOptionalParameters, homepage = DefaultValueForOptionalParameters, private = DefaultValueForOptionalParameters, has_issues = DefaultValueForOptionalParameters, has_wiki = DefaultValueForOptionalParameters, has_downloads = DefaultValueForOptionalParameters ): @@ -275,6 +287,8 @@ class AuthenticatedUser( object ): None, post_parameters ) + if self.__requester.isFailureStatus( status ): # pragma no branch + raise GithubException( status, data ) # pragma no cover return Repository.Repository( self.__requester, data, completed = True ) def edit( self, name = DefaultValueForOptionalParameters, email = DefaultValueForOptionalParameters, blog = DefaultValueForOptionalParameters, company = DefaultValueForOptionalParameters, location = DefaultValueForOptionalParameters, hireable = DefaultValueForOptionalParameters, bio = DefaultValueForOptionalParameters ): @@ -313,6 +327,8 @@ class AuthenticatedUser( object ): None, post_parameters ) + if self.__requester.isFailureStatus( status ): # pragma no branch + raise GithubException( status, data ) # pragma no cover self.__useAttributes( data ) def get_authorization( self, id ): @@ -323,6 +339,8 @@ class AuthenticatedUser( object ): None, None ) + if self.__requester.isFailureStatus( status ): # pragma no branch + raise GithubException( status, data ) # pragma no cover return Authorization.Authorization( self.__requester, data, completed = True ) def get_authorizations( self ): @@ -332,6 +350,8 @@ class AuthenticatedUser( object ): None, None ) + if self.__requester.isFailureStatus( status ): # pragma no branch + raise GithubException( status, data ) # pragma no cover return PaginatedList.PaginatedList( Authorization.Authorization, self.__requester, @@ -346,6 +366,8 @@ class AuthenticatedUser( object ): None, None ) + if self.__requester.isFailureStatus( status ): # pragma no branch + raise GithubException( status, data ) # pragma no cover return data def get_events( self ): @@ -355,6 +377,8 @@ class AuthenticatedUser( object ): None, None ) + if self.__requester.isFailureStatus( status ): # pragma no branch + raise GithubException( status, data ) # pragma no cover return PaginatedList.PaginatedList( Event.Event, self.__requester, @@ -369,6 +393,8 @@ class AuthenticatedUser( object ): None, None ) + if self.__requester.isFailureStatus( status ): # pragma no branch + raise GithubException( status, data ) # pragma no cover return PaginatedList.PaginatedList( NamedUser.NamedUser, self.__requester, @@ -383,6 +409,8 @@ class AuthenticatedUser( object ): None, None ) + if self.__requester.isFailureStatus( status ): # pragma no branch + raise GithubException( status, data ) # pragma no cover return PaginatedList.PaginatedList( NamedUser.NamedUser, self.__requester, @@ -397,6 +425,8 @@ class AuthenticatedUser( object ): None, None ) + if self.__requester.isFailureStatus( status ): # pragma no branch + raise GithubException( status, data ) # pragma no cover return PaginatedList.PaginatedList( Gist.Gist, self.__requester, @@ -411,6 +441,8 @@ class AuthenticatedUser( object ): None, None ) + if self.__requester.isFailureStatus( status ): # pragma no branch + raise GithubException( status, data ) # pragma no cover return PaginatedList.PaginatedList( Issue.Issue, self.__requester, @@ -426,6 +458,8 @@ class AuthenticatedUser( object ): None, None ) + if self.__requester.isFailureStatus( status ): # pragma no branch + raise GithubException( status, data ) # pragma no cover return UserKey.UserKey( self.__requester, data, completed = True ) def get_keys( self ): @@ -435,6 +469,8 @@ class AuthenticatedUser( object ): None, None ) + if self.__requester.isFailureStatus( status ): # pragma no branch + raise GithubException( status, data ) # pragma no cover return PaginatedList.PaginatedList( UserKey.UserKey, self.__requester, @@ -450,6 +486,8 @@ class AuthenticatedUser( object ): None, None ) + if self.__requester.isFailureStatus( status ): # pragma no branch + raise GithubException( status, data ) # pragma no cover return PaginatedList.PaginatedList( Event.Event, self.__requester, @@ -464,6 +502,8 @@ class AuthenticatedUser( object ): None, None ) + if self.__requester.isFailureStatus( status ): # pragma no branch + raise GithubException( status, data ) # pragma no cover return PaginatedList.PaginatedList( Organization.Organization, self.__requester, @@ -479,6 +519,8 @@ class AuthenticatedUser( object ): None, None ) + if self.__requester.isFailureStatus( status ): # pragma no branch + raise GithubException( status, data ) # pragma no cover return Repository.Repository( self.__requester, data, completed = True ) def get_repos( self, type = DefaultValueForOptionalParameters, sort = DefaultValueForOptionalParameters, direction = DefaultValueForOptionalParameters ): @@ -501,6 +543,8 @@ class AuthenticatedUser( object ): url_parameters, None ) + if self.__requester.isFailureStatus( status ): # pragma no branch + raise GithubException( status, data ) # pragma no cover return PaginatedList.PaginatedList( Repository.Repository, self.__requester, @@ -515,6 +559,8 @@ class AuthenticatedUser( object ): None, None ) + if self.__requester.isFailureStatus( status ): # pragma no branch + raise GithubException( status, data ) # pragma no cover return PaginatedList.PaginatedList( Gist.Gist, self.__requester, @@ -529,6 +575,8 @@ class AuthenticatedUser( object ): None, None ) + if self.__requester.isFailureStatus( status ): # pragma no branch + raise GithubException( status, data ) # pragma no cover return PaginatedList.PaginatedList( Repository.Repository, self.__requester, @@ -565,6 +613,8 @@ class AuthenticatedUser( object ): None, post_parameters ) + if self.__requester.isFailureStatus( status ): # pragma no branch + raise GithubException( status, data ) # pragma no cover def remove_from_following( self, following ): assert isinstance( following, NamedUser.NamedUser ), following @@ -574,6 +624,8 @@ class AuthenticatedUser( object ): None, None ) + if self.__requester.isFailureStatus( status ): # pragma no branch + raise GithubException( status, data ) # pragma no cover def remove_from_watched( self, watched ): assert isinstance( watched, Repository.Repository ), watched @@ -583,6 +635,8 @@ class AuthenticatedUser( object ): None, None ) + if self.__requester.isFailureStatus( status ): # pragma no branch + raise GithubException( status, data ) # pragma no cover def __initAttributes( self ): self.__avatar_url = None diff --git a/src/github/Authorization.py b/src/github/Authorization.py index ad288380..bf9442dc 100644 --- a/src/github/Authorization.py +++ b/src/github/Authorization.py @@ -64,6 +64,8 @@ class Authorization( object ): None, None ) + if self.__requester.isFailureStatus( status ): # pragma no branch + raise GithubException( status, data ) # pragma no cover def edit( self, scopes = DefaultValueForOptionalParameters, add_scopes = DefaultValueForOptionalParameters, remove_scopes = DefaultValueForOptionalParameters, note = DefaultValueForOptionalParameters, note_url = DefaultValueForOptionalParameters ): if scopes is not DefaultValueForOptionalParameters: @@ -93,6 +95,8 @@ class Authorization( object ): None, post_parameters ) + if self.__requester.isFailureStatus( status ): # pragma no branch + raise GithubException( status, data ) # pragma no cover self.__useAttributes( data ) def __initAttributes( self ): diff --git a/src/github/Commit.py b/src/github/Commit.py index fee85395..6d341b66 100644 --- a/src/github/Commit.py +++ b/src/github/Commit.py @@ -80,6 +80,8 @@ class Commit( object ): None, post_parameters ) + if self.__requester.isFailureStatus( status ): # pragma no branch + raise GithubException( status, data ) # pragma no cover return CommitComment.CommitComment( self.__requester, data, completed = True ) def get_comments( self ): @@ -89,6 +91,8 @@ class Commit( object ): None, None ) + if self.__requester.isFailureStatus( status ): # pragma no branch + raise GithubException( status, data ) # pragma no cover return PaginatedList.PaginatedList( CommitComment.CommitComment, self.__requester, diff --git a/src/github/CommitComment.py b/src/github/CommitComment.py index 070cb2dc..6cc52e36 100644 --- a/src/github/CommitComment.py +++ b/src/github/CommitComment.py @@ -74,6 +74,8 @@ class CommitComment( object ): None, None ) + if self.__requester.isFailureStatus( status ): # pragma no branch + raise GithubException( status, data ) # pragma no cover def edit( self, body ): assert isinstance( body, ( str, unicode ) ), body @@ -86,6 +88,8 @@ class CommitComment( object ): None, post_parameters ) + if self.__requester.isFailureStatus( status ): # pragma no branch + raise GithubException( status, data ) # pragma no cover self.__useAttributes( data ) def __initAttributes( self ): diff --git a/src/github/Download.py b/src/github/Download.py index 5849eade..9478107c 100644 --- a/src/github/Download.py +++ b/src/github/Download.py @@ -118,6 +118,8 @@ class Download( object ): None, None ) + if self.__requester.isFailureStatus( status ): # pragma no branch + raise GithubException( status, data ) # pragma no cover def __initAttributes( self ): self.__accesskeyid = None diff --git a/src/github/Gist.py b/src/github/Gist.py index 2edcdd71..b932b694 100644 --- a/src/github/Gist.py +++ b/src/github/Gist.py @@ -101,6 +101,8 @@ class Gist( object ): None, post_parameters ) + if self.__requester.isFailureStatus( status ): # pragma no branch + raise GithubException( status, data ) # pragma no cover return GistComment.GistComment( self.__requester, data, completed = True ) def create_fork( self ): @@ -110,6 +112,8 @@ class Gist( object ): None, None ) + if self.__requester.isFailureStatus( status ): # pragma no branch + raise GithubException( status, data ) # pragma no cover return Gist( self.__requester, data, completed = True ) def delete( self ): @@ -119,6 +123,8 @@ class Gist( object ): None, None ) + if self.__requester.isFailureStatus( status ): # pragma no branch + raise GithubException( status, data ) # pragma no cover def edit( self, description = DefaultValueForOptionalParameters, files = DefaultValueForOptionalParameters ): if description is not DefaultValueForOptionalParameters: @@ -134,6 +140,8 @@ class Gist( object ): None, post_parameters ) + if self.__requester.isFailureStatus( status ): # pragma no branch + raise GithubException( status, data ) # pragma no cover self.__useAttributes( data ) def get_comment( self, id ): @@ -144,6 +152,8 @@ class Gist( object ): None, None ) + if self.__requester.isFailureStatus( status ): # pragma no branch + raise GithubException( status, data ) # pragma no cover return GistComment.GistComment( self.__requester, data, completed = True ) def get_comments( self ): @@ -153,6 +163,8 @@ class Gist( object ): None, None ) + if self.__requester.isFailureStatus( status ): # pragma no branch + raise GithubException( status, data ) # pragma no cover return PaginatedList.PaginatedList( GistComment.GistComment, self.__requester, @@ -176,6 +188,8 @@ class Gist( object ): None, None ) + if self.__requester.isFailureStatus( status ): # pragma no branch + raise GithubException( status, data ) # pragma no cover def set_starred( self ): status, headers, data = self.__requester.request( @@ -184,6 +198,8 @@ class Gist( object ): None, None ) + if self.__requester.isFailureStatus( status ): # pragma no branch + raise GithubException( status, data ) # pragma no cover def __initAttributes( self ): self.__comments = None diff --git a/src/github/GistComment.py b/src/github/GistComment.py index 1de42f72..26b331e4 100644 --- a/src/github/GistComment.py +++ b/src/github/GistComment.py @@ -42,6 +42,8 @@ class GistComment( object ): None, None ) + if self.__requester.isFailureStatus( status ): # pragma no branch + raise GithubException( status, data ) # pragma no cover def edit( self, body ): assert isinstance( body, ( str, unicode ) ), body @@ -54,6 +56,8 @@ class GistComment( object ): None, post_parameters ) + if self.__requester.isFailureStatus( status ): # pragma no branch + raise GithubException( status, data ) # pragma no cover self.__useAttributes( data ) def __initAttributes( self ): diff --git a/src/github/GitRef.py b/src/github/GitRef.py index deb68b71..a8eca8da 100644 --- a/src/github/GitRef.py +++ b/src/github/GitRef.py @@ -30,6 +30,8 @@ class GitRef( object ): None, None ) + if self.__requester.isFailureStatus( status ): # pragma no branch + raise GithubException( status, data ) # pragma no cover def edit( self, sha, force = DefaultValueForOptionalParameters ): assert isinstance( sha, ( str, unicode ) ), sha @@ -46,6 +48,8 @@ class GitRef( object ): None, post_parameters ) + if self.__requester.isFailureStatus( status ): # pragma no branch + raise GithubException( status, data ) # pragma no cover self.__useAttributes( data ) def __initAttributes( self ): diff --git a/src/github/Hook.py b/src/github/Hook.py index 79b1468a..7dbefeb1 100644 --- a/src/github/Hook.py +++ b/src/github/Hook.py @@ -54,6 +54,8 @@ class Hook( object ): None, None ) + if self.__requester.isFailureStatus( status ): # pragma no branch + raise GithubException( status, data ) # pragma no cover def edit( self, name, config, events = DefaultValueForOptionalParameters, add_events = DefaultValueForOptionalParameters, remove_events = DefaultValueForOptionalParameters, active = DefaultValueForOptionalParameters ): assert isinstance( name, ( str, unicode ) ), name @@ -83,6 +85,8 @@ class Hook( object ): None, post_parameters ) + if self.__requester.isFailureStatus( status ): # pragma no branch + raise GithubException( status, data ) # pragma no cover self.__useAttributes( data ) def test( self ): @@ -92,6 +96,8 @@ class Hook( object ): None, None ) + if self.__requester.isFailureStatus( status ): # pragma no branch + raise GithubException( status, data ) # pragma no cover def __initAttributes( self ): self.__active = None diff --git a/src/github/Issue.py b/src/github/Issue.py index 42a06264..6e02b07f 100644 --- a/src/github/Issue.py +++ b/src/github/Issue.py @@ -117,6 +117,8 @@ class Issue( object ): None, post_parameters ) + if self.__requester.isFailureStatus( status ): # pragma no branch + raise GithubException( status, data ) # pragma no cover def create_comment( self, body ): assert isinstance( body, ( str, unicode ) ), body @@ -129,6 +131,8 @@ class Issue( object ): None, post_parameters ) + if self.__requester.isFailureStatus( status ): # pragma no branch + raise GithubException( status, data ) # pragma no cover return IssueComment.IssueComment( self.__requester, data, completed = True ) def delete_labels( self ): @@ -138,6 +142,8 @@ class Issue( object ): None, None ) + if self.__requester.isFailureStatus( status ): # pragma no branch + raise GithubException( status, data ) # pragma no cover def edit( self, title = DefaultValueForOptionalParameters, body = DefaultValueForOptionalParameters, assignee = DefaultValueForOptionalParameters, state = DefaultValueForOptionalParameters, milestone = DefaultValueForOptionalParameters, labels = DefaultValueForOptionalParameters ): if title is not DefaultValueForOptionalParameters: @@ -171,6 +177,8 @@ class Issue( object ): None, post_parameters ) + if self.__requester.isFailureStatus( status ): # pragma no branch + raise GithubException( status, data ) # pragma no cover self.__useAttributes( data ) def get_comment( self, id ): @@ -181,6 +189,8 @@ class Issue( object ): None, None ) + if self.__requester.isFailureStatus( status ): # pragma no branch + raise GithubException( status, data ) # pragma no cover return IssueComment.IssueComment( self.__requester, data, completed = True ) def get_comments( self ): @@ -190,6 +200,8 @@ class Issue( object ): None, None ) + if self.__requester.isFailureStatus( status ): # pragma no branch + raise GithubException( status, data ) # pragma no cover return PaginatedList.PaginatedList( IssueComment.IssueComment, self.__requester, @@ -204,6 +216,8 @@ class Issue( object ): None, None ) + if self.__requester.isFailureStatus( status ): # pragma no branch + raise GithubException( status, data ) # pragma no cover return PaginatedList.PaginatedList( IssueEvent.IssueEvent, self.__requester, @@ -218,6 +232,8 @@ class Issue( object ): None, None ) + if self.__requester.isFailureStatus( status ): # pragma no branch + raise GithubException( status, data ) # pragma no cover return PaginatedList.PaginatedList( Label.Label, self.__requester, @@ -233,6 +249,8 @@ class Issue( object ): None, None ) + if self.__requester.isFailureStatus( status ): # pragma no branch + raise GithubException( status, data ) # pragma no cover def set_labels( self, *labels ): assert all( isinstance( label, Label.Label ) for label in labels ), labels @@ -243,6 +261,8 @@ class Issue( object ): None, post_parameters ) + if self.__requester.isFailureStatus( status ): # pragma no branch + raise GithubException( status, data ) # pragma no cover def __initAttributes( self ): self.__assignee = None diff --git a/src/github/IssueComment.py b/src/github/IssueComment.py index ba3ddac7..6b0e7c89 100644 --- a/src/github/IssueComment.py +++ b/src/github/IssueComment.py @@ -42,6 +42,8 @@ class IssueComment( object ): None, None ) + if self.__requester.isFailureStatus( status ): # pragma no branch + raise GithubException( status, data ) # pragma no cover def edit( self, body ): assert isinstance( body, ( str, unicode ) ), body @@ -54,6 +56,8 @@ class IssueComment( object ): None, post_parameters ) + if self.__requester.isFailureStatus( status ): # pragma no branch + raise GithubException( status, data ) # pragma no cover self.__useAttributes( data ) def __initAttributes( self ): diff --git a/src/github/Label.py b/src/github/Label.py index 2a0c6c57..af9a83ac 100644 --- a/src/github/Label.py +++ b/src/github/Label.py @@ -30,6 +30,8 @@ class Label( object ): None, None ) + if self.__requester.isFailureStatus( status ): # pragma no branch + raise GithubException( status, data ) # pragma no cover def edit( self, name, color ): assert isinstance( name, ( str, unicode ) ), name @@ -44,6 +46,8 @@ class Label( object ): None, post_parameters ) + if self.__requester.isFailureStatus( status ): # pragma no branch + raise GithubException( status, data ) # pragma no cover self.__useAttributes( data ) @property diff --git a/src/github/Milestone.py b/src/github/Milestone.py index 1138b3d0..359e696c 100644 --- a/src/github/Milestone.py +++ b/src/github/Milestone.py @@ -63,6 +63,8 @@ class Milestone( object ): None, None ) + if self.__requester.isFailureStatus( status ): # pragma no branch + raise GithubException( status, data ) # pragma no cover def edit( self, title, state = DefaultValueForOptionalParameters, description = DefaultValueForOptionalParameters, due_on = DefaultValueForOptionalParameters ): assert isinstance( title, ( str, unicode ) ), title @@ -87,6 +89,8 @@ class Milestone( object ): None, post_parameters ) + if self.__requester.isFailureStatus( status ): # pragma no branch + raise GithubException( status, data ) # pragma no cover self.__useAttributes( data ) def get_labels( self ): @@ -96,6 +100,8 @@ class Milestone( object ): None, None ) + if self.__requester.isFailureStatus( status ): # pragma no branch + raise GithubException( status, data ) # pragma no cover return PaginatedList.PaginatedList( Label.Label, self.__requester, diff --git a/src/github/NamedUser.py b/src/github/NamedUser.py index 965c293a..bd5c6a87 100644 --- a/src/github/NamedUser.py +++ b/src/github/NamedUser.py @@ -163,6 +163,8 @@ class NamedUser( object ): None, post_parameters ) + if self.__requester.isFailureStatus( status ): # pragma no branch + raise GithubException( status, data ) # pragma no cover return Gist.Gist( self.__requester, data, completed = True ) def get_events( self ): @@ -172,6 +174,8 @@ class NamedUser( object ): None, None ) + if self.__requester.isFailureStatus( status ): # pragma no branch + raise GithubException( status, data ) # pragma no cover return PaginatedList.PaginatedList( Event.Event, self.__requester, @@ -186,6 +190,8 @@ class NamedUser( object ): None, None ) + if self.__requester.isFailureStatus( status ): # pragma no branch + raise GithubException( status, data ) # pragma no cover return PaginatedList.PaginatedList( NamedUser, self.__requester, @@ -200,6 +206,8 @@ class NamedUser( object ): None, None ) + if self.__requester.isFailureStatus( status ): # pragma no branch + raise GithubException( status, data ) # pragma no cover return PaginatedList.PaginatedList( NamedUser, self.__requester, @@ -214,6 +222,8 @@ class NamedUser( object ): None, None ) + if self.__requester.isFailureStatus( status ): # pragma no branch + raise GithubException( status, data ) # pragma no cover return PaginatedList.PaginatedList( Gist.Gist, self.__requester, @@ -228,6 +238,8 @@ class NamedUser( object ): None, None ) + if self.__requester.isFailureStatus( status ): # pragma no branch + raise GithubException( status, data ) # pragma no cover return PaginatedList.PaginatedList( Organization.Organization, self.__requester, @@ -242,6 +254,8 @@ class NamedUser( object ): None, None ) + if self.__requester.isFailureStatus( status ): # pragma no branch + raise GithubException( status, data ) # pragma no cover return PaginatedList.PaginatedList( Event.Event, self.__requester, @@ -256,6 +270,8 @@ class NamedUser( object ): None, None ) + if self.__requester.isFailureStatus( status ): # pragma no branch + raise GithubException( status, data ) # pragma no cover return PaginatedList.PaginatedList( Event.Event, self.__requester, @@ -270,6 +286,8 @@ class NamedUser( object ): None, None ) + if self.__requester.isFailureStatus( status ): # pragma no branch + raise GithubException( status, data ) # pragma no cover return PaginatedList.PaginatedList( Event.Event, self.__requester, @@ -285,6 +303,8 @@ class NamedUser( object ): None, None ) + if self.__requester.isFailureStatus( status ): # pragma no branch + raise GithubException( status, data ) # pragma no cover return Repository.Repository( self.__requester, data, completed = True ) def get_repos( self, type = DefaultValueForOptionalParameters ): @@ -299,6 +319,8 @@ class NamedUser( object ): url_parameters, None ) + if self.__requester.isFailureStatus( status ): # pragma no branch + raise GithubException( status, data ) # pragma no cover return PaginatedList.PaginatedList( Repository.Repository, self.__requester, @@ -313,6 +335,8 @@ class NamedUser( object ): None, None ) + if self.__requester.isFailureStatus( status ): # pragma no branch + raise GithubException( status, data ) # pragma no cover return PaginatedList.PaginatedList( Repository.Repository, self.__requester, diff --git a/src/github/Organization.py b/src/github/Organization.py index 5a8ccf70..66eac690 100644 --- a/src/github/Organization.py +++ b/src/github/Organization.py @@ -144,6 +144,8 @@ class Organization( object ): None, None ) + if self.__requester.isFailureStatus( status ): # pragma no branch + raise GithubException( status, data ) # pragma no cover def create_fork( self, repo ): assert isinstance( repo, Repository.Repository ), repo @@ -156,6 +158,8 @@ class Organization( object ): url_parameters, None ) + if self.__requester.isFailureStatus( status ): # pragma no branch + raise GithubException( status, data ) # pragma no cover return Repository.Repository( self.__requester, data, completed = True ) def create_repo( self, name, description = DefaultValueForOptionalParameters, homepage = DefaultValueForOptionalParameters, private = DefaultValueForOptionalParameters, has_issues = DefaultValueForOptionalParameters, has_wiki = DefaultValueForOptionalParameters, has_downloads = DefaultValueForOptionalParameters, team_id = DefaultValueForOptionalParameters ): @@ -197,6 +201,8 @@ class Organization( object ): None, post_parameters ) + if self.__requester.isFailureStatus( status ): # pragma no branch + raise GithubException( status, data ) # pragma no cover return Repository.Repository( self.__requester, data, completed = True ) def create_team( self, name, repo_names = DefaultValueForOptionalParameters, permission = DefaultValueForOptionalParameters ): @@ -218,6 +224,8 @@ class Organization( object ): None, post_parameters ) + if self.__requester.isFailureStatus( status ): # pragma no branch + raise GithubException( status, data ) # pragma no cover return Team.Team( self.__requester, data, completed = True ) def edit( self, billing_email = DefaultValueForOptionalParameters, blog = DefaultValueForOptionalParameters, company = DefaultValueForOptionalParameters, email = DefaultValueForOptionalParameters, location = DefaultValueForOptionalParameters, name = DefaultValueForOptionalParameters ): @@ -252,6 +260,8 @@ class Organization( object ): None, post_parameters ) + if self.__requester.isFailureStatus( status ): # pragma no branch + raise GithubException( status, data ) # pragma no cover self.__useAttributes( data ) def get_events( self ): @@ -261,6 +271,8 @@ class Organization( object ): None, None ) + if self.__requester.isFailureStatus( status ): # pragma no branch + raise GithubException( status, data ) # pragma no cover return PaginatedList.PaginatedList( Event.Event, self.__requester, @@ -275,6 +287,8 @@ class Organization( object ): None, None ) + if self.__requester.isFailureStatus( status ): # pragma no branch + raise GithubException( status, data ) # pragma no cover return PaginatedList.PaginatedList( NamedUser.NamedUser, self.__requester, @@ -289,6 +303,8 @@ class Organization( object ): None, None ) + if self.__requester.isFailureStatus( status ): # pragma no branch + raise GithubException( status, data ) # pragma no cover return PaginatedList.PaginatedList( NamedUser.NamedUser, self.__requester, @@ -304,6 +320,8 @@ class Organization( object ): None, None ) + if self.__requester.isFailureStatus( status ): # pragma no branch + raise GithubException( status, data ) # pragma no cover return Repository.Repository( self.__requester, data, completed = True ) def get_repos( self, type = DefaultValueForOptionalParameters ): @@ -318,6 +336,8 @@ class Organization( object ): url_parameters, None ) + if self.__requester.isFailureStatus( status ): # pragma no branch + raise GithubException( status, data ) # pragma no cover return PaginatedList.PaginatedList( Repository.Repository, self.__requester, @@ -333,6 +353,8 @@ class Organization( object ): None, None ) + if self.__requester.isFailureStatus( status ): # pragma no branch + raise GithubException( status, data ) # pragma no cover return Team.Team( self.__requester, data, completed = True ) def get_teams( self ): @@ -342,6 +364,8 @@ class Organization( object ): None, None ) + if self.__requester.isFailureStatus( status ): # pragma no branch + raise GithubException( status, data ) # pragma no cover return PaginatedList.PaginatedList( Team.Team, self.__requester, @@ -377,6 +401,8 @@ class Organization( object ): None, None ) + if self.__requester.isFailureStatus( status ): # pragma no branch + raise GithubException( status, data ) # pragma no cover def remove_from_public_members( self, public_member ): assert isinstance( public_member, NamedUser.NamedUser ), public_member @@ -386,6 +412,8 @@ class Organization( object ): None, None ) + if self.__requester.isFailureStatus( status ): # pragma no branch + raise GithubException( status, data ) # pragma no cover def __initAttributes( self ): self.__avatar_url = None diff --git a/src/github/PullRequest.py b/src/github/PullRequest.py index 8bda68b5..4f750f07 100644 --- a/src/github/PullRequest.py +++ b/src/github/PullRequest.py @@ -163,6 +163,8 @@ class PullRequest( object ): None, post_parameters ) + if self.__requester.isFailureStatus( status ): # pragma no branch + raise GithubException( status, data ) # pragma no cover return PullRequestComment.PullRequestComment( self.__requester, data, completed = True ) def edit( self, title = DefaultValueForOptionalParameters, body = DefaultValueForOptionalParameters, state = DefaultValueForOptionalParameters ): @@ -185,6 +187,8 @@ class PullRequest( object ): None, post_parameters ) + if self.__requester.isFailureStatus( status ): # pragma no branch + raise GithubException( status, data ) # pragma no cover self.__useAttributes( data ) def get_comment( self, id ): @@ -195,6 +199,8 @@ class PullRequest( object ): None, None ) + if self.__requester.isFailureStatus( status ): # pragma no branch + raise GithubException( status, data ) # pragma no cover return PullRequestComment.PullRequestComment( self.__requester, data, completed = True ) def get_comments( self ): @@ -204,6 +210,8 @@ class PullRequest( object ): None, None ) + if self.__requester.isFailureStatus( status ): # pragma no branch + raise GithubException( status, data ) # pragma no cover return PaginatedList.PaginatedList( PullRequestComment.PullRequestComment, self.__requester, @@ -218,6 +226,8 @@ class PullRequest( object ): None, None ) + if self.__requester.isFailureStatus( status ): # pragma no branch + raise GithubException( status, data ) # pragma no cover return PaginatedList.PaginatedList( Commit.Commit, self.__requester, @@ -232,6 +242,8 @@ class PullRequest( object ): None, None ) + if self.__requester.isFailureStatus( status ): # pragma no branch + raise GithubException( status, data ) # pragma no cover return PaginatedList.PaginatedList( PullRequestFile.PullRequestFile, self.__requester, @@ -260,6 +272,8 @@ class PullRequest( object ): None, post_parameters ) + if self.__requester.isFailureStatus( status ): # pragma no branch + raise GithubException( status, data ) # pragma no cover return PullRequestMergeStatus.PullRequestMergeStatus( self.__requester, data, completed = True ) def __initAttributes( self ): diff --git a/src/github/PullRequestComment.py b/src/github/PullRequestComment.py index 088e796e..3984f4e0 100644 --- a/src/github/PullRequestComment.py +++ b/src/github/PullRequestComment.py @@ -74,6 +74,8 @@ class PullRequestComment( object ): None, None ) + if self.__requester.isFailureStatus( status ): # pragma no branch + raise GithubException( status, data ) # pragma no cover def edit( self, body ): assert isinstance( body, ( str, unicode ) ), body @@ -86,6 +88,8 @@ class PullRequestComment( object ): None, post_parameters ) + if self.__requester.isFailureStatus( status ): # pragma no branch + raise GithubException( status, data ) # pragma no cover self.__useAttributes( data ) def __initAttributes( self ): diff --git a/src/github/Repository.py b/src/github/Repository.py index 55b0c947..3a59abe7 100644 --- a/src/github/Repository.py +++ b/src/github/Repository.py @@ -193,6 +193,8 @@ class Repository( object ): None, None ) + if self.__requester.isFailureStatus( status ): # pragma no branch + raise GithubException( status, data ) # pragma no cover def compare( self, base, head ): assert isinstance( base, ( str, unicode ) ), base @@ -203,6 +205,8 @@ class Repository( object ): None, None ) + if self.__requester.isFailureStatus( status ): # pragma no branch + raise GithubException( status, data ) # pragma no cover return data def create_download( self, name, size, description = DefaultValueForOptionalParameters, content_type = DefaultValueForOptionalParameters ): @@ -226,6 +230,8 @@ class Repository( object ): None, post_parameters ) + if self.__requester.isFailureStatus( status ): # pragma no branch + raise GithubException( status, data ) # pragma no cover return Download.Download( self.__requester, data, completed = True ) def create_git_blob( self, content, encoding ): @@ -241,6 +247,8 @@ class Repository( object ): None, post_parameters ) + if self.__requester.isFailureStatus( status ): # pragma no branch + raise GithubException( status, data ) # pragma no cover return GitBlob.GitBlob( self.__requester, data, completed = True ) def create_git_commit( self, message, tree, parents, author = DefaultValueForOptionalParameters, committer = DefaultValueForOptionalParameters ): @@ -261,6 +269,8 @@ class Repository( object ): None, post_parameters ) + if self.__requester.isFailureStatus( status ): # pragma no branch + raise GithubException( status, data ) # pragma no cover return GitCommit.GitCommit( self.__requester, data, completed = True ) def create_git_ref( self, ref, sha ): @@ -276,6 +286,8 @@ class Repository( object ): None, post_parameters ) + if self.__requester.isFailureStatus( status ): # pragma no branch + raise GithubException( status, data ) # pragma no cover return GitRef.GitRef( self.__requester, data, completed = True ) def create_git_tag( self, tag, message, object, type, tagger = DefaultValueForOptionalParameters ): @@ -297,6 +309,8 @@ class Repository( object ): None, post_parameters ) + if self.__requester.isFailureStatus( status ): # pragma no branch + raise GithubException( status, data ) # pragma no cover return GitTag.GitTag( self.__requester, data, completed = True ) def create_git_tree( self, tree, base_tree = DefaultValueForOptionalParameters ): @@ -311,6 +325,8 @@ class Repository( object ): None, post_parameters ) + if self.__requester.isFailureStatus( status ): # pragma no branch + raise GithubException( status, data ) # pragma no cover return GitTree.GitTree( self.__requester, data, completed = True ) def create_hook( self, name, config, events = DefaultValueForOptionalParameters, active = DefaultValueForOptionalParameters ): @@ -333,6 +349,8 @@ class Repository( object ): None, post_parameters ) + if self.__requester.isFailureStatus( status ): # pragma no branch + raise GithubException( status, data ) # pragma no cover return Hook.Hook( self.__requester, data, completed = True ) def create_issue( self, title, body = DefaultValueForOptionalParameters, assignee = DefaultValueForOptionalParameters, milestone = DefaultValueForOptionalParameters, labels = DefaultValueForOptionalParameters ): @@ -362,6 +380,8 @@ class Repository( object ): None, post_parameters ) + if self.__requester.isFailureStatus( status ): # pragma no branch + raise GithubException( status, data ) # pragma no cover return Issue.Issue( self.__requester, data, completed = True ) def create_key( self, title, key ): @@ -377,6 +397,8 @@ class Repository( object ): None, post_parameters ) + if self.__requester.isFailureStatus( status ): # pragma no branch + raise GithubException( status, data ) # pragma no cover return RepositoryKey.RepositoryKey( self.__requester, data, completed = True, repoUrl = self.url ) def create_label( self, name, color ): @@ -392,6 +414,8 @@ class Repository( object ): None, post_parameters ) + if self.__requester.isFailureStatus( status ): # pragma no branch + raise GithubException( status, data ) # pragma no cover return Label.Label( self.__requester, data, completed = True ) def create_milestone( self, title, state = DefaultValueForOptionalParameters, description = DefaultValueForOptionalParameters, due_on = DefaultValueForOptionalParameters ): @@ -417,6 +441,8 @@ class Repository( object ): None, post_parameters ) + if self.__requester.isFailureStatus( status ): # pragma no branch + raise GithubException( status, data ) # pragma no cover return Milestone.Milestone( self.__requester, data, completed = True ) def create_pull( self, *args, **kwds ): @@ -442,6 +468,8 @@ class Repository( object ): None, post_parameters ) + if self.__requester.isFailureStatus( status ): # pragma no branch + raise GithubException( status, data ) # pragma no cover return PullRequest.PullRequest( self.__requester, data, completed = True ) def edit( self, name, description = DefaultValueForOptionalParameters, homepage = DefaultValueForOptionalParameters, public = DefaultValueForOptionalParameters, has_issues = DefaultValueForOptionalParameters, has_wiki = DefaultValueForOptionalParameters, has_downloads = DefaultValueForOptionalParameters ): @@ -479,6 +507,8 @@ class Repository( object ): None, post_parameters ) + if self.__requester.isFailureStatus( status ): # pragma no branch + raise GithubException( status, data ) # pragma no cover self.__useAttributes( data ) def get_branches( self ): @@ -488,6 +518,8 @@ class Repository( object ): None, None ) + if self.__requester.isFailureStatus( status ): # pragma no branch + raise GithubException( status, data ) # pragma no cover return PaginatedList.PaginatedList( Branch.Branch, self.__requester, @@ -502,6 +534,8 @@ class Repository( object ): None, None ) + if self.__requester.isFailureStatus( status ): # pragma no branch + raise GithubException( status, data ) # pragma no cover return PaginatedList.PaginatedList( NamedUser.NamedUser, self.__requester, @@ -517,6 +551,8 @@ class Repository( object ): None, None ) + if self.__requester.isFailureStatus( status ): # pragma no branch + raise GithubException( status, data ) # pragma no cover return CommitComment.CommitComment( self.__requester, data, completed = True ) def get_comments( self ): @@ -526,6 +562,8 @@ class Repository( object ): None, None ) + if self.__requester.isFailureStatus( status ): # pragma no branch + raise GithubException( status, data ) # pragma no cover return PaginatedList.PaginatedList( CommitComment.CommitComment, self.__requester, @@ -541,6 +579,8 @@ class Repository( object ): None, None ) + if self.__requester.isFailureStatus( status ): # pragma no branch + raise GithubException( status, data ) # pragma no cover return Commit.Commit( self.__requester, data, completed = True ) def get_commits( self, sha = DefaultValueForOptionalParameters, path = DefaultValueForOptionalParameters ): @@ -559,6 +599,8 @@ class Repository( object ): url_parameters, None ) + if self.__requester.isFailureStatus( status ): # pragma no branch + raise GithubException( status, data ) # pragma no cover return PaginatedList.PaginatedList( Commit.Commit, self.__requester, @@ -573,6 +615,8 @@ class Repository( object ): None, None ) + if self.__requester.isFailureStatus( status ): # pragma no branch + raise GithubException( status, data ) # pragma no cover return PaginatedList.PaginatedList( NamedUser.NamedUser, self.__requester, @@ -588,6 +632,8 @@ class Repository( object ): None, None ) + if self.__requester.isFailureStatus( status ): # pragma no branch + raise GithubException( status, data ) # pragma no cover return Download.Download( self.__requester, data, completed = True ) def get_downloads( self ): @@ -597,6 +643,8 @@ class Repository( object ): None, None ) + if self.__requester.isFailureStatus( status ): # pragma no branch + raise GithubException( status, data ) # pragma no cover return PaginatedList.PaginatedList( Download.Download, self.__requester, @@ -611,6 +659,8 @@ class Repository( object ): None, None ) + if self.__requester.isFailureStatus( status ): # pragma no branch + raise GithubException( status, data ) # pragma no cover return PaginatedList.PaginatedList( Event.Event, self.__requester, @@ -625,6 +675,8 @@ class Repository( object ): None, None ) + if self.__requester.isFailureStatus( status ): # pragma no branch + raise GithubException( status, data ) # pragma no cover return PaginatedList.PaginatedList( Repository, self.__requester, @@ -640,6 +692,8 @@ class Repository( object ): None, None ) + if self.__requester.isFailureStatus( status ): # pragma no branch + raise GithubException( status, data ) # pragma no cover return GitBlob.GitBlob( self.__requester, data, completed = True ) def get_git_commit( self, sha ): @@ -650,6 +704,8 @@ class Repository( object ): None, None ) + if self.__requester.isFailureStatus( status ): # pragma no branch + raise GithubException( status, data ) # pragma no cover return GitCommit.GitCommit( self.__requester, data, completed = True ) def get_git_ref( self, ref ): @@ -660,6 +716,8 @@ class Repository( object ): None, None ) + if self.__requester.isFailureStatus( status ): # pragma no branch + raise GithubException( status, data ) # pragma no cover return GitRef.GitRef( self.__requester, data, completed = True ) def get_git_refs( self ): @@ -669,6 +727,8 @@ class Repository( object ): None, None ) + if self.__requester.isFailureStatus( status ): # pragma no branch + raise GithubException( status, data ) # pragma no cover return PaginatedList.PaginatedList( GitRef.GitRef, self.__requester, @@ -684,6 +744,8 @@ class Repository( object ): None, None ) + if self.__requester.isFailureStatus( status ): # pragma no branch + raise GithubException( status, data ) # pragma no cover return GitTag.GitTag( self.__requester, data, completed = True ) def get_git_tree( self, sha, recursive = DefaultValueForOptionalParameters ): @@ -699,6 +761,8 @@ class Repository( object ): url_parameters, None ) + if self.__requester.isFailureStatus( status ): # pragma no branch + raise GithubException( status, data ) # pragma no cover return GitTree.GitTree( self.__requester, data, completed = True ) def get_hook( self, id ): @@ -709,6 +773,8 @@ class Repository( object ): None, None ) + if self.__requester.isFailureStatus( status ): # pragma no branch + raise GithubException( status, data ) # pragma no cover return Hook.Hook( self.__requester, data, completed = True ) def get_hooks( self ): @@ -718,6 +784,8 @@ class Repository( object ): None, None ) + if self.__requester.isFailureStatus( status ): # pragma no branch + raise GithubException( status, data ) # pragma no cover return PaginatedList.PaginatedList( Hook.Hook, self.__requester, @@ -733,6 +801,8 @@ class Repository( object ): None, None ) + if self.__requester.isFailureStatus( status ): # pragma no branch + raise GithubException( status, data ) # pragma no cover return Issue.Issue( self.__requester, data, completed = True ) def get_issues( self, milestone = DefaultValueForOptionalParameters, state = DefaultValueForOptionalParameters, assignee = DefaultValueForOptionalParameters, mentioned = DefaultValueForOptionalParameters, labels = DefaultValueForOptionalParameters, sort = DefaultValueForOptionalParameters, direction = DefaultValueForOptionalParameters, since = DefaultValueForOptionalParameters ): @@ -775,6 +845,8 @@ class Repository( object ): url_parameters, None ) + if self.__requester.isFailureStatus( status ): # pragma no branch + raise GithubException( status, data ) # pragma no cover return PaginatedList.PaginatedList( Issue.Issue, self.__requester, @@ -790,6 +862,8 @@ class Repository( object ): None, None ) + if self.__requester.isFailureStatus( status ): # pragma no branch + raise GithubException( status, data ) # pragma no cover return IssueEvent.IssueEvent( self.__requester, data, completed = True ) def get_issues_events( self ): @@ -799,6 +873,8 @@ class Repository( object ): None, None ) + if self.__requester.isFailureStatus( status ): # pragma no branch + raise GithubException( status, data ) # pragma no cover return PaginatedList.PaginatedList( IssueEvent.IssueEvent, self.__requester, @@ -814,6 +890,8 @@ class Repository( object ): None, None ) + if self.__requester.isFailureStatus( status ): # pragma no branch + raise GithubException( status, data ) # pragma no cover return RepositoryKey.RepositoryKey( self.__requester, data, completed = True, repoUrl = self.url ) def get_keys( self ): @@ -823,6 +901,8 @@ class Repository( object ): None, None ) + if self.__requester.isFailureStatus( status ): # pragma no branch + raise GithubException( status, data ) # pragma no cover return PaginatedList.PaginatedList( lambda r, d, completed: RepositoryKey.RepositoryKey( r, d, completed, repoUrl = self.url ), self.__requester, @@ -838,6 +918,8 @@ class Repository( object ): None, None ) + if self.__requester.isFailureStatus( status ): # pragma no branch + raise GithubException( status, data ) # pragma no cover return Label.Label( self.__requester, data, completed = True ) def get_labels( self ): @@ -847,6 +929,8 @@ class Repository( object ): None, None ) + if self.__requester.isFailureStatus( status ): # pragma no branch + raise GithubException( status, data ) # pragma no cover return PaginatedList.PaginatedList( Label.Label, self.__requester, @@ -861,6 +945,8 @@ class Repository( object ): None, None ) + if self.__requester.isFailureStatus( status ): # pragma no branch + raise GithubException( status, data ) # pragma no cover return data def get_milestone( self, number ): @@ -871,6 +957,8 @@ class Repository( object ): None, None ) + if self.__requester.isFailureStatus( status ): # pragma no branch + raise GithubException( status, data ) # pragma no cover return Milestone.Milestone( self.__requester, data, completed = True ) def get_milestones( self, state = DefaultValueForOptionalParameters, sort = DefaultValueForOptionalParameters, direction = DefaultValueForOptionalParameters ): @@ -893,6 +981,8 @@ class Repository( object ): url_parameters, None ) + if self.__requester.isFailureStatus( status ): # pragma no branch + raise GithubException( status, data ) # pragma no cover return PaginatedList.PaginatedList( Milestone.Milestone, self.__requester, @@ -907,6 +997,8 @@ class Repository( object ): None, None ) + if self.__requester.isFailureStatus( status ): # pragma no branch + raise GithubException( status, data ) # pragma no cover return PaginatedList.PaginatedList( Event.Event, self.__requester, @@ -922,6 +1014,8 @@ class Repository( object ): None, None ) + if self.__requester.isFailureStatus( status ): # pragma no branch + raise GithubException( status, data ) # pragma no cover return PullRequest.PullRequest( self.__requester, data, completed = True ) def get_pulls( self, state = DefaultValueForOptionalParameters ): @@ -936,6 +1030,8 @@ class Repository( object ): url_parameters, None ) + if self.__requester.isFailureStatus( status ): # pragma no branch + raise GithubException( status, data ) # pragma no cover return PaginatedList.PaginatedList( PullRequest.PullRequest, self.__requester, @@ -950,6 +1046,8 @@ class Repository( object ): None, None ) + if self.__requester.isFailureStatus( status ): # pragma no branch + raise GithubException( status, data ) # pragma no cover return PaginatedList.PaginatedList( Tag.Tag, self.__requester, @@ -964,6 +1062,8 @@ class Repository( object ): None, None ) + if self.__requester.isFailureStatus( status ): # pragma no branch + raise GithubException( status, data ) # pragma no cover return PaginatedList.PaginatedList( Team.Team, self.__requester, @@ -978,6 +1078,8 @@ class Repository( object ): None, None ) + if self.__requester.isFailureStatus( status ): # pragma no branch + raise GithubException( status, data ) # pragma no cover return PaginatedList.PaginatedList( NamedUser.NamedUser, self.__requester, @@ -1003,6 +1105,8 @@ class Repository( object ): None, None ) + if self.__requester.isFailureStatus( status ): # pragma no branch + raise GithubException( status, data ) # pragma no cover @property def _identity( self ): diff --git a/src/github/RepositoryKey.py b/src/github/RepositoryKey.py index 99a0a721..4b6af4e3 100644 --- a/src/github/RepositoryKey.py +++ b/src/github/RepositoryKey.py @@ -48,6 +48,8 @@ class RepositoryKey( object ): None, None ) + if self.__requester.isFailureStatus( status ): # pragma no branch + raise GithubException( status, data ) # pragma no cover def edit( self, title = DefaultValueForOptionalParameters, key = DefaultValueForOptionalParameters ): if title is not DefaultValueForOptionalParameters: @@ -65,6 +67,8 @@ class RepositoryKey( object ): None, post_parameters ) + if self.__requester.isFailureStatus( status ): # pragma no branch + raise GithubException( status, data ) # pragma no cover self.__useAttributes( data ) def __initAttributes( self ): diff --git a/src/github/Team.py b/src/github/Team.py index b0b46abe..355e67e3 100644 --- a/src/github/Team.py +++ b/src/github/Team.py @@ -51,6 +51,8 @@ class Team( object ): None, None ) + if self.__requester.isFailureStatus( status ): # pragma no branch + raise GithubException( status, data ) # pragma no cover def add_to_repos( self, repo ): assert isinstance( repo, Repository.Repository ), repo @@ -60,6 +62,8 @@ class Team( object ): None, None ) + if self.__requester.isFailureStatus( status ): # pragma no branch + raise GithubException( status, data ) # pragma no cover def delete( self ): status, headers, data = self.__requester.request( @@ -68,6 +72,8 @@ class Team( object ): None, None ) + if self.__requester.isFailureStatus( status ): # pragma no branch + raise GithubException( status, data ) # pragma no cover def edit( self, name, permission = DefaultValueForOptionalParameters ): assert isinstance( name, ( str, unicode ) ), name @@ -84,6 +90,8 @@ class Team( object ): None, post_parameters ) + if self.__requester.isFailureStatus( status ): # pragma no branch + raise GithubException( status, data ) # pragma no cover self.__useAttributes( data ) def get_members( self ): @@ -93,6 +101,8 @@ class Team( object ): None, None ) + if self.__requester.isFailureStatus( status ): # pragma no branch + raise GithubException( status, data ) # pragma no cover return PaginatedList.PaginatedList( NamedUser.NamedUser, self.__requester, @@ -107,6 +117,8 @@ class Team( object ): None, None ) + if self.__requester.isFailureStatus( status ): # pragma no branch + raise GithubException( status, data ) # pragma no cover return PaginatedList.PaginatedList( Repository.Repository, self.__requester, @@ -142,6 +154,8 @@ class Team( object ): None, None ) + if self.__requester.isFailureStatus( status ): # pragma no branch + raise GithubException( status, data ) # pragma no cover def remove_from_repos( self, repo ): assert isinstance( repo, Repository.Repository ), repo @@ -151,6 +165,8 @@ class Team( object ): None, None ) + if self.__requester.isFailureStatus( status ): # pragma no branch + raise GithubException( status, data ) # pragma no cover def __initAttributes( self ): self.__id = None diff --git a/src/github/UserKey.py b/src/github/UserKey.py index 907ec95d..a1174fda 100644 --- a/src/github/UserKey.py +++ b/src/github/UserKey.py @@ -43,6 +43,8 @@ class UserKey( object ): None, None ) + if self.__requester.isFailureStatus( status ): # pragma no branch + raise GithubException( status, data ) # pragma no cover def edit( self, title = DefaultValueForOptionalParameters, key = DefaultValueForOptionalParameters ): if title is not DefaultValueForOptionalParameters: @@ -60,6 +62,8 @@ class UserKey( object ): None, post_parameters ) + if self.__requester.isFailureStatus( status ): # pragma no branch + raise GithubException( status, data ) # pragma no cover self.__useAttributes( data ) def __initAttributes( self ):