Cleanup use of (app) auth required decorator

westphahl · westphahl · commit 392f68b40b92 · 2019-01-10T13:10:30.000+01:00
diff --git a/src/github3/checks.py b/src/github3/checks.py
@@ -216,6 +216,7 @@ def _update_attributes(self, suite):
     def _repr(self):
         return "<{s.class_name} [{s.id}:{s.status}]>".format(s=self)
 
+    @decorators.requires_auth
     def rerequest(self):
         """Rerequest the check suite.
 
diff --git a/src/github3/repos/commit.py b/src/github3/repos/commit.py
@@ -3,7 +3,7 @@
 from __future__ import unicode_literals
 
 from . import status
-from .. import checks, git, models, users, decorators
+from .. import checks, git, models, users
 from .comment import RepoComment
 
 
@@ -36,7 +36,6 @@ def _update_attributes(self, commit):
     def _repr(self):
         return "<{0} [{1}]>".format(self.class_name, self.sha[:7])
 
-    @decorators.requires_app_installation_auth
     def check_runs(self):
         """Retrieve the check runs for this commit.
 
@@ -52,7 +51,6 @@ def check_runs(self):
             -1, url, checks.CheckRun, headers=checks.CheckRun.CUSTOM_HEADERS
         )
 
-    @decorators.requires_app_installation_auth
     def check_suites(self):
         """Retrieve the check suites for this commit.
 
diff --git a/src/github3/repos/repo.py b/src/github3/repos/repo.py
@@ -263,7 +263,6 @@ def branches(self, number=-1, protected=False, etag=None):
             headers=branch.Branch.PREVIEW_HEADERS,
         )
 
-    @decorators.requires_app_installation_auth
     def check_run(self, id):
         """Return a single check run.
 
@@ -284,7 +283,6 @@ def check_run(self, id):
             )
         return self._instance_or_null(checks.CheckRun, data)
 
-    @decorators.requires_app_installation_auth
     def check_suite(self, id):
         """Return a single check suite.
 
diff --git a/tests/unit/test_repos_commit.py b/tests/unit/test_repos_commit.py
@@ -1,8 +1,5 @@
 """Unit tests for Repository Commits."""
 import github3
-import pytest
-
-from github3.exceptions import GitHubException
 
 from . import helper
 
@@ -66,19 +63,3 @@ def test_check_suits(self):
             params={"per_page": 100},
             headers=github3.checks.CheckSuite.CUSTOM_HEADERS,
         )
-
-
-class TestRepoCommitRequiresAuth(helper.UnitRequiresAuthenticationHelper):
-
-    """Unit test for auth required Repository Commit methods."""
-
-    described_class = github3.repos.commit.RepoCommit
-    example_data = example_commit_data
-
-    def test_check_runs(self):
-        with pytest.raises(GitHubException):
-            self.instance.check_runs()
-
-    def test_check_suites(self):
-        with pytest.raises(GitHubException):
-            self.instance.check_suites()

Original file line number	Diff line number	Diff line change
`@@ -263,7 +263,6 @@ def branches(self, number=-1, protected=False, etag=None):`
`263`	`263`	`headers=branch.Branch.PREVIEW_HEADERS,`
`264`	`264`	`)`
`265`	`265`
`266`		`- @decorators.requires_app_installation_auth`
`267`	`266`	`def check_run(self, id):`
`268`	`267`	`"""Return a single check run.`
`269`	`268`
`@@ -284,7 +283,6 @@ def check_run(self, id):`
`284`	`283`	`)`
`285`	`284`	`return self._instance_or_null(checks.CheckRun, data)`
`286`	`285`
`287`		`- @decorators.requires_app_installation_auth`
`288`	`286`	`def check_suite(self, id):`
`289`	`287`	`"""Return a single check suite.`
`290`	`288`