wip: trying to find code that doesn't return a dict

JohnVillalovos · JohnVillalovos · commit 2e88cd4cd92d · 2022-07-22T23:22:04.000-07:00
diff --git a/.github/workflows/test.yml b/.github/workflows/test.yml
@@ -73,7 +73,7 @@ jobs:
       - name: Run tests
         env:
           TOXENV: ${{ matrix.toxenv }}
-        run: tox -- --override-ini='log_cli=True'
+        run: tox -- -vv --override-ini='log_cli=True'
       - name: Upload codecov coverage
         uses: codecov/codecov-action@v3
         with:
diff --git a/gitlab/exceptions.py b/gitlab/exceptions.py
@@ -16,6 +16,7 @@
 # along with this program.  If not, see <http://www.gnu.org/licenses/>.
 
 import functools
+import inspect
 from typing import Any, Callable, cast, Optional, Type, TYPE_CHECKING, TypeVar, Union
 
 
@@ -331,10 +332,23 @@ def wrap(f: __F) -> __F:
         @functools.wraps(f)
         def wrapped_f(*args: Any, **kwargs: Any) -> Any:
             try:
-                return f(*args, **kwargs)
+                result = f(*args, **kwargs)
             except GitlabHttpError as e:
                 raise error(e.error_message, e.response_code, e.response_body) from e
 
+            annotations = inspect.get_annotations(f)
+            return_type = annotations.get("return", "")
+            if (
+                "typing.Union[typing.Dict[str, typing.Any], "
+                "typing.List[typing.Any], requests.models.Response]"
+            ) in str(return_type):
+                if not isinstance(result, dict):
+                    print(f, annotations)
+                    print(f"str: {str(return_type)!r}")
+                    print(f"{type(result)} {result!r}")
+                    assert 0
+            return result
+
         return cast(__F, wrapped_f)
 
     return wrap
