Do not assert that we know all the attributes we receive

src/github/AuthenticatedUser.py

@@ -564,9 +564,6 @@ class AuthenticatedUser( object ):
         self.__completed = True
 
     def __useAttributes( self, attributes ):
-        # @todo Remove this debug weakness: we shall assume that github will add new attributes
-        for attribute in attributes:
-            assert attribute in [ "avatar_url", "bio", "blog", "collaborators", "company", "created_at", "disk_usage", "email", "followers", "following", "gravatar_id", "hireable", "html_url", "id", "location", "login", "name", "owned_private_repos", "plan", "private_gists", "public_gists", "public_repos", "total_private_repos", "type", "url", ], attribute
         # @todo No need to check if attribute is in attributes when attribute is mandatory
         if "avatar_url" in attributes and attributes[ "avatar_url" ] is not None: # pragma no branch
             assert isinstance( attributes[ "avatar_url" ], ( str, unicode ) )

src/github/Authorization.py

@@ -113,9 +113,6 @@ class Authorization( object ):
         self.__completed = True
 
     def __useAttributes( self, attributes ):
-        # @todo Remove this debug weakness: we shall assume that github will add new attributes
-        for attribute in attributes:
-            assert attribute in [ "app", "created_at", "id", "note", "note_url", "scopes", "token", "updated_at", "url", ], attribute
         # @todo No need to check if attribute is in attributes when attribute is mandatory
         if "app" in attributes and attributes[ "app" ] is not None: # pragma no branch
             self.__app = attributes[ "app" ]

src/github/Branch.py

@@ -24,9 +24,6 @@ class Branch( object ):
         self.__name = None
 
     def __useAttributes( self, attributes ):
-        # @todo Remove this debug weakness: we shall assume that github will add new attributes
-        for attribute in attributes:
-            assert attribute in [ "commit", "name", ], attribute
         # @todo No need to check if attribute is in attributes when attribute is mandatory
         if "commit" in attributes and attributes[ "commit" ] is not None: # pragma no branch
             assert isinstance( attributes[ "commit" ], dict )

src/github/Commit.py

@@ -116,9 +116,6 @@ class Commit( object ):
         self.__completed = True
 
     def __useAttributes( self, attributes ):
-        # @todo Remove this debug weakness: we shall assume that github will add new attributes
-        for attribute in attributes:
-            assert attribute in [ "author", "commit", "committer", "files", "parents", "sha", "stats", "url", ], attribute
         # @todo No need to check if attribute is in attributes when attribute is mandatory
         if "author" in attributes and attributes[ "author" ] is not None: # pragma no branch
             assert isinstance( attributes[ "author" ], dict )

src/github/CommitComment.py

@@ -117,9 +117,6 @@ class CommitComment( object ):
         self.__completed = True
 
     def __useAttributes( self, attributes ):
-        # @todo Remove this debug weakness: we shall assume that github will add new attributes
-        for attribute in attributes:
-            assert attribute in [ "body", "commit_id", "created_at", "html_url", "id", "line", "path", "position", "updated_at", "url", "user", ], attribute
         # @todo No need to check if attribute is in attributes when attribute is mandatory
         if "body" in attributes and attributes[ "body" ] is not None: # pragma no branch
             assert isinstance( attributes[ "body" ], ( str, unicode ) )

src/github/CommitFile.py

@@ -58,9 +58,6 @@ class CommitFile( object ):
         self.__status = None
 
     def __useAttributes( self, attributes ):
-        # @todo Remove this debug weakness: we shall assume that github will add new attributes
-        for attribute in attributes:
-            assert attribute in [ "additions", "blob_url", "changes", "deletions", "filename", "patch", "raw_url", "sha", "status", ], attribute
         # @todo No need to check if attribute is in attributes when attribute is mandatory
         if "additions" in attributes and attributes[ "additions" ] is not None: # pragma no branch
             self.__additions = attributes[ "additions" ]

src/github/CommitStats.py

@@ -28,9 +28,6 @@ class CommitStats( object ):
         self.__total = None
 
     def __useAttributes( self, attributes ):
-        # @todo Remove this debug weakness: we shall assume that github will add new attributes
-        for attribute in attributes:
-            assert attribute in [ "additions", "deletions", "total", ], attribute
         # @todo No need to check if attribute is in attributes when attribute is mandatory
         if "additions" in attributes and attributes[ "additions" ] is not None: # pragma no branch
             assert isinstance( attributes[ "additions" ], int )

src/github/Download.py

@@ -158,9 +158,6 @@ class Download( object ):
         self.__completed = True
 
     def __useAttributes( self, attributes ):
-        # @todo Remove this debug weakness: we shall assume that github will add new attributes
-        for attribute in attributes:
-            assert attribute in [ "accesskeyid", "acl", "bucket", "content_type", "created_at", "description", "download_count", "expirationdate", "html_url", "id", "mime_type", "name", "path", "policy", "prefix", "redirect", "s3_url", "signature", "size", "url", "x-amz-meta-content-disposition" ], attribute
         # @todo No need to check if attribute is in attributes when attribute is mandatory
         if "accesskeyid" in attributes and attributes[ "accesskeyid" ] is not None: # pragma no branch
             self.__accesskeyid = attributes[ "accesskeyid" ]

src/github/Event.py

@@ -56,9 +56,6 @@ class Event( object ):
         self.__type = None
 
     def __useAttributes( self, attributes ):
-        # @todo Remove this debug weakness: we shall assume that github will add new attributes
-        for attribute in attributes:
-            assert attribute in [ "actor", "created_at", "id", "org", "payload", "public", "repo", "type", ], attribute
         # @todo No need to check if attribute is in attributes when attribute is mandatory
         if "actor" in attributes and attributes[ "actor" ] is not None: # pragma no branch
             assert isinstance( attributes[ "actor" ], dict )

src/github/Gist.py

@@ -216,9 +216,6 @@ class Gist( object ):
         self.__completed = True
 
     def __useAttributes( self, attributes ):
-        # @todo Remove this debug weakness: we shall assume that github will add new attributes
-        for attribute in attributes:
-            assert attribute in [ "comments", "created_at", "description", "files", "fork_of", "forks", "git_pull_url", "git_push_url", "history", "html_url", "id", "public", "updated_at", "url", "user", ], attribute
         # @todo No need to check if attribute is in attributes when attribute is mandatory
         if "comments" in attributes and attributes[ "comments" ] is not None: # pragma no branch
             assert isinstance( attributes[ "comments" ], int )

src/github/GistComment.py

@@ -64,9 +64,6 @@ class GistComment( object ):
         self.__user = None
 
     def __useAttributes( self, attributes ):
-        # @todo Remove this debug weakness: we shall assume that github will add new attributes
-        for attribute in attributes:
-            assert attribute in [ "body", "created_at", "id", "updated_at", "url", "user", ], attribute
         # @todo No need to check if attribute is in attributes when attribute is mandatory
         if "body" in attributes and attributes[ "body" ] is not None: # pragma no branch
             self.__body = attributes[ "body" ]

src/github/GistHistoryState.py

@@ -40,9 +40,6 @@ class GistHistoryState( object ):
         self.__version = None
 
     def __useAttributes( self, attributes ):
-        # @todo Remove this debug weakness: we shall assume that github will add new attributes
-        for attribute in attributes:
-            assert attribute in [ "change_status", "committed_at", "url", "user", "version", ], attribute
         # @todo No need to check if attribute is in attributes when attribute is mandatory
         if "change_status" in attributes and attributes[ "change_status" ] is not None: # pragma no branch
             assert isinstance( attributes[ "change_status" ], dict )

src/github/GitAuthor.py

@@ -28,9 +28,6 @@ class GitAuthor( object ):
         self.__name = None
 
     def __useAttributes( self, attributes ):
-        # @todo Remove this debug weakness: we shall assume that github will add new attributes
-        for attribute in attributes:
-            assert attribute in [ "date", "email", "name", ], attribute
         # @todo No need to check if attribute is in attributes when attribute is mandatory
         if "date" in attributes and attributes[ "date" ] is not None: # pragma no branch
             assert isinstance( attributes[ "date" ], ( str, unicode ) )

src/github/GitBlob.py

@@ -38,9 +38,6 @@ class GitBlob( object ):
         self.__url = None
 
     def __useAttributes( self, attributes ):
-        # @todo Remove this debug weakness: we shall assume that github will add new attributes
-        for attribute in attributes:
-            assert attribute in [ "content", "encoding", "sha", "size", "url", ], attribute
         # @todo No need to check if attribute is in attributes when attribute is mandatory
         if "content" in attributes and attributes[ "content" ] is not None: # pragma no branch
             assert isinstance( attributes[ "content" ], ( str, unicode ) )

src/github/GitCommit.py

@@ -51,9 +51,6 @@ class GitCommit( object ):
         self.__url = None
 
     def __useAttributes( self, attributes ):
-        # @todo Remove this debug weakness: we shall assume that github will add new attributes
-        for attribute in attributes:
-            assert attribute in [ "author", "committer", "message", "parents", "sha", "tree", "url", ], attribute
         # @todo No need to check if attribute is in attributes when attribute is mandatory
         if "author" in attributes and attributes[ "author" ] is not None: # pragma no branch
             assert isinstance( attributes[ "author" ], dict )

src/github/GitObject.py

@@ -28,9 +28,6 @@ class GitObject( object ):
         self.__url = None
 
     def __useAttributes( self, attributes ):
-        # @todo Remove this debug weakness: we shall assume that github will add new attributes
-        for attribute in attributes:
-            assert attribute in [ "sha", "type", "url", ], attribute
         # @todo No need to check if attribute is in attributes when attribute is mandatory
         if "sha" in attributes and attributes[ "sha" ] is not None: # pragma no branch
             assert isinstance( attributes[ "sha" ], ( str, unicode ) )

src/github/GitRef.py

@@ -51,9 +51,6 @@ class GitRef( object ):
         self.__url = None
 
     def __useAttributes( self, attributes ):
-        # @todo Remove this debug weakness: we shall assume that github will add new attributes
-        for attribute in attributes:
-            assert attribute in [ "object", "ref", "url", ], attribute
         # @todo No need to check if attribute is in attributes when attribute is mandatory
         if "object" in attributes and attributes[ "object" ] is not None: # pragma no branch
             assert isinstance( attributes[ "object" ], dict )

src/github/GitTag.py

@@ -45,9 +45,6 @@ class GitTag( object ):
         self.__url = None
 
     def __useAttributes( self, attributes ):
-        # @todo Remove this debug weakness: we shall assume that github will add new attributes
-        for attribute in attributes:
-            assert attribute in [ "message", "object", "sha", "tag", "tagger", "url", ], attribute
         # @todo No need to check if attribute is in attributes when attribute is mandatory
         if "message" in attributes and attributes[ "message" ] is not None: # pragma no branch
             assert isinstance( attributes[ "message" ], ( str, unicode ) )

src/github/GitTree.py

@@ -29,9 +29,6 @@ class GitTree( object ):
         self.__url = None
 
     def __useAttributes( self, attributes ):
-        # @todo Remove this debug weakness: we shall assume that github will add new attributes
-        for attribute in attributes:
-            assert attribute in [ "sha", "tree", "url", ], attribute
         # @todo No need to check if attribute is in attributes when attribute is mandatory
         if "sha" in attributes and attributes[ "sha" ] is not None: # pragma no branch
             assert isinstance( attributes[ "sha" ], ( str, unicode ) )

src/github/GitTreeElement.py

@@ -43,9 +43,6 @@ class GitTreeElement( object ):
         self.__url = None
 
     def __useAttributes( self, attributes ):
-        # @todo Remove this debug weakness: we shall assume that github will add new attributes
-        for attribute in attributes:
-            assert attribute in [ "mode", "path", "sha", "size", "type", "url", ], attribute
         # @todo No need to check if attribute is in attributes when attribute is mandatory
         if "mode" in attributes and attributes[ "mode" ] is not None: # pragma no branch
             assert isinstance( attributes[ "mode" ], ( str, unicode ) )

src/github/Hook.py

@@ -95,9 +95,6 @@ class Hook( object ):
         self.__url = None
 
     def __useAttributes( self, attributes ):
-        # @todo Remove this debug weakness: we shall assume that github will add new attributes
-        for attribute in attributes:
-            assert attribute in [ "active", "config", "created_at", "events", "id", "last_response", "name", "updated_at", "url", ], attribute
         # @todo No need to check if attribute is in attributes when attribute is mandatory
         if "active" in attributes and attributes[ "active" ] is not None: # pragma no branch
             self.__active = attributes[ "active" ]

src/github/Issue.py

@@ -264,9 +264,6 @@ class Issue( object ):
         self.__completed = True
 
     def __useAttributes( self, attributes ):
-        # @todo Remove this debug weakness: we shall assume that github will add new attributes
-        for attribute in attributes:
-            assert attribute in [ "assignee", "body", "closed_at", "closed_by", "comments", "created_at", "html_url", "id", "labels", "milestone", "number", "pull_request", "repository", "state", "title", "updated_at", "url", "user", ], attribute
         # @todo No need to check if attribute is in attributes when attribute is mandatory
         if "assignee" in attributes and attributes[ "assignee" ] is not None: # pragma no branch
             assert isinstance( attributes[ "assignee" ], dict )

src/github/IssueComment.py

@@ -64,9 +64,6 @@ class IssueComment( object ):
         self.__user = None
 
     def __useAttributes( self, attributes ):
-        # @todo Remove this debug weakness: we shall assume that github will add new attributes
-        for attribute in attributes:
-            assert attribute in [ "body", "created_at", "id", "updated_at", "url", "user", ], attribute
         # @todo No need to check if attribute is in attributes when attribute is mandatory
         if "body" in attributes and attributes[ "body" ] is not None: # pragma no branch
             self.__body = attributes[ "body" ]

src/github/IssueEvent.py

@@ -74,9 +74,6 @@ class IssueEvent( object ):
         self.__completed = True
 
     def __useAttributes( self, attributes ):
-        # @todo Remove this debug weakness: we shall assume that github will add new attributes
-        for attribute in attributes:
-            assert attribute in [ "actor", "commit_id", "created_at", "event", "id", "issue", "url", ], attribute
         # @todo No need to check if attribute is in attributes when attribute is mandatory
         if "actor" in attributes and attributes[ "actor" ] is not None: # pragma no branch
             assert isinstance( attributes[ "actor" ], dict )

src/github/Label.py

@@ -56,9 +56,6 @@ class Label( object ):
         self.__url = None
 
     def __useAttributes( self, attributes ):
-        # @todo Remove this debug weakness: we shall assume that github will add new attributes
-        for attribute in attributes:
-            assert attribute in [ "color", "name", "url", ], attribute
         # @todo No need to check if attribute is in attributes when attribute is mandatory
         if "color" in attributes and attributes[ "color" ] is not None: # pragma no branch
             self.__color = attributes[ "color" ]

src/github/Milestone.py

@@ -110,9 +110,6 @@ class Milestone( object ):
         self.__url = None
 
     def __useAttributes( self, attributes ):
-        # @todo Remove this debug weakness: we shall assume that github will add new attributes
-        for attribute in attributes:
-            assert attribute in [ "closed_issues", "created_at", "creator", "description", "due_on", "id", "number", "open_issues", "state", "title", "url", ], attribute
         # @todo No need to check if attribute is in attributes when attribute is mandatory
         if "closed_issues" in attributes and attributes[ "closed_issues" ] is not None: # pragma no branch
             assert isinstance( attributes[ "closed_issues" ], int )

src/github/NamedUser.py

@@ -361,9 +361,6 @@ class NamedUser( object ):
         self.__completed = True
 
     def __useAttributes( self, attributes ):
-        # @todo Remove this debug weakness: we shall assume that github will add new attributes
-        for attribute in attributes:
-            assert attribute in [ "avatar_url", "bio", "blog", "collaborators", "company", "contributions", "created_at", "disk_usage", "email", "followers", "following", "gravatar_id", "hireable", "html_url", "id", "location", "login", "name", "owned_private_repos", "plan", "private_gists", "public_gists", "public_repos", "total_private_repos", "type", "url", ], attribute
         # @todo No need to check if attribute is in attributes when attribute is mandatory
         if "avatar_url" in attributes and attributes[ "avatar_url" ] is not None: # pragma no branch
             assert isinstance( attributes[ "avatar_url" ], ( str, unicode ) )

src/github/Organization.py

@@ -386,9 +386,6 @@ class Organization( object ):
         self.__completed = True
 
     def __useAttributes( self, attributes ):
-        # @todo Remove this debug weakness: we shall assume that github will add new attributes
-        for attribute in attributes:
-            assert attribute in [ "avatar_url", "billing_email", "blog", "collaborators", "company", "created_at", "disk_usage", "email", "followers", "following", "gravatar_id", "html_url", "id", "location", "login", "name", "owned_private_repos", "plan", "private_gists", "public_gists", "public_repos", "total_private_repos", "type", "url", ], attribute
         # @todo No need to check if attribute is in attributes when attribute is mandatory
         if "avatar_url" in attributes and attributes[ "avatar_url" ] is not None: # pragma no branch
             assert isinstance( attributes[ "avatar_url" ], ( str, unicode ) )

src/github/Permissions.py

@@ -28,9 +28,6 @@ class Permissions( object ):
         self.__push = None
 
     def __useAttributes( self, attributes ):
-        # @todo Remove this debug weakness: we shall assume that github will add new attributes
-        for attribute in attributes:
-            assert attribute in [ "admin", "pull", "push", ], attribute
         # @todo No need to check if attribute is in attributes when attribute is mandatory
         if "admin" in attributes and attributes[ "admin" ] is not None: # pragma no branch
             assert isinstance( attributes[ "admin" ], bool )

src/github/Plan.py

@@ -33,9 +33,6 @@ class Plan( object ):
         self.__space = None
 
     def __useAttributes( self, attributes ):
-        # @todo Remove this debug weakness: we shall assume that github will add new attributes
-        for attribute in attributes:
-            assert attribute in [ "collaborators", "name", "private_repos", "space", ], attribute
         # @todo No need to check if attribute is in attributes when attribute is mandatory
         if "collaborators" in attributes and attributes[ "collaborators" ] is not None: # pragma no branch
             assert isinstance( attributes[ "collaborators" ], int )

src/github/PullRequest.py

@@ -290,9 +290,6 @@ class PullRequest( object ):
         self.__completed = True
 
     def __useAttributes( self, attributes ):
-        # @todo Remove this debug weakness: we shall assume that github will add new attributes
-        for attribute in attributes:
-            assert attribute in [ "additions", "base", "body", "changed_files", "closed_at", "comments", "commits", "created_at", "deletions", "diff_url", "head", "html_url", "id", "issue_url", "mergeable", "merged", "merged_at", "merged_by", "number", "patch_url", "review_comments", "state", "title", "updated_at", "url", "user", "_links", ], attribute
         # @todo No need to check if attribute is in attributes when attribute is mandatory
         if "additions" in attributes and attributes[ "additions" ] is not None: # pragma no branch
             self.__additions = attributes[ "additions" ]

src/github/PullRequestComment.py

@@ -117,9 +117,6 @@ class PullRequestComment( object ):
         self.__completed = True
 
     def __useAttributes( self, attributes ):
-        # @todo Remove this debug weakness: we shall assume that github will add new attributes
-        for attribute in attributes:
-            assert attribute in [ "body", "commit_id", "created_at", "id", "original_commit_id", "original_position", "path", "position", "updated_at", "url", "user", "_links", ], attribute
         # @todo No need to check if attribute is in attributes when attribute is mandatory
         if "body" in attributes and attributes[ "body" ] is not None: # pragma no branch
             self.__body = attributes[ "body" ]

src/github/PullRequestFile.py

@@ -58,9 +58,6 @@ class PullRequestFile( object ):
         self.__status = None
 
     def __useAttributes( self, attributes ):
-        # @todo Remove this debug weakness: we shall assume that github will add new attributes
-        for attribute in attributes:
-            assert attribute in [ "additions", "blob_url", "changes", "deletions", "filename", "patch", "raw_url", "sha", "status", ], attribute
         # @todo No need to check if attribute is in attributes when attribute is mandatory
         if "additions" in attributes and attributes[ "additions" ] is not None: # pragma no branch
             self.__additions = attributes[ "additions" ]

src/github/Repository.py

@@ -917,9 +917,6 @@ class Repository( object ):
         self.__completed = True
 
     def __useAttributes( self, attributes ):
-        # @todo Remove this debug weakness: we shall assume that github will add new attributes
-        for attribute in attributes:
-            assert attribute in [ "clone_url", "created_at", "description", "fork", "forks", "full_name", "git_url", "has_downloads", "has_issues", "has_wiki", "homepage", "html_url", "id", "language", "master_branch", "mirror_url", "name", "open_issues", "organization", "owner", "parent", "permissions", "private", "pushed_at", "size", "source", "ssh_url", "svn_url", "updated_at", "url", "watchers", ], attribute
         # @todo No need to check if attribute is in attributes when attribute is mandatory
         if "clone_url" in attributes and attributes[ "clone_url" ] is not None: # pragma no branch
             assert isinstance( attributes[ "clone_url" ], ( str, unicode ) )

src/github/RepositoryKey.py

@@ -83,9 +83,6 @@ class RepositoryKey( object ):
         self.__completed = True
 
     def __useAttributes( self, attributes ):
-        # @todo Remove this debug weakness: we shall assume that github will add new attributes
-        for attribute in attributes:
-            assert attribute in [ "id", "key", "title", "url", "verified", ], attribute
         # @todo No need to check if attribute is in attributes when attribute is mandatory
         if "id" in attributes and attributes[ "id" ] is not None: # pragma no branch
             self.__id = attributes[ "id" ]

src/github/Tag.py

@@ -34,9 +34,6 @@ class Tag( object ):
         self.__zipball_url = None
 
     def __useAttributes( self, attributes ):
-        # @todo Remove this debug weakness: we shall assume that github will add new attributes
-        for attribute in attributes:
-            assert attribute in [ "commit", "name", "tarball_url", "zipball_url", ], attribute
         # @todo No need to check if attribute is in attributes when attribute is mandatory
         if "commit" in attributes and attributes[ "commit" ] is not None: # pragma no branch
             assert isinstance( attributes[ "commit" ], dict )

src/github/Team.py

@@ -168,9 +168,6 @@ class Team( object ):
         self.__completed = True
 
     def __useAttributes( self, attributes ):
-        # @todo Remove this debug weakness: we shall assume that github will add new attributes
-        for attribute in attributes:
-            assert attribute in [ "id", "members_count", "name", "permission", "repos_count", "url", ], attribute
         # @todo No need to check if attribute is in attributes when attribute is mandatory
         if "id" in attributes and attributes[ "id" ] is not None: # pragma no branch
             self.__id = attributes[ "id" ]

src/github/UserKey.py

@@ -83,9 +83,6 @@ class UserKey( object ):
         self.__completed = True
 
     def __useAttributes( self, attributes ):
-        # @todo Remove this debug weakness: we shall assume that github will add new attributes
-        for attribute in attributes:
-            assert attribute in [ "id", "key", "title", "url", "verified", ], attribute
         # @todo No need to check if attribute is in attributes when attribute is mandatory
         if "id" in attributes and attributes[ "id" ] is not None: # pragma no branch
             self.__id = attributes[ "id" ]