removed some unnecessary functions from reflect module

Author: eadwinCode

ellar/common/__init__.py

@@ -8,6 +8,7 @@
     Module,
     command,
     exception_handler,
+    extra_args,
     file,
     guards,
     middleware,
@@ -98,6 +99,7 @@
     "Http",
     "UploadFile",
     "file",
+    "extra_args",
 ]
 
 

ellar/common/decorators/__init__.py

@@ -4,6 +4,7 @@
 from .command import command
 from .controller import Controller
 from .exception import exception_handler
+from .extra_args import extra_args
 from .file import file
 from .guards import guards
 from .html import render, template_filter, template_global
@@ -31,6 +32,7 @@
     "middleware",
     "openapi_info",
     "Module",
+    "extra_args",
 ]
 
 

ellar/common/decorators/base.py

@@ -8,13 +8,12 @@
 def set_metadata(
     meta_key: t.Any,
     meta_value: t.Optional[t.Any] = NOT_SET,
-    default_value: t.Any = None,
 ) -> t.Callable:
     if meta_value is NOT_SET:
         return partial(set_metadata, meta_key)
 
     def _decorator(target: t.Union[t.Callable, t.Any]) -> t.Union[t.Callable, t.Any]:
-        reflect.define_metadata(meta_key, meta_value, target, default_value)
+        reflect.define_metadata(meta_key, meta_value, target)
         return target
 
     return _decorator

ellar/common/decorators/controller.py

@@ -58,9 +58,8 @@ def reflect_all_controller_type_routes(cls: t.Type[ControllerBase]) -> None:
                 reflect.define_metadata(CONTROLLER_CLASS_KEY, cls, endpoint_func)
                 reflect.define_metadata(
                     CONTROLLER_OPERATION_HANDLER_KEY,
-                    operation,
+                    [operation],
                     cls,
-                    default_value=[],
                 )
 
 

ellar/common/decorators/extra_args.py

@@ -0,0 +1,18 @@
+import typing as t
+
+from ellar.constants import EXTRA_ROUTE_ARGS_KEY
+from ellar.core.params import ExtraEndpointArg
+
+from .base import set_metadata as set_meta
+
+
+def extra_args(*args: ExtraEndpointArg) -> t.Callable:
+    """
+    =========FUNCTION DECORATOR ==============
+
+    Programmatically adds extra route function parameter.
+    see https://github.com/eadwinCode/ellar/blob/main/tests/test_routing/test_extra_args.py for usages
+    :param args: Collection ExtraEndpointArg
+    :return:
+    """
+    return set_meta(EXTRA_ROUTE_ARGS_KEY, list(args))

ellar/common/decorators/guards.py

@@ -18,4 +18,4 @@ def guards(
     :param _guards: Guard Type or Instance
     :return:
     """
-    return set_meta(GUARDS_KEY, _guards, default_value=[])
+    return set_meta(GUARDS_KEY, list(_guards))

ellar/common/decorators/modules.py

@@ -90,14 +90,14 @@ def Module(
     """
     kwargs = AttributeDict(
         name=name,
-        controllers=controllers,
+        controllers=list(controllers),
         base_directory=base_directory,
         static_folder=static_folder,
-        routers=routers,
-        providers=providers,
+        routers=list(routers),
+        providers=list(providers),
         template_folder=template_folder,
-        modules=modules,
-        commands=commands,
+        modules=list(modules),
+        commands=list(commands),
     )
 
     return partial(

ellar/common/decorators/openapi.py

@@ -33,5 +33,4 @@ def openapi_info(
             deprecated=deprecated,
             tags=tags,
         ),
-        default_value=AttributeDict(),
     )

ellar/common/decorators/versioning.py

@@ -13,6 +13,4 @@ def version(*_version: str) -> t.Callable:
     :param _version: allowed versions
     :return:
     """
-    return set_meta(
-        VERSIONING_KEY, set([str(i) for i in _version]), default_value=set()
-    )
+    return set_meta(VERSIONING_KEY, set([str(i) for i in _version]))

ellar/core/modules/builder.py

@@ -46,18 +46,15 @@ def exception_config(self, exception_dict: t.Dict) -> None:
             func = CallableExceptionHandler(
                 self._cls, callable_exception_handler=v, exc_class_or_status_code=k
             )
-            reflect.define_metadata(
-                EXCEPTION_HANDLERS_KEY, func, self._cls, default_value=[]
-            )
+            reflect.define_metadata(EXCEPTION_HANDLERS_KEY, [func], self._cls)
 
     @t.no_type_check
     def middleware_config(self, middleware: "MiddlewareSchema") -> None:
         middleware.dispatch = module_callable_factory(middleware.dispatch, self._cls)
         reflect.define_metadata(
             MIDDLEWARE_HANDLERS_KEY,
-            middleware.create_middleware(),
+            [middleware.create_middleware()],
             self._cls,
-            default_value=[],
         )
 
     def on_request_shut_down_config(self, on_shutdown_event: EventHandler) -> None:
@@ -66,9 +63,8 @@ def on_request_shut_down_config(self, on_shutdown_event: EventHandler) -> None:
         )
         reflect.define_metadata(
             ON_REQUEST_SHUTDOWN_KEY,
-            on_shutdown_event,
+            [on_shutdown_event],
             self._cls,
-            default_value=[],
         )
 
     def on_request_startup_config(self, on_startup_event: EventHandler) -> None:
@@ -77,9 +73,8 @@ def on_request_startup_config(self, on_startup_event: EventHandler) -> None:
         )
         reflect.define_metadata(
             ON_REQUEST_STARTUP_KEY,
-            on_startup_event,
+            [on_startup_event],
             self._cls,
-            default_value=[],
         )
 
     def template_filter_config(self, template_filter: "TemplateFunctionData") -> None:
@@ -91,7 +86,6 @@ def template_filter_config(self, template_filter: "TemplateFunctionData") -> Non
                 )
             },
             self._cls,
-            default_value={},
         )
 
     def template_global_config(self, template_filter: "TemplateFunctionData") -> None:
@@ -103,7 +97,6 @@ def template_global_config(self, template_filter: "TemplateFunctionData") -> Non
                 )
             },
             self._cls,
-            default_value={},
         )
 
     def build(self, namespace: t.Dict) -> None: