refactor: remove unused imports (#11)

* refactor: remove unused imports

An object has been imported but is not used anywhere in the file.
It should either be used or the import should be removed.

* other fixes

* style: format code with Black and isort

This commit fixes the style issues introduced in f2740df according to the output
from Black and isort.

Details: #11

---------

Co-authored-by: deepsource-autofix[bot] <62050782+deepsource-autofix[bot]@users.noreply.github.com>
Co-authored-by: Heitor Polidoro <[email protected]>

github_app/LazyCompletableGithubObject.py

@@ -10,6 +10,11 @@
 
 
 class LazyCompletableGithubObject(CompletableGithubObject):
+    """
+    A lazy CompletableGithubObject that will only initialize when it is accessed.
+    In the initialization will create a github.Requester.Requester
+    """
+
     def __init__(
         self,
         requester: "Requester" = None,
@@ -33,7 +38,9 @@ def __init__(
     def lazy_requester(self):
         if self._lazy_requester is None:
             token = (
-                GithubIntegration(auth=AppAuth(681139, os.getenv("PRIVATE_KEY")))
+                GithubIntegration(
+                    auth=AppAuth(os.getenv("GITHUB_APP_ID"), os.getenv("PRIVATE_KEY"))
+                )
                 .get_access_token(Event.installation_id)
                 .token
             )
@@ -50,6 +57,7 @@ def lazy_requester(self):
         return self._lazy_requester
 
     def __getattribute__(self, item):
+        """If the value is None, makes a request to update the object."""
         value = super().__getattribute__(item)
         if (
             not item.startswith("_lazy")
@@ -66,7 +74,10 @@ def __getattribute__(self, item):
         return value
 
     @staticmethod
-    def get_lazy_instance(cls, attributes):
-        if LazyCompletableGithubObject not in cls.__bases__:
-            cls.__bases__ = tuple([LazyCompletableGithubObject] + list(cls.__bases__))
-        return cls(attributes=attributes)
+    def get_lazy_instance(clazz, attributes):
+        """Makes the clazz a subclass of LazyCompletableGithubObject"""
+        if LazyCompletableGithubObject not in clazz.__bases__:
+            clazz.__bases__ = tuple(
+                [LazyCompletableGithubObject] + list(clazz.__bases__)
+            )
+        return clazz(attributes=attributes)

github_app/ReleaseEvent.py

@@ -1,6 +1,4 @@
 from github.GitRelease import GitRelease
-from github.Issue import Issue
-from github.IssueComment import IssueComment
 from github.NamedUser import NamedUser
 from github.Repository import Repository
 
@@ -9,6 +7,8 @@
 
 
 class ReleaseEvent(Event):
+    """This class represents a generic release event."""
+
     name = "release"
 
     def __init__(self, release, repository, sender, **kwargs):
@@ -25,8 +25,12 @@ def __init__(self, release, repository, sender, **kwargs):
 
 
 class ReleaseReleasedEvent(ReleaseEvent):
+    """This class represents an event when a release is released."""
+
     action = "released"
 
 
 class ReleaseCreatedEvent(ReleaseEvent):
+    """This class represents an event when a release is created."""
+
     action = "created"

tests/conftest.py

@@ -1,27 +1 @@
-import os
-import pathlib
 
-import pytest
-from cryptography.hazmat.backends import default_backend as crypto_default_backend
-from cryptography.hazmat.primitives import serialization as crypto_serialization
-from cryptography.hazmat.primitives.asymmetric import rsa
-
-
-@pytest.fixture(scope="session")
-def private_key():
-    if os.path.exists("private-key.pem"):
-        return pathlib.Path("private-key.pem").read_text()
-    key = rsa.generate_private_key(
-        backend=crypto_default_backend(), public_exponent=65537, key_size=2048
-    )
-
-    return key.private_bytes(
-        crypto_serialization.Encoding.PEM,
-        crypto_serialization.PrivateFormat.PKCS8,
-        crypto_serialization.NoEncryption(),
-    ).decode()
-
-
-@pytest.fixture(scope="session")
-def vcr_cassette_name(request):
-    return "github_requests.yaml"