Merge pull request #1350 from JohnVillalovos/jlvillal/isinstance

nejch · web-flow · commit c530f75a3f35 · 2021-03-06T18:57:08.000+01:00
chore: Put assert statements inside 'if TYPE_CHECKING:'
diff --git a/gitlab/client.py b/gitlab/client.py
@@ -17,7 +17,7 @@
 """Wrapper for the GitLab API."""
 
 import time
-from typing import cast, Any, Dict, List, Optional, Tuple, Union
+from typing import cast, Any, Dict, List, Optional, Tuple, TYPE_CHECKING, Union
 
 import requests
 import requests.utils
@@ -266,7 +266,8 @@ def lint(self, content: str, **kwargs: Any) -> Tuple[bool, List[str]]:
         """
         post_data = {"content": content}
         data = self.http_post("/ci/lint", post_data=post_data, **kwargs)
-        assert isinstance(data, dict)
+        if TYPE_CHECKING:
+            assert not isinstance(data, requests.Response)
         return (data["status"] == "valid", data["errors"])
 
     @gitlab.exceptions.on_http_error(gitlab.exceptions.GitlabMarkdownError)
@@ -294,7 +295,8 @@ def markdown(
         if project is not None:
             post_data["project"] = project
         data = self.http_post("/markdown", post_data=post_data, **kwargs)
-        assert isinstance(data, dict)
+        if TYPE_CHECKING:
+            assert not isinstance(data, requests.Response)
         return data["html"]
 
     @gitlab.exceptions.on_http_error(gitlab.exceptions.GitlabLicenseError)
@@ -333,7 +335,8 @@ def set_license(self, license: str, **kwargs: Any) -> Dict[str, Any]:
         """
         data = {"license": license}
         result = self.http_post("/license", post_data=data, **kwargs)
-        assert isinstance(result, dict)
+        if TYPE_CHECKING:
+            assert not isinstance(result, requests.Response)
         return result
 
     def _set_auth_info(self) -> None:
@@ -855,7 +858,8 @@ def _query(
     @property
     def current_page(self) -> int:
         """The current page number."""
-        assert self._current_page is not None
+        if TYPE_CHECKING:
+            assert self._current_page is not None
         return int(self._current_page)
 
     @property
@@ -877,19 +881,22 @@ def next_page(self) -> Optional[int]:
     @property
     def per_page(self) -> int:
         """The number of items per page."""
-        assert self._per_page is not None
+        if TYPE_CHECKING:
+            assert self._per_page is not None
         return int(self._per_page)
 
     @property
     def total_pages(self) -> int:
         """The total number of pages."""
-        assert self._total_pages is not None
+        if TYPE_CHECKING:
+            assert self._total_pages is not None
         return int(self._total_pages)
 
     @property
     def total(self) -> int:
         """The total number of items."""
-        assert self._total is not None
+        if TYPE_CHECKING:
+            assert self._total is not None
         return int(self._total)
 
     def __iter__(self) -> "GitlabList":
