fix: return type Self for functions that return self (#2468)

Author: myakove

class_generator/class_generator.py

@@ -636,7 +636,7 @@ def get_client_binary() -> str:
         sys.exit(1)
 
 
-def update_kind_schema():
+def update_kind_schema() -> None:
     openapi2jsonschema_str: str = "openapi2jsonschema"
     client = get_client_binary()
 

class_generator/scripts/tests/test_scripts.py

@@ -1,7 +1,7 @@
 from class_generator.scripts.tools import get_generated_files
 
 
-def test_get_generated_files():
+def test_get_generated_files() -> None:
     _files: dict[str, dict[str, str]] = get_generated_files()
     for _key in ("with_end_comment", "without_end_comment", "not_generated"):
         assert _files.get(_key), f"{_key} is missing in generated files"

ocp_resources/project_request.py

@@ -45,7 +45,7 @@ def to_dict(self) -> None:
 
     # End of generated code
 
-    def deploy(self, wait: bool = False) -> Project:
+    def deploy(self, wait: bool = False) -> Project:  # type: ignore[override]
         super().deploy(wait=wait)
 
         project = Project(

ocp_resources/resource.py

@@ -11,7 +11,7 @@
 from io import StringIO
 from signal import SIGINT, signal
 from types import TracebackType
-from typing import Any, Type
+from typing import Any, Self, Type
 from urllib.parse import parse_qs, urlencode, urlparse
 
 import jsonschema
@@ -752,7 +752,7 @@ def _sigint_handler(self, signal_received: int, frame: Any) -> None:
         self.__exit__()
         sys.exit(signal_received)
 
-    def deploy(self, wait: bool = False) -> "Resource | NamespacedResource":
+    def deploy(self, wait: bool = False) -> Self:
         """
         For debug, export REUSE_IF_RESOURCE_EXISTS to skip resource create.
         Spaces are important in the export dict
@@ -1734,7 +1734,7 @@ def update(self, backup_resources: bool = False) -> None:
     def restore(self) -> None:
         self._apply_patches_sampler(patches=self._backups, action_text="Restoring", action=self.action)
 
-    def __enter__(self) -> "ResourceEditor":
+    def __enter__(self) -> Self:
         self.update(backup_resources=True)
         return self
 
@@ -1863,11 +1863,11 @@ class BaseResourceList(ABC):
     iteration, indexing, deployment, and cleanup operations.
     """
 
-    def __init__(self, client: DynamicClient):
+    def __init__(self, client: DynamicClient) -> None:
         self.resources: list[Resource] = []
         self.client = client
 
-    def __enter__(self):
+    def __enter__(self) -> Self:
         """Enters the runtime context and deploys all resources."""
         self.deploy()
         return self